Skip to content

pwm: tegra: Cleanups and .get_state()

Uwe Kleine-König linux-pwm linux-tegra 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest6 patches 9 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6pwm: tegra: Check for match_data being NULLdrivers/pwm/pwm-tegra.c+7 -02 replies
2/6pwm: tegra: Make use of dev_err_probe()drivers/pwm/pwm-tegra.c+26 -181 reply
3/6pwm: tegra: Use devm function for pm_runtime_enable()drivers/pwm/pwm-tegra.c+4 -41 reply
4/6pwm: tegra: Simplify using devm_reset_control_get_exclusive_deasserted()drivers/pwm/pwm-tegra.c+4 -103 replies
5/6pwm: tegra: Simplify using devm_pwmchip_add()drivers/pwm/pwm-tegra.c+1 -90 replies
6/6pwm: tegra: Implement .get_state()drivers/pwm/pwm-tegra.c+48 -02 replies

Reviews trailers per patch

2 of 6 patches have a Reviewed-by or Acked-by.

PatchMikko PerttunenOla Chr. Vaage
1/6
2/6R-b
3/6
4/6
5/6
6/6R-b T-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH v1 0/6] pwm: tegra: Cleanups and .get_state()

Hello,

the tegra PWM driver annoyed me for some time. This patch set addresses some of the concerns. The implementation of .get_state() is the check if I understood the hardware correctly. If so, the next step is to fix the rounding in .apply() and/or convert to the waveform callbacks.

The first 5 patches should be fine (unless my understanding of pm_runtime is wrong, which is quite possible), for the 6th patch I'd like someone to test it before it gets applied.

Best regards
Uwe

Uwe Kleine-König (6):
pwm: tegra: Check for match_data being NULL
pwm: tegra: Make use of dev_err_probe()
pwm: tegra: Use devm function for pm_runtime_enable()
pwm: tegra: Simplify using
devm_reset_control_get_exclusive_deasserted()
pwm: tegra: Simplify using devm_pwmchip_add()
pwm: tegra: Implement .get_state()

 drivers/pwm/pwm-tegra.c | 123 ++++++++++++++++++++++++++++------------
 1 file changed, 86 insertions(+), 37 deletions(-)

base-commit: 82dcd68f7246eeacbc2ef614b82e284a50b2b8bb

Signature · 1 line
2.55.0.11.g153666a7d9bb