On Sun, Apr 8, 2018 at 6:40 PM, Niklas Söderlund <niklas.soderlund@xxxxxxxxxxxx> wrote: > On 2018-04-06 22:14:37 +0300, Laurent Pinchart wrote: >> --- a/arch/arm/boot/dts/r8a7791-koelsch.dts >> +++ b/arch/arm/boot/dts/r8a7791-koelsch.dts >> @@ -470,8 +470,7 @@ >> >> clocks = <&cpg CPG_MOD 724>, <&cpg CPG_MOD 723>, <&cpg CPG_MOD 726>, >> <&x13_clk>, <&x2_clk>; > > I think you forgot to delete <&cpg CPG_MOD 726> from the clocks, with > that fixed: > > Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> > >> - clock-names = "du.0", "du.1", "lvds.0", >> - "dclkin.0", "dclkin.1"; >> + clock-names = "du.0", "du.1", "dclkin.0", "dclkin.1"; Obviously we need a check in dtc for matching the lengths of the clocks and clock-names (etc.) properties. 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