Message-ID: <20230306031609.GH143566@dragon>
Date:   Mon, 6 Mar 2023 11:16:09 +0800
From:   Shawn Guo <shawnguo@...nel.org>
To:     Marcel Ziswiler <marcel@...wiler.com>
Cc:     devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        Liu Ying <victor.liu@....com>, linux-imx@....com,
        linux-arm-kernel@...ts.infradead.org,
        Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Marcel Ziswiler <marcel.ziswiler@...adex.com>,
        Clark Wang <xiaoning.wang@....com>,
        Denys Drozdov <denys.drozdov@...adex.com>,
        Fabio Estevam <festevam@...x.de>,
        Fabio Estevam <festevam@...il.com>,
        Frank Li <Frank.Li@....com>,
        Frieder Schrempf <frieder.schrempf@...tron.de>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Li Yang <leoyang.li@....com>, Marek Vasut <marex@...x.de>,
        Matthias Schiffer <matthias.schiffer@...group.com>,
        Max Krummenacher <max.krummenacher@...adex.com>,
        Ming Qian <ming.qian@....com>,
        Pengutronix Kernel Team <kernel@...gutronix.de>,
        Rob Herring <robh+dt@...nel.org>,
        Sascha Hauer <s.hauer@...gutronix.de>,
        Shijie Qin <shijie.qin@....com>,
        Tim Harvey <tharvey@...eworks.com>
Subject: Re: [PATCH v7 00/10] arm64: dts: freescale: prepare and add apalis
 imx8 support

On Wed, Feb 08, 2023 at 07:56:31AM +0100, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <marcel.ziswiler@...adex.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 v7:
> - Re-order device tree properties according to various more or less
>   undocumented rules as e.g. pointed out by Shawn. Thanks!
> 
> 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

Applied all, thanks!

Powered by blists - more mailing lists