On Wed, Apr 20, 2022 at 02:12:34PM +0200, Uwe Kleine-König wrote: > Hello, > > I reworked the series to improve the renesas-tpu PWM driver after > feedback I got from Geert. The changes are: > > - Add Geert's tags for the first two patches > (I didn't add them for the other patches as I changed these enough to > not carry the tags forward.) > - Use tpu_pwm_disable instead chip->ops->disable (which is NULL now) > fixing a crash in v1. > - The calculation now uses a switch instead of ilog2. > Looking at the compiled code for ARM this is more effective and I > assume the same holds true for other platforms. > > Thanks again to Geert for his valuable feedback and testing. > > Best regards > Uwe > > Uwe Kleine-König (6): > pwm: renesas-tpu: Make use of dev_err_probe() > pwm: renesas-tpu: Make use of devm functions > pwm: renesas-tpu: Implement .apply() callback > pwm: renesas-tpu: Rename variables to match the usual naming > pwm: renesas-tpu: Improve maths to compute register settings > pwm: renesas-tpu: Improve precision of period and duty_cycle > calculation > > drivers/pwm/pwm-renesas-tpu.c | 317 +++++++++++++++++++--------------- > 1 file changed, 180 insertions(+), 137 deletions(-) Applied, with the checkpatch warning in patch 5/6 addressed (I opted to replace the tabs by spaces). Thanks, Thierry
Attachment:
signature.asc
Description: PGP signature
