| Message ID | 20241126125158.37744-1-vaishnav.a@ti.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v2,1/2] dt-bindings: dma: ti: k3-bcdma: Add J722S CSI BCDMA | expand |
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@ti.com> > --- > > 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@ti.com/ > > 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
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 then: properties: ti,sci-rm-range-bchan: false
J722S CSI BCDMA is similar to J721S2 CSI BCDMA and supports both RX and TX channels. Add an entry for J722S CSIRX BCDMA. Signed-off-by: Vaishnav Achath <vaishnav.a@ti.com> --- 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@ti.com/ Documentation/devicetree/bindings/dma/ti/k3-bcdma.yaml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)