[PATCH v1 0/5] Add IBM Huygens BMC machine for AST2700

Mikail Sadic posted 5 patches 1 week, 3 days ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260715155037.2237011-1-mikail.sadic@ibm.com
Maintainers: "Cédric Le Goater" <clg@kaod.org>, Peter Maydell <peter.maydell@linaro.org>, Steven Lee <steven_lee@aspeedtech.com>, Troy Lee <leetroy@gmail.com>, Jamin Lin <jamin_lin@aspeedtech.com>, Kane Chen <kane_chen@aspeedtech.com>, Andrew Jeffery <andrew@codeconstruct.com.au>, Joel Stanley <joel@jms.id.au>, Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>, Ninad Palsule <ninad@linux.ibm.com>, Paolo Bonzini <pbonzini@redhat.com>, "Philippe Mathieu-Daudé" <philmd@mailo.com>, Titus Rwantare <titusr@google.com>, Jeuk Kim <jeuk20.kim@samsung.com>
MAINTAINERS                                   |    2 +
docs/specs/aspeed-ufs.rst                     |   69 ++
docs/specs/fsi.rst                            |   58 +
docs/specs/index.rst                          |    2 +
docs/specs/ucd90320.rst                       |   36 +
docs/system/arm/aspeed.rst                    |   41 +-
include/hw/arm/aspeed_soc.h                   |    3 +
include/hw/fsi/cfam-s.h                       |   26 +
include/hw/fsi/fsi-master.h                   |    2 +
include/hw/ufs/aspeed_ufs.h                   |   66 +
hw/arm/aspeed.c                               |   12 +
hw/arm/aspeed_ast27x0.c                       |   31 +
hw/arm/aspeed_ast27x0_huygens.c               |  242 ++++
hw/fsi/cfam-s.c                               |  145 +++
hw/fsi/fsi-master.c                           |   10 +-
hw/i2c/aspeed_i2c.c                           |    7 +
hw/sensor/ucd90320.c                          |  144 +++
hw/ufs/aspeed_ufs.c                           | 1095 +++++++++++++++++
hw/arm/Kconfig                                |    1 +
hw/arm/meson.build                            |    1 +
hw/fsi/meson.build                            |    2 +-
hw/sensor/Kconfig                             |    4 +
hw/sensor/meson.build                         |    1 +
hw/ufs/meson.build                            |    1 +
tests/functional/aarch64/meson.build          |    2 +
.../functional/aarch64/test_aspeed_huygens.py |   49 +
26 files changed, 2047 insertions(+), 5 deletions(-)
create mode 100644 docs/specs/aspeed-ufs.rst
create mode 100644 docs/specs/ucd90320.rst
create mode 100644 include/hw/fsi/cfam-s.h
create mode 100644 include/hw/ufs/aspeed_ufs.h
create mode 100644 hw/arm/aspeed_ast27x0_huygens.c
create mode 100644 hw/fsi/cfam-s.c
create mode 100644 hw/sensor/ucd90320.c
create mode 100644 hw/ufs/aspeed_ufs.c
create mode 100755 tests/functional/aarch64/test_aspeed_huygens.py
[PATCH v1 0/5] Add IBM Huygens BMC machine for AST2700
Posted by Mikail Sadic 1 week, 3 days ago
This series adds initial support for the IBM Huygens BMC platform based
on the ASPEED AST2700 A2 SoC.

The series introduces a new Huygens machine model, adds a dedicated
CFAM-S implementation for the AST2700 FSI responder framework, provides
a PMBus model for the TI UCD90320 power sequencer, and implements the
AST2700 UFS host controller required for booting from UFS storage.
It also includes an initial functional test covering the Huygens boot 
flow.

Patch summary:

1. Add the AST2700 Huygens machine.
2. Add the CFAM-S model for AST2700.
3. Add the TI UCD90320 PMBus device model.
4. Add the AST2700 UFS host controller.
5. Add an initial Huygens functional boot test.

Testing done:

* Booted OpenBMC using UFS storage.
* Verified OpenBMC reaches the login prompt & Active/Ready state.
* Verified `make check` test cases all pass.

Note: checkpatch has some warns about MAINTAINERS for patches 1, 2, 
and 5, but the new files should already be covered by existing 
wildcard entries in the ASPEED BMCs and FSI sections.

Mikail Sadic (5):
  arm/aspeed: Add AST2700 Huygens machine
  fsi/cfam: Add CFAM-S model for AST2700
  hw/sensor: Add UCD90320 model
  ufs/aspeed: Add AST2700 UFS host controller
  tests/functional: Add Huygens BMC boot test

 MAINTAINERS                                   |    2 +
 docs/specs/aspeed-ufs.rst                     |   69 ++
 docs/specs/fsi.rst                            |   58 +
 docs/specs/index.rst                          |    2 +
 docs/specs/ucd90320.rst                       |   36 +
 docs/system/arm/aspeed.rst                    |   41 +-
 include/hw/arm/aspeed_soc.h                   |    3 +
 include/hw/fsi/cfam-s.h                       |   26 +
 include/hw/fsi/fsi-master.h                   |    2 +
 include/hw/ufs/aspeed_ufs.h                   |   66 +
 hw/arm/aspeed.c                               |   12 +
 hw/arm/aspeed_ast27x0.c                       |   31 +
 hw/arm/aspeed_ast27x0_huygens.c               |  242 ++++
 hw/fsi/cfam-s.c                               |  145 +++
 hw/fsi/fsi-master.c                           |   10 +-
 hw/i2c/aspeed_i2c.c                           |    7 +
 hw/sensor/ucd90320.c                          |  144 +++
 hw/ufs/aspeed_ufs.c                           | 1095 +++++++++++++++++
 hw/arm/Kconfig                                |    1 +
 hw/arm/meson.build                            |    1 +
 hw/fsi/meson.build                            |    2 +-
 hw/sensor/Kconfig                             |    4 +
 hw/sensor/meson.build                         |    1 +
 hw/ufs/meson.build                            |    1 +
 tests/functional/aarch64/meson.build          |    2 +
 .../functional/aarch64/test_aspeed_huygens.py |   49 +
 26 files changed, 2047 insertions(+), 5 deletions(-)
 create mode 100644 docs/specs/aspeed-ufs.rst
 create mode 100644 docs/specs/ucd90320.rst
 create mode 100644 include/hw/fsi/cfam-s.h
 create mode 100644 include/hw/ufs/aspeed_ufs.h
 create mode 100644 hw/arm/aspeed_ast27x0_huygens.c
 create mode 100644 hw/fsi/cfam-s.c
 create mode 100644 hw/sensor/ucd90320.c
 create mode 100644 hw/ufs/aspeed_ufs.c
 create mode 100755 tests/functional/aarch64/test_aspeed_huygens.py

-- 
2.53.0
Re: [PATCH v1 0/5] Add IBM Huygens BMC machine for AST2700
Posted by Glenn Miles 1 week, 3 days ago
Hi Mikail,

Could you use my milesg@linux.ibm.com email address for patches please?

Thanks,

Glenn
________________________________
From: Mikail Sadic <mikail.sadic@ibm.com>
Sent: Wednesday, July 15, 2026 10:50 AM
To: clg@kaod.org <clg@kaod.org>; peter.maydell@linaro.org <peter.maydell@linaro.org>
Cc: Mikail Sadic <Mikail.Sadic@ibm.com>; Paolo Bonzini <pbonzini@redhat.com>; ninad@linux.ibm.com <ninad@linux.ibm.com>; titusr@google.com <titusr@google.com>; jeuk20.kim@samsung.com <jeuk20.kim@samsung.com>; philmd@mailo.com <philmd@mailo.com>; steven_lee@aspeedtech.com <steven_lee@aspeedtech.com>; leetroy@gmail.com <leetroy@gmail.com>; jamin_lin@aspeedtech.com <jamin_lin@aspeedtech.com>; kane_chen@aspeedtech.com <kane_chen@aspeedtech.com>; andrew@codeconstruct.com.au <andrew@codeconstruct.com.au>; joel@jms.id.au <joel@jms.id.au>; CALEB SCHLOSSIN <calebs@us.ibm.com>; Glenn Miles <milesg@ibm.com>; qemu-arm@nongnu.org <qemu-arm@nongnu.org>; qemu-devel@nongnu.org <qemu-devel@nongnu.org>
Subject: [PATCH v1 0/5] Add IBM Huygens BMC machine for AST2700

This series adds initial support for the IBM Huygens BMC platform based
on the ASPEED AST2700 A2 SoC.

The series introduces a new Huygens machine model, adds a dedicated
CFAM-S implementation for the AST2700 FSI responder framework, provides
a PMBus model for the TI UCD90320 power sequencer, and implements the
AST2700 UFS host controller required for booting from UFS storage.
It also includes an initial functional test covering the Huygens boot
flow.

Patch summary:

1. Add the AST2700 Huygens machine.
2. Add the CFAM-S model for AST2700.
3. Add the TI UCD90320 PMBus device model.
4. Add the AST2700 UFS host controller.
5. Add an initial Huygens functional boot test.

Testing done:

* Booted OpenBMC using UFS storage.
* Verified OpenBMC reaches the login prompt & Active/Ready state.
* Verified `make check` test cases all pass.

Note: checkpatch has some warns about MAINTAINERS for patches 1, 2,
and 5, but the new files should already be covered by existing
wildcard entries in the ASPEED BMCs and FSI sections.

Mikail Sadic (5):
  arm/aspeed: Add AST2700 Huygens machine
  fsi/cfam: Add CFAM-S model for AST2700
  hw/sensor: Add UCD90320 model
  ufs/aspeed: Add AST2700 UFS host controller
  tests/functional: Add Huygens BMC boot test

 MAINTAINERS                                   |    2 +
 docs/specs/aspeed-ufs.rst                     |   69 ++
 docs/specs/fsi.rst                            |   58 +
 docs/specs/index.rst                          |    2 +
 docs/specs/ucd90320.rst                       |   36 +
 docs/system/arm/aspeed.rst                    |   41 +-
 include/hw/arm/aspeed_soc.h                   |    3 +
 include/hw/fsi/cfam-s.h                       |   26 +
 include/hw/fsi/fsi-master.h                   |    2 +
 include/hw/ufs/aspeed_ufs.h                   |   66 +
 hw/arm/aspeed.c                               |   12 +
 hw/arm/aspeed_ast27x0.c                       |   31 +
 hw/arm/aspeed_ast27x0_huygens.c               |  242 ++++
 hw/fsi/cfam-s.c                               |  145 +++
 hw/fsi/fsi-master.c                           |   10 +-
 hw/i2c/aspeed_i2c.c                           |    7 +
 hw/sensor/ucd90320.c                          |  144 +++
 hw/ufs/aspeed_ufs.c                           | 1095 +++++++++++++++++
 hw/arm/Kconfig                                |    1 +
 hw/arm/meson.build                            |    1 +
 hw/fsi/meson.build                            |    2 +-
 hw/sensor/Kconfig                             |    4 +
 hw/sensor/meson.build                         |    1 +
 hw/ufs/meson.build                            |    1 +
 tests/functional/aarch64/meson.build          |    2 +
 .../functional/aarch64/test_aspeed_huygens.py |   49 +
 26 files changed, 2047 insertions(+), 5 deletions(-)
 create mode 100644 docs/specs/aspeed-ufs.rst
 create mode 100644 docs/specs/ucd90320.rst
 create mode 100644 include/hw/fsi/cfam-s.h
 create mode 100644 include/hw/ufs/aspeed_ufs.h
 create mode 100644 hw/arm/aspeed_ast27x0_huygens.c
 create mode 100644 hw/fsi/cfam-s.c
 create mode 100644 hw/sensor/ucd90320.c
 create mode 100644 hw/ufs/aspeed_ufs.c
 create mode 100755 tests/functional/aarch64/test_aspeed_huygens.py

--
2.53.0

Re: [PATCH v1 0/5] Add IBM Huygens BMC machine for AST2700
Posted by CALEB SCHLOSSIN 1 week, 2 days ago
Mikail,

Same for me, please use calebs@linux.ibm.com.

Thanks,
Caleb Schlossin
----------------------------------------
Power Simulation Architect
IBM Rochester, MN
----------------------------------------
________________________________
From: Glenn Miles <milesg@ibm.com>
Sent: Wednesday, July 15, 2026 11:01 AM
To: Mikail Sadic <Mikail.Sadic@ibm.com>; clg@kaod.org <clg@kaod.org>; peter.maydell@linaro.org <peter.maydell@linaro.org>
Cc: Paolo Bonzini <pbonzini@redhat.com>; ninad@linux.ibm.com <ninad@linux.ibm.com>; titusr@google.com <titusr@google.com>; jeuk20.kim@samsung.com <jeuk20.kim@samsung.com>; philmd@mailo.com <philmd@mailo.com>; steven_lee@aspeedtech.com <steven_lee@aspeedtech.com>; leetroy@gmail.com <leetroy@gmail.com>; jamin_lin@aspeedtech.com <jamin_lin@aspeedtech.com>; kane_chen@aspeedtech.com <kane_chen@aspeedtech.com>; andrew@codeconstruct.com.au <andrew@codeconstruct.com.au>; joel@jms.id.au <joel@jms.id.au>; CALEB SCHLOSSIN <calebs@us.ibm.com>; qemu-arm@nongnu.org <qemu-arm@nongnu.org>; qemu-devel@nongnu.org <qemu-devel@nongnu.org>
Subject: Re: [PATCH v1 0/5] Add IBM Huygens BMC machine for AST2700

Hi Mikail,

Could you use my milesg@linux.ibm.com email address for patches please?

Thanks,

Glenn
________________________________
From: Mikail Sadic <mikail.sadic@ibm.com>
Sent: Wednesday, July 15, 2026 10:50 AM
To: clg@kaod.org <clg@kaod.org>; peter.maydell@linaro.org <peter.maydell@linaro.org>
Cc: Mikail Sadic <Mikail.Sadic@ibm.com>; Paolo Bonzini <pbonzini@redhat.com>; ninad@linux.ibm.com <ninad@linux.ibm.com>; titusr@google.com <titusr@google.com>; jeuk20.kim@samsung.com <jeuk20.kim@samsung.com>; philmd@mailo.com <philmd@mailo.com>; steven_lee@aspeedtech.com <steven_lee@aspeedtech.com>; leetroy@gmail.com <leetroy@gmail.com>; jamin_lin@aspeedtech.com <jamin_lin@aspeedtech.com>; kane_chen@aspeedtech.com <kane_chen@aspeedtech.com>; andrew@codeconstruct.com.au <andrew@codeconstruct.com.au>; joel@jms.id.au <joel@jms.id.au>; CALEB SCHLOSSIN <calebs@us.ibm.com>; Glenn Miles <milesg@ibm.com>; qemu-arm@nongnu.org <qemu-arm@nongnu.org>; qemu-devel@nongnu.org <qemu-devel@nongnu.org>
Subject: [PATCH v1 0/5] Add IBM Huygens BMC machine for AST2700

This series adds initial support for the IBM Huygens BMC platform based
on the ASPEED AST2700 A2 SoC.

The series introduces a new Huygens machine model, adds a dedicated
CFAM-S implementation for the AST2700 FSI responder framework, provides
a PMBus model for the TI UCD90320 power sequencer, and implements the
AST2700 UFS host controller required for booting from UFS storage.
It also includes an initial functional test covering the Huygens boot
flow.

Patch summary:

1. Add the AST2700 Huygens machine.
2. Add the CFAM-S model for AST2700.
3. Add the TI UCD90320 PMBus device model.
4. Add the AST2700 UFS host controller.
5. Add an initial Huygens functional boot test.

Testing done:

* Booted OpenBMC using UFS storage.
* Verified OpenBMC reaches the login prompt & Active/Ready state.
* Verified `make check` test cases all pass.

Note: checkpatch has some warns about MAINTAINERS for patches 1, 2,
and 5, but the new files should already be covered by existing
wildcard entries in the ASPEED BMCs and FSI sections.

Mikail Sadic (5):
  arm/aspeed: Add AST2700 Huygens machine
  fsi/cfam: Add CFAM-S model for AST2700
  hw/sensor: Add UCD90320 model
  ufs/aspeed: Add AST2700 UFS host controller
  tests/functional: Add Huygens BMC boot test

 MAINTAINERS                                   |    2 +
 docs/specs/aspeed-ufs.rst                     |   69 ++
 docs/specs/fsi.rst                            |   58 +
 docs/specs/index.rst                          |    2 +
 docs/specs/ucd90320.rst                       |   36 +
 docs/system/arm/aspeed.rst                    |   41 +-
 include/hw/arm/aspeed_soc.h                   |    3 +
 include/hw/fsi/cfam-s.h                       |   26 +
 include/hw/fsi/fsi-master.h                   |    2 +
 include/hw/ufs/aspeed_ufs.h                   |   66 +
 hw/arm/aspeed.c                               |   12 +
 hw/arm/aspeed_ast27x0.c                       |   31 +
 hw/arm/aspeed_ast27x0_huygens.c               |  242 ++++
 hw/fsi/cfam-s.c                               |  145 +++
 hw/fsi/fsi-master.c                           |   10 +-
 hw/i2c/aspeed_i2c.c                           |    7 +
 hw/sensor/ucd90320.c                          |  144 +++
 hw/ufs/aspeed_ufs.c                           | 1095 +++++++++++++++++
 hw/arm/Kconfig                                |    1 +
 hw/arm/meson.build                            |    1 +
 hw/fsi/meson.build                            |    2 +-
 hw/sensor/Kconfig                             |    4 +
 hw/sensor/meson.build                         |    1 +
 hw/ufs/meson.build                            |    1 +
 tests/functional/aarch64/meson.build          |    2 +
 .../functional/aarch64/test_aspeed_huygens.py |   49 +
 26 files changed, 2047 insertions(+), 5 deletions(-)
 create mode 100644 docs/specs/aspeed-ufs.rst
 create mode 100644 docs/specs/ucd90320.rst
 create mode 100644 include/hw/fsi/cfam-s.h
 create mode 100644 include/hw/ufs/aspeed_ufs.h
 create mode 100644 hw/arm/aspeed_ast27x0_huygens.c
 create mode 100644 hw/fsi/cfam-s.c
 create mode 100644 hw/sensor/ucd90320.c
 create mode 100644 hw/ufs/aspeed_ufs.c
 create mode 100755 tests/functional/aarch64/test_aspeed_huygens.py

--
2.53.0