Re: [PATCH 1/2] dt-bindings: ata: rockchip-dwc-ahci: add RK3576 compatible

Niklas Cassel
2025-04-25T08:02:22Z

On Thu, Apr 24, 2025 at 08:52:22PM +0200, Nicolas Frattaroli wrote:
> The Rockchip RK3576 has two SATA controllers. They work the same as the
> RK3568 SATA controllers, having the same number of clocks and ports.
>
> Signed-off-by: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
> ---
> Documentation/devicetree/bindings/ata/rockchip,dwc-ahci.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/ata/rockchip,dwc-ahci.yaml b/Documentation/devicetree/bindings/ata/rockchip,dwc-ahci.yaml
> index 13eaa8d9a16e5a4bd43b3e184f9277494acf27a1..b5ecaabfe2e2537afe6093558fb0ab975dcf6058 100644
> --- a/Documentation/devicetree/bindings/ata/rockchip,dwc-ahci.yaml
> +++ b/Documentation/devicetree/bindings/ata/rockchip,dwc-ahci.yaml
> @@ -20,6 +20,7 @@ select:
> contains:
> enum:
> - rockchip,rk3568-dwc-ahci
> + - rockchip,rk3576-dwc-ahci
> - rockchip,rk3588-dwc-ahci
> required:
> - compatible
> @@ -29,6 +30,7 @@ properties:
> items:
> - enum:
> - rockchip,rk3568-dwc-ahci
> + - rockchip,rk3576-dwc-ahci
> - rockchip,rk3588-dwc-ahci
> - const: snps,dwc-ahci
>
> @@ -83,6 +85,7 @@ allOf:
> contains:
> enum:
> - rockchip,rk3568-dwc-ahci
> + - rockchip,rk3576-dwc-ahci
> then:
> properties:
> clocks:
>
> --
> 2.49.0
>

Looks good to me:
Reviewed-by: Niklas Cassel <cassel@kernel.org>