[v1] Add support for gb200-bmc machine

[PATCH 0/4] Add support for gb200-bmc machine

Ed Tanous posted 4 patches 4 months, 2 weeks ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20250701203400.71677-1-etanous@nvidia.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>, Andrew Jeffery <andrew@codeconstruct.com.au>, Joel Stanley <joel@jms.id.au>, Paolo Bonzini <pbonzini@redhat.com>

There is a newer version of this series

docs/system/arm/aspeed.rst |  4 +-
hw/arm/Kconfig             |  1 +
hw/arm/aspeed.c            | 81 ++++++++++++++++++++++++++++++++++++++
hw/arm/aspeed_eeprom.c     | 21 ++++++++++
hw/arm/aspeed_eeprom.h     |  3 ++
include/hw/arm/aspeed.h    |  2 +
6 files changed, 110 insertions(+), 2 deletions(-)

Expand all Fold all

[PATCH 0/4] Add support for gb200-bmc machine

Posted by Ed Tanous 4 months, 2 weeks ago

This patch series adds support for gb200-bmc, a baseboard management controller
module based on an Aspeed 2600 SOC.

Ed Tanous (4):
  hw/arm: Add PCA9554 to ARM target
  hw/arm/aspeed: Add second SPI chip to Aspeed model
  docs: add support for gb200-bmc
  hw/arm/aspeed: Add GB200 BMC target

 docs/system/arm/aspeed.rst |  4 +-
 hw/arm/Kconfig             |  1 +
 hw/arm/aspeed.c            | 81 ++++++++++++++++++++++++++++++++++++++
 hw/arm/aspeed_eeprom.c     | 21 ++++++++++
 hw/arm/aspeed_eeprom.h     |  3 ++
 include/hw/arm/aspeed.h    |  2 +
 6 files changed, 110 insertions(+), 2 deletions(-)

-- 
2.43.0

Re: [PATCH 0/4] Add support for gb200-bmc machine

Posted by Cédric Le Goater 4 months, 2 weeks ago

On 7/1/25 22:33, Ed Tanous wrote:
> This patch series adds support for gb200-bmc, a baseboard management controller
> module based on an Aspeed 2600 SOC.
> 
> Ed Tanous (4):
>    hw/arm: Add PCA9554 to ARM target
>    hw/arm/aspeed: Add second SPI chip to Aspeed model
>    docs: add support for gb200-bmc
>    hw/arm/aspeed: Add GB200 BMC target
> 
>   docs/system/arm/aspeed.rst |  4 +-
>   hw/arm/Kconfig             |  1 +
>   hw/arm/aspeed.c            | 81 ++++++++++++++++++++++++++++++++++++++
>   hw/arm/aspeed_eeprom.c     | 21 ++++++++++
>   hw/arm/aspeed_eeprom.h     |  3 ++
>   include/hw/arm/aspeed.h    |  2 +
>   6 files changed, 110 insertions(+), 2 deletions(-)
> 

Could you provide a functional test for the gb200nvl-bmc machine too ?
See tests/functional/test_*aspeed* files.

Thanks,

C.

Re: [PATCH 0/4] Add support for gb200-bmc machine

Posted by etanous via 4 months, 2 weeks ago

On Wed, Jul 02, 2025 at 09:00:53AM +0200, Cédric Le Goater wrote:
> External email: Use caution opening links or attachments
> 
> 
> On 7/1/25 22:33, Ed Tanous wrote:
> > This patch series adds support for gb200-bmc, a baseboard management controller
> > module based on an Aspeed 2600 SOC.
> > 
> > Ed Tanous (4):
> >    hw/arm: Add PCA9554 to ARM target
> >    hw/arm/aspeed: Add second SPI chip to Aspeed model
> >    docs: add support for gb200-bmc
> >    hw/arm/aspeed: Add GB200 BMC target
> > 
> >   docs/system/arm/aspeed.rst |  4 +-
> >   hw/arm/Kconfig             |  1 +
> >   hw/arm/aspeed.c            | 81 ++++++++++++++++++++++++++++++++++++++
> >   hw/arm/aspeed_eeprom.c     | 21 ++++++++++
> >   hw/arm/aspeed_eeprom.h     |  3 ++
> >   include/hw/arm/aspeed.h    |  2 +
> >   6 files changed, 110 insertions(+), 2 deletions(-)
> > 
> 
> Could you provide a functional test for the gb200nvl-bmc machine too ?
> See tests/functional/test_*aspeed* files.
> 

Can do.  It looks like images are pushed to
https://github.com/legoater/qemu-aspeed-boot and accepted via Github PR?
Will open PR shortly.

> Thanks,
> 
> C.
> 
>

Re: [PATCH 0/4] Add support for gb200-bmc machine

Posted by Cédric Le Goater 4 months, 2 weeks ago

On 7/2/25 21:31, etanous wrote:
> On Wed, Jul 02, 2025 at 09:00:53AM +0200, Cédric Le Goater wrote:
>> External email: Use caution opening links or attachments
>>
>>
>> On 7/1/25 22:33, Ed Tanous wrote:
>>> This patch series adds support for gb200-bmc, a baseboard management controller
>>> module based on an Aspeed 2600 SOC.
>>>
>>> Ed Tanous (4):
>>>     hw/arm: Add PCA9554 to ARM target
>>>     hw/arm/aspeed: Add second SPI chip to Aspeed model
>>>     docs: add support for gb200-bmc
>>>     hw/arm/aspeed: Add GB200 BMC target
>>>
>>>    docs/system/arm/aspeed.rst |  4 +-
>>>    hw/arm/Kconfig             |  1 +
>>>    hw/arm/aspeed.c            | 81 ++++++++++++++++++++++++++++++++++++++
>>>    hw/arm/aspeed_eeprom.c     | 21 ++++++++++
>>>    hw/arm/aspeed_eeprom.h     |  3 ++
>>>    include/hw/arm/aspeed.h    |  2 +
>>>    6 files changed, 110 insertions(+), 2 deletions(-)
>>>
>>
>> Could you provide a functional test for the gb200nvl-bmc machine too ?
>> See tests/functional/test_*aspeed* files.
>>
> 
> Can do.  It looks like images are pushed to
> https://github.com/legoater/qemu-aspeed-boot and accepted via Github PR?

Yes. please do that. It seems the easiest way to collect FW images.

Please provide some info on how it was built, openbmc version, repo, etc.

Thanks,

C.