mbox series

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

Message ID 20230202153221.197308-1-marcel@ziswiler.com (mailing list archive)
Headers show
Series arm64: dts: freescale: prepare and add apalis imx8 support | expand

Message

Marcel Ziswiler Feb. 2, 2023, 3:32 p.m. UTC 
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!

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

Comments

Marcel Ziswiler Feb. 3, 2023, 1:16 p.m. UTC | #1 
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