On Fri, Apr 25, 2014 at 9:05 AM, Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: >> + scif0: serial@ffe40000 { >> + compatible = "renesas,scif", "renesas,scif-r8a7779"; >> + reg = <0xffe40000 265>; >> + interrupt-parent = <&gic>; >> + interrupts = <0 88 IRQ_TYPE_LEVEL_HIGH>; >> + clocks = <&cpg_clocks R8A7779_CLK_P>; >> + clock-names = "sci_ick"; > > Clock handling in the sh-sci driver should probably be improved. The driver > currently requires an "sci_ick" interface clock and supports an optional > "sci_fck" functional clock. In practice, as far as I can see, platforms that > provide both sci_ick and sci_fck set the two clocks to the same source. That's right. As a consequence, the clock's enable count is incremented 3 times: - once for fck, - once for ick, - once for generic Runtime PM using the "NULL" clock. 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 linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html