Re: [PATCH v6 00/10] arm64: dts: freescale: prepare and add apalis imx8 support

[Marcel Ziswiler <marcel.ziswiler@toradex.com>]

On Thu, 2023-02-02 at 16:32 +0100, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> 
> Add support for lsio_pwm0-3, add io-channel-cells property for ADC
> nodes, set lpspi0 max frequency to 60MHz, add missed LVDS lpi2c and PWM
> power domains, add flexcan in adma, add can node in devicetree, add vpu
> decoder and encoder, add toradex,apalis-imx8 et al. to dt-bindings and
> finally, add initial support for Apalis iMX8 split into module and
> carrier board parts.
> 
> Changes in v6:
> - Add Alexander's acked-by. Thanks!
> - Add Alexander's tested-by. Thanks!
> - Incorporate Shawn's review feedback. Thanks!

Sorry, I missed the generally when Shawn was suggesting to put generic properties before device specific ones.
Let me re-work the device trees with this in mind adhering to hopefully all the rules which apply and send a v7
ASAP.


> Changes in v5:
> - Update subject prefix as pointed out by Krzysztof. Thanks!
> - Remove LVDS PWM support waiting for Liu's patches to land first.
> - Remove bkl1_pwm functionality depending on the above.
> - Squashing all Apalis iMX8 specific device tree patches. As outlined by
>   Krzysztof reviewers may simply use b4 diff.
> 
> Changes in v4:
> - New patch inspired by the following downstream patch:
>   commit 0c36c5b63c1e ("LF-3569-1 firmware: imx: scu-pd: add missed lvds lpi2c and pwm power domains")
>   Avoiding the following being reported upon boot:
>   [    1.309776] lvds0_pwm0_clk: failed to attached the power domain -2
>   [    1.334097] lvds1_pwm0_clk: failed to attached the power domain -2
>   ...
>   [   15.281767] platform 57244000.pwm: deferred probe pending
>   ...
>   [   15.292630] platform backlight: deferred probe pending
> - New patch combining the following downstream patches:
>   commit e8fe3f57223a ("arm64: dts: imx8qxp: add FlexCAN in adma")
>   commit 4e90361f1ed3 ("arm64: dts: imx8qxp: add multi-pd support for CAN1/2")
>   commit 899f516e61f8 ("arm64: dts: imx8: dma: fully switched to new clk binding")
>   commit 8a28ca15a058 ("arm64: dts: imx8qxp: drop multi-pd for CAN device")
>   commit c493402197dd ("arm64: dts: imx8: update CAN fsl,clk-source and fsl,scu-index property")
> - New patch inspired by the following downstream patch:
>   commit 117607e6a7b5 ("arm64: dts: imx8qm: add CAN node in devicetree")
> - New patch combining the following downstream patches:
>   commit 4f2147ce6f0e ("arm64: dts: imx8qm: add vpu decoder and encoder")
>   commit 0c9f9b64d27d ("LF-6575: imx8q: vpu: switch to amphion upstream driver")
> - Shorten subject.
> 
> Changes in v3:
> - Fix subject as pointed out by Shawn. Thanks!
> - Properly accommodate for -v1.1 modules curtsey Francesco.
> - Only use V1.1 compatible for V1.1 module dtsi.
> - Split patch into separate module and carrier boards parts as suggested
>   by Shawn.
> - Put reg after compatible as requested by Shawn.
> - Move atmel_mxt_ts and rtc_i2c into module dtsi to save such
>   duplications as suggested by Shawn.
> - Change iomuxc pinctrl indent style as suggested by Shawn.
> - Change led node names to the preferred first form as suggested by
>   Shawn. While at it also add color (yikes), default-state and function
>   properties and remove the deprecated label property.
> - Put enable-active-high properties after the gpio ones as suggested by
>   Shawn. Thanks!
> - Remove adc node's vref-supply and accompanying reg_vref_1v8 regulator
>   node.
> - Rename gpio-hogs adherring to dt schema naming convention.
> 
> Changes in v2:
> - Fixed missing space in the comment V1.1Module curtsey Max.
> - Added Rob's ack.
> 
> Clark Wang (1):
>   firmware: imx: scu-pd: add missed lvds lpi2c and pwm power domains
> 
> Joakim Zhang (2):
>   arm64: dts: imx8qxp: add flexcan in adma
>   arm64: dts: imx8qm: add can node in devicetree
> 
> Marcel Ziswiler (3):
>   dt-bindings: arm: fsl: add toradex,apalis-imx8 et al.
>   arm64: dts: freescale: add initial apalis imx8 aka quadmax module
>     support
>   arm64: dts: freescale: add apalis imx8 aka quadmax carrier board
>     support
> 
> Max Krummenacher (1):
>   arm64: dts: imx8-ss-dma: add io-channel-cells to adc nodes
> 
> Philippe Schenker (2):
>   arm64: dts: freescale: imx8-ss-lsio: add support for lsio_pwm0-3
>   arm64: dts: freescale: imx8-ss-dma: set lpspi0 max frequency to 60mhz
> 
> Zhou Peng (1):
>   arm64: dts: imx8qm: add vpu decoder and encoder
> 
>  .../devicetree/bindings/arm/fsl.yaml          |   19 +
>  arch/arm64/boot/dts/freescale/Makefile        |    5 +
>  .../boot/dts/freescale/imx8-apalis-eval.dtsi  |  144 ++
>  .../dts/freescale/imx8-apalis-ixora-v1.1.dtsi |  220 +++
>  .../dts/freescale/imx8-apalis-ixora-v1.2.dtsi |  270 +++
>  .../boot/dts/freescale/imx8-apalis-v1.1.dtsi  | 1484 +++++++++++++++++
>  .../arm64/boot/dts/freescale/imx8-ss-dma.dtsi |   76 +-
>  .../boot/dts/freescale/imx8-ss-lsio.dtsi      |   48 +
>  .../boot/dts/freescale/imx8qm-apalis-eval.dts |   16 +
>  .../freescale/imx8qm-apalis-ixora-v1.1.dts    |   16 +
>  .../dts/freescale/imx8qm-apalis-v1.1-eval.dts |   16 +
>  .../imx8qm-apalis-v1.1-ixora-v1.1.dts         |   16 +
>  .../imx8qm-apalis-v1.1-ixora-v1.2.dts         |   16 +
>  .../dts/freescale/imx8qm-apalis-v1.1.dtsi     |   16 +
>  .../boot/dts/freescale/imx8qm-apalis.dtsi     |  340 ++++
>  .../boot/dts/freescale/imx8qm-ss-dma.dtsi     |   44 +
>  arch/arm64/boot/dts/freescale/imx8qm.dtsi     |    4 +
>  drivers/firmware/imx/scu-pd.c                 |    4 +
>  18 files changed, 2753 insertions(+), 1 deletion(-)
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8-apalis-eval.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8-apalis-v1.1.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8qm-apalis-eval.dts
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8qm-apalis-ixora-v1.1.dts
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8qm-apalis-v1.1-eval.dts
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8qm-apalis-v1.1-ixora-v1.1.dts
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8qm-apalis-v1.1-ixora-v1.2.dts
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8qm-apalis-v1.1.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8qm-apalis.dtsi