| Message ID | 41ced8db-caeb-4deb-9193-9802fc603180@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [GIT,PULL] amlogic ARM64 DT updates for v6.15 | expand |
On Mon, Mar 17, 2025, at 17:13, Neil Armstrong wrote: > Hi, > > Here's a changeset for Amlogic ARM64 DT, also switching to the new PWM bindings, > adding pinctrl support for Amlogic A4 SoC with a brand new binding style then > finally gpio_intc node for Amlogic A4 & A5 SoCs. I'll skip this one for 6.15, seeing that the branch contains both driver and devicetree changes, which messes up the way I send the contents. From the description I also can't tell to what degree there is backwards compatibility in the new binding, I assume the changed PWM and pinctrl drivers continue to work find with the old dts but not the other way round, which is why you sent them together? Arnd
Hi Arnd, On Wed, Mar 19, 2025 at 9:58 PM Arnd Bergmann <arnd@arndb.de> wrote: > > On Mon, Mar 17, 2025, at 17:13, Neil Armstrong wrote: > > Hi, > > > > Here's a changeset for Amlogic ARM64 DT, also switching to the new PWM bindings, > > adding pinctrl support for Amlogic A4 SoC with a brand new binding style then > > finally gpio_intc node for Amlogic A4 & A5 SoCs. > > I'll skip this one for 6.15, seeing that the branch contains both > driver and devicetree changes, which messes up the way I send > the contents. There's an immutable branch in the linux-pinctrl tree from Linus. I'm not sure on the exact process here (whether the contents of the tag are fine and we're "just" missing a description, ...) so I'll let Neil comment on this. > From the description I also can't tell to what degree there is > backwards compatibility in the new binding, I assume the changed > PWM and pinctrl drivers continue to work find with the old dts > but not the other way round, which is why you sent them together? The PWM part is backwards compatible. The bindings are upstream since commit 5fd61cc28171 ("dt-bindings: pwm: amlogic: Add a new binding for meson8 pwm types"). The mentioned pinctrl binding / driver is completely new for the A4 SoCs, so there's nothing to be backwards compatible with. Neil, for me it's fine to skip this one for 6.15 if it causes too much work on your side. Using the old PWM bindings for another release won't hurt anyone. And I think the same goes for A4 pinctrl support as well as GPIO interrupt controller nodes for A4/S5. Best regards, Martin [0] https://lore.kernel.org/linux-gpio/CACRpkdZg-9Zr3f7zG36x7jGvT46FRfRJ40khNZDdYb01-BGigw@mail.gmail.com/
Hi, On 19/03/2025 21:57, Arnd Bergmann wrote: > On Mon, Mar 17, 2025, at 17:13, Neil Armstrong wrote: >> Hi, >> >> Here's a changeset for Amlogic ARM64 DT, also switching to the new PWM bindings, >> adding pinctrl support for Amlogic A4 SoC with a brand new binding style then >> finally gpio_intc node for Amlogic A4 & A5 SoCs. > > I'll skip this one for 6.15, seeing that the branch contains both > driver and devicetree changes, which messes up the way I send > the contents. Sure, I can strip out the pinctrl DT changes, since they can definitely wait another cycle. > > From the description I also can't tell to what degree there is > backwards compatibility in the new binding, I assume the changed > PWM and pinctrl drivers continue to work find with the old dts > but not the other way round, which is why you sent them together? For the PWM bindings, they are in -next for a long time already, and everything was done to keep the old bindings still work over time. So we're pretty confident it works fine since it has run over CI for multiple months. I'll resent a new PR without the new A4 pinctrl changes. Thanks, Neil > > Arnd
Hi, Here's a changeset for Amlogic ARM64 DT, also switching to the new PWM bindings, adding pinctrl support for Amlogic A4 SoC with a brand new binding style then finally gpio_intc node for Amlogic A4 & A5 SoCs. Thanks, Neil The following changes since commit 2014c95afecee3e76ca4a56956a936e23283f05b: Linux 6.14-rc1 (2025-02-02 15:39:26 -0800) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/amlogic/linux.git tags/amlogic-arm64-dt-for-v6.15 for you to fetch changes up to 454e0353b226a47f3c3369b17c7a4893f74e29cd: arm64: dts: Add gpio_intc node for Amlogic A5 SoCs (2025-03-17 08:38:58 +0100) ---------------------------------------------------------------- Amlogic ARM64 DT changes for v6.15: - Add pinctrl support for Amlogic A4 SoC with new bindings - switch to the new PWM controller binding - Add gpio_intc node for Amlogic A4 & A5 SoCs ---------------------------------------------------------------- Martin Blumenstingl (3): arm64: dts: amlogic: gx: switch to the new PWM controller binding arm64: dts: amlogic: axg: switch to the new PWM controller binding arm64: dts: amlogic: g12: switch to the new PWM controller binding Neil Armstrong (1): Merge branch 'ib-amlogic-a4' of git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-pinctrl into HEAD Xianwei Zhao (7): dt-bindings: pinctrl: Add support for Amlogic A4 SoC pinctrl: pinconf-generic: Add API for pinmux propertity in DTS file pinctrl: Add driver support for Amlogic SoCs MAINTAINERS: Add an entry for Amlogic pinctrl driver arm64: dts: amlogic: a4: add pinctrl node arm64: dts: Add gpio_intc node for Amlogic A4 SoCs arm64: dts: Add gpio_intc node for Amlogic A5 SoCs .../bindings/pinctrl/amlogic,pinctrl-a4.yaml | 126 +++ MAINTAINERS | 8 + arch/arm64/boot/dts/amlogic/amlogic-a4.dtsi | 139 +++ arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi | 12 + arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 24 +- arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi | 33 +- arch/arm64/boot/dts/amlogic/meson-g12a-fbx8am.dts | 4 - .../boot/dts/amlogic/meson-g12a-radxa-zero.dts | 4 - arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts | 4 - arch/arm64/boot/dts/amlogic/meson-g12a-u200.dts | 2 - arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts | 4 - .../dts/amlogic/meson-g12b-a311d-libretech-cc.dts | 2 - .../boot/dts/amlogic/meson-g12b-bananapi-cm4.dtsi | 4 - .../boot/dts/amlogic/meson-g12b-bananapi.dtsi | 4 - .../boot/dts/amlogic/meson-g12b-khadas-vim3.dtsi | 4 - arch/arm64/boot/dts/amlogic/meson-g12b-odroid.dtsi | 4 - .../boot/dts/amlogic/meson-g12b-radxa-zero2.dts | 8 - arch/arm64/boot/dts/amlogic/meson-g12b-w400.dtsi | 6 - .../boot/dts/amlogic/meson-gx-libretech-pc.dtsi | 6 - .../arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi | 2 - arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 8 +- .../boot/dts/amlogic/meson-gxbb-nanopi-k2.dts | 2 - .../boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts | 2 - arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi | 2 - .../boot/dts/amlogic/meson-gxbb-vega-s95.dtsi | 2 - arch/arm64/boot/dts/amlogic/meson-gxbb-wetek.dtsi | 2 - arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 25 + .../boot/dts/amlogic/meson-gxl-s805x-p241.dts | 2 - .../amlogic/meson-gxl-s905w-jethome-jethub-j80.dts | 2 - .../dts/amlogic/meson-gxl-s905x-hwacom-amazetv.dts | 2 - .../dts/amlogic/meson-gxl-s905x-khadas-vim.dts | 2 - .../dts/amlogic/meson-gxl-s905x-nexbox-a95x.dts | 2 - .../boot/dts/amlogic/meson-gxl-s905x-p212.dtsi | 2 - arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 25 + .../boot/dts/amlogic/meson-gxm-khadas-vim2.dts | 4 - arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts | 2 - .../dts/amlogic/meson-libretech-cottonwood.dtsi | 6 - arch/arm64/boot/dts/amlogic/meson-sm1-ac2xx.dtsi | 6 - .../arm64/boot/dts/amlogic/meson-sm1-bananapi.dtsi | 2 - .../boot/dts/amlogic/meson-sm1-khadas-vim3l.dts | 2 - arch/arm64/boot/dts/amlogic/meson-sm1-odroid.dtsi | 2 - arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts | 6 - drivers/pinctrl/meson/Kconfig | 11 + drivers/pinctrl/meson/Makefile | 1 + drivers/pinctrl/meson/pinctrl-amlogic-a4.c | 1053 ++++++++++++++++++++ drivers/pinctrl/pinconf-generic.c | 130 +++ drivers/pinctrl/pinconf.h | 4 + include/dt-bindings/pinctrl/amlogic,pinctrl.h | 46 + include/linux/pinctrl/pinconf-generic.h | 4 + 49 files changed, 1636 insertions(+), 123 deletions(-) create mode 100644 Documentation/devicetree/bindings/pinctrl/amlogic,pinctrl-a4.yaml create mode 100644 drivers/pinctrl/meson/pinctrl-amlogic-a4.c create mode 100644 include/dt-bindings/pinctrl/amlogic,pinctrl.h