linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: lee.jones@linaro.org (Lee Jones)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] Immutable branch between MFD, ARM, IIO and PWM due for the v4.10 merge window
Date: Wed, 25 Jan 2017 15:33:52 +0000	[thread overview]
Message-ID: <20170125153352.GA19242@dell> (raw)
In-Reply-To: <20170125105511.GA9344@ulmo.ba.sec>

On Wed, 25 Jan 2017, Thierry Reding wrote:

> On Wed, Jan 25, 2017 at 10:08:58AM +0000, Lee Jones wrote:
> > Jonathan, Thierry,
> > 
> > On Tue, 24 Jan 2017, Olof Johansson wrote:
> > > On Tue, Jan 24, 2017 at 12:12 AM, Lee Jones <lee.jones@linaro.org> wrote:
> > > > On Mon, 23 Jan 2017, Olof Johansson wrote:
> > > >> On Mon, Jan 23, 2017 at 4:12 AM, Lee Jones <lee.jones@linaro.org> wrote:
> > > >> > Arm, IIO and PWM Maintainers,
> > > >> >
> > > >> > Enjoy!
> > > >> >
> > > >> > The following changes since commit 7ce7d89f48834cefece7804d38fc5d85382edf77:
> > > >> >
> > > >> >   Linux 4.10-rc1 (2016-12-25 16:13:08 -0800)
> > > >> >
> > > >> > are available in the git repository at:
> > > >> >
> > > >> >   git://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git ib-mfd-arm-iio-pwm-v4.10
> > > >> >
> > > >> > for you to fetch changes up to 94df449273475980a39f1bf47f1e3d2b07ce6577:
> > > >> >
> > > >> >   ARM: dts: stm32: Enable pwm1 and pwm3 for stm32f469-disco (2017-01-23 12:06:28 +0000)
> > > >> >
> > > >> > ----------------------------------------------------------------
> > > >> > Immutable branch between MFD, ARM, IIO and PWM due for the v4.10 merge window
> > > >> >
> > > >> > ----------------------------------------------------------------
> > > >> > Benjamin Gaignard (8):
> > > >> >       dt-bindings: mfd: Add bindings for STM32 Timers driver
> > > >> >       mfd: Add STM32 Timers driver
> > > >> >       dt-bindings: pwm: Add STM32 bindings
> > > >> >       pwm: Add driver for STM32 plaftorm
> > > >> >       iio: Add bindings for STM32 timer trigger driver
> > > >> >       iio: Add STM32 timer trigger driver
> > > >> >       ARM: dts: stm32: add Timers driver for stm32f429 MCU
> > > >> >       ARM: dts: stm32: Enable pwm1 and pwm3 for stm32f469-disco
> > > >> >
> > > >> >  .../ABI/testing/sysfs-bus-iio-timer-stm32          |  29 ++
> > > >> >  .../bindings/iio/timer/stm32-timer-trigger.txt     |  23 ++
> > > >> >  .../devicetree/bindings/mfd/stm32-timers.txt       |  46 +++
> > > >> >  .../devicetree/bindings/pwm/pwm-stm32.txt          |  35 ++
> > > >> >  arch/arm/boot/dts/stm32f429.dtsi                   | 275 ++++++++++++++
> > > >> >  arch/arm/boot/dts/stm32f469-disco.dts              |  28 ++
> > > >> >  drivers/iio/trigger/Kconfig                        |   9 +
> > > >> >  drivers/iio/trigger/Makefile                       |   1 +
> > > >> >  drivers/iio/trigger/stm32-timer-trigger.c          | 342 ++++++++++++++++++
> > > >> >  drivers/mfd/Kconfig                                |  11 +
> > > >> >  drivers/mfd/Makefile                               |   2 +
> > > >> >  drivers/mfd/stm32-timers.c                         |  80 +++++
> > > >> >  drivers/pwm/Kconfig                                |   9 +
> > > >> >  drivers/pwm/Makefile                               |   1 +
> > > >> >  drivers/pwm/pwm-stm32.c                            | 398 +++++++++++++++++++++
> > > >> >  include/linux/iio/timer/stm32-timer-trigger.h      |  62 ++++
> > > >> >  include/linux/mfd/stm32-timers.h                   |  71 ++++
> > > >> >  17 files changed, 1422 insertions(+)
> > > >> >  create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-timer-stm32
> > > >> >  create mode 100644 Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
> > > >> >  create mode 100644 Documentation/devicetree/bindings/mfd/stm32-timers.txt
> > > >> >  create mode 100644 Documentation/devicetree/bindings/pwm/pwm-stm32.txt
> > > >> >  create mode 100644 drivers/iio/trigger/stm32-timer-trigger.c
> > > >> >  create mode 100644 drivers/mfd/stm32-timers.c
> > > >> >  create mode 100644 drivers/pwm/pwm-stm32.c
> > > >> >  create mode 100644 include/linux/iio/timer/stm32-timer-trigger.h
> > > >> >  create mode 100644 include/linux/mfd/stm32-timers.h
> > > >>
> > > >> May I ask why you're creating an immutable branch of all this contents?
> > > >>
> > > >> We should get the dts/dtsi contents like usual from any maintainer;
> > > >> just the DT updates in a branch.
> > > >>
> > > >> Sometimes (but less and less due to us discouraging the use) we need
> > > >> an immutable branch that includes dt-include updates. But I don't see
> > > >> any of those in this contents.
> > > >>
> > > >> I.e. I guess I can see the need between mfd and downstream trees
> > > >> (iio/pwm), but not the DTS contents at this time. Or am I missing
> > > >> something here?
> > > >
> > > > I agree with you.  However, since one of the STM32 Maintainers Acked
> > > > the DTS patches, and the other one hasn't been heard from for months,
> > > > I took that to mean he wanted me to apply and take them through the
> > > > MFD tree.
> > > >
> > > > A misunderstanding or misaction perhaps, but this is the normal
> > > > assumption.
> > > 
> > > It seems to be a misunderstanding in this case, since there are other
> > > additions in the stm32 tree this release cycle so we should only pick
> > > up DTS changes through our tree.
> > > 
> > > By now, I think it should be pretty well known that we nearly never
> > > want DTS contents merged through any other tree. Ever.
> > > 
> > > So, please revoke your branch and take those out. We'll take them
> > > through the normal flow. Thanks!
> > 
> > That okay with you?
> 
> Do you still want to carry the PWM patches in your tree? Or should they
> all just be picked up into the respective subsystem trees, then?
> 
> Either way is fine with me.

I'm also happy either way.  For ease, I guess we'll keep things as
they are.  I'll just drop the Arm patches and re-send the
pull-request.  This also gives us the opportunity to roll in Ben's
recent fix.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

  reply	other threads:[~2017-01-25 15:33 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-20  9:15 [PATCH v9 0/8] Add PWM and IIO timer drivers for STM32 Benjamin Gaignard
2017-01-20  9:15 ` [PATCH v9 1/8] MFD: add bindings for STM32 Timers driver Benjamin Gaignard
2017-01-20  9:15 ` [PATCH v9 2/8] MFD: add " Benjamin Gaignard
2017-01-20  9:15 ` [PATCH v9 3/8] dt-bindings: pwm: Add STM32 bindings Benjamin Gaignard
2017-01-20  9:15 ` [PATCH v9 4/8] pwm: add driver for STM32 plaftorm Benjamin Gaignard
2017-01-20  9:15 ` [PATCH v9 5/8] IIO: add bindings for STM32 timer trigger driver Benjamin Gaignard
2017-01-20  9:15 ` [PATCH v9 6/8] IIO: add " Benjamin Gaignard
2017-01-20  9:15 ` [PATCH v9 7/8] ARM: dts: stm32: add Timers driver for stm32f429 MCU Benjamin Gaignard
2017-01-20 10:04   ` Alexandre Torgue
2017-01-20  9:15 ` [PATCH v9 8/8] ARM: dts: stm32: Enable pwm1 and pwm3 for stm32f469-disco Benjamin Gaignard
2017-01-20 10:03   ` Alexandre Torgue
2017-01-23 12:12 ` [GIT PULL] Immutable branch between MFD, ARM, IIO and PWM due for the v4.10 merge window Lee Jones
2017-01-24  6:42   ` Olof Johansson
2017-01-24  8:12     ` Lee Jones
2017-01-24 19:54       ` Olof Johansson
2017-01-25 10:08         ` Lee Jones
2017-01-25 10:55           ` Thierry Reding
2017-01-25 15:33             ` Lee Jones [this message]
2017-01-25 18:23           ` Jonathan Cameron
2017-01-25 16:40   ` Lee Jones

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=20170125153352.GA19242@dell \
    --to=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).