On Thu, Aug 02, 2018 at 01:31:52PM +0200, Geert Uytterhoeven wrote: > Hi Simon, > > On Thu, Aug 2, 2018 at 1:13 PM Simon Horman <horms@xxxxxxxxxxxx> wrote: > > On Mon, Jul 30, 2018 at 09:33:38AM +0200, Wolfram Sang wrote: > > > From: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> > > > This patch adds SATA controller node for the R8A77965 SoC. > > > > > > Signed-off-by: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> > > > [wsa: rebased to upstream base] > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> > > > --- > > > > > > Changes since v1: > > > * sorted it according to address (Thanks, Geert!) > > > * added Geert's tag > > > > > > arch/arm64/boot/dts/renesas/r8a77965.dtsi | 11 +++++++++++ > > > 1 file changed, 11 insertions(+) > > > > > > diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi > > > index 0cd44461a0bd..bf22efb82cbf 100644 > > > --- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi > > > +++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi > > > @@ -1507,6 +1507,17 @@ > > > status = "disabled"; > > > }; > > > > > > + sata: sata@ee300000 { > > > + compatible = "renesas,sata-r8a77965", > > > + "renesas,rcar-gen3-sata"; > > > + reg = <0 0xee300000 0 0x200000>; > > > + interrupts = <GIC_SPI 105 IRQ_TYPE_LEVEL_HIGH>; > > > + clocks = <&cpg CPG_MOD 815>; > > > + power-domains = <&sysc R8A77965_PD_ALWAYS_ON>; > > > + resets = <&cpg 815>; > > > + status = "disabled"; > > > > Is it intentional that there is no iommus property here? > > Because none of the other device nodes have iommus properties yet? That is my unstated assumption too. I've gone ahead and applied this patch for v4.20.