Hi Mike, Stephen, As the .round_rate() callback returns a long clock rate, it cannot return clock rates that do not fit in signed long, but do fit in unsigned long. The newer .determine_rate() callback does not suffer from this limitation. In addition, .determine_rate() provides the ability to specify a rate range. Hence this patch series switches the Z (CPU) and SD clocks in the R-Car Gen2 and Gen3 clock drivers from the .round_rate() to the .determine_rate() callback. Note that the "div6" clock driver hasn't been converted yet, so div6 clocks still use .round_rate(). This has been tested on R-Car M2-W and R-Car M3-N, and should have no behavioral impact. To be queued in clk-renesas-for-v5.3, if approved. Thanks for your comments! Geert Uytterhoeven (5): clk: renesas: rcar-gen2-legacy: Switch Z clock to .determine_rate() clk: renesas: rcar-gen2: Switch Z clock to .determine_rate() clk: renesas: rcar-gen3: Switch Z clocks to .determine_rate() clk: renesas: rcar-gen3: Avoid double table iteration in SD .set_rate() clk: renesas: rcar-gen3: Switch SD clocks to .determine_rate() drivers/clk/renesas/clk-rcar-gen2.c | 23 ++++++----- drivers/clk/renesas/rcar-gen2-cpg.c | 23 ++++++----- drivers/clk/renesas/rcar-gen3-cpg.c | 64 ++++++++++++++++------------- 3 files changed, 61 insertions(+), 49 deletions(-) -- 2.17.1 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