mbox series

[v2,0/3] pwm: stm32: Minor cleanups

Message ID 20191016110601.1765415-1-thierry.reding@gmail.com
Headers show
Series pwm: stm32: Minor cleanups | expand

Message

Thierry Reding Oct. 16, 2019, 11:05 a.m. UTC 
Hi,

Looking at Fabrice's STM32 patches I noticed that we're now passing the
breakinput values (u32) into a function via int parameters. The easiest
way to fix this inconsistency is by just passing a pointer to the break
input structure. There's some preparatory work here that makes the code
slightly more readable, in my opinion. I ended up squashing two patches
into one in v2 because the second patch from v1 is already addressed in
patch 1 of v2. I've added a patch in v2 that validates device tree data
for breakinput as suggested by Uwe.

Lee, patch 1 of this small series touches the MFD header for the STM32
timers, but there's no good way to separate the patches, so if you could
provide an Acked-by on that patch so that I can take it through the PWM
tree along with the rest, that'd be great.

Thierry

Thierry Reding (3):
  pwm: stm32: Remove clutter from ternary operator
  pwm: stm32: Pass breakinput instead of its values
  pwm: stm32: Validate breakinput data from DT

 drivers/pwm/pwm-stm32.c          | 36 +++++++++++++++++---------------
 include/linux/mfd/stm32-timers.h | 12 ++++-------
 2 files changed, 23 insertions(+), 25 deletions(-)