mbox series

[v3,00/19] dt-bindings / arm64: Cleanup of i.MX 8 bindings

Message ID 20200825193536.7332-1-krzk@kernel.org (mailing list archive)
Headers show
Series dt-bindings / arm64: Cleanup of i.MX 8 bindings | expand

Message

Krzysztof Kozlowski Aug. 25, 2020, 7:35 p.m. UTC
Hi,

This is a v3 of cleanup of i.XM 8 bindings and DTSes.

It is separate patchset from i.MX 8 pin configuration cleanup, which
also touch the bindings [1]. No dependencies (although in my tree this
comes first).

Merging
=======
There are no dependencies, so dt-bindings could go through Rob's tree,
DTS through SoC. I think there is no point to push dt-bindings changes
through subsystem maintainers (gpio, pwm, watchdog, mtd etc). Usually
Rob has been picking them up.

Changes since v2:
=================
1. Add Rob's review,
2. Correct things pointed during review (see individual patches and
their change logs).

[1] dt-bindings: mtd: gpmi-nand: Fix matching of clocks on different SoC

Best regards,
Krzysztof


Krzysztof Kozlowski (19):
  dt-bindings: gpio: fsl-imx-gpio: Add i.MX 8 compatibles
  dt-bindings: gpio: fsl-imx-gpio: Add gpio-ranges property
  dt-bindings: gpio: fsl-imx-gpio: Add parsing of hogs
  dt-bindings: gpio: fsl-imx-gpio: Add power-domains
  dt-bindings: perf: fsl-imx-ddr: Add i.MX 8M compatibles
  dt-bindings: pwm: imx-pwm: Add i.MX 8M compatibles
  dt-bindings: serial: fsl-imx-uart: Add i.MX 8M compatibles
  dt-bindings: watchdog: fsl-imx-wdt: Add i.MX 8M compatibles
  dt-bindings: mtd: gpmi-nand: Add i.MX 8M compatibles
  dt-bindings: reset: fsl,imx7-src: Add i.MX 8M compatibles
  dt-bindings: thermal: imx8mm-thermal: Add i.MX 8M Nano compatible
  dt-bindings: mmc: fsl-imx-esdhc: Fix i.MX 8 compatible matching
  dt-bindings: nvmem: imx-ocotp: Update i.MX 8M compatibles
  dt-bindings: arm: fsl: Fix Toradex Colibri i.MX 8 binding
  dt-bindings: arm: fsl: Add ZII Ultra boards binding
  dt-bindings: interrupt-controller: fsl,irqsteer: Fix compatible
    matching
  dt-bindings: serial: fsl-lpuart: Fix compatible matching
  arm64: dts: imx8mq-evk: Add hog suffix to wl-reg-on
  arm64: dts: imx8mq-zii-ultra: Add hog suffixes to GPIO hogs

 .../devicetree/bindings/arm/fsl.yaml          | 14 ++++++
 .../bindings/gpio/fsl-imx-gpio.yaml           | 43 ++++++++++++++++---
 .../interrupt-controller/fsl,irqsteer.yaml    |  8 ++--
 .../bindings/mmc/fsl-imx-esdhc.yaml           | 37 ++++++++--------
 .../devicetree/bindings/mtd/gpmi-nand.yaml    | 18 +++++---
 .../devicetree/bindings/nvmem/imx-ocotp.yaml  | 38 +++++++++-------
 .../devicetree/bindings/perf/fsl-imx-ddr.yaml | 16 +++++--
 .../devicetree/bindings/pwm/imx-pwm.yaml      | 14 ++++--
 .../bindings/reset/fsl,imx7-src.yaml          | 19 +++++---
 .../bindings/serial/fsl-imx-uart.yaml         |  4 ++
 .../bindings/serial/fsl-lpuart.yaml           | 17 +++++---
 .../bindings/thermal/imx8mm-thermal.yaml      | 10 +++--
 .../bindings/watchdog/fsl-imx-wdt.yaml        | 11 ++++-
 arch/arm64/boot/dts/freescale/imx8mq-evk.dts  |  2 +-
 .../boot/dts/freescale/imx8mq-zii-ultra.dtsi  |  8 ++--
 15 files changed, 182 insertions(+), 77 deletions(-)

Comments

Linus Walleij Aug. 28, 2020, 12:51 p.m. UTC | #1
On Tue, Aug 25, 2020 at 9:35 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:

> This is a v3 of cleanup of i.XM 8 bindings and DTSes.

If you are going to be working a lot on Freescale SoC code going forward
I wouldn't mind if you could add yourself as maintainer for the
Freescale pin controller and GPIO at least, I already have high trust
in you in general so if the Freescale maintainers also have that I think you
should just sign up as maintainer. This makes it easier to do pull requests
and things like that.

Yours,
Linus Walleij
Krzysztof Kozlowski Aug. 28, 2020, 1:08 p.m. UTC | #2
On Fri, Aug 28, 2020 at 02:51:20PM +0200, Linus Walleij wrote:
> On Tue, Aug 25, 2020 at 9:35 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> 
> > This is a v3 of cleanup of i.XM 8 bindings and DTSes.
> 
> If you are going to be working a lot on Freescale SoC code going forward
> I wouldn't mind if you could add yourself as maintainer for the
> Freescale pin controller and GPIO at least, I already have high trust
> in you in general so if the Freescale maintainers also have that I think you
> should just sign up as maintainer. This makes it easier to do pull requests
> and things like that.

Thanks for encouragement.  Indeed I am planning to do more work around
i.MX 8M platforms from NXP/Freescale. However there are already four
maintainers for Freescale pin controller drivers so I doubt there is
need for fifth entry :).

Different question is the GPIO driver which apparently lacks entry in
Maintainers file.

Best regards,
Krzysztof
Linus Walleij Aug. 28, 2020, 2:51 p.m. UTC | #3
On Fri, Aug 28, 2020 at 3:08 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> On Fri, Aug 28, 2020 at 02:51:20PM +0200, Linus Walleij wrote:
> > On Tue, Aug 25, 2020 at 9:35 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> >
> > > This is a v3 of cleanup of i.XM 8 bindings and DTSes.
> >
> > If you are going to be working a lot on Freescale SoC code going forward
> > I wouldn't mind if you could add yourself as maintainer for the
> > Freescale pin controller and GPIO at least, I already have high trust
> > in you in general so if the Freescale maintainers also have that I think you
> > should just sign up as maintainer. This makes it easier to do pull requests
> > and things like that.
>
> Thanks for encouragement.  Indeed I am planning to do more work around
> i.MX 8M platforms from NXP/Freescale. However there are already four
> maintainers for Freescale pin controller drivers so I doubt there is
> need for fifth entry :).

I beg to differ. As subsystem maintainer it has happened to me more
than once that the i.MX pin control patches went unreviewed for weeks.
Everyone listed for this driver is a kitchen sink maintainer that get
way too much traffic and things fall over the edge.

> Different question is the GPIO driver which apparently lacks entry in
> Maintainers file.

That's probably just an oversight. Feel free to add it!

Yours,
Linus Walleij