Hi Laurent, On Fri, Oct 23, 2015 at 1:21 PM, Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: >> +#define R8A7795_CLK_R 45 >> +#define R8A7795_CLK_OSC 46 > > Those two clocks are called RCLK and OSCCLK in the datasheet, shouldn't we use > those names ? ..._CLK_RCLK and ..._CLK_OSCCLK sound a bit heavy to me. Besides, all other clocks in the datasheet have a "φ" suffix, which also means "CLK", and which we don't retain. 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html