[v3,5/5] dt-bindings: qcom,pdc: Introduce pdc bindings for QDU1000 and QRU1000

Message ID 20221026190441.4002212-6-quic_molvera@quicinc.com
State New
Headers
Series clk: qcom: Add clocks for the QDU1000 and QRU1000 SoCs |

Commit Message

Melody Olvera Oct. 26, 2022, 7:04 p.m. UTC
  Add compatible fields for QDU1000 and QRU1000 pdcs.

Signed-off-by: Melody Olvera <quic_molvera@quicinc.com>
---
 .../devicetree/bindings/interrupt-controller/qcom,pdc.yaml       | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Krzysztof Kozlowski Oct. 27, 2022, 3:58 p.m. UTC | #1
On 26/10/2022 15:04, Melody Olvera wrote:
> Add compatible fields for QDU1000 and QRU1000 pdcs.
> 
> Signed-off-by: Melody Olvera <quic_molvera@quicinc.com>

Looks ok, except usual stuff to fix...


Best regards,
Krzysztof
  
Stephen Boyd Oct. 27, 2022, 6:22 p.m. UTC | #2
Quoting Melody Olvera (2022-10-26 12:04:41)
> Add compatible fields for QDU1000 and QRU1000 pdcs.
> 
> Signed-off-by: Melody Olvera <quic_molvera@quicinc.com>
> ---

Is there a reason why this patch continues to be included in what is
otherwise a clk driver patch series? Can this patch be sent separately
from the clk patches (and not Cced to clk maintainers/list) in the
future?
  
Melody Olvera Oct. 27, 2022, 9:31 p.m. UTC | #3
On 10/27/2022 11:22 AM, Stephen Boyd wrote:
> Quoting Melody Olvera (2022-10-26 12:04:41)
>> Add compatible fields for QDU1000 and QRU1000 pdcs.
>>
>> Signed-off-by: Melody Olvera <quic_molvera@quicinc.com>
>> ---
> Is there a reason why this patch continues to be included in what is
> otherwise a clk driver patch series? Can this patch be sent separately
> from the clk patches (and not Cced to clk maintainers/list) in the
> future?
Sure thing. On reflection, this belongs with the misc support patches.

Thanks,
Melody
  
Stephen Boyd Oct. 27, 2022, 9:38 p.m. UTC | #4
Quoting Melody Olvera (2022-10-27 14:31:08)
> 
> 
> On 10/27/2022 11:22 AM, Stephen Boyd wrote:
> > Quoting Melody Olvera (2022-10-26 12:04:41)
> >> Add compatible fields for QDU1000 and QRU1000 pdcs.
> >>
> >> Signed-off-by: Melody Olvera <quic_molvera@quicinc.com>
> >> ---
> > Is there a reason why this patch continues to be included in what is
> > otherwise a clk driver patch series? Can this patch be sent separately
> > from the clk patches (and not Cced to clk maintainers/list) in the
> > future?
> Sure thing. On reflection, this belongs with the misc support patches.
> 

Thanks!
  
Bjorn Andersson Nov. 7, 2022, 5:16 p.m. UTC | #5
On Thu, Oct 27, 2022 at 02:31:08PM -0700, Melody Olvera wrote:
> 
> 
> On 10/27/2022 11:22 AM, Stephen Boyd wrote:
> > Quoting Melody Olvera (2022-10-26 12:04:41)
> >> Add compatible fields for QDU1000 and QRU1000 pdcs.
> >>
> >> Signed-off-by: Melody Olvera <quic_molvera@quicinc.com>
> >> ---
> > Is there a reason why this patch continues to be included in what is
> > otherwise a clk driver patch series? Can this patch be sent separately
> > from the clk patches (and not Cced to clk maintainers/list) in the
> > future?
> Sure thing. On reflection, this belongs with the misc support patches.
> 

Even better, the "misc" series goes through my tree but this patch is
unrelated. So if you send it on it's own it could just be picked up by
the PDC maintainer (or Rob).

Regards,
Bjorn
  
Melody Olvera Nov. 7, 2022, 11:20 p.m. UTC | #6
On 11/7/2022 9:16 AM, Bjorn Andersson wrote:
> On Thu, Oct 27, 2022 at 02:31:08PM -0700, Melody Olvera wrote:
>>
>> On 10/27/2022 11:22 AM, Stephen Boyd wrote:
>>> Quoting Melody Olvera (2022-10-26 12:04:41)
>>>> Add compatible fields for QDU1000 and QRU1000 pdcs.
>>>>
>>>> Signed-off-by: Melody Olvera <quic_molvera@quicinc.com>
>>>> ---
>>> Is there a reason why this patch continues to be included in what is
>>> otherwise a clk driver patch series? Can this patch be sent separately
>>> from the clk patches (and not Cced to clk maintainers/list) in the
>>> future?
>> Sure thing. On reflection, this belongs with the misc support patches.
>>
> Even better, the "misc" series goes through my tree but this patch is
> unrelated. So if you send it on it's own it could just be picked up by
> the PDC maintainer (or Rob).
>
>
Sure thing. I'll just send each on their own.

Thanks,
Melody
  

Patch

diff --git a/Documentation/devicetree/bindings/interrupt-controller/qcom,pdc.yaml b/Documentation/devicetree/bindings/interrupt-controller/qcom,pdc.yaml
index b6f56cf5fbe3..afeed7b8c0fa 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/qcom,pdc.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/qcom,pdc.yaml
@@ -26,6 +26,7 @@  properties:
   compatible:
     items:
       - enum:
+          - qcom,qdu1000-pdc
           - qcom,sc7180-pdc
           - qcom,sc7280-pdc
           - qcom,sdm845-pdc