Re: [PATCH v2 0/3] ARM: dts: aspeed: Add Balcones system

[Andrew Jeffery <andrew@codeconstruct.com.au>]

On Fri, 2025-02-28 at 11:23 -0600, Rob Herring wrote:
> On Fri, Feb 28, 2025 at 10:46 AM Eddie James <eajames@linux.ibm.com>
> wrote:
> > 
> > 
> > On 2/24/25 22:34, Andrew Jeffery wrote:
> > > On Fri, 2025-02-21 at 12:11 -0600, Rob Herring (Arm) wrote:
> > > > On Thu, 20 Feb 2025 14:59:45 -0600, Eddie James wrote:
> > > > > The Balcones system is similar to Bonnell but with a POWER11
> > > > > processor.
> > > > > 
> > > > > Changes since v1:
> > > > >   - Add all the ucd9000 driver supported compatible strings
> > > > >   - Fix node ordering in Balcones device tree
> > > > >   - Improve commit message to explain addition of ibm-
> > > > > power11-dual.dtsi
> > > > > 
> > > > > Eddie James (3):
> > > > >    dt-bindings: arm: aspeed: add IBM Balcones board
> > > > >    dt-bindings: hwmon: ucd90320: Add additional compatible
> > > > > strings
> > > > >    ARM: dts: aspeed: Add Balcones system
> > > > > 
> > > > >   .../bindings/arm/aspeed/aspeed.yaml           |   1 +
> > > > >   .../bindings/hwmon/pmbus/ti,ucd90320.yaml     |   6 +
> > > > >   arch/arm/boot/dts/aspeed/Makefile             |   1 +
> > > > >   .../dts/aspeed/aspeed-bmc-ibm-balcones.dts    | 594
> > > > > +++++++++++++
> > > > >   .../arm/boot/dts/aspeed/ibm-power11-dual.dtsi | 779
> > > > > ++++++++++++++++++
> > > > >   .../arm/boot/dts/aspeed/ibm-power11-quad.dtsi | 769 +------
> > > > > ----------
> > > > >   6 files changed, 1383 insertions(+), 767 deletions(-)
> > > > >   create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-
> > > > > balcones.dts
> > > > >   create mode 100644 arch/arm/boot/dts/aspeed/ibm-power11-
> > > > > dual.dtsi
> > > > > 
> > > > > --
> > > > > 2.43.5
> > > > > 
> > > > > 
> > > > > 
> > > > 
> > > > My bot found new DTB warnings on the .dts files added or
> > > > changed in this
> > > > series.
> > > > 
> > > ...
> > > 
> > > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb:
> > > > gpio@1e780000: 'usb-power-hog' does not match any of the
> > > > regexes: 'pinctrl-[0-9]+'
> > > >          from schema $id:
> > > > http://devicetree.org/schemas/gpio/aspeed,ast2400-gpio.yaml#
> > > Ah, can you address this one?
> > > 
> > > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb:
> > > > /ahb/apb/bus@1e78a000/i2c@400/pwm@53: failed to match any
> > > > schema with compatible: ['maxim,max31785a']
> > > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dtb:
> > > > pressure-sensor@76: '#io-channel-cells' does not match any of
> > > > the regexes: 'pinctrl-[0-9]+'
> > > >          from schema $id:
> > > > http://devicetree.org/schemas/trivial-devices.yaml#
> > > And these also?
> > > 
> > > The rest looked like the usual noise.
> > 
> > 
> > Well, all of these errors are also present in Blueridge. I would
> > need to
> > write a new binding for dps310, move max31785 to yaml format, and
> > I'm
> > not sure how to fix the gpio hog one. I don't see that error when I
> > run
> > CHECK_DTBS myself... any suggestions?
> 
> I think the hog one got fixed recently. The tests run on top of
> whatever b4 picks for the base or linux-next if it can't figure out
> the base. So that might be the difference.

Eddie: Yeah, disregard the hog one. An alternative to
writing/converting the bindings is dropping the relevant nodes for now,
and adding them back once the bindings have been dealt with. Up to you.

Andrew