On Thu, Nov 26, 2015 at 11:59 PM, Wolfram Sang wrote:
> --- a/drivers/i2c/busses/i2c-rcar.c
> +++ b/drivers/i2c/busses/i2c-rcar.c
> @@ -211,12 +214,12 @@ static int rcar_i2c_clock_calculate(struct
> rcar_i2c_priv *priv, struct i2c_timin
> * it is impossible to calculate large scale
> * number on u32. separate it
> *
> -* F[(ticf + tr + intd) * ick]
> -* = F[(35 + 200 + 50)ns * ick]
> -* = F[285 * ick / 10]
> -* = F[(ick / 100) * 285 / 1000]
> +* F[(ticf + tr + intd) * ick] with sum = (ticf + tr + intd)
> +* = F[sum * ick / 10]
> +* = F[(ick / 100) * sum / 1000]
> */
> - round = (ick + 50) / 100 * 285;
> + sum = t->scl_fall_ns + t->scl_rise_ns + t->scl_int_delay_ns;
> + round = (ick + 50) / 100 * sum;
If you're sure it cannot overflow, doing the "* sum" before the division
improves accuracy.
> round = (round + 500) / 1000;
You can use DIV_ROUND_CLOSEST everywhere, e.g.
round = DIV_ROUND_CLOSEST(round, 1000)
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org
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-i2c" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
