Message ID | 20170123121222.GJ16020@dell |
---|---|
State | Not Applicable |
Headers | show |
Hi Lee, 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? -Olof -- To unsubscribe from this list: send the line "unsubscribe linux-pwm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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.
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! -Olof -- To unsubscribe from this list: send the line "unsubscribe linux-pwm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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?
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. Thierry
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.
On 25 January 2017 10:08:58 GMT+00:00, Lee Jones <lee.jones@linaro.org> 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? Bit late from me but fine. I tend to only pull in these branches when stuff to go on top turns up. Not happened with this set yet! J