Hi Magnus,

Thanks for your patch!

On Sun, Oct 21, 2018 at 7:56 PM Magnus Damm <[email protected]> wrote:
> From: Magnus Damm <[email protected]>
>
> For testing purpose enable IPMMU for Ethernet-AVB on R-Car M3-N/V3H/E3.
>
> Not for upstream merge.
>
> Not-Yet-Signed-off-by: Magnus Damm <[email protected]>
> ---
>
>  Applies on top of renesas-devel-20181019-v4.19-rc8
>
>  drivers/iommu/ipmmu-vmsa.c |    4 ++++
>  1 file changed, 4 insertions(+)
>
> --- 0001/drivers/iommu/ipmmu-vmsa.c
> +++ work/drivers/iommu/ipmmu-vmsa.c     2018-10-22 02:46:30.139880557 +0900
> @@ -756,6 +756,10 @@ static int ipmmu_init_platform_device(st
>
>  static bool ipmmu_slave_whitelist(struct device *dev)
>  {
> +       /* R-Car M3-N/V3H/E3 Ethernet-AVB */
> +       if (!strcmp(dev_name(dev), "e6800000.ethernet"))
> +               return true;

I'm afraid the whitelisting doesn't work that way: with the above check, it will
be enabled on all R-Car Gen3 SoCs.

> +
>         /* By default, do not allow use of IPMMU */
>         return false;
>  }

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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