On Wed, Nov 1, 2017 at 11:35 AM, Simon Horman <horms+renesas@xxxxxxxxxxxx> wrote: > Add r8a77970 IPMMU nodes and keep all disabled by default. > > Based on work for the r8a7796 by Magnus Damm > > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> With the compatible value for ipmmu_ds1 fixed: Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> My comments for r8a7795.dtsi about IPMMU-* comments and power-domains properties are applicable here, too. > --- a/arch/arm64/boot/dts/renesas/r8a77970.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a77970.dtsi [...] > + ipmmu_ds1: mmu@e7740000 { > + compatible = "renesas,ipmmu-r8a7796"; "renesas,ipmmu-r8a77970" > + reg = <0 0xe7740000 0 0x1000>; /* IPMMU-DS1 */ > + renesas,ipmmu-main = <&ipmmu_mm 1>; > + #iommu-cells = <1>; > + status = "disabled"; > + }; 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