On 8/28/21 6:00 PM, Peter Delevoryas wrote:
> Actually yes! I should have included a link to it:
> https://github.com/facebook/openbmc-uboot/blob/openbmc/helium/v2019.04/arch/arm/dts/aspeed-bmc-facebook-fuji.dts
(On technical lists, it's best to avoid top-posting, and to
instead reply inline to make the conversation easier to follow).
Can you add a test for your board please? See examples
in tests/acceptance/boot_linux_console.py:
- test_arm_ast2500_romulus_openbmc_v2_9_0
- test_arm_ast2600_debian
> *From: *Cédric Le Goater <c...@kaod.org>
> *Date: *Saturday, August 28, 2021 at 1:28 AM
> *To: *Peter Delevoryas <p...@fb.com>
> *Cc: *j...@jms.id.au <j...@jms.id.au>, qemu-devel@nongnu.org
> <qemu-devel@nongnu.org>, qemu-...@nongnu.org <qemu-...@nongnu.org>
> *Subject: *Re: [PATCH 3/5] hw/arm/aspeed: Add fuji machine type
>
> On 8/27/21 11:04 PM, p...@fb.com wrote:
>> From: Peter Delevoryas <p...@fb.com>
>>
>> Fuji uses the AST2600, so it's very similar to `ast2600-evb`, but it has
>> a few slight differences, such as using UART1 for the serial console.
>
> Do we have a public DTS for this board ?
>
> Thanks,
>
> C.
>
>>
>> Signed-off-by: Peter Delevoryas <p...@fb.com>
>> ---
>> hw/arm/aspeed.c | 13 +++++++++++++
>> 1 file changed, 13 insertions(+)
>>
>> diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
>> index ff53d12395..d2eb516a1d 100644
>> --- a/hw/arm/aspeed.c
>> +++ b/hw/arm/aspeed.c
>> @@ -1029,6 +1029,15 @@ static void
> aspeed_machine_rainier_class_init(ObjectClass *oc, void *data)
>> aspeed_soc_num_cpus(amc->soc_name);
>> };
>>
>> +static void aspeed_machine_fuji_class_init(ObjectClass *oc, void *data)
>> +{
>> + MachineClass *mc = MACHINE_CLASS(oc);
>> + AspeedMachineClass *amc = ASPEED_MACHINE_CLASS(oc);
>> +
>> + mc->desc = "Facebook Fuji BMC (Aspeed AST2600, Cortex-A7)";
>> + amc->serial_dev = ASPEED_DEV_UART1;
>> +}
>> +
>> static const TypeInfo aspeed_machine_types[] = {
>> {
>> .name = MACHINE_TYPE_NAME("palmetto-bmc"),
>> @@ -1078,6 +1087,10 @@ static const TypeInfo aspeed_machine_types[] = {
>> .name = MACHINE_TYPE_NAME("rainier-bmc"),
>> .parent = TYPE_ASPEED_MACHINE,
>> .class_init = aspeed_machine_rainier_class_init,
>> + }, {
>> + .name = MACHINE_TYPE_NAME("fuji"),
>> + .parent = MACHINE_TYPE_NAME("ast2600-evb"),
>> + .class_init = aspeed_machine_fuji_class_init,
>> }, {
>> .name = TYPE_ASPEED_MACHINE,
>> .parent = TYPE_MACHINE,
>>
>
