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

[Michal Simek <michal.simek@amd.com>]

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@amd.com:
> 
> 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