lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <6e611196a52f7884cad8ccd030280d384d0e403b.camel@toradex.com>
Date:   Fri, 3 Feb 2023 13:16:27 +0000
From:   Marcel Ziswiler <marcel.ziswiler@...adex.com>
To:     "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
CC:     "krzysztof.kozlowski@...aro.org" <krzysztof.kozlowski@...aro.org>,
        "s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        Philippe Schenker <philippe.schenker@...adex.com>,
        "viorel.suman@....com" <viorel.suman@....com>,
        "abelvesa@...nel.org" <abelvesa@...nel.org>,
        "oliver.graute@...oconnector.com" <oliver.graute@...oconnector.com>,
        Max Krummenacher <max.krummenacher@...adex.com>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "festevam@...x.de" <festevam@...x.de>,
        "eagle.zhou@....com" <eagle.zhou@....com>,
        "clin@...e.com" <clin@...e.com>,
        "frieder.schrempf@...tron.de" <frieder.schrempf@...tron.de>,
        "qiangqing.zhang@....com" <qiangqing.zhang@....com>,
        "krzysztof.kozlowski+dt@...aro.org" 
        <krzysztof.kozlowski+dt@...aro.org>,
        "reinhold.mueller@...rion.com" <reinhold.mueller@...rion.com>,
        "xiaoning.wang@....com" <xiaoning.wang@....com>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        "peng.fan@....com" <peng.fan@....com>,
        "marex@...x.de" <marex@...x.de>,
        "ming.qian@....com" <ming.qian@....com>,
        "leoyang.li@....com" <leoyang.li@....com>,
        "matthias.schiffer@...group.com" <matthias.schiffer@...group.com>,
        "festevam@...il.com" <festevam@...il.com>,
        "linux-imx@....com" <linux-imx@....com>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "victor.liu@....com" <victor.liu@....com>,
        "Frank.Li@....com" <Frank.Li@....com>,
        "tharvey@...eworks.com" <tharvey@...eworks.com>,
        "denys.drozdov@...adex.com" <denys.drozdov@...adex.com>,
        "pierre.gondois@....com" <pierre.gondois@....com>
Subject: Re: [PATCH v6 00/10] arm64: dts: freescale: prepare and add apalis
 imx8 support

On Thu, 2023-02-02 at 16:32 +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 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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ