2023-12-06 18:53:43

by Brandon Brnich

[permalink] [raw]
Subject: [PATCH v2 1/2] dt-bindings: media: Remove K3 Family Prefix from Compatible

K3 family prefix is not included in other TI compatible strings. Remove
this prefix to keep naming convention consistent.

Fixes: de4b9f7e371a ("dt-bindings: media: wave5: add yaml devicetree bindings")
Reported-by: Geert Uytterhoeven <[email protected]>
Closes: https://lore.kernel.org/all/CAMuHMdUYOq=q1j=d+Eac28hthOUAaNUkuvxmRu-mUN1pLKq69g@mail.gmail.com/
Signed-off-by: Brandon Brnich <[email protected]>
---

Bindings currently reside in linux-next, meaning that the ABI will not be
broken with the removal of K3 prefix. Important to get this merged in
prior to being moved to master so that K3 prefix does not need to be
permanately included.

Documentation/devicetree/bindings/media/cnm,wave521c.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/media/cnm,wave521c.yaml b/Documentation/devicetree/bindings/media/cnm,wave521c.yaml
index 6d5569e77b7a..471840e858c9 100644
--- a/Documentation/devicetree/bindings/media/cnm,wave521c.yaml
+++ b/Documentation/devicetree/bindings/media/cnm,wave521c.yaml
@@ -17,7 +17,7 @@ properties:
compatible:
items:
- enum:
- - ti,k3-j721s2-wave521c
+ - ti,j721s2-wave521c
- const: cnm,wave521c

reg:
--
2.34.1


2023-12-06 18:57:36

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v2 1/2] dt-bindings: media: Remove K3 Family Prefix from Compatible

On 06/12/2023 19:52, Brandon Brnich wrote:
> K3 family prefix is not included in other TI compatible strings. Remove
> this prefix to keep naming convention consistent.
>
> Fixes: de4b9f7e371a ("dt-bindings: media: wave5: add yaml devicetree bindings")
> Reported-by: Geert Uytterhoeven <[email protected]>
> Closes: https://lore.kernel.org/all/CAMuHMdUYOq=q1j=d+Eac28hthOUAaNUkuvxmRu-mUN1pLKq69g@mail.gmail.com/
> Signed-off-by: Brandon Brnich <[email protected]>
> ---
>

Acked-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof

2023-12-06 21:37:55

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH v2 1/2] dt-bindings: media: Remove K3 Family Prefix from Compatible

On 12:52-20231206, Brandon Brnich wrote:
> K3 family prefix is not included in other TI compatible strings. Remove
> this prefix to keep naming convention consistent.
>
> Fixes: de4b9f7e371a ("dt-bindings: media: wave5: add yaml devicetree bindings")
> Reported-by: Geert Uytterhoeven <[email protected]>
> Closes: https://lore.kernel.org/all/CAMuHMdUYOq=q1j=d+Eac28hthOUAaNUkuvxmRu-mUN1pLKq69g@mail.gmail.com/
> Signed-off-by: Brandon Brnich <[email protected]>
> ---
>
> Bindings currently reside in linux-next, meaning that the ABI will not be
> broken with the removal of K3 prefix. Important to get this merged in
> prior to being moved to master so that K3 prefix does not need to be
> permanately included.
>
> Documentation/devicetree/bindings/media/cnm,wave521c.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/media/cnm,wave521c.yaml b/Documentation/devicetree/bindings/media/cnm,wave521c.yaml
> index 6d5569e77b7a..471840e858c9 100644
> --- a/Documentation/devicetree/bindings/media/cnm,wave521c.yaml
> +++ b/Documentation/devicetree/bindings/media/cnm,wave521c.yaml
> @@ -17,7 +17,7 @@ properties:
> compatible:
> items:
> - enum:
> - - ti,k3-j721s2-wave521c
> + - ti,j721s2-wave521c
> - const: cnm,wave521c
>
> reg:
> --
> 2.34.1
>

Reviewed-by: Nishanth Menon <[email protected]>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D

2023-12-07 12:45:49

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH v2 1/2] dt-bindings: media: Remove K3 Family Prefix from Compatible

On 12:52-20231206, Brandon Brnich wrote:
> K3 family prefix is not included in other TI compatible strings. Remove
> this prefix to keep naming convention consistent.
>
> Fixes: de4b9f7e371a ("dt-bindings: media: wave5: add yaml devicetree bindings")
> Reported-by: Geert Uytterhoeven <[email protected]>
> Closes: https://lore.kernel.org/all/CAMuHMdUYOq=q1j=d+Eac28hthOUAaNUkuvxmRu-mUN1pLKq69g@mail.gmail.com/
> Signed-off-by: Brandon Brnich <[email protected]>
> ---
>
> Bindings currently reside in linux-next, meaning that the ABI will not be
> broken with the removal of K3 prefix. Important to get this merged in
> prior to being moved to master so that K3 prefix does not need to be
> permanately included.
>
> Documentation/devicetree/bindings/media/cnm,wave521c.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/media/cnm,wave521c.yaml b/Documentation/devicetree/bindings/media/cnm,wave521c.yaml
> index 6d5569e77b7a..471840e858c9 100644
> --- a/Documentation/devicetree/bindings/media/cnm,wave521c.yaml
> +++ b/Documentation/devicetree/bindings/media/cnm,wave521c.yaml
> @@ -17,7 +17,7 @@ properties:
> compatible:
> items:
> - enum:
> - - ti,k3-j721s2-wave521c
> + - ti,j721s2-wave521c
> - const: cnm,wave521c


Apologies, I just noticed. you failed to update the vpu example for the
same in the bindings doc.

>
> reg:
> --
> 2.34.1
>

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D