On 26/11/2024 13:51, Vaishnav Achath wrote: > J722S CSI BCDMA is similar to J721S2 CSI BCDMA and > supports both RX and TX channels. Add an entry for > J722S CSIRX BCDMA. Please wrap commit message according to Linux coding style / submission process (neither too early nor over the limit): https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst#L597 > > Signed-off-by: Vaishnav Achath <vaishnav.a@xxxxxx> > --- > > V1->V2: > * Address review from Conor to add new J722S compatible > * J722S BCDMA is more similar to J721S2 in terms of RX/TX support, > add an entry alongside J721S2 instead of modifying AM62A. > > V1: https://lore.kernel.org/all/20241125083914.2934815-1-vaishnav.a@xxxxxx/ > > Documentation/devicetree/bindings/dma/ti/k3-bcdma.yaml | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/dma/ti/k3-bcdma.yaml b/Documentation/devicetree/bindings/dma/ti/k3-bcdma.yaml > index 27b8e1636560..37832c71bd8e 100644 > --- a/Documentation/devicetree/bindings/dma/ti/k3-bcdma.yaml > +++ b/Documentation/devicetree/bindings/dma/ti/k3-bcdma.yaml > @@ -196,7 +196,9 @@ allOf: > properties: > compatible: > contains: > - const: ti,j721s2-dmss-bcdma-csi > + enum: > + - ti,j721s2-dmss-bcdma-csi > + - ti,j722s-dmss-bcdma-csi This compatible was never documented. There is no dependency here, no cover letter explaining where is this compatible introduced. Best regards, Krzysztof
