2023-09-27 14:57:46

by Sridharan S N

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

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 <[email protected]>
---
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

2023-09-27 18:14:23

by Bryan O'Donoghue

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

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 <[email protected]>
> ---
> 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

2023-10-03 05:09:37

by Sridharan S N

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

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 <[email protected]>
>> ---
>>   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

2023-10-06 15:36:47

by Rob Herring (Arm)

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

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 <[email protected]>
> ---
> Documentation/devicetree/bindings/arm/qcom.yaml | 14 --------------
> 1 file changed, 14 deletions(-)
>

Acked-by: Rob Herring <[email protected]>

2023-12-03 04:54:26

by Bjorn Andersson

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

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 <[email protected]>