Re: [PATCH] dt-bindings: ata: ahci-ceva: Cover all 4 iommus entries

From: Damien Le Moal
Date: Wed May 03 2023 - 22:34:12 EST


On 5/2/23 22:48, Michal Simek wrote:
> Current only one entry is enabled but IP itself is using 4 different IDs
> which are already listed in zynqmp.dtsi.
>
> sata: ahci@fd0c0000 {
> compatible = "ceva,ahci-1v84";
> ...
> iommus = <&smmu 0x4c0>, <&smmu 0x4c1>,
> <&smmu 0x4c2>, <&smmu 0x4c3>;
> };
>
> Signed-off-by: Michal Simek <michal.simek@xxxxxxx>
> ---
>
> Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml b/Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml
> index 9b31f864e071..71364c6081ff 100644
> --- a/Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml
> +++ b/Documentation/devicetree/bindings/ata/ceva,ahci-1v84.yaml
> @@ -32,7 +32,7 @@ properties:
> maxItems: 1
>
> iommus:
> - maxItems: 1
> + maxItems: 4
>
> power-domains:
> maxItems: 1

Is this a bug fix ? If yes, what about adding Fixes & Cc: stable tags ?