Re: [PATCH v4 1/2] arm64: dts: rockchip: Add rk3588-orangepi-5b device tree and refactor

[Jimmy Hon]

> New warnings running 'make CHECK_DTBS=y rockchip/rk3588s-orangepi-5.dtb rockchip/rk3588s-orangepi-5b.dtb' for 20241014113026.78853-1-cenk.uluisik@xxxxxxxxxxxxxx:
>
> arch/arm64/boot/dts/rockchip/rk3588s-orangepi-5b.dtb: gpio-func: leds-gpio: {'rockchip,pins': [[0, 2, 0, 245]], 'phandle': 253} is not of type 'array'
>         from schema $id: http://devicetree.org/schemas/gpio/gpio-consumer.yaml#
Regarding the "leds-gpio" warning, DTS checking script might not like
that id. You could probably avoid it by naming it "green-led". Similar
to what the Orange Pi 5 Plus does. This would be a separate bugfix
commit before any of the other changes in this series.

> arch/arm64/boot/dts/rockchip/rk3588s-orangepi-5b.dtb: vcc-3v3-sd-s0-regulator: Unevaluated properties are not allowed ('enable-active-low' was unexpected)
>         from schema $id: http://devicetree.org/schemas/regulator/fixed-regulator.yaml#
The enable-active-low has already been addressed in
https://git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git/commit/?h=v6.12-armsoc/dtsfixes&id=f4d29bebaa6118c1e51e8f1c21ce2b34f43e1479
So you can avoid the error if you start with v6.12-armsoc as your baseline.

Regarding the "leds-gpio" warning, DTS checking script might not like
that id. You could probably avoid it by naming it "green-led". Similar
to what the Orange Pi 5 Plus does. This would be a separate bugfix
commit before any of the other changes.

Jimmy