Messages in this thread | ![/](/images/icornerl.gif) | | Date | Tue, 12 Dec 2023 09:06:34 -0600 | From | Nishanth Menon <> | Subject | Re: [PATCH v3 1/2] dt-bindings: media: Remove K3 Family Prefix from Compatible |
| |
On 14:59-20231211, 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 <geert@linux-m68k.org> > Closes: https://lore.kernel.org/all/CAMuHMdUYOq=q1j=d+Eac28hthOUAaNUkuvxmRu-mUN1pLKq69g@mail.gmail.com/ > Signed-off-by: Brandon Brnich <b-brnich@ti.com> > --- > V2 -> V3: Update compatible in example > > 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 | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/cnm,wave521c.yaml b/Documentation/devicetree/bindings/media/cnm,wave521c.yaml > index 6d5569e77b7a..6a11c1d11fb5 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: > @@ -53,7 +53,7 @@ additionalProperties: false > examples: > - | > vpu: video-codec@12345678 { > - compatible = "ti,k3-j721s2-wave521c", "cnm,wave521c"; > + compatible = "ti,j721s2-wave521c", "cnm,wave521c"; > reg = <0x12345678 0x1000>; > clocks = <&clks 42>; > interrupts = <42>; > -- > 2.34.1 >
Reviewed-by: Nishanth Menon <nm@ti.com>
-- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
| ![\](/images/icornerr.gif) |