dt-bindings: arm: qcom: drop the IPQ board types

[PATCH] dt-bindings: arm: qcom: drop the IPQ board types

Posted by Sridharan S N 2 years, 4 months ago

IPQ bootloaders do not need these information to select the
the DTB blob. So dropping the board names from board section.

Signed-off-by: Sridharan S N <quic_sridsn@quicinc.com>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 7f80f48a0954..5b56f3d90d4d 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -91,24 +91,10 @@ description: |
   The 'board' element must be one of the following strings:
 
         adp
-        ap-al02-c2
-        ap-al02-c6
-        ap-al02-c7
-        ap-al02-c8
-        ap-al02-c9
-        ap-mi01.2
-        ap-mi01.3
-        ap-mi01.6
-        ap-mi01.9
         cdp
-        cp01-c1
         dragonboard
-        hk01
-        hk10-c1
-        hk10-c2
         idp
         liquid
-        rdp432-c2
         mtp
         qrd
         rb2
-- 
2.34.1

Re: [PATCH] dt-bindings: arm: qcom: drop the IPQ board types

Posted by Bjorn Andersson 2 years, 2 months ago

On Wed, 27 Sep 2023 15:01:30 +0530, Sridharan S N wrote:
> IPQ bootloaders do not need these information to select the
> the DTB blob. So dropping the board names from board section.
> 
> 

Applied, thanks!

[1/1] dt-bindings: arm: qcom: drop the IPQ board types
      commit: 14259fcdaf72d3ce93ad9c2b12cc0e96ed5a0c4e

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>

Re: [PATCH] dt-bindings: arm: qcom: drop the IPQ board types

Posted by Rob Herring 2 years, 4 months ago

On Wed, 27 Sep 2023 15:01:30 +0530, Sridharan S N wrote:
> IPQ bootloaders do not need these information to select the
> the DTB blob. So dropping the board names from board section.
> 
> Signed-off-by: Sridharan S N <quic_sridsn@quicinc.com>
> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 14 --------------
>  1 file changed, 14 deletions(-)
> 

Acked-by: Rob Herring <robh@kernel.org>

Re: [PATCH] dt-bindings: arm: qcom: drop the IPQ board types

Posted by Bryan O'Donoghue 2 years, 4 months ago

On 27/09/2023 10:31, Sridharan S N wrote:
> IPQ bootloaders do not need these information to select the
> the DTB blob. So dropping the board names from board section.
> 
> Signed-off-by: Sridharan S N <quic_sridsn@quicinc.com>
> ---
>   Documentation/devicetree/bindings/arm/qcom.yaml | 14 --------------
>   1 file changed, 14 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index 7f80f48a0954..5b56f3d90d4d 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -91,24 +91,10 @@ description: |
>     The 'board' element must be one of the following strings:
>   
>           adp
> -        ap-al02-c2
> -        ap-al02-c6
> -        ap-al02-c7
> -        ap-al02-c8
> -        ap-al02-c9
> -        ap-mi01.2
> -        ap-mi01.3
> -        ap-mi01.6
> -        ap-mi01.9
>           cdp
> -        cp01-c1
>           dragonboard
> -        hk01
> -        hk10-c1
> -        hk10-c2
>           idp
>           liquid
> -        rdp432-c2
>           mtp
>           qrd
>           rb2

All bootloaders ? You are 100% sure that all versions of downstream 
u-boot don't rely on these strings ? Upstream u-boot too ?

---
bod

Re: [PATCH] dt-bindings: arm: qcom: drop the IPQ board types

Posted by Sridharan S N 2 years, 4 months ago

On 9/27/2023 5:01 PM, Bryan O'Donoghue wrote:
> On 27/09/2023 10:31, Sridharan S N wrote:
>> IPQ bootloaders do not need these information to select the
>> the DTB blob. So dropping the board names from board section.
>>
>> Signed-off-by: Sridharan S N <quic_sridsn@quicinc.com>
>> ---
>>   Documentation/devicetree/bindings/arm/qcom.yaml | 14 --------------
>>   1 file changed, 14 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml 
>> b/Documentation/devicetree/bindings/arm/qcom.yaml
>> index 7f80f48a0954..5b56f3d90d4d 100644
>> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
>> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
>> @@ -91,24 +91,10 @@ description: |
>>     The 'board' element must be one of the following strings:
>>             adp
>> -        ap-al02-c2
>> -        ap-al02-c6
>> -        ap-al02-c7
>> -        ap-al02-c8
>> -        ap-al02-c9
>> -        ap-mi01.2
>> -        ap-mi01.3
>> -        ap-mi01.6
>> -        ap-mi01.9
>>           cdp
>> -        cp01-c1
>>           dragonboard
>> -        hk01
>> -        hk10-c1
>> -        hk10-c2
>>           idp
>>           liquid
>> -        rdp432-c2
>>           mtp
>>           qrd
>>           rb2
>
> All bootloaders ? You are 100% sure that all versions of downstream 
> u-boot don't rely on these strings ? Upstream u-boot too ?
>
> ---
> bod

Yes , we don't rely on these strings , instead downstream u-boot uses 
machid to identify the DTB.

Thanks,

Sridharan S N