Hi Laurent, Thanks for your patch! On Fri, Jun 8, 2018 at 1:18 PM Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> wrote: > The R8A77995 VSP and FCP nodes have overlapping register ranges, as the > SoC integrates the FCP devices in the memory range usually used by the > VSP LUT and CLUT, which are not present. Fix this by shortening the VSP > registers range to 0x5000. According to datasheet rev. 1.00, the CLUT is present on R-Car D3, the LUT is not. > Fixes: 295952a183d3 ("arm64: dts: renesas: r8a77995: add VSP instances") > Reported-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> > Reported-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> Apart from my comment above: Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> 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