Re: [PATCH v7 6/7] ARM: dts: aspeed: Add Balcones system
From: Andrew Jeffery <andrew@codeconstruct.com.au>
Date: 2025-09-26 00:03:19
Also in:
linux-devicetree
On Mon, 2025-09-22 at 12:58 -0500, Eddie James wrote:
The Balcones system is similar to Bonnell but with a POWER11 processor. Like POWER10, the POWER11 is a dual-chip module, so a dual chip FSI tree is needed. Therefore, split up the quad chip FSI tree. Signed-off-by: Eddie James <eajames@linux.ibm.com> --- arch/arm/boot/dts/aspeed/Makefile | 1 + .../dts/aspeed/aspeed-bmc-ibm-balcones.dts | 609 ++++++++++++++
Looking at Balcones specifically, there's still an immediate concern:
/home/andrew/src/kernel.org/linux/origin/build.arm.aspeed_g5/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: pressure-sensor@76 (infineon,dps310): '#io-channel-cells' does not match any of the regexes: '^pinctrl-[0-9]+$'
from schema $id: http://devicetree.org/schemas/trivial-devices.yaml#
On the whole though it's much cleaner than other IBM devicetrees, just
the DTSI warnings remain. I'd appreciate it if you have some time for
cleanups to Rainier, Everest and Bonnell, the number of warnings is
really quite large :(
Andrew