mbox series

[v2,0/5] gpio: mvebu: pwm fixes and improvements

Message ID cover.1610529002.git.baruch@tkos.co.il
Headers show
Series gpio: mvebu: pwm fixes and improvements | expand

Message

Baruch Siach Jan. 13, 2021, 9:22 a.m. UTC
This series adds a few related fixes to the pwm .apply and .get_state 
callbacks.

The first patch was originally part of the series adding Armada 8K/7K pwm 
support. I split it out to a separate series following review comments from 
Uwe Kleine-König who spotted a few more issues. There is no dependency between 
this and the Armada 8K/7K series.

v2:

Address Uwe Kleine-König comments.

  * Improve patch 1/5 summary line

  * Add more information to patch 1/5 description

  * Add more information to patch 2/5 description

  * Don't round period/duty_cycle up in .apply (patch 3/5)

  * Expand the comment in path 5/5 based on RMK's analysis of hardware 
    behaviour

  * Add Uwe's Reviewed-by tags

Baruch Siach (5):
  gpio: mvebu: fix pwm .get_state period calculation
  gpio: mvebu: improve pwm period calculation accuracy
  gpio: mvebu: make pwm .get_state closer to idempotent
  gpio: mvebu: don't limit pwm period/duty_cycle to UINT_MAX
  gpio: mvebu: document zero pwm duty cycle limitation

 drivers/gpio/gpio-mvebu.c | 33 +++++++++++++++------------------
 1 file changed, 15 insertions(+), 18 deletions(-)

Comments

Linus Walleij Jan. 18, 2021, 1:45 p.m. UTC | #1
On Wed, Jan 13, 2021 at 10:23 AM Baruch Siach <baruch@tkos.co.il> wrote:

> This series adds a few related fixes to the pwm .apply and .get_state

> callbacks.

>

> The first patch was originally part of the series adding Armada 8K/7K pwm

> support. I split it out to a separate series following review comments from

> Uwe Kleine-König who spotted a few more issues. There is no dependency between

> this and the Armada 8K/7K series.


This version looks really good and +/- minor tweaks as indicated by
Uwe:
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>


Yours,
Linus Walleij
Uwe Kleine-König Jan. 18, 2021, 2:59 p.m. UTC | #2
Hello Linus,

On Mon, Jan 18, 2021 at 02:45:14PM +0100, Linus Walleij wrote:
> On Wed, Jan 13, 2021 at 10:23 AM Baruch Siach <baruch@tkos.co.il> wrote:

> 

> > This series adds a few related fixes to the pwm .apply and .get_state

> > callbacks.

> >

> > The first patch was originally part of the series adding Armada 8K/7K pwm

> > support. I split it out to a separate series following review comments from

> > Uwe Kleine-König who spotted a few more issues. There is no dependency between

> > this and the Armada 8K/7K series.

> 

> This version looks really good and +/- minor tweaks as indicated by

> Uwe:

> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>


The most recent version of this series is v4.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |