[PATCH 1/3] dt-bindings: arm: ti: Add bindings for BeaglePlay

Nishanth Menon posted 3 patches 2 years, 11 months ago
There is a newer version of this series
[PATCH 1/3] dt-bindings: arm: ti: Add bindings for BeaglePlay
Posted by Nishanth Menon 2 years, 11 months ago
From: Robert Nelson <robertcnelson@gmail.com>

This board is based on ti,am625

https://beagleplay.org/
https://git.beagleboard.org/beagleplay/beagleplay

Co-developed-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Robert Nelson <robertcnelson@gmail.com>
---
 Documentation/devicetree/bindings/arm/ti/k3.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml
index a60a4065caa8..b4703ac88f04 100644
--- a/Documentation/devicetree/bindings/arm/ti/k3.yaml
+++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml
@@ -28,6 +28,7 @@ properties:
       - description: K3 AM625 SoC
         items:
           - enum:
+              - beagle,am625-beagleplay
               - ti,am625-sk
           - const: ti,am625
 
-- 
2.37.2
Re: [PATCH 1/3] dt-bindings: arm: ti: Add bindings for BeaglePlay
Posted by Krzysztof Kozlowski 2 years, 11 months ago
On 11/03/2023 12:10, Nishanth Menon wrote:
> From: Robert Nelson <robertcnelson@gmail.com>
> 
> This board is based on ti,am625

Subject: drop second/last, redundant "bindings for". The "dt-bindings"
prefix is already stating that these are bindings.

> 
> https://beagleplay.org/
> https://git.beagleboard.org/beagleplay/beagleplay
> 
> Co-developed-by: Nishanth Menon <nm@ti.com>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> Signed-off-by: Robert Nelson <robertcnelson@gmail.com>

Your chain order is not correct. If Robert is the author, his SoB is first.

> ---
>  Documentation/devicetree/bindings/arm/ti/k3.yaml | 1 +
>  1 file changed, 1 insertion(+)

Best regards,
Krzysztof
Re: [PATCH 1/3] dt-bindings: arm: ti: Add bindings for BeaglePlay
Posted by Nishanth Menon 2 years, 11 months ago
On 17:00-20230311, Krzysztof Kozlowski wrote:
> On 11/03/2023 12:10, Nishanth Menon wrote:
> > From: Robert Nelson <robertcnelson@gmail.com>
> > 
> > This board is based on ti,am625
> 
> Subject: drop second/last, redundant "bindings for". The "dt-bindings"
> prefix is already stating that these are bindings.
> 
> > 
> > https://beagleplay.org/
> > https://git.beagleboard.org/beagleplay/beagleplay
> > 
> > Co-developed-by: Nishanth Menon <nm@ti.com>
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> > Signed-off-by: Robert Nelson <robertcnelson@gmail.com>
> 
> Your chain order is not correct. If Robert is the author, his SoB is first.

Thanks for reviewing. will fixup and redo the series.
-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D