[PATCH 0/4] pwm: Ensure .polarity is set in .get_state()

Uwe Kleine-König posted 4 patches 2 years, 6 months ago
drivers/pwm/pwm-cros-ec.c | 1 +
drivers/pwm/pwm-hibvt.c   | 1 +
drivers/pwm/pwm-iqs620a.c | 1 +
drivers/pwm/pwm-sprd.c    | 1 +
4 files changed, 4 insertions(+)
[PATCH 0/4] pwm: Ensure .polarity is set in .get_state()
Posted by Uwe Kleine-König 2 years, 6 months ago
Hello,

in the context of the discussion with Munehisa Kamata about the meson driver
not setting .polarity in .get_state() I checked the other drivers for the same
issue. The identified drivers are fixed here. For three of them
zero-initializing *state would be good enough, still I think setting the
polarity explicitly is a good idea. For the hibvt driver (that supports
both polarities) the change is a real improvement.

Note I didn't touch the meson driver here that requires some non-trivial
updates that I don't want to tackle without hardware to test.

Best regards
Uwe

Uwe Kleine-König (4):
  pwm: hibvt: Explicitly set .polarity in .get_state()
  pwm: cros-ec: Explicitly set .polarity in .get_state()
  pwm: iqs620a: Explicitly set .polarity in .get_state()
  pwm: sprd: Explicitly set .polarity in .get_state()

 drivers/pwm/pwm-cros-ec.c | 1 +
 drivers/pwm/pwm-hibvt.c   | 1 +
 drivers/pwm/pwm-iqs620a.c | 1 +
 drivers/pwm/pwm-sprd.c    | 1 +
 4 files changed, 4 insertions(+)


base-commit: 7a77daf8223e772a225d6aa6202a5b1ae2392caf
-- 
2.39.1