Re: [PATCH v3 0/5] pwm: support backward compatibility of DTB extending PWM args

Hello Thierry,

On 01.12.2014 16:50, Vladimir Zapolskiy wrote:

Hello Thierry,

On 18.11.2014 21:37, Vladimir Zapolskiy wrote:

quoted

Please review this alternative in-house version of supporting backward
compatibility of DTB files (new kernel/PWM driver and old DTS), overcoming
a problem that pwms arguments of existing PWM drivers are to be extended
with polarity field.

Unfortunately I've firstly created the change and only after that
found that Lothar Waßmann is working on his version, so I won't insist
on formal preference, but still I like to share my variant since it
technically different from Lothar's one.

Changes from v2 to v3:
- improved commit messages
- added two more patches removing of_pwm_xlate_with_flags()

Changes from v1 to v2:
- corrected the proper kernel log level, thanks to Lothar Waßmann.

Vladimir Zapolskiy (5):
  pwm: process pwm polarity argument in of_pwm_simple_xlate()
  pwm: push up dynamic printk level on errors in of_pwm_get()
  pwm: set the actual number of pwms arguments defined in board dts
  pwm: do not set of_pwm_xlate_with_flags() as a callback
  pwm: remove of_pwm_xlate_with_flags() function

 drivers/pwm/core.c            |   42 ++++++++++++-----------------------------
 drivers/pwm/pwm-atmel-hlcdc.c |    2 --
 drivers/pwm/pwm-atmel-tcb.c   |    2 --
 drivers/pwm/pwm-atmel.c       |    6 ------
 drivers/pwm/pwm-bcm-kona.c    |    2 --
 drivers/pwm/pwm-fsl-ftm.c     |    2 --
 drivers/pwm/pwm-renesas-tpu.c |    2 --
 drivers/pwm/pwm-rockchip.c    |    5 -----
 drivers/pwm/pwm-samsung.c     |    3 ---
 drivers/pwm/pwm-tiecap.c      |    2 --
 drivers/pwm/pwm-tiehrpwm.c    |    2 --
 drivers/pwm/pwm-vt8500.c      |    2 --
 include/linux/pwm.h           |    3 ---
 13 files changed, 12 insertions(+), 63 deletions(-)

could you please review this PWM DTB backward compatibility change?

Thank you in advance.

the first three patches are unmodified since October 2014, but still not
reviewed...

Could you please review the changeset?

Thank you in advance.

--
With best wishes,
Vladimir