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

[Krzysztof Kozlowski]

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