diff mbox series

[1/3] dt-bindings: arm: ti: k3: Add compatible for AM62x LP SK

Message ID 20230320051304.2730692-2-vigneshr@ti.com (mailing list archive)
State New, archived
Headers show
Series arm64: dts: ti: Introduce AM62x LP SK board support | expand

Commit Message

Vignesh Raghavendra March 20, 2023, 5:13 a.m. UTC
Add compatible for AM62x SoC based Low Power Starter Kit board[1]

[1] https://www.ti.com/tool/SK-AM62-LP

Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
---
 Documentation/devicetree/bindings/arm/ti/k3.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski March 20, 2023, 3:20 p.m. UTC | #1
On 20/03/2023 06:13, Vignesh Raghavendra wrote:
> Add compatible for AM62x SoC based Low Power Starter Kit board[1]
> 
> [1] https://www.ti.com/tool/SK-AM62-LP
> 
> Signed-off-by: Vignesh Raghavendra <vigneshr@ti.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..56fdd94bf899 100644
> --- a/Documentation/devicetree/bindings/arm/ti/k3.yaml
> +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml
> @@ -29,6 +29,7 @@ properties:
>          items:
>            - enum:
>                - ti,am625-sk
> +              - ti,am62x-lp-sk

Don't use wildcards in compatibles... but judging by
https://www.ti.com/tool/SK-AM62 it looks like model name. If it is truly
model name, not wildcard, then:

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

Best regards,
Krzysztof
Vignesh Raghavendra March 21, 2023, 4:09 a.m. UTC | #2
On 20/03/23 20:50, Krzysztof Kozlowski wrote:
> On 20/03/2023 06:13, Vignesh Raghavendra wrote:
>> Add compatible for AM62x SoC based Low Power Starter Kit board[1]
>>
>> [1] https://www.ti.com/tool/SK-AM62-LP
>>
>> Signed-off-by: Vignesh Raghavendra <vigneshr@ti.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..56fdd94bf899 100644
>> --- a/Documentation/devicetree/bindings/arm/ti/k3.yaml
>> +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml
>> @@ -29,6 +29,7 @@ properties:
>>          items:
>>            - enum:
>>                - ti,am625-sk
>> +              - ti,am62x-lp-sk
> 
> Don't use wildcards in compatibles... but judging by
> https://www.ti.com/tool/SK-AM62 it looks like model name. If it is truly
> model name, not wildcard, then:

I used the name on the EVM's silk screen. But I think its better to drop
the wildcard. Will respin with that fixed.

FWIW Correct URL is https://www.ti.com/tool/SK-AM62-LP (Boards look very
identical though ;) )

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

Thanks
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml
index a60a4065caa8..56fdd94bf899 100644
--- a/Documentation/devicetree/bindings/arm/ti/k3.yaml
+++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml
@@ -29,6 +29,7 @@  properties:
         items:
           - enum:
               - ti,am625-sk
+              - ti,am62x-lp-sk
           - const: ti,am625
 
       - description: K3 AM642 SoC