Re: [PATCH v4 10/15] arm64: dts: renesas: r8a7795: Point VSPI via FCPVI to IPMMU-VP0/1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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+renesas@xxxxxxxxxxxx> wrote:
> > From: Magnus Damm <damm+renesas@xxxxxxxxxxxxx>
> >
> > Hook up the FCPVI devices to allow use of VSPI with
> > IPMMU-VP0 and IPMMU-VP1.
> >
> > Signed-off-by: Magnus Damm <damm+renesas@xxxxxxxxxxxxx>
> > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> 
> > --- 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 -- geert@xxxxxxxxxxxxxx
> 
> 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
> 



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux