Re: [PATCH 1/3] dt-bindings: arm: rockchip: Add ArmSoM Sige7s
From: Jianfeng Liu
Date: Tue Oct 27 2026 - 21:51:17 EST
Hi Quentin,
On Tue, 27 Oct 2026 17:03:02 +0100, Quentin Schulz wrote:
> From U-Boot's perspective, I would have preferred there be a simple
>difference we can detect at runtime and based on that decide which
>device tree to load between coolpi-4b and armsom-sige7s.
Maybe we can ask ArmSom to write some OEM strings into the OTP of SoC.
So we can use one defconfig for two boards. I will check this method
with ArmSoM and current OTP support in u-boot.
>I don't know Heiko's or Krzysztof's opinion here, but I think it would
>better reflect the actual context to NOT use a dtsi and simply have
>arch/arm64/boot/dts/rockchip/rk3588s-armsom-sige7s.dts include
>arch/arm64/boot/dts/rockchip/rk3588s-coolpi-4b.dts and update the model
>and compatible properties.
Thanks for the suggestion. I checked the existing mainline DTS files and
found similar examples, such as stm32mp157c-ev1.dts including
stm32mp157c-ed1.dts directly, and the ARM Juno SCMI variants including
their base board DTS files.
I agree that directly including rk3588s-coolpi-4b.dts would better reflect
the relationship between the two boards and avoid unnecessary changes to
the existing CoolPi DTS.
I'll use this approach in v2 unless Heiko prefers otherwise.
Best regards,
Jianfeng