[PATCH 1/2] dt-bindings: arm: ti: k3: Sort the am654 board enums

Nishanth Menon posted 2 patches 3 years, 7 months ago
[PATCH 1/2] dt-bindings: arm: ti: k3: Sort the am654 board enums
Posted by Nishanth Menon 3 years, 7 months ago
Use alphabetical sort to organize the am654 board names.

Reported-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
 Documentation/devicetree/bindings/arm/ti/k3.yaml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml
index 61c6ab4f52e2..7e93e87dcdf4 100644
--- a/Documentation/devicetree/bindings/arm/ti/k3.yaml
+++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml
@@ -22,11 +22,11 @@ properties:
       - description: K3 AM654 SoC
         items:
           - enum:
-              - ti,am654-evm
-              - siemens,iot2050-basic
-              - siemens,iot2050-basic-pg2
               - siemens,iot2050-advanced
               - siemens,iot2050-advanced-pg2
+              - siemens,iot2050-basic
+              - siemens,iot2050-basic-pg2
+              - ti,am654-evm
           - const: ti,am654
 
       - description: K3 J721E SoC
-- 
2.31.1
Re: [PATCH 1/2] dt-bindings: arm: ti: k3: Sort the am654 board enums
Posted by Krzysztof Kozlowski 3 years, 7 months ago
On 30/08/2022 19:05, Nishanth Menon wrote:
> Use alphabetical sort to organize the am654 board names.
> 
> Reported-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

It's not a bug, so it should be rather "Suggested-by".

> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  Documentation/devicetree/bindings/arm/ti/k3.yaml | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)


Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
Re: [PATCH 1/2] dt-bindings: arm: ti: k3: Sort the am654 board enums
Posted by Nishanth Menon 3 years, 7 months ago
On 19:50-20220830, Krzysztof Kozlowski wrote:
> On 30/08/2022 19:05, Nishanth Menon wrote:
> > Use alphabetical sort to organize the am654 board names.
> > 
> > Reported-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> It's not a bug, so it should be rather "Suggested-by".
> 
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> > ---
> >  Documentation/devicetree/bindings/arm/ti/k3.yaml | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> 
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> Best regards,
> Krzysztof

Thanks - Vignesh: let me know if you'd like a v2 with
s/Reported-by/Suggested-by update.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
Re: [PATCH 1/2] dt-bindings: arm: ti: k3: Sort the am654 board enums
Posted by Vignesh Raghavendra 3 years, 7 months ago

On 30/08/22 23:08, Nishanth Menon wrote:
> On 19:50-20220830, Krzysztof Kozlowski wrote:
>> On 30/08/2022 19:05, Nishanth Menon wrote:
>>> Use alphabetical sort to organize the am654 board names.
>>>
>>> Reported-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>
>> It's not a bug, so it should be rather "Suggested-by".
>>
>>> Signed-off-by: Nishanth Menon <nm@ti.com>
>>> ---
>>>  Documentation/devicetree/bindings/arm/ti/k3.yaml | 6 +++---
>>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>
>>
>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>
>> Best regards,
>> Krzysztof
> 
> Thanks - Vignesh: let me know if you'd like a v2 with
> s/Reported-by/Suggested-by update.
> 

No need, I can take care of this locally before applying.

-- 
Regards
Vignesh