| Message ID | 1555580267-29299-1-git-send-email-fabrice.gasnier@st.com |
|---|---|
| Headers | show |
| Series | Add PM support to STM32 LP Timer drivers | expand |
On 4/18/19 11:37 AM, Fabrice Gasnier wrote: > This patch series adds power management support for STM32 LP Timer: > - PWM driver > - Document the pinctrl states for sleep mode > > It also adds device link between the PWM consumer and the PWM provider. > This allows proper sequencing for suspend/resume (e.g. user will likely > do a pwm_disable() before the PWM provider suspend executes), see [1]. > > [1] https://lkml.org/lkml/2019/2/5/770 > Hi Thierry, Please let me know if you have some more comments on this series. It's been under review since quite some time now. Thanks in advance, Best Regards, Fabrice > --- > resend v5: > - update collected acks > > Changes in v5: > - improve a warning message, fix a style issue. > > Changes in v4: > - improve error handling when adding the PWM consumer device link. > > Changes in v3: > - Move the device_link_add() call to of_pwm_get() as discussed with Uwe. > > Changes in v2: > - Don't disable PWM channel in PWM provider: rather refuse to suspend > and report an error as suggested by Uwe and Thierry. > - Add patch 3/3 to propose device link addition. > - No updates for STM32 LP Timer IIO driver. Patches can be send separately. > > Fabrice Gasnier (3): > dt-bindings: pwm-stm32-lp: document pinctrl sleep state > pwm: stm32-lp: Add power management support > pwm: core: add consumer device link > > .../devicetree/bindings/pwm/pwm-stm32-lp.txt | 9 ++-- > drivers/pwm/core.c | 50 ++++++++++++++++++++-- > drivers/pwm/pwm-stm32-lp.c | 25 +++++++++++ > include/linux/pwm.h | 6 ++- > 4 files changed, 82 insertions(+), 8 deletions(-) >
Le ven. 10 mai 2019 à 09:51, Fabrice Gasnier <fabrice.gasnier@st.com> a écrit : > > On 4/18/19 11:37 AM, Fabrice Gasnier wrote: > > This patch series adds power management support for STM32 LP Timer: > > - PWM driver > > - Document the pinctrl states for sleep mode > > > > It also adds device link between the PWM consumer and the PWM provider. > > This allows proper sequencing for suspend/resume (e.g. user will likely > > do a pwm_disable() before the PWM provider suspend executes), see [1]. > > > > [1] https://lkml.org/lkml/2019/2/5/770 > > > > Hi Thierry, > > Please let me know if you have some more comments on this series. It's > been under review since quite some time now. > Hi Thierry, Does something is blocking on this series ? How can we progress on it ? Regards, Benjamin > Thanks in advance, > Best Regards, > Fabrice > > > --- > > resend v5: > > - update collected acks > > > > Changes in v5: > > - improve a warning message, fix a style issue. > > > > Changes in v4: > > - improve error handling when adding the PWM consumer device link. > > > > Changes in v3: > > - Move the device_link_add() call to of_pwm_get() as discussed with Uwe. > > > > Changes in v2: > > - Don't disable PWM channel in PWM provider: rather refuse to suspend > > and report an error as suggested by Uwe and Thierry. > > - Add patch 3/3 to propose device link addition. > > - No updates for STM32 LP Timer IIO driver. Patches can be send separately. > > > > Fabrice Gasnier (3): > > dt-bindings: pwm-stm32-lp: document pinctrl sleep state > > pwm: stm32-lp: Add power management support > > pwm: core: add consumer device link > > > > .../devicetree/bindings/pwm/pwm-stm32-lp.txt | 9 ++-- > > drivers/pwm/core.c | 50 ++++++++++++++++++++-- > > drivers/pwm/pwm-stm32-lp.c | 25 +++++++++++ > > include/linux/pwm.h | 6 ++- > > 4 files changed, 82 insertions(+), 8 deletions(-) > > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel