Hi Simon,

On Thu, Aug 2, 2018 at 1:13 PM Simon Horman <ho...@verge.net.au> wrote:
> On Mon, Jul 30, 2018 at 09:33:38AM +0200, Wolfram Sang wrote:
> > From: Takeshi Kihara <takeshi.kihara...@renesas.com>
> > This patch adds SATA controller node for the R8A77965 SoC.
> >
> > Signed-off-by: Takeshi Kihara <takeshi.kihara...@renesas.com>
> > [wsa: rebased to upstream base]
> > Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be>
> > Signed-off-by: Wolfram Sang <wsa+rene...@sang-engineering.com>
> > ---
> >
> > 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?

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Reply via email to