Re: [PATCH 00/15] ARM: zynq: Sync DTs with U-Boot

[Michal Simek]

On 12/11/24 16:54, Rob Herring (Arm) wrote:
> On Wed, 11 Dec 2024 13:41:19 +0100, Michal Simek wrote:
> > Hi,
> >
> > over years there were some changes pushed to U-Boot which were never merged
> > back to Linux. U-Boot introduced new option OF_UPSTREAM and start to sync
> > up DTs from Linux back to U-Boot.
> > This series is addressing differences.
> > There are still 3 more differences but they should be addressed separately.
> >
> > Thanks,
> > Michal
> >
> >
> > Michal Simek (14):
> >    ARM: zynq: Remove deprecated device_type property
> >    ARM: zynq: DT: List OCM memory for all platforms
> >    ARM: zynq: Mark boot-phase-specific device nodes
> >    ARM: zynq: Do not define address/size-cells for nand-controller
> >    ARM: zynq: Wire smcc with nand/nor memories on zc770 platform
> >    ARM: zynq: Add ethernet phy reset information to DT(zc702)
> >    ARM: zynq: Define u-boot bootscrip addr via DT
> >    ARM: zynq: Point via nvmem0 alias to eeprom on zc702/zc706
> >    ARM: zynq: Define rtc alias on zc702/zc706
> >    ARM: zynq: Rename i2c?-gpio to i2c?-gpio-grp
> >    ARM: zynq: Fix fpga region DT nodes name
> >    ARM: zynq: Enable QSPIs on platforms
> >    ARM: zynq: Add sdhci to alias node
> >    ARM: zynq: Remove ethernet0 alias from Microzed
> >
> > Sai Krishna Potthuri (1):
> >    ARM: zynq: Replace 'io-standard' with 'power-source' property
> >
> >   arch/arm/boot/dts/xilinx/zynq-7000.dtsi       | 33 ++++++-
> >   arch/arm/boot/dts/xilinx/zynq-cc108.dts       | 41 ++++++++-
> >   arch/arm/boot/dts/xilinx/zynq-microzed.dts    | 10 ++-
> >   arch/arm/boot/dts/xilinx/zynq-parallella.dts  |  1 -
> >   arch/arm/boot/dts/xilinx/zynq-zc702.dts       | 87 +++++++++++++------
> >   arch/arm/boot/dts/xilinx/zynq-zc706.dts       | 67 +++++++++++---
> >   arch/arm/boot/dts/xilinx/zynq-zc770-xm010.dts | 39 ++++++++-
> >   arch/arm/boot/dts/xilinx/zynq-zc770-xm011.dts | 31 +++++++
> >   arch/arm/boot/dts/xilinx/zynq-zc770-xm012.dts | 35 ++++++++
> >   arch/arm/boot/dts/xilinx/zynq-zc770-xm013.dts | 41 ++++++++-
> >   arch/arm/boot/dts/xilinx/zynq-zed.dts         | 43 ++++++++-
> >   .../boot/dts/xilinx/zynq-zturn-common.dtsi    |  8 ++
> >   arch/arm/boot/dts/xilinx/zynq-zybo-z7.dts     | 10 ++-
> >   arch/arm/boot/dts/xilinx/zynq-zybo.dts        |  9 +-
> >   14 files changed, 404 insertions(+), 51 deletions(-)
> >
> > --
> > 2.43.0
>
>
> My bot found new DTB warnings on the .dts files added or changed in this
> series.
>
> Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
> are fixed by another series. Ultimately, it is up to the platform
> maintainer whether these warnings are acceptable or not. No need to reply
> unless the platform maintainer has comments.
>
> If you already ran DT checks and didn't see these error(s), then
> make sure dt-schema is up to date:
>
>    pip3 install dtschema --upgrade
>
>
> New warnings running 'make CHECK_DTBS=y xilinx/zynq-cc108.dtb xilinx/zynq-microzed.dtb xilinx/zynq-parallella.dtb xilinx/zynq-zc702.dtb xilinx/zynq-zc706.dtb xilinx/zynq-zc770-xm010.dtb xilinx/zynq-zc770-xm011.dtb xilinx/zynq-zc770-xm012.dtb xilinx/zynq-zc770-xm013.dtb xilinx/zynq-zed.dtb xilinx/zynq-zybo-z7.dtb xilinx/zynq-zybo.dtb' for cover.1733920873.git.michal.simek@xxxxxxx:
>
> arch/arm/boot/dts/xilinx/zynq-parallella.dtb: ethernet@e000b000: ethernet-phy@0: Unevaluated properties are not allowed ('marvell,reg-init' was unexpected)
> 	from schema $id: http://devicetree.org/schemas/net/cdns,macb.yaml#

This is not issue caused by this series.


> arch/arm/boot/dts/xilinx/zynq-zc702.dtb: ethernet@e000b000: Unevaluated properties are not allowed ('phy-reset-active-low', 'phy-reset-gpio' were unexpected)
> 	from schema $id: http://devicetree.org/schemas/net/cdns,macb.yaml#

This has been reported by Andrew already and the patch will be removed.

Thanks,
Michal