Re: [PATCH v5 4/7] pwm: ntxec: Add driver for PWM function in Netronix EC

2020-12-01 Thread Jonathan Neuschäfer
On Tue, Dec 01, 2020 at 08:20:26AM +0100, Uwe Kleine-König wrote: > Hello Jonathan, > > very nice driver, just a few minor comments below. > > On Tue, Dec 01, 2020 at 02:15:10AM +0100, Jonathan Neuschäfer wrote: > > +static struct ntxec_pwm *pwmchip_to_priv(struct pwm_chip *chip) > > a function

Re: [PATCH v5 4/7] pwm: ntxec: Add driver for PWM function in Netronix EC

2020-11-30 Thread Uwe Kleine-König
Hello Jonathan, very nice driver, just a few minor comments below. On Tue, Dec 01, 2020 at 02:15:10AM +0100, Jonathan Neuschäfer wrote: > +static struct ntxec_pwm *pwmchip_to_priv(struct pwm_chip *chip) a function prefix would be great here, I'd pick ntxec_pwm_from_chip as name. > +{ > + re

[PATCH v5 4/7] pwm: ntxec: Add driver for PWM function in Netronix EC

2020-11-30 Thread Jonathan Neuschäfer
The Netronix EC provides a PWM output which is used for the backlight on some ebook readers. This patches adds a driver for the PWM output. The .get_state callback is not implemented, because the PWM state can't be read back from the hardware. Signed-off-by: Jonathan Neuschäfer --- v5: - Avoid