Received-SPF: pass (google.com: domain of linux-kernel+bounces-203500-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <b40486cf90de44bad177f034d13d8ab69ad9aac8.camel@codeconstruct.com.au>
Subject: Re: [PATCH v6 00/20] ARM: dts: aspeed: Add IBM P11 BMC systems
From: Andrew Jeffery <andrew@codeconstruct.com.au>
To: Eddie James <eajames@linux.ibm.com>, linux-fsi@lists.ozlabs.org
Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, 
 ninad@linux.ibm.com, lakshmiy@us.ibm.com, linux-i2c@vger.kernel.org, 
 linux-spi@vger.kernel.org, linux-aspeed@lists.ozlabs.org, joel@jms.id.au, 
 robh@kernel.org, conor+dt@kernel.org, krzk+dt@kernel.org,
 andi.shyti@kernel.org,  broonie@kernel.org
Date: Thu, 06 Jun 2024 11:01:41 +0930
In-Reply-To: <f2f70d62-3edb-4273-b40e-430d789f19dc@linux.ibm.com>
References: <20240522192524.3286237-1-eajames@linux.ibm.com>
	 <f2f70d62-3edb-4273-b40e-430d789f19dc@linux.ibm.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.46.4-2 
Precedence: bulk
MIME-Version: 1.0

Hi Eddie,

On Wed, 2024-06-05 at 10:47 -0500, Eddie James wrote:
> On 5/22/24 14:25, Eddie James wrote:
> > Add the Blueridge and Fuji BMC systems. Document many missing FSI relat=
ed
> > properties, and fix existing warnings. Make some minor fixes in OCC and
> > SCOM drivers for the updated bindings.
>=20
>=20
> Hi Joel/Andrew, what else needs to be fixed before this can be merged=20
> (minus Huygens patch which I will resend)? I believe all the patches=20
> have been reviewed.
>=20

Firstly, thanks for your work here on the FSI bindings.

However, the series is a bit awkward, as it sandwiches Aspeed
devicetree patches that should go through Joel's bmc tree between the
bindings and driver fixes that should go through the FSI tree.

This is potentially less of a problem for Joel as he's the maintainer
for both, but it's not my place to be touching the FSI tree.=20

For now I've applied the dts patches and pushed them here after
dropping the Huygens patch:

https://github.com/amboar/linux/commits/for/bmc/dt-6.11/

But I would appreciate it if you split series by subsystem in the
future (see my comments on the other FSI series you have out for
review).

Andrew

>=20
> Thanks,
>=20
> Eddie
>=20
>=20
> >=20
> > Changes since v5:
> >   - Switch from clock-frequency to bus-frequency for common FSI control=
ler
> >     properties
> >   - Add reg properties for AST2700 FSI controller
> >   - Fix patternProperties for i2c bus nodes under FSI-based I2C control=
ler
> >   - Add bus-frequency for P11 FSI device tree node
> >   - Change model name from Blueridge to Blueridge 2U
> >   - Add missing reset gpio to led controller on Fuji
> >   - Add Huygens (Rainier with modified FSI wiring)
> >=20
> > Eddie James (20):
> >    spi: dt-bindings: Document the IBM FSI-attached SPI controller
> >    dt-bindings: fsi: fsi2spi: Document SPI controller child nodes
> >    dt-bindings: fsi: Document the IBM SCOM engine
> >    dt-bindings: fsi: p9-occ: Convert to json-schema
> >    dt-bindings: fsi: Document the IBM SBEFIFO engine
> >    dt-bindings: fsi: Document the FSI controller common properties
> >    dt-bindings: fsi: ibm,i2cr-fsi-master: Reference common FSI controll=
er
> >    dt-bindings: fsi: ast2600-fsi-master: Convert to json-schema
> >    dt-bindings: fsi: Document the AST2700 FSI controller
> >    dt-bindings: fsi: Document the FSI Hub Controller
> >    dt-bindings: i2c: i2c-fsi: Convert to json-schema
> >    dt-bindings: arm: aspeed: add IBM P11 BMC boards
> >    ARM: dts: aspeed: Add IBM P11 FSI devices
> >    ARM: dts: aspeed: Add IBM P11 Blueridge BMC system
> >    ARM: dts: aspeed: Add IBM P11 Blueridge 4U BMC system
> >    ARM: dts: aspeed: Add IBM P11 Fuji BMC system
> >    ARM: dts: aspeed: Add IBM Huygens BMC system
> >    fsi: occ: Get device number from FSI minor number API
> >    fsi: occ: Find next available child rather than node name match
> >    fsi: scom: Update compatible string to match documentation
> >=20
> >   .../bindings/arm/aspeed/aspeed.yaml           |    2 +
> >   .../fsi/aspeed,ast2600-fsi-master.yaml        |  121 +
> >   .../bindings/fsi/fsi-controller.yaml          |   66 +
> >   .../bindings/fsi/fsi-master-aspeed.txt        |   36 -
> >   .../devicetree/bindings/fsi/ibm,fsi2spi.yaml  |   36 +-
> >   .../bindings/fsi/ibm,i2cr-fsi-master.yaml     |    5 +-
> >   .../bindings/fsi/ibm,p9-fsi-controller.yaml   |   45 +
> >   .../devicetree/bindings/fsi/ibm,p9-occ.txt    |   16 -
> >   .../devicetree/bindings/fsi/ibm,p9-occ.yaml   |   40 +
> >   .../bindings/fsi/ibm,p9-sbefifo.yaml          |   46 +
> >   .../devicetree/bindings/fsi/ibm,p9-scom.yaml  |   37 +
> >   .../devicetree/bindings/i2c/i2c-fsi.txt       |   40 -
> >   .../devicetree/bindings/i2c/ibm,i2c-fsi.yaml  |   76 +
> >   .../devicetree/bindings/spi/ibm,spi-fsi.yaml  |   55 +
> >   MAINTAINERS                                   |    2 +-
> >   arch/arm/boot/dts/aspeed/Makefile             |    3 +
> >   .../aspeed/aspeed-bmc-ibm-blueridge-4u.dts    |   21 +
> >   .../dts/aspeed/aspeed-bmc-ibm-blueridge.dts   | 1691 +++++++
> >   .../boot/dts/aspeed/aspeed-bmc-ibm-fuji.dts   | 3881 ++++++++++++++++=
+
> >   .../dts/aspeed/aspeed-bmc-ibm-huygens.dts     |   23 +
> >   .../arm/boot/dts/aspeed/ibm-power11-quad.dtsi | 1539 +++++++
> >   drivers/fsi/fsi-occ.c                         |   49 +-
> >   drivers/fsi/fsi-scom.c                        |    1 +
> >   23 files changed, 7694 insertions(+), 137 deletions(-)
> >   create mode 100644 Documentation/devicetree/bindings/fsi/aspeed,ast26=
00-fsi-master.yaml
> >   create mode 100644 Documentation/devicetree/bindings/fsi/fsi-controll=
er.yaml
> >   delete mode 100644 Documentation/devicetree/bindings/fsi/fsi-master-a=
speed.txt
> >   create mode 100644 Documentation/devicetree/bindings/fsi/ibm,p9-fsi-c=
ontroller.yaml
> >   delete mode 100644 Documentation/devicetree/bindings/fsi/ibm,p9-occ.t=
xt
> >   create mode 100644 Documentation/devicetree/bindings/fsi/ibm,p9-occ.y=
aml
> >   create mode 100644 Documentation/devicetree/bindings/fsi/ibm,p9-sbefi=
fo.yaml
> >   create mode 100644 Documentation/devicetree/bindings/fsi/ibm,p9-scom.=
yaml
> >   delete mode 100644 Documentation/devicetree/bindings/i2c/i2c-fsi.txt
> >   create mode 100644 Documentation/devicetree/bindings/i2c/ibm,i2c-fsi.=
yaml
> >   create mode 100644 Documentation/devicetree/bindings/spi/ibm,spi-fsi.=
yaml
> >   create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-blueridge-=
4u.dts
> >   create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-blueridge.=
dts
> >   create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-fuji.dts
> >   create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-huygens.dt=
s
> >   create mode 100644 arch/arm/boot/dts/aspeed/ibm-power11-quad.dtsi
> >=20