Re: [PATCH 1/2] ARM: dts: r8a7793: Add SATA controller node

[Simon Horman]

On Wed, Mar 16, 2016 at 06:11:07PM +0100, Ulrich Hecht wrote:
> Adds SATA device node to r8a7793.dtsi.
> 
> Signed-off-by: Ulrich Hecht <ulrich.hecht+rene...@gmail.com>
> ---
>  arch/arm/boot/dts/r8a7793.dtsi | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi
> index 3ebe7d4..abb6851 100644
> --- a/arch/arm/boot/dts/r8a7793.dtsi
> +++ b/arch/arm/boot/dts/r8a7793.dtsi
> @@ -775,6 +775,22 @@
>               status = "disabled";
>       };
>  
> +     sata0: sata@ee300000 {
> +             compatible = "renesas,sata-r8a7793";
> +             reg = <0 0xee300000 0 0x2000>;
> +             interrupts = <0 105 IRQ_TYPE_LEVEL_HIGH>;
> +             clocks = <&mstp8_clks R8A7793_CLK_SATA0>;

I think you also need a power domains property:

                power-domains = <&cpg_clocks>;

> +             status = "disabled";
> +     };
> +
> +     sata1: sata@ee500000 {
> +             compatible = "renesas,sata-r8a7793";
> +             reg = <0 0xee500000 0 0x2000>;
> +             interrupts = <0 106 IRQ_TYPE_LEVEL_HIGH>;
> +             clocks = <&mstp8_clks R8A7793_CLK_SATA1>;

Here too.

> +             status = "disabled";
> +     };
> +
>       qspi: spi@e6b10000 {
>               compatible = "renesas,qspi-r8a7793", "renesas,qspi";
>               reg = <0 0xe6b10000 0 0x2c>;
> -- 
> 2.6.4