Re: [PATCH v8 2/2] ARM: dts: aspeed: ventura2: Add Meta ventura2 BMC

On Mon, 2026-06-15 at 10:46 +0800, Kyle Hsieh wrote:
> Ventura2 is a Rack Management Controller. It is a modular
> device designed to manage liquid cooling systems and monitor hardware
> states within an IT rack. The system uses an AST2600 BMC for management.
>=20
> RMCv2 serves several critical roles:
> - Detects liquid leakage at both tray and rack levels.
> - Communicates with and controls liquid cooling equipment.
> - Manages leakage events and executes system recovery protocols.
>=20
> Key hardware features include:
> - An extensive I2C and IO expander topology to support comprehensive
> =C2=A0 sensor monitoring and backward compatibility with legacy trays.
> - MCTP over I2C support for asynchronous device communications.
> - A dual-flash design for BMC firmware redundancy.
>=20
> Signed-off-by: Kyle Hsieh <[email protected]>
> ---
> =C2=A0arch/arm/boot/dts/aspeed/Makefile=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 |=C2=
=A0=C2=A0=C2=A0 1 +
> =C2=A0.../dts/aspeed/aspeed-bmc-facebook-ventura2.dts=C2=A0=C2=A0=C2=A0 |=
 2903 ++++++++++++++++++++
> =C2=A02 files changed, 2904 insertions(+)

We now have scripts/dtc/dt-check-style. Running it over the dts with '-
-mode relaxed' yields 49 instances of 'mixed-indent-chars' - these must
be fixed. Further, can you please look at the output with '--mode
strict', address all the warnings that are reasonable, and then discuss
what's left?

Otherwise, there are still some concerns reported against the bindings:

   $ make CHECK_DTBS=3Dy aspeed/aspeed-bmc-facebook-ventura2.dtb W=3D1
   ...
     DTC [C] arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb
   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: /ahb/apb/lpc@=
1e789000/lhc@a0: failed to match any schema with compatible: ['aspeed,ast26=
00-lhc']
   /home/andrew/src/kernel.org/linux/origin/build.arm.aspeed_g5/arch/arm/bo=
ot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: fan-controller@20 (maxim,ma=
x31790): 'channel@2', 'channel@5' do not match any of the regexes: '^fan@[0=
-9]+$', '^pinctrl-[0-9]+$'
           from schema $id: http://devicetree.org/schemas/hwmon/maxim,max31=
790.yaml
   /home/andrew/src/kernel.org/linux/origin/build.arm.aspeed_g5/arch/arm/bo=
ot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: i2c@400 (aspeed,ast2600-i2c=
-bus): Unevaluated properties are not allowed ('aspeed,hw-timeout-ms' was u=
nexpected)
           from schema $id: http://devicetree.org/schemas/i2c/aspeed,i2c.ya=
ml
   /home/andrew/src/kernel.org/linux/origin/build.arm.aspeed_g5/arch/arm/bo=
ot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: adc@48 (ti,ads1015): '#addr=
ess-cells' is a required property
           from schema $id: http://devicetree.org/schemas/iio/adc/ti,ads101=
5.yaml
   /home/andrew/src/kernel.org/linux/origin/build.arm.aspeed_g5/arch/arm/bo=
ot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: adc@48 (ti,ads1015): '#size=
-cells' is a required property
           from schema $id: http://devicetree.org/schemas/iio/adc/ti,ads101=
5.yaml
   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: /ahb/apb/bus@=
1e78a000/i2c@580/power-monitor@69: failed to match any schema with compatib=
le: ['pmbus']
   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dtb: /ahb/apb/bus@=
1e7a0000/syscon@0: failed to match any schema with compatible: ['aspeed,ast=
2600-i3c-global', 'syscon']

Can you please tidy these up?

Thanks,

Andrew