Hi Shimoda-san,
On Wed, Nov 28, 2018 at 7:10 AM Yoshihiro Shimoda
<yoshihiro.shimoda...@renesas.com> wrote:
> To avoid adding copy and pasted strcmp codes in the future,
> this patch adds an array "rcar_gen3_slave_whitelist" to check
> whether the device can work with the IPMMU or not.
>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda...@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be>
One small comment below.
> --- a/drivers/iommu/ipmmu-vmsa.c
> +++ b/drivers/iommu/ipmmu-vmsa.c
> @@ -773,8 +773,13 @@ static int ipmmu_init_platform_device(struct device *dev,
> { /* sentinel */ }
> };
>
> +static const char * const rcar_gen3_slave_whitelist[] = {
> +};
> +
> static bool ipmmu_slave_whitelist(struct device *dev)
> {
> + int i;
unsigned int
> +
> /*
> * For R-Car Gen3 use a white list to opt-in slave devices.
> * For Other SoCs, this returns true anyway.
> @@ -786,7 +791,13 @@ static bool ipmmu_slave_whitelist(struct device *dev)
> if (!soc_device_match(soc_rcar_gen3_whitelist))
> return false;
>
> - /* By default, do not allow use of IPMMU */
> + /* Check whether this slave device can work with the IPMMU */
> + for (i = 0; i < ARRAY_SIZE(rcar_gen3_slave_whitelist); i++) {
> + if (!strcmp(dev_name(dev), rcar_gen3_slave_whitelist[i]))
> + return true;
> + }
> +
> + /* Otherwise, do not allow use of IPMMU */
> return false;
> }
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
_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu
