[PATCH 0/2] dt-bindings: arm: ti: k3: Sort alphabetically

[PATCH 0/2] dt-bindings: arm: ti: k3: Sort alphabetically

[Nishanth Menon]

Inspired by the discussion in [1] - sort the yaml to keep things clean
for the new SoCs and boards that are getting added in.

[1] https://lore.kernel.org/all/d7303503-70ef-7307-8cc0-1bc334c9e6af@linaro.org/

Nishanth Menon (2):
  dt-bindings: arm: ti: k3: Sort the am654 board enums
  dt-bindings: arm: ti: k3: Sort the SoC definitions alphabetically

 .../devicetree/bindings/arm/ti/k3.yaml        | 48 +++++++++----------
 1 file changed, 24 insertions(+), 24 deletions(-)

-- 
2.31.1

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

[Nishanth Menon]

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

[Krzysztof Kozlowski]

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

[Nishanth Menon]

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

[Vignesh Raghavendra]

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

[PATCH 2/2] dt-bindings: arm: ti: k3: Sort the SoC definitions alphabetically

[Nishanth Menon]

Use alphabetical sort to organize the SoCs

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

diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml
index 7e93e87dcdf4..aeef800eb6ec 100644
--- a/Documentation/devicetree/bindings/arm/ti/k3.yaml
+++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml
@@ -19,6 +19,19 @@ properties:
   compatible:
     oneOf:
 
+      - description: K3 AM625 SoC
+        items:
+          - enum:
+              - ti,am625-sk
+          - const: ti,am625
+
+      - description: K3 AM642 SoC
+        items:
+          - enum:
+              - ti,am642-evm
+              - ti,am642-sk
+          - const: ti,am642
+
       - description: K3 AM654 SoC
         items:
           - enum:
@@ -29,15 +42,6 @@ properties:
               - ti,am654-evm
           - const: ti,am654
 
-      - description: K3 J721E SoC
-        oneOf:
-          - const: ti,j721e
-          - items:
-              - enum:
-                  - ti,j721e-evm
-                  - ti,j721e-sk
-              - const: ti,j721e
-
       - description: K3 J7200 SoC
         oneOf:
           - const: ti,j7200
@@ -46,18 +50,14 @@ properties:
                   - ti,j7200-evm
               - const: ti,j7200
 
-      - description: K3 AM625 SoC
-        items:
-          - enum:
-              - ti,am625-sk
-          - const: ti,am625
-
-      - description: K3 AM642 SoC
-        items:
-          - enum:
-              - ti,am642-evm
-              - ti,am642-sk
-          - const: ti,am642
+      - description: K3 J721E SoC
+        oneOf:
+          - const: ti,j721e
+          - items:
+              - enum:
+                  - ti,j721e-evm
+                  - ti,j721e-sk
+              - const: ti,j721e
 
       - description: K3 J721s2 SoC
         items:
-- 
2.31.1

Re: [PATCH 2/2] dt-bindings: arm: ti: k3: Sort the SoC definitions alphabetically

[Krzysztof Kozlowski]

On 30/08/2022 19:05, Nishanth Menon wrote:
> Use alphabetical sort to organize the SoCs
> 
> Reported-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Suggested-by

> Signed-off-by: Nishanth Menon <nm@ti.com>


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


Best regards,
Krzysztof

Re: [PATCH 0/2] dt-bindings: arm: ti: k3: Sort alphabetically

[Vignesh Raghavendra]

Hi Nishanth Menon,

On Tue, 30 Aug 2022 11:05:05 -0500, Nishanth Menon wrote:
> Inspired by the discussion in [1] - sort the yaml to keep things clean
> for the new SoCs and boards that are getting added in.
> 
> [1] https://lore.kernel.org/all/d7303503-70ef-7307-8cc0-1bc334c9e6af@linaro.org/
> 
> Nishanth Menon (2):
>   dt-bindings: arm: ti: k3: Sort the am654 board enums
>   dt-bindings: arm: ti: k3: Sort the SoC definitions alphabetically
> 
> [...]

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/2] dt-bindings: arm: ti: k3: Sort the am654 board enums
      commit: 5f120a4dc7a71187fdae0a11f6c65b7e2cf7a2d7
[2/2] dt-bindings: arm: ti: k3: Sort the SoC definitions alphabetically
      commit: b7365658294304fac75bf33d11824d3a949122a2

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
--
Vignesh