pwm: renesas-tpu: Improve precision of period and duty_cycle calculation
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Wed, 20 Apr 2022 12:12:40 +0000 (14:12 +0200)
committerThierry Reding <thierry.reding@gmail.com>
Fri, 20 May 2022 14:18:40 +0000 (16:18 +0200)
commit615f4e84461b71e5fed01d9f6d9d98ef3dd1d452
treed66b85a1e12e2f88f2e26106c1a21b742c98d799
parent3c173376efc461dc670b02ba2846c2a533491104
pwm: renesas-tpu: Improve precision of period and duty_cycle calculation

Dividing by the result of a division looses precision. Consider for example
clk_rate = 33000000 and period_ns = 500001. Then

clk_rate / (NSEC_PER_SEC / period_ns)

has the exact value 16500.033, but in C this evaluates to 16508. It gets
worse for even bigger values of period_ns, so with period_ns = 500000001,
the exact result is 16500000.033 while in C we get 33000000.

For that reason use

clk_rate * period_ns / NSEC_PER_SEC

instead which doesn't suffer from this problem. To ensure this doesn't
overflow add a safeguard check for clk_rate.

Note that duty > period can never happen, so the respective check can be
dropped.

Incidentally this fixes a division by zero if period_ns > NSEC_PER_SEC.
Another side effect is that values bigger than INT_MAX for period and
duty_cyle are not wrongly discarded any more.

Fixes: 99b82abb0a35 ("pwm: Add Renesas TPU PWM driver")
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
drivers/pwm/pwm-renesas-tpu.c