Hi Ramesh, On Mon, Feb 29, 2016 at 10:21 AM, Ramesh Shanmugasundaram <ramesh.shanmugasundaram@xxxxxxxxxxxxxx> wrote: > --- a/drivers/net/can/rcar_can.c > +++ b/drivers/net/can/rcar_can.c > @@ -808,6 +808,10 @@ static int rcar_can_probe(struct platform_device *pdev) > priv->regs = addr; > priv->clock_select = clock_select; > priv->can.clock.freq = clk_get_rate(priv->can_clk); > + > + /* Use the rounded clock value (nearest kHz) */ > + priv->can.clock.freq = 1000 * DIV_ROUND_UP(priv->can.clock.freq, 1000); The comment and the implementation don't match. To get the nearest kHz, you have to useDIV_ROUND_CLOSEST(). 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