linux-kernel - Re: [RESEND PATCH v2 0/8] Add support for PWM input capture on STM32

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20180329085837.GA16952@ulmo>
Date:   Thu, 29 Mar 2018 10:58:37 +0200
From:   Thierry Reding <thierry.reding@...il.com>
To:     Lee Jones <lee.jones@...aro.org>
Cc:     Fabrice Gasnier <fabrice.gasnier@...com>, alexandre.torgue@...com,
        benjamin.gaignard@...aro.org, robh+dt@...nel.org,
        mark.rutland@....com, linux@...linux.org.uk,
        mcoquelin.stm32@...il.com, benjamin.gaignard@...com,
        devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org, linux-pwm@...r.kernel.org
Subject: Re: [RESEND PATCH v2 0/8] Add support for PWM input capture on STM32

On Wed, Mar 28, 2018 at 11:03:09AM +0100, Lee Jones wrote:
> On Wed, 28 Mar 2018, Thierry Reding wrote:
> 
> > On Wed, Feb 14, 2018 at 11:04:31AM +0100, Fabrice Gasnier wrote:
> > > This series adds support for capture to stm32-pwm driver.
> > > Capture is based on DMAs.
> > > - First two patches are precursor patches
> > > - Subsequent two patches add support for requesting DMAs to MFD core
> > > - Next three patches add support for capture to stm32-pwm driver
> > > - This has been tested on stm32429i-eval board.
> > > 
> > > ---
> > > Resend v2:
> > > - Add collected Acks
> > > 
> > > Changes in v2:
> > > - Abstract DMA handling from child driver: move it to MFD core
> > > - Rework pwm capture routines to adopt this change
> > > - Comment on optional dma support, beautify DMAs probe
> > > 
> > > Fabrice Gasnier (8):
> > >   pwm: stm32: fix, remove unused struct device
> > >   pwm: stm32: protect common prescaler for all channels
> > >   dt-bindings: mfd: stm32-timers: add support for dmas
> > >   mfd: stm32-timers: add support for dmas
> > >   pwm: stm32: add capture support
> > >   pwm: stm32: improve capture by tuning counter prescaler
> > >   pwm: stm32: use input prescaler to improve period capture
> > >   ARM: dts: stm32: Enable pwm3 input capture on stm32f429i-eval
> > > 
> > >  .../devicetree/bindings/mfd/stm32-timers.txt       |  20 ++
> > >  arch/arm/boot/dts/stm32429i-eval.dts               |   3 +
> > >  drivers/mfd/stm32-timers.c                         | 215 +++++++++++++++-
> > >  drivers/pwm/pwm-stm32.c                            | 276 ++++++++++++++++++++-
> > >  include/linux/mfd/stm32-timers.h                   |  39 +++
> > >  5 files changed, 547 insertions(+), 6 deletions(-)
> > 
> > I can't apply patches 1-2 and 5-6 since they depend on patches 3-4 for
> > which I'd need an Acked-by: from Lee if I'm to pick them up. Same goes
> > for patch 7.
> > 
> > By the looks of it there are minor conflicts with the MFD tree, but no
> > major ones. Perhaps it'd be better for Lee to pick up 3-4 for v4.17-rc1
> > and ack patch 7, then I can take the rest after v4.17-rc1?
> 
> Sounds like a faff.
> 
> Why don't I just pick them all up (except the ARM patch)?

Fine with me. For all the PWM patches in this series:

Acked-by: Thierry Reding <thierry.reding@...il.com>

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)