diff mbox series

[1/2] dt-bindings: arm: qcom: Update the compatible

Message ID 20230720072938.31546-2-quic_harihk@quicinc.com (mailing list archive)
State Superseded
Headers show
Series Add common RDP dtsi file for ipq5018 | expand

Commit Message

Hariharan K July 20, 2023, 7:29 a.m. UTC
Update the compatible in device tree binding to
use model names instead of rdp numbers for IPQ5018
platform.

Signed-off-by: Hariharan K <quic_harihk@quicinc.com>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Krzysztof Kozlowski July 20, 2023, 8:42 a.m. UTC | #1
On 20/07/2023 09:29, Hariharan K wrote:
> Update the compatible in device tree binding to
> use model names instead of rdp numbers for IPQ5018
> platform.
> 
> Signed-off-by: Hariharan K <quic_harihk@quicinc.com>
> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index e94ce1e26fbf..922feb912594 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -97,6 +97,7 @@ description: |
>          ap-mi01.3
>          ap-mi01.6
>          ap-mi01.9
> +        ap-mp03.5-c2
>          cdp
>          cp01-c1
>          dragonboard
> @@ -105,7 +106,6 @@ description: |
>          hk10-c2
>          idp
>          liquid
> -        rdp432-c2
>          mtp
>          qrd
>          rb2
> @@ -343,7 +343,7 @@ properties:
>  
>        - items:
>            - enum:
> -              - qcom,ipq5018-rdp432-c2


There is no such compatible. Don't send patches with wrong names and
before they got merged send a follow up.

Fix the original submission.

Best regards,
Krzysztof
Hariharan K July 20, 2023, 11:04 a.m. UTC | #2
On 7/20/2023 2:12 PM, Krzysztof Kozlowski wrote:
> On 20/07/2023 09:29, Hariharan K wrote:
>> Update the compatible in device tree binding to
>> use model names instead of rdp numbers for IPQ5018
>> platform.
>>
>> Signed-off-by: Hariharan K <quic_harihk@quicinc.com>
>> ---
>>   Documentation/devicetree/bindings/arm/qcom.yaml | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
>> index e94ce1e26fbf..922feb912594 100644
>> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
>> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
>> @@ -97,6 +97,7 @@ description: |
>>           ap-mi01.3
>>           ap-mi01.6
>>           ap-mi01.9
>> +        ap-mp03.5-c2
>>           cdp
>>           cp01-c1
>>           dragonboard
>> @@ -105,7 +106,6 @@ description: |
>>           hk10-c2
>>           idp
>>           liquid
>> -        rdp432-c2
>>           mtp
>>           qrd
>>           rb2
>> @@ -343,7 +343,7 @@ properties:
>>   
>>         - items:
>>             - enum:
>> -              - qcom,ipq5018-rdp432-c2
> 
> 
> There is no such compatible. Don't send patches with wrong names and
> before they got merged send a follow up.
> 
> Fix the original submission.

Sorry, this patch is in-correct. Names used in the original submission 
are the right ones. Will drop this patch and also fix it in other 
patches that i have posted.

Best regards,
Hariharan K

> 
> Best regards,
> Krzysztof
>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index e94ce1e26fbf..922feb912594 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -97,6 +97,7 @@  description: |
         ap-mi01.3
         ap-mi01.6
         ap-mi01.9
+        ap-mp03.5-c2
         cdp
         cp01-c1
         dragonboard
@@ -105,7 +106,6 @@  description: |
         hk10-c2
         idp
         liquid
-        rdp432-c2
         mtp
         qrd
         rb2
@@ -343,7 +343,7 @@  properties:
 
       - items:
           - enum:
-              - qcom,ipq5018-rdp432-c2
+              - qcom,ipq5018-ap-mp03.5-c2
           - const: qcom,ipq5018
 
       - items: