All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vladimir Zapolskiy <vladimir_zapolskiy@mentor.com>
To: Thierry Reding <thierry.reding@gmail.com>
Cc: "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	devicetree@vger.kernel.org, linux-pwm@vger.kernel.org,
	"Lothar Waßmann" <LW@KARO-electronics.de>,
	"Sascha Hauer" <s.hauer@pengutronix.de>,
	"Heiko Stuebner" <heiko@sntech.de>,
	"Tony Prisk" <linux@prisktech.co.nz>,
	linux-arm-kernel@lists.infradead.org,
	linux-rockchip@lists.infradead.org,
	"Shawn Guo" <shawn.guo@linaro.org>
Subject: Re: [PATCH v3 0/5] pwm: support backward compatibility of DTB extending PWM args
Date: Wed, 21 Jan 2015 15:16:35 +0200	[thread overview]
Message-ID: <54BFA6B3.3070702@mentor.com> (raw)
In-Reply-To: <547C801F.2010307@mentor.com>

Hello Thierry,

On 01.12.2014 16:50, Vladimir Zapolskiy wrote:
> Hello Thierry,
> 
> On 18.11.2014 21:37, Vladimir Zapolskiy wrote:
>> 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

WARNING: multiple messages have this Message-ID (diff)
From: vladimir_zapolskiy@mentor.com (Vladimir Zapolskiy)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/5] pwm: support backward compatibility of DTB extending PWM args
Date: Wed, 21 Jan 2015 15:16:35 +0200	[thread overview]
Message-ID: <54BFA6B3.3070702@mentor.com> (raw)
In-Reply-To: <547C801F.2010307@mentor.com>

Hello Thierry,

On 01.12.2014 16:50, Vladimir Zapolskiy wrote:
> Hello Thierry,
> 
> On 18.11.2014 21:37, Vladimir Zapolskiy wrote:
>> 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

  reply	other threads:[~2015-01-21 13:16 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-18 19:37 [PATCH v3 0/5] pwm: support backward compatibility of DTB extending PWM args Vladimir Zapolskiy
2014-11-18 19:37 ` Vladimir Zapolskiy
2014-11-18 19:38 ` [PATCH 1/5] pwm: process pwm polarity argument in of_pwm_simple_xlate() Vladimir Zapolskiy
2014-11-18 19:38   ` [PATCH 2/5] pwm: push up dynamic printk level on errors in of_pwm_get() Vladimir Zapolskiy
2014-11-18 19:38   ` [PATCH 3/5] pwm: set the actual number of pwms arguments defined in board dts Vladimir Zapolskiy
2014-11-18 19:38   ` [PATCH 4/5] pwm: do not set of_pwm_xlate_with_flags() as a callback Vladimir Zapolskiy
2014-11-18 19:38     ` Vladimir Zapolskiy
2014-11-18 19:38   ` [PATCH 5/5] pwm: remove of_pwm_xlate_with_flags() function Vladimir Zapolskiy
2014-12-01 14:50 ` [PATCH v3 0/5] pwm: support backward compatibility of DTB extending PWM args Vladimir Zapolskiy
2014-12-01 14:50   ` Vladimir Zapolskiy
2015-01-21 13:16   ` Vladimir Zapolskiy [this message]
2015-01-21 13:16     ` Vladimir Zapolskiy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54BFA6B3.3070702@mentor.com \
    --to=vladimir_zapolskiy@mentor.com \
    --cc=LW@KARO-electronics.de \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=heiko@sntech.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-pwm@vger.kernel.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=linux@prisktech.co.nz \
    --cc=s.hauer@pengutronix.de \
    --cc=shawn.guo@linaro.org \
    --cc=thierry.reding@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.