On Tue, Nov 07, 2017 at 10:35:23AM +0100, Geert Uytterhoeven wrote:
> On Wed, Nov 1, 2017 at 11:34 AM, Simon Horman
> <horms+rene...@verge.net.au> wrote:
> > From: Magnus Damm <damm+rene...@opensource.se>
> >
> > Hook up the FCPVI devices to allow use of VSPI with
> > IPMMU-VP0 and IPMMU-VP1.
> >
> > Signed-off-by: Magnus Damm <damm+rene...@opensource.se>
> > Signed-off-by: Simon Horman <horms+rene...@verge.net.au>
> 
> Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be>
> 
> > --- a/arch/arm64/boot/dts/renesas/r8a7795-es1.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r8a7795-es1.dtsi
> > @@ -155,6 +155,10 @@
> >         iommus = <&ipmmu_vp0 1>;
> >  };
> >
> > +&fcpvi1 {
> > +       iommus = <&ipmmu_vp0 9>;
> > +};
> 
> Do we sort labels alphabetically?
> Or do we follow the order of the original targets?
> 
> (Random comment that may apply elsewhere, too)

In these IPMMU patches I tried to sort them in the same order
as they appear in r8a7795.dtsi. I'm happy to change that if
we decided on a scheme to adopt.

> 
> > +
> >  &fcpvd2 {
> >         iommus = <&ipmmu_vi0 10>;
> >  };
> 
> 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