Re: [PATCH v2 1/2] dt-bindings: dma: ti: k3-bcdma: Add J722S CSI BCDMA

From: Krzysztof Kozlowski
Date: Tue Nov 26 2024 - 09:19:30 EST


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