Hi Rob, On Fri, 2025-08-15 at 18:08 -0500, Rob Herring (Arm) wrote: > The ASpeed DTS files have various I2C devices with missing or incorrect > vendor prefixes in their compatible strings. This hasn't really mattered > and doesn't impact ABI compatibility as I2C devices get matched with their > vendor prefix stripped. > > With this, the "maxim,max31790" nodes now validate and have some > warnings. Remove the spurious "#address-cells" and "#size-cells" > properties to fix the warnings. > > Signed-off-by: Rob Herring (Arm) <robh@xxxxxxxxxx> > --- > .../aspeed-bmc-arm-stardragon4800-rep2.dts | 2 +- > .../dts/aspeed/aspeed-bmc-facebook-harma.dts | 8 ++----- > .../aspeed/aspeed-bmc-facebook-minerva.dts | 24 +++++-------------- > .../aspeed/aspeed-bmc-facebook-tiogapass.dts | 2 +- > .../dts/aspeed/aspeed-bmc-lenovo-hr855xg2.dts | 2 +- > .../dts/aspeed/aspeed-bmc-opp-palmetto.dts | 2 +- > .../boot/dts/aspeed/aspeed-bmc-quanta-s6q.dts | 4 ++-- > 7 files changed, 14 insertions(+), 30 deletions(-) > I see you've applied this to your tree. Sorry for being slow on the up- take, I've been focusing on other things recently. Happy to take it though, if you're happy to drop it? Andrew