[PATCH v7 6/7] ARM: dts: aspeed: Add Balcones system

[Eddie James]

On 9/25/25 7:03 PM, Andrew Jeffery wrote:
> 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 at 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 at 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#


Hmm, patch 3 in the series fixes this. Did you apply that one? Here is 
my output from CHECK_DTBS for balcones:


$ make CHECK_DTBS=y dtbs
   DTC [C] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb
/home/eajames/development/linux/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
timer (arm,armv7-timer): 'clocks' does not match any of the regexes: 
'^pinctrl-[0-9]+$'
         from schema $id: 
http://devicetree.org/schemas/timer/arm,arch_timer.yaml#
/home/eajames/development/linux/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
sdram at 1e6e0000 (aspeed,ast2600-sdram-edac): compatible: 
['aspeed,ast2600-sdram-edac', 'syscon'] is too long
         from schema $id: 
http://devicetree.org/schemas/edac/aspeed,ast2400-sdram-edac.yaml#
/home/eajames/development/linux/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
bus at 1e600000 (aspeed,ast2600-ahbc): compatible: ['aspeed,ast2600-ahbc', 
'syscon'] is too long
         from schema $id: 
http://devicetree.org/schemas/bus/aspeed,ast2600-ahbc.yaml#
/home/eajames/development/linux/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
syscon at 1e6e2000 (aspeed,ast2600-scu): 'smp-memram at 180' does not match 
any of the regexes: '^interrupt-controller@[0-9a-f]+$', 
'^p2a-control@[0-9a-f]+$', '^pinctrl(@[0-9a-f]+)?$', '^pinctrl-[0-9]+$', 
'^silicon-id@[0-9a-f]+$'
         from schema $id: 
http://devicetree.org/schemas/mfd/aspeed,ast2x00-scu.yaml#
arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
/ahb/apb/syscon at 1e6e2000/smp-memram at 180: failed to match any schema with 
compatible: ['aspeed,ast2600-smpmem']
/home/eajames/development/linux/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
adc at 1e6e9000 (aspeed,ast2600-adc0): 'interrupts' does not match any of 
the regexes: '^pinctrl-[0-9]+$'
         from schema $id: 
http://devicetree.org/schemas/iio/adc/aspeed,ast2600-adc.yaml#
/home/eajames/development/linux/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
adc at 1e6e9100 (aspeed,ast2600-adc1): 'interrupts' does not match any of 
the regexes: '^pinctrl-[0-9]+$'
         from schema $id: 
http://devicetree.org/schemas/iio/adc/aspeed,ast2600-adc.yaml#
/home/eajames/development/linux/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
crypto at 1e6fa000 (aspeed,ast2600-acry): 'aspeed,ahbc' does not match any 
of the regexes: '^pinctrl-[0-9]+$'
         from schema $id: 
http://devicetree.org/schemas/crypto/aspeed,ast2600-acry.yaml#
arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
/ahb/apb/lpc at 1e789000/lhc at a0: failed to match any schema with 
compatible: ['aspeed,ast2600-lhc']
arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
/ahb/apb/lpc at 1e789000/ibt at 140: failed to match any schema with 
compatible: ['aspeed,ast2600-ibt-bmc']
arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb: 
/ahb/apb/dma-controller at 1e79e000: failed to match any schema with 
compatible: ['aspeed,ast2600-udma']



>
> 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 :(


Yea, I can try and get to it eventually :)


Thanks!

Eddie


>
> Andrew
>