diff mbox series

[v3,12/16] aspeed/soc: Add AST2700 support

Message ID 20240416091904.935283-13-jamin_lin@aspeedtech.com (mailing list archive)
State New, archived
Headers show
Series Add AST2700 support | expand

Commit Message

Jamin Lin April 16, 2024, 9:18 a.m. UTC
Initial definitions for a simple machine using an AST2700 SOC (Cortex-a35 CPU).

AST2700 SOC and its interrupt controller are too complex to handle
in the common Aspeed SoC framework. We introduce a new ast2700
class with instance_init and realize handlers.

AST2700 is a 64 bits quad core cpus and support 8 watchdog.
Update maximum ASPEED_CPUS_NUM to 4 and ASPEED_WDTS_NUM to 8.
In addition, update AspeedSocState to support scuio, sli, sliio and intc.

Add TYPE_ASPEED27X0_SOC machine type.

The SDMC controller is unlocked at SPL stage.
At present, only supports to emulate booting
start from u-boot stage. Set SDMC controller
unlocked by default.

In INTC, each interrupt of INT 128 to INT 136 combines 32 interrupts.
It connect GICINT IRQ GPIO-OUTPUT pins to GIC device with irq 128 to 136.
And, if a device irq is 128 to 136, its irq GPIO-OUTPUT pin is connected to
GICINT or-gates instead of GIC device.

Signed-off-by: Troy Lee <troy_lee@aspeedtech.com>
Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
---
 hw/arm/aspeed_ast27x0.c     | 554 ++++++++++++++++++++++++++++++++++++
 hw/arm/meson.build          |   1 +
 include/hw/arm/aspeed_soc.h |  26 +-
 3 files changed, 579 insertions(+), 2 deletions(-)
 create mode 100644 hw/arm/aspeed_ast27x0.c

Comments

Cédric Le Goater April 19, 2024, 7:10 a.m. UTC | #1
On 4/16/24 11:18, Jamin Lin wrote:
> Initial definitions for a simple machine using an AST2700 SOC (Cortex-a35 CPU).
> 
> AST2700 SOC and its interrupt controller are too complex to handle
> in the common Aspeed SoC framework. We introduce a new ast2700
> class with instance_init and realize handlers.
> 
> AST2700 is a 64 bits quad core cpus and support 8 watchdog.
> Update maximum ASPEED_CPUS_NUM to 4 and ASPEED_WDTS_NUM to 8.
> In addition, update AspeedSocState to support scuio, sli, sliio and intc.
> 
> Add TYPE_ASPEED27X0_SOC machine type.
> 
> The SDMC controller is unlocked at SPL stage.
> At present, only supports to emulate booting
> start from u-boot stage. Set SDMC controller
> unlocked by default.
> 
> In INTC, each interrupt of INT 128 to INT 136 combines 32 interrupts.
> It connect GICINT IRQ GPIO-OUTPUT pins to GIC device with irq 128 to 136.
> And, if a device irq is 128 to 136, its irq GPIO-OUTPUT pin is connected to
> GICINT or-gates instead of GIC device.
> 
> Signed-off-by: Troy Lee <troy_lee@aspeedtech.com>
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>

Before I forget, please see a little comment below regarding user creatable
devices.

The model looks fine. The interrupt controller part is more complex than
the previous SoCs so I will come back to it later when I have more time.

> ---
>   hw/arm/aspeed_ast27x0.c     | 554 ++++++++++++++++++++++++++++++++++++
>   hw/arm/meson.build          |   1 +
>   include/hw/arm/aspeed_soc.h |  26 +-
>   3 files changed, 579 insertions(+), 2 deletions(-)
>   create mode 100644 hw/arm/aspeed_ast27x0.c
> 
> diff --git a/hw/arm/aspeed_ast27x0.c b/hw/arm/aspeed_ast27x0.c
> new file mode 100644
> index 0000000000..754c963230
> --- /dev/null
> +++ b/hw/arm/aspeed_ast27x0.c
> @@ -0,0 +1,554 @@
> +/*
> + * ASPEED SoC 27x0 family
> + *
> + * Copyright (C) 2024 ASPEED Technology Inc.
> + *
> + * This code is licensed under the GPL version 2 or later.  See
> + * the COPYING file in the top-level directory.
> + *
> + * Implementation extracted from the AST2600 and adapted for AST27x0.
> + */
> +
> +#include "qemu/osdep.h"
> +#include "qapi/error.h"
> +#include "hw/misc/unimp.h"
> +#include "hw/arm/aspeed_soc.h"
> +#include "qemu/module.h"
> +#include "qemu/error-report.h"
> +#include "hw/i2c/aspeed_i2c.h"
> +#include "net/net.h"
> +#include "sysemu/sysemu.h"
> +#include "hw/intc/arm_gicv3.h"
> +#include "qapi/qmp/qlist.h"
> +
> +static const hwaddr aspeed_soc_ast2700_memmap[] = {
> +    [ASPEED_DEV_SPI_BOOT]  =  0x400000000,
> +    [ASPEED_DEV_SRAM]      =  0x10000000,
> +    [ASPEED_DEV_SDMC]      =  0x12C00000,
> +    [ASPEED_DEV_SCU]       =  0x12C02000,
> +    [ASPEED_DEV_SCUIO]     =  0x14C02000,
> +    [ASPEED_DEV_UART0]     =  0X14C33000,
> +    [ASPEED_DEV_UART1]     =  0X14C33100,
> +    [ASPEED_DEV_UART2]     =  0X14C33200,
> +    [ASPEED_DEV_UART3]     =  0X14C33300,
> +    [ASPEED_DEV_UART4]     =  0X12C1A000,
> +    [ASPEED_DEV_UART5]     =  0X14C33400,
> +    [ASPEED_DEV_UART6]     =  0X14C33500,
> +    [ASPEED_DEV_UART7]     =  0X14C33600,
> +    [ASPEED_DEV_UART8]     =  0X14C33700,
> +    [ASPEED_DEV_UART9]     =  0X14C33800,
> +    [ASPEED_DEV_UART10]    =  0X14C33900,
> +    [ASPEED_DEV_UART11]    =  0X14C33A00,
> +    [ASPEED_DEV_UART12]    =  0X14C33B00,
> +    [ASPEED_DEV_WDT]       =  0x14C37000,
> +    [ASPEED_DEV_VUART]     =  0X14C30000,
> +    [ASPEED_DEV_FMC]       =  0x14000000,
> +    [ASPEED_DEV_SPI0]      =  0x14010000,
> +    [ASPEED_DEV_SPI1]      =  0x14020000,
> +    [ASPEED_DEV_SPI2]      =  0x14030000,
> +    [ASPEED_DEV_SDRAM]     =  0x400000000,
> +    [ASPEED_DEV_MII1]      =  0x14040000,
> +    [ASPEED_DEV_MII2]      =  0x14040008,
> +    [ASPEED_DEV_MII3]      =  0x14040010,
> +    [ASPEED_DEV_ETH1]      =  0x14050000,
> +    [ASPEED_DEV_ETH2]      =  0x14060000,
> +    [ASPEED_DEV_ETH3]      =  0x14070000,
> +    [ASPEED_DEV_EMMC]      =  0x12090000,
> +    [ASPEED_DEV_INTC]      =  0x12100000,
> +    [ASPEED_DEV_SLI]       =  0x12C17000,
> +    [ASPEED_DEV_SLIIO]     =  0x14C1E000,
> +    [ASPEED_GIC_DIST]      =  0x12200000,
> +    [ASPEED_GIC_REDIST]    =  0x12280000,
> +};
> +
> +#define AST2700_MAX_IRQ 288
> +
> +/* Shared Peripheral Interrupt values below are offset by -32 from datasheet */
> +static const int aspeed_soc_ast2700_irqmap[] = {
> +    [ASPEED_DEV_UART0]     = 132,
> +    [ASPEED_DEV_UART1]     = 132,
> +    [ASPEED_DEV_UART2]     = 132,
> +    [ASPEED_DEV_UART3]     = 132,
> +    [ASPEED_DEV_UART4]     = 8,
> +    [ASPEED_DEV_UART5]     = 132,
> +    [ASPEED_DEV_UART6]     = 132,
> +    [ASPEED_DEV_UART7]     = 132,
> +    [ASPEED_DEV_UART8]     = 132,
> +    [ASPEED_DEV_UART9]     = 132,
> +    [ASPEED_DEV_UART10]    = 132,
> +    [ASPEED_DEV_UART11]    = 132,
> +    [ASPEED_DEV_UART12]    = 132,
> +    [ASPEED_DEV_FMC]       = 131,
> +    [ASPEED_DEV_SDMC]      = 0,
> +    [ASPEED_DEV_SCU]       = 12,
> +    [ASPEED_DEV_ADC]       = 130,
> +    [ASPEED_DEV_XDMA]      = 5,
> +    [ASPEED_DEV_EMMC]      = 15,
> +    [ASPEED_DEV_GPIO]      = 11,
> +    [ASPEED_DEV_GPIO_1_8V] = 130,
> +    [ASPEED_DEV_RTC]       = 13,
> +    [ASPEED_DEV_TIMER1]    = 16,
> +    [ASPEED_DEV_TIMER2]    = 17,
> +    [ASPEED_DEV_TIMER3]    = 18,
> +    [ASPEED_DEV_TIMER4]    = 19,
> +    [ASPEED_DEV_TIMER5]    = 20,
> +    [ASPEED_DEV_TIMER6]    = 21,
> +    [ASPEED_DEV_TIMER7]    = 22,
> +    [ASPEED_DEV_TIMER8]    = 23,
> +    [ASPEED_DEV_WDT]       = 131,
> +    [ASPEED_DEV_PWM]       = 131,
> +    [ASPEED_DEV_LPC]       = 128,
> +    [ASPEED_DEV_IBT]       = 128,
> +    [ASPEED_DEV_I2C]       = 130,
> +    [ASPEED_DEV_PECI]      = 133,
> +    [ASPEED_DEV_ETH1]      = 132,
> +    [ASPEED_DEV_ETH2]      = 132,
> +    [ASPEED_DEV_ETH3]      = 132,
> +    [ASPEED_DEV_HACE]      = 4,
> +    [ASPEED_DEV_KCS]       = 128,
> +    [ASPEED_DEV_DP]        = 28,
> +    [ASPEED_DEV_I3C]       = 131,
> +};
> +
> +/* GICINT 128 */
> +static const int aspeed_soc_ast2700_gic128_intcmap[] = {
> +    [ASPEED_DEV_LPC]       = 0,
> +    [ASPEED_DEV_IBT]       = 2,
> +    [ASPEED_DEV_KCS]       = 4,
> +};
> +
> +/* GICINT 130 */
> +static const int aspeed_soc_ast2700_gic130_intcmap[] = {
> +    [ASPEED_DEV_I2C]        = 0,
> +    [ASPEED_DEV_ADC]        = 16,
> +    [ASPEED_DEV_GPIO_1_8V]  = 18,
> +};
> +
> +/* GICINT 131 */
> +static const int aspeed_soc_ast2700_gic131_intcmap[] = {
> +    [ASPEED_DEV_I3C]       = 0,
> +    [ASPEED_DEV_WDT]       = 16,
> +    [ASPEED_DEV_FMC]       = 25,
> +    [ASPEED_DEV_PWM]       = 29,
> +};
> +
> +/* GICINT 132 */
> +static const int aspeed_soc_ast2700_gic132_intcmap[] = {
> +    [ASPEED_DEV_ETH1]      = 0,
> +    [ASPEED_DEV_ETH2]      = 1,
> +    [ASPEED_DEV_ETH3]      = 2,
> +    [ASPEED_DEV_UART0]     = 7,
> +    [ASPEED_DEV_UART1]     = 8,
> +    [ASPEED_DEV_UART2]     = 9,
> +    [ASPEED_DEV_UART3]     = 10,
> +    [ASPEED_DEV_UART5]     = 11,
> +    [ASPEED_DEV_UART6]     = 12,
> +    [ASPEED_DEV_UART7]     = 13,
> +    [ASPEED_DEV_UART8]     = 14,
> +    [ASPEED_DEV_UART9]     = 15,
> +    [ASPEED_DEV_UART10]    = 16,
> +    [ASPEED_DEV_UART11]    = 17,
> +    [ASPEED_DEV_UART12]    = 18,
> +};
> +
> +/* GICINT 133 */
> +static const int aspeed_soc_ast2700_gic133_intcmap[] = {
> +    [ASPEED_DEV_PECI]      = 4,
> +};
> +
> +/* GICINT 128 ~ 136 */
> +struct gic_intc_irq_info {
> +    int irq;
> +    const int *ptr;
> +};
> +
> +static const struct gic_intc_irq_info aspeed_soc_ast2700_gic_intcmap[] = {
> +    {128,  aspeed_soc_ast2700_gic128_intcmap},
> +    {129,  NULL},
> +    {130,  aspeed_soc_ast2700_gic130_intcmap},
> +    {131,  aspeed_soc_ast2700_gic131_intcmap},
> +    {132,  aspeed_soc_ast2700_gic132_intcmap},
> +    {133,  aspeed_soc_ast2700_gic133_intcmap},
> +    {134,  NULL},
> +    {135,  NULL},
> +    {136,  NULL},
> +};
> +
> +static qemu_irq aspeed_soc_ast2700_get_irq(AspeedSoCState *s, int dev)
> +{
> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(s);
> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> +    int i;
> +
> +    for (i = 0; i < ARRAY_SIZE(aspeed_soc_ast2700_gic_intcmap); i++) {
> +        if (sc->irqmap[dev] == aspeed_soc_ast2700_gic_intcmap[i].irq) {
> +            assert(aspeed_soc_ast2700_gic_intcmap[i].ptr);
> +            return qdev_get_gpio_in(DEVICE(&a->intc.gicint_orgate[i]),
> +                aspeed_soc_ast2700_gic_intcmap[i].ptr[dev]);
> +        }
> +    }
> +
> +    return qdev_get_gpio_in(a->intc.gic, sc->irqmap[dev]);
> +}
> +
> +static void aspeed_soc_ast2700_init(Object *obj)
> +{
> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(obj);
> +    AspeedSoCState *s = ASPEED_SOC(obj);
> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> +    int i;
> +    char socname[8];
> +    char typename[64];
> +
> +    if (sscanf(sc->name, "%7s", socname) != 1) {
> +        g_assert_not_reached();
> +    }
> +
> +    for (i = 0; i < sc->num_cpus; i++) {
> +        object_initialize_child(obj, "cpu[*]", &a->cpu[i],
> +                                aspeed_soc_cpu_type(sc));
> +    }
> +
> +    object_initialize_child(obj, "scu", &s->scu, TYPE_ASPEED_2700_SCU);
> +    qdev_prop_set_uint32(DEVICE(&s->scu), "silicon-rev",
> +                         sc->silicon_rev);
> +    object_property_add_alias(obj, "hw-strap1", OBJECT(&s->scu),
> +                              "hw-strap1");
> +    object_property_add_alias(obj, "hw-strap2", OBJECT(&s->scu),
> +                              "hw-strap2");
> +    object_property_add_alias(obj, "hw-prot-key", OBJECT(&s->scu),
> +                              "hw-prot-key");
> +
> +    object_initialize_child(obj, "scuio", &s->scuio, TYPE_ASPEED_2700_SCUIO);
> +    qdev_prop_set_uint32(DEVICE(&s->scuio), "silicon-rev",
> +                         sc->silicon_rev);
> +
> +    snprintf(typename, sizeof(typename), "aspeed.fmc-%s", socname);
> +    object_initialize_child(obj, "fmc", &s->fmc, typename);
> +
> +    for (i = 0; i < sc->spis_num; i++) {
> +        snprintf(typename, sizeof(typename), "aspeed.spi%d-%s", i, socname);
> +        object_initialize_child(obj, "spi[*]", &s->spi[i], typename);
> +    }
> +
> +    snprintf(typename, sizeof(typename), "aspeed.sdmc-%s", socname);
> +    object_initialize_child(obj, "sdmc", &s->sdmc, typename);
> +    object_property_add_alias(obj, "ram-size", OBJECT(&s->sdmc),
> +                              "ram-size");
> +
> +    for (i = 0; i < sc->wdts_num; i++) {
> +        snprintf(typename, sizeof(typename), "aspeed.wdt-%s", socname);
> +        object_initialize_child(obj, "wdt[*]", &s->wdt[i], typename);
> +    }
> +
> +    for (i = 0; i < sc->macs_num; i++) {
> +        object_initialize_child(obj, "ftgmac100[*]", &s->ftgmac100[i],
> +                                TYPE_FTGMAC100);
> +
> +        object_initialize_child(obj, "mii[*]", &s->mii[i], TYPE_ASPEED_MII);
> +    }
> +
> +    for (i = 0; i < sc->uarts_num; i++) {
> +        object_initialize_child(obj, "uart[*]", &s->uart[i], TYPE_SERIAL_MM);
> +    }
> +
> +    object_initialize_child(obj, "sli", &s->sli, TYPE_ASPEED_2700_SLI);
> +    object_initialize_child(obj, "sliio", &s->sliio, TYPE_ASPEED_2700_SLIIO);
> +    object_initialize_child(obj, "intc", &a->intc, TYPE_ASPEED_INTC);
> +}
> +
> +/*
> + * ASPEED ast2700 has 0x0 as cluster ID
> + *
> + * https://developer.arm.com/documentation/100236/0100/register-descriptions/aarch64-system-registers/multiprocessor-affinity-register--el1
> + */
> +static uint64_t aspeed_calc_affinity(int cpu)
> +{
> +    return (0x0 << ARM_AFF1_SHIFT) | cpu;
> +}
> +
> +static void aspeed_soc_ast2700_gic(DeviceState *dev, Error **errp)
> +{
> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
> +    AspeedSoCState *s = ASPEED_SOC(dev);
> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> +    SysBusDevice *gicbusdev;
> +    QList *redist_region_count;
> +    int i;
> +
> +    a->intc.gic = qdev_new(gicv3_class_name());
> +    qdev_prop_set_uint32(a->intc.gic, "revision", 3);
> +    qdev_prop_set_uint32(a->intc.gic, "num-cpu", sc->num_cpus);
> +    qdev_prop_set_uint32(a->intc.gic, "num-irq", AST2700_MAX_IRQ);
> +
> +    redist_region_count = qlist_new();
> +    qlist_append_int(redist_region_count, sc->num_cpus);
> +    qdev_prop_set_array(a->intc.gic, "redist-region-count",
> +                            redist_region_count);
> +
> +    gicbusdev = SYS_BUS_DEVICE(a->intc.gic);
> +    sysbus_realize_and_unref(gicbusdev, errp);
> +    sysbus_mmio_map(gicbusdev, 0, sc->memmap[ASPEED_GIC_DIST]);
> +    sysbus_mmio_map(gicbusdev, 1, sc->memmap[ASPEED_GIC_REDIST]);
> +
> +    for (i = 0; i < sc->num_cpus; i++) {
> +        DeviceState *cpudev = DEVICE(qemu_get_cpu(i));
> +        int NUM_IRQS = 256, ARCH_GIC_MAINT_IRQ = 9, VIRTUAL_PMU_IRQ = 7;
> +        int ppibase = NUM_IRQS + i * GIC_INTERNAL + GIC_NR_SGIS;
> +
> +        const int timer_irq[] = {
> +            [GTIMER_PHYS] = 14,
> +            [GTIMER_VIRT] = 11,
> +            [GTIMER_HYP]  = 10,
> +            [GTIMER_SEC]  = 13,
> +        };
> +        int j;
> +
> +        for (j = 0; j < ARRAY_SIZE(timer_irq); j++) {
> +            qdev_connect_gpio_out(cpudev, j,
> +                    qdev_get_gpio_in(a->intc.gic, ppibase + timer_irq[j]));
> +        }
> +
> +        qemu_irq irq = qdev_get_gpio_in(a->intc.gic,
> +                                        ppibase + ARCH_GIC_MAINT_IRQ);
> +        qdev_connect_gpio_out_named(cpudev, "gicv3-maintenance-interrupt",
> +                                    0, irq);
> +        qdev_connect_gpio_out_named(cpudev, "pmu-interrupt", 0,
> +                qdev_get_gpio_in(a->intc.gic, ppibase + VIRTUAL_PMU_IRQ));
> +
> +        sysbus_connect_irq(gicbusdev, i, qdev_get_gpio_in(cpudev, ARM_CPU_IRQ));
> +        sysbus_connect_irq(gicbusdev, i + sc->num_cpus,
> +                           qdev_get_gpio_in(cpudev, ARM_CPU_FIQ));
> +        sysbus_connect_irq(gicbusdev, i + 2 * sc->num_cpus,
> +                           qdev_get_gpio_in(cpudev, ARM_CPU_VIRQ));
> +        sysbus_connect_irq(gicbusdev, i + 3 * sc->num_cpus,
> +                           qdev_get_gpio_in(cpudev, ARM_CPU_VFIQ));
> +    }
> +}
> +
> +static void aspeed_soc_ast2700_realize(DeviceState *dev, Error **errp)
> +{
> +    int i;
> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
> +    AspeedSoCState *s = ASPEED_SOC(dev);
> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> +    g_autofree char *sram_name = NULL;
> +
> +    /* Default boot region (SPI memory or ROMs) */
> +    memory_region_init(&s->spi_boot_container, OBJECT(s),
> +                       "aspeed.spi_boot_container", 0x400000000);
> +    memory_region_add_subregion(s->memory, sc->memmap[ASPEED_DEV_SPI_BOOT],
> +                                &s->spi_boot_container);
> +
> +    /* CPU */
> +    for (i = 0; i < sc->num_cpus; i++) {
> +        object_property_set_int(OBJECT(&a->cpu[i]), "mp-affinity",
> +                                aspeed_calc_affinity(i), &error_abort);
> +
> +        object_property_set_int(OBJECT(&a->cpu[i]), "cntfrq", 1125000000,
> +                                &error_abort);
> +        object_property_set_link(OBJECT(&a->cpu[i]), "memory",
> +                                 OBJECT(s->memory), &error_abort);
> +
> +        if (!qdev_realize(DEVICE(&a->cpu[i]), NULL, errp)) {
> +            return;
> +        }
> +    }
> +
> +    /* GIC */
> +    aspeed_soc_ast2700_gic(dev, errp);

Hmm, we should return.

> +
> +    /* INTC */
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&a->intc), errp)) {
> +        return;
> +    }
> +
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&a->intc), 0,
> +                    sc->memmap[ASPEED_DEV_INTC]);
> +
> +    /* GICINT orgate -> INTC -> GIC */
> +    for (i = 0; i < ASPEED_INTC_NR_GICS; i++) {
> +        if (!qdev_realize(DEVICE(&a->intc.gicint_orgate[i]), NULL, errp)) {
> +            return;
> +        }
> +
> +        qdev_connect_gpio_out(DEVICE(&a->intc.gicint_orgate[i]), 0,
> +                              qdev_get_gpio_in(DEVICE(&a->intc), i));
> +
> +        sysbus_connect_irq(SYS_BUS_DEVICE(&a->intc), i,
> +                           qdev_get_gpio_in(a->intc.gic,
> +                                aspeed_soc_ast2700_gic_intcmap[i].irq));
> +    }
> +
> +    /* SRAM */
> +    sram_name = g_strdup_printf("aspeed.sram.%d", CPU(&a->cpu[0])->cpu_index);
> +    if (!memory_region_init_ram(&s->sram, OBJECT(s), sram_name, sc->sram_size,
> +                                 errp)) {
> +        return;
> +    }
> +    memory_region_add_subregion(s->memory,
> +                                sc->memmap[ASPEED_DEV_SRAM], &s->sram);
> +
> +    /* SCU */
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scu), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scu), 0, sc->memmap[ASPEED_DEV_SCU]);
> +
> +    /* SCU1 */
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scuio), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scuio), 0,
> +                    sc->memmap[ASPEED_DEV_SCUIO]);
> +
> +    /* UART */
> +    if (!aspeed_soc_uart_realize(s, errp)) {
> +        return;
> +    }
> +
> +    /* FMC, The number of CS is set at the board level */
> +    object_property_set_link(OBJECT(&s->fmc), "dram", OBJECT(s->dram_mr),
> +                             &error_abort);
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->fmc), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 0, sc->memmap[ASPEED_DEV_FMC]);
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 1,
> +                    ASPEED_SMC_GET_CLASS(&s->fmc)->flash_window_base);
> +    sysbus_connect_irq(SYS_BUS_DEVICE(&s->fmc), 0,
> +                       aspeed_soc_get_irq(s, ASPEED_DEV_FMC));
> +
> +    /* Set up an alias on the FMC CE0 region (boot default) */
> +    MemoryRegion *fmc0_mmio = &s->fmc.flashes[0].mmio;
> +    memory_region_init_alias(&s->spi_boot, OBJECT(s), "aspeed.spi_boot",
> +                             fmc0_mmio, 0, memory_region_size(fmc0_mmio));
> +    memory_region_add_subregion(&s->spi_boot_container, 0x0, &s->spi_boot);
> +
> +    /* SPI */
> +    for (i = 0; i < sc->spis_num; i++) {
> +        object_property_set_link(OBJECT(&s->spi[i]), "dram",
> +                                 OBJECT(s->dram_mr), &error_abort);
> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->spi[i]), errp)) {
> +            return;
> +        }
> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 0,
> +                        sc->memmap[ASPEED_DEV_SPI0 + i]);
> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 1,
> +                        ASPEED_SMC_GET_CLASS(&s->spi[i])->flash_window_base);
> +    }
> +
> +    /*
> +     * SDMC - SDRAM Memory Controller
> +     * The SDMC controller is unlocked at SPL stage.
> +     * At present, only supports to emulate booting
> +     * start from u-boot stage. Set SDMC controller
> +     * unlocked by default. It is a temporarily solution.
> +     */
> +    object_property_set_bool(OBJECT(&s->sdmc), "unlocked", true,
> +                                 &error_abort);
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sdmc), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sdmc), 0,
> +                    sc->memmap[ASPEED_DEV_SDMC]);
> +
> +    /* RAM */
> +    if (!aspeed_soc_dram_init(s, errp)) {
> +        return;
> +    }
> +
> +    for (i = 0; i < sc->macs_num; i++) {
> +        object_property_set_bool(OBJECT(&s->ftgmac100[i]), "aspeed", true,
> +                                 &error_abort);
> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->ftgmac100[i]), errp)) {
> +            return;
> +        }
> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
> +                        sc->memmap[ASPEED_DEV_ETH1 + i]);
> +        sysbus_connect_irq(SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
> +                           aspeed_soc_get_irq(s, ASPEED_DEV_ETH1 + i));
> +
> +        object_property_set_link(OBJECT(&s->mii[i]), "nic",
> +                                 OBJECT(&s->ftgmac100[i]), &error_abort);
> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->mii[i]), errp)) {
> +            return;
> +        }
> +
> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->mii[i]), 0,
> +                        sc->memmap[ASPEED_DEV_MII1 + i]);
> +    }
> +
> +    /* Watch dog */
> +    for (i = 0; i < sc->wdts_num; i++) {
> +        AspeedWDTClass *awc = ASPEED_WDT_GET_CLASS(&s->wdt[i]);
> +        hwaddr wdt_offset = sc->memmap[ASPEED_DEV_WDT] + i * awc->iosize;
> +
> +        object_property_set_link(OBJECT(&s->wdt[i]), "scu", OBJECT(&s->scu),
> +                                 &error_abort);
> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->wdt[i]), errp)) {
> +            return;
> +        }
> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->wdt[i]), 0, wdt_offset);
> +    }
> +
> +    /* SLI */
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sli), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sli), 0, sc->memmap[ASPEED_DEV_SLI]);
> +
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sliio), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sliio), 0,
> +                    sc->memmap[ASPEED_DEV_SLIIO]);
> +
> +    create_unimplemented_device("ast2700.dpmcu", 0x11000000, 0x40000);
> +    create_unimplemented_device("ast2700.iomem0", 0x12000000, 0x01000000);
> +    create_unimplemented_device("ast2700.iomem1", 0x14000000, 0x01000000);
> +    create_unimplemented_device("ast2700.ltpi", 0x30000000, 0x1000000);
> +    create_unimplemented_device("ast2700.io", 0x0, 0x4000000);
> +}
> +
> +static void aspeed_soc_ast2700_class_init(ObjectClass *oc, void *data)
> +{
> +    static const char * const valid_cpu_types[] = {
> +        ARM_CPU_TYPE_NAME("cortex-a35"),
> +        NULL
> +    };
> +    DeviceClass *dc = DEVICE_CLASS(oc);
> +    AspeedSoCClass *sc = ASPEED_SOC_CLASS(oc);
> +
> +    dc->realize      = aspeed_soc_ast2700_realize;


Please add :

     /* Reason: The Aspeed SoC can only be instantiated from a board */
     dc->user_creatable = false;


Thanks,

C.


> +
> +    sc->name         = "ast2700-a0";
> +    sc->valid_cpu_types = valid_cpu_types;
> +    sc->silicon_rev  = AST2700_A0_SILICON_REV;
> +    sc->sram_size    = 0x20000;
> +    sc->spis_num     = 3;
> +    sc->wdts_num     = 8;
> +    sc->macs_num     = 1;
> +    sc->uarts_num    = 13;
> +    sc->num_cpus     = 4;
> +    sc->uarts_base   = ASPEED_DEV_UART0;
> +    sc->irqmap       = aspeed_soc_ast2700_irqmap;
> +    sc->memmap       = aspeed_soc_ast2700_memmap;
> +    sc->get_irq      = aspeed_soc_ast2700_get_irq;
> +}
> +
> +static const TypeInfo aspeed_soc_ast27x0_types[] = {
> +    {
> +        .name           = TYPE_ASPEED27X0_SOC,
> +        .parent         = TYPE_ASPEED_SOC,
> +        .instance_size  = sizeof(Aspeed27x0SoCState),
> +        .abstract       = true,
> +    }, {
> +        .name           = "ast2700-a0",
> +        .parent         = TYPE_ASPEED27X0_SOC,
> +        .instance_init  = aspeed_soc_ast2700_init,
> +        .class_init     = aspeed_soc_ast2700_class_init,
> +    },
> +};
> +
> +DEFINE_TYPES(aspeed_soc_ast27x0_types)
> diff --git a/hw/arm/meson.build b/hw/arm/meson.build
> index 6808135c1f..1e3295a423 100644
> --- a/hw/arm/meson.build
> +++ b/hw/arm/meson.build
> @@ -46,6 +46,7 @@ arm_ss.add(when: 'CONFIG_ASPEED_SOC', if_true: files(
>     'aspeed_soc_common.c',
>     'aspeed_ast2400.c',
>     'aspeed_ast2600.c',
> +  'aspeed_ast27x0.c',
>     'aspeed_ast10x0.c',
>     'aspeed_eeprom.c',
>     'fby35.c'))
> diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h
> index c60fac900a..9f177b6037 100644
> --- a/include/hw/arm/aspeed_soc.h
> +++ b/include/hw/arm/aspeed_soc.h
> @@ -15,6 +15,7 @@
>   #include "hw/cpu/a15mpcore.h"
>   #include "hw/arm/armv7m.h"
>   #include "hw/intc/aspeed_vic.h"
> +#include "hw/intc/aspeed_intc.h"
>   #include "hw/misc/aspeed_scu.h"
>   #include "hw/adc/aspeed_adc.h"
>   #include "hw/misc/aspeed_sdmc.h"
> @@ -26,6 +27,7 @@
>   #include "hw/ssi/aspeed_smc.h"
>   #include "hw/misc/aspeed_hace.h"
>   #include "hw/misc/aspeed_sbc.h"
> +#include "hw/misc/aspeed_sli.h"
>   #include "hw/watchdog/wdt_aspeed.h"
>   #include "hw/net/ftgmac100.h"
>   #include "target/arm/cpu.h"
> @@ -41,8 +43,8 @@
>   
>   #define ASPEED_SPIS_NUM  2
>   #define ASPEED_EHCIS_NUM 2
> -#define ASPEED_WDTS_NUM  4
> -#define ASPEED_CPUS_NUM  2
> +#define ASPEED_WDTS_NUM  8
> +#define ASPEED_CPUS_NUM  4
>   #define ASPEED_MACS_NUM  4
>   #define ASPEED_UARTS_NUM 13
>   #define ASPEED_JTAG_NUM  2
> @@ -61,6 +63,7 @@ struct AspeedSoCState {
>       AspeedI2CState i2c;
>       AspeedI3CState i3c;
>       AspeedSCUState scu;
> +    AspeedSCUState scuio;
>       AspeedHACEState hace;
>       AspeedXDMAState xdma;
>       AspeedADCState adc;
> @@ -68,6 +71,8 @@ struct AspeedSoCState {
>       AspeedSMCState spi[ASPEED_SPIS_NUM];
>       EHCISysBusState ehci[ASPEED_EHCIS_NUM];
>       AspeedSBCState sbc;
> +    AspeedSLIState sli;
> +    AspeedSLIState sliio;
>       MemoryRegion secsram;
>       UnimplementedDeviceState sbc_unimplemented;
>       AspeedSDMCState sdmc;
> @@ -117,6 +122,16 @@ struct Aspeed2600SoCState {
>   #define TYPE_ASPEED2600_SOC "aspeed2600-soc"
>   OBJECT_DECLARE_SIMPLE_TYPE(Aspeed2600SoCState, ASPEED2600_SOC)
>   
> +struct Aspeed27x0SoCState {
> +    AspeedSoCState parent;
> +
> +    ARMCPU cpu[ASPEED_CPUS_NUM];
> +    AspeedINTCState intc;
> +};
> +
> +#define TYPE_ASPEED27X0_SOC "aspeed27x0-soc"
> +OBJECT_DECLARE_SIMPLE_TYPE(Aspeed27x0SoCState, ASPEED27X0_SOC)
> +
>   struct Aspeed10x0SoCState {
>       AspeedSoCState parent;
>   
> @@ -168,11 +183,13 @@ enum {
>       ASPEED_DEV_UART13,
>       ASPEED_DEV_VUART,
>       ASPEED_DEV_FMC,
> +    ASPEED_DEV_SPI0,
>       ASPEED_DEV_SPI1,
>       ASPEED_DEV_SPI2,
>       ASPEED_DEV_EHCI1,
>       ASPEED_DEV_EHCI2,
>       ASPEED_DEV_VIC,
> +    ASPEED_DEV_INTC,
>       ASPEED_DEV_SDMC,
>       ASPEED_DEV_SCU,
>       ASPEED_DEV_ADC,
> @@ -222,6 +239,11 @@ enum {
>       ASPEED_DEV_JTAG1,
>       ASPEED_DEV_FSI1,
>       ASPEED_DEV_FSI2,
> +    ASPEED_DEV_SCUIO,
> +    ASPEED_DEV_SLI,
> +    ASPEED_DEV_SLIIO,
> +    ASPEED_GIC_DIST,
> +    ASPEED_GIC_REDIST,
>   };
>   
>   qemu_irq aspeed_soc_get_irq(AspeedSoCState *s, int dev);
Jamin Lin April 19, 2024, 7:58 a.m. UTC | #2
Hi Cedric, 
> On 4/16/24 11:18, Jamin Lin wrote:
> > Initial definitions for a simple machine using an AST2700 SOC (Cortex-a35
> CPU).
> >
> > AST2700 SOC and its interrupt controller are too complex to handle in
> > the common Aspeed SoC framework. We introduce a new ast2700 class with
> > instance_init and realize handlers.
> >
> > AST2700 is a 64 bits quad core cpus and support 8 watchdog.
> > Update maximum ASPEED_CPUS_NUM to 4 and ASPEED_WDTS_NUM to 8.
> > In addition, update AspeedSocState to support scuio, sli, sliio and intc.
> >
> > Add TYPE_ASPEED27X0_SOC machine type.
> >
> > The SDMC controller is unlocked at SPL stage.
> > At present, only supports to emulate booting start from u-boot stage.
> > Set SDMC controller unlocked by default.
> >
> > In INTC, each interrupt of INT 128 to INT 136 combines 32 interrupts.
> > It connect GICINT IRQ GPIO-OUTPUT pins to GIC device with irq 128 to 136.
> > And, if a device irq is 128 to 136, its irq GPIO-OUTPUT pin is
> > connected to GICINT or-gates instead of GIC device.
> >
> > Signed-off-by: Troy Lee <troy_lee@aspeedtech.com>
> > Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> 
> Before I forget, please see a little comment below regarding user creatable
> devices.
> 
> The model looks fine. The interrupt controller part is more complex than the
> previous SoCs so I will come back to it later when I have more time. 
Thanks for your kindly support.
> > ---
> >   hw/arm/aspeed_ast27x0.c     | 554
> ++++++++++++++++++++++++++++++++++++
> >   hw/arm/meson.build          |   1 +
> >   include/hw/arm/aspeed_soc.h |  26 +-
> >   3 files changed, 579 insertions(+), 2 deletions(-)
> >   create mode 100644 hw/arm/aspeed_ast27x0.c
> >
> > diff --git a/hw/arm/aspeed_ast27x0.c b/hw/arm/aspeed_ast27x0.c new
> > file mode 100644 index 0000000000..754c963230
> > --- /dev/null
> > +++ b/hw/arm/aspeed_ast27x0.c
> > @@ -0,0 +1,554 @@
> > +/*
> > + * ASPEED SoC 27x0 family
> > + *
> > + * Copyright (C) 2024 ASPEED Technology Inc.
> > + *
> > + * This code is licensed under the GPL version 2 or later.  See
> > + * the COPYING file in the top-level directory.
> > + *
> > + * Implementation extracted from the AST2600 and adapted for AST27x0.
> > + */
> > +
> > +#include "qemu/osdep.h"
> > +#include "qapi/error.h"
> > +#include "hw/misc/unimp.h"
> > +#include "hw/arm/aspeed_soc.h"
> > +#include "qemu/module.h"
> > +#include "qemu/error-report.h"
> > +#include "hw/i2c/aspeed_i2c.h"
> > +#include "net/net.h"
> > +#include "sysemu/sysemu.h"
> > +#include "hw/intc/arm_gicv3.h"
> > +#include "qapi/qmp/qlist.h"
> > +
> > +static const hwaddr aspeed_soc_ast2700_memmap[] = {
> > +    [ASPEED_DEV_SPI_BOOT]  =  0x400000000,
> > +    [ASPEED_DEV_SRAM]      =  0x10000000,
> > +    [ASPEED_DEV_SDMC]      =  0x12C00000,
> > +    [ASPEED_DEV_SCU]       =  0x12C02000,
> > +    [ASPEED_DEV_SCUIO]     =  0x14C02000,
> > +    [ASPEED_DEV_UART0]     =  0X14C33000,
> > +    [ASPEED_DEV_UART1]     =  0X14C33100,
> > +    [ASPEED_DEV_UART2]     =  0X14C33200,
> > +    [ASPEED_DEV_UART3]     =  0X14C33300,
> > +    [ASPEED_DEV_UART4]     =  0X12C1A000,
> > +    [ASPEED_DEV_UART5]     =  0X14C33400,
> > +    [ASPEED_DEV_UART6]     =  0X14C33500,
> > +    [ASPEED_DEV_UART7]     =  0X14C33600,
> > +    [ASPEED_DEV_UART8]     =  0X14C33700,
> > +    [ASPEED_DEV_UART9]     =  0X14C33800,
> > +    [ASPEED_DEV_UART10]    =  0X14C33900,
> > +    [ASPEED_DEV_UART11]    =  0X14C33A00,
> > +    [ASPEED_DEV_UART12]    =  0X14C33B00,
> > +    [ASPEED_DEV_WDT]       =  0x14C37000,
> > +    [ASPEED_DEV_VUART]     =  0X14C30000,
> > +    [ASPEED_DEV_FMC]       =  0x14000000,
> > +    [ASPEED_DEV_SPI0]      =  0x14010000,
> > +    [ASPEED_DEV_SPI1]      =  0x14020000,
> > +    [ASPEED_DEV_SPI2]      =  0x14030000,
> > +    [ASPEED_DEV_SDRAM]     =  0x400000000,
> > +    [ASPEED_DEV_MII1]      =  0x14040000,
> > +    [ASPEED_DEV_MII2]      =  0x14040008,
> > +    [ASPEED_DEV_MII3]      =  0x14040010,
> > +    [ASPEED_DEV_ETH1]      =  0x14050000,
> > +    [ASPEED_DEV_ETH2]      =  0x14060000,
> > +    [ASPEED_DEV_ETH3]      =  0x14070000,
> > +    [ASPEED_DEV_EMMC]      =  0x12090000,
> > +    [ASPEED_DEV_INTC]      =  0x12100000,
> > +    [ASPEED_DEV_SLI]       =  0x12C17000,
> > +    [ASPEED_DEV_SLIIO]     =  0x14C1E000,
> > +    [ASPEED_GIC_DIST]      =  0x12200000,
> > +    [ASPEED_GIC_REDIST]    =  0x12280000,
> > +};
> > +
> > +#define AST2700_MAX_IRQ 288
> > +
> > +/* Shared Peripheral Interrupt values below are offset by -32 from
> > +datasheet */ static const int aspeed_soc_ast2700_irqmap[] = {
> > +    [ASPEED_DEV_UART0]     = 132,
> > +    [ASPEED_DEV_UART1]     = 132,
> > +    [ASPEED_DEV_UART2]     = 132,
> > +    [ASPEED_DEV_UART3]     = 132,
> > +    [ASPEED_DEV_UART4]     = 8,
> > +    [ASPEED_DEV_UART5]     = 132,
> > +    [ASPEED_DEV_UART6]     = 132,
> > +    [ASPEED_DEV_UART7]     = 132,
> > +    [ASPEED_DEV_UART8]     = 132,
> > +    [ASPEED_DEV_UART9]     = 132,
> > +    [ASPEED_DEV_UART10]    = 132,
> > +    [ASPEED_DEV_UART11]    = 132,
> > +    [ASPEED_DEV_UART12]    = 132,
> > +    [ASPEED_DEV_FMC]       = 131,
> > +    [ASPEED_DEV_SDMC]      = 0,
> > +    [ASPEED_DEV_SCU]       = 12,
> > +    [ASPEED_DEV_ADC]       = 130,
> > +    [ASPEED_DEV_XDMA]      = 5,
> > +    [ASPEED_DEV_EMMC]      = 15,
> > +    [ASPEED_DEV_GPIO]      = 11,
> > +    [ASPEED_DEV_GPIO_1_8V] = 130,
> > +    [ASPEED_DEV_RTC]       = 13,
> > +    [ASPEED_DEV_TIMER1]    = 16,
> > +    [ASPEED_DEV_TIMER2]    = 17,
> > +    [ASPEED_DEV_TIMER3]    = 18,
> > +    [ASPEED_DEV_TIMER4]    = 19,
> > +    [ASPEED_DEV_TIMER5]    = 20,
> > +    [ASPEED_DEV_TIMER6]    = 21,
> > +    [ASPEED_DEV_TIMER7]    = 22,
> > +    [ASPEED_DEV_TIMER8]    = 23,
> > +    [ASPEED_DEV_WDT]       = 131,
> > +    [ASPEED_DEV_PWM]       = 131,
> > +    [ASPEED_DEV_LPC]       = 128,
> > +    [ASPEED_DEV_IBT]       = 128,
> > +    [ASPEED_DEV_I2C]       = 130,
> > +    [ASPEED_DEV_PECI]      = 133,
> > +    [ASPEED_DEV_ETH1]      = 132,
> > +    [ASPEED_DEV_ETH2]      = 132,
> > +    [ASPEED_DEV_ETH3]      = 132,
> > +    [ASPEED_DEV_HACE]      = 4,
> > +    [ASPEED_DEV_KCS]       = 128,
> > +    [ASPEED_DEV_DP]        = 28,
> > +    [ASPEED_DEV_I3C]       = 131,
> > +};
> > +
> > +/* GICINT 128 */
> > +static const int aspeed_soc_ast2700_gic128_intcmap[] = {
> > +    [ASPEED_DEV_LPC]       = 0,
> > +    [ASPEED_DEV_IBT]       = 2,
> > +    [ASPEED_DEV_KCS]       = 4,
> > +};
> > +
> > +/* GICINT 130 */
> > +static const int aspeed_soc_ast2700_gic130_intcmap[] = {
> > +    [ASPEED_DEV_I2C]        = 0,
> > +    [ASPEED_DEV_ADC]        = 16,
> > +    [ASPEED_DEV_GPIO_1_8V]  = 18,
> > +};
> > +
> > +/* GICINT 131 */
> > +static const int aspeed_soc_ast2700_gic131_intcmap[] = {
> > +    [ASPEED_DEV_I3C]       = 0,
> > +    [ASPEED_DEV_WDT]       = 16,
> > +    [ASPEED_DEV_FMC]       = 25,
> > +    [ASPEED_DEV_PWM]       = 29,
> > +};
> > +
> > +/* GICINT 132 */
> > +static const int aspeed_soc_ast2700_gic132_intcmap[] = {
> > +    [ASPEED_DEV_ETH1]      = 0,
> > +    [ASPEED_DEV_ETH2]      = 1,
> > +    [ASPEED_DEV_ETH3]      = 2,
> > +    [ASPEED_DEV_UART0]     = 7,
> > +    [ASPEED_DEV_UART1]     = 8,
> > +    [ASPEED_DEV_UART2]     = 9,
> > +    [ASPEED_DEV_UART3]     = 10,
> > +    [ASPEED_DEV_UART5]     = 11,
> > +    [ASPEED_DEV_UART6]     = 12,
> > +    [ASPEED_DEV_UART7]     = 13,
> > +    [ASPEED_DEV_UART8]     = 14,
> > +    [ASPEED_DEV_UART9]     = 15,
> > +    [ASPEED_DEV_UART10]    = 16,
> > +    [ASPEED_DEV_UART11]    = 17,
> > +    [ASPEED_DEV_UART12]    = 18,
> > +};
> > +
> > +/* GICINT 133 */
> > +static const int aspeed_soc_ast2700_gic133_intcmap[] = {
> > +    [ASPEED_DEV_PECI]      = 4,
> > +};
> > +
> > +/* GICINT 128 ~ 136 */
> > +struct gic_intc_irq_info {
> > +    int irq;
> > +    const int *ptr;
> > +};
> > +
> > +static const struct gic_intc_irq_info aspeed_soc_ast2700_gic_intcmap[] = {
> > +    {128,  aspeed_soc_ast2700_gic128_intcmap},
> > +    {129,  NULL},
> > +    {130,  aspeed_soc_ast2700_gic130_intcmap},
> > +    {131,  aspeed_soc_ast2700_gic131_intcmap},
> > +    {132,  aspeed_soc_ast2700_gic132_intcmap},
> > +    {133,  aspeed_soc_ast2700_gic133_intcmap},
> > +    {134,  NULL},
> > +    {135,  NULL},
> > +    {136,  NULL},
> > +};
> > +
> > +static qemu_irq aspeed_soc_ast2700_get_irq(AspeedSoCState *s, int
> > +dev) {
> > +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(s);
> > +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> > +    int i;
> > +
> > +    for (i = 0; i < ARRAY_SIZE(aspeed_soc_ast2700_gic_intcmap); i++) {
> > +        if (sc->irqmap[dev] == aspeed_soc_ast2700_gic_intcmap[i].irq) {
> > +            assert(aspeed_soc_ast2700_gic_intcmap[i].ptr);
> > +            return qdev_get_gpio_in(DEVICE(&a->intc.gicint_orgate[i]),
> > +                aspeed_soc_ast2700_gic_intcmap[i].ptr[dev]);
> > +        }
> > +    }
> > +
> > +    return qdev_get_gpio_in(a->intc.gic, sc->irqmap[dev]); }
> > +
> > +static void aspeed_soc_ast2700_init(Object *obj) {
> > +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(obj);
> > +    AspeedSoCState *s = ASPEED_SOC(obj);
> > +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> > +    int i;
> > +    char socname[8];
> > +    char typename[64];
> > +
> > +    if (sscanf(sc->name, "%7s", socname) != 1) {
> > +        g_assert_not_reached();
> > +    }
> > +
> > +    for (i = 0; i < sc->num_cpus; i++) {
> > +        object_initialize_child(obj, "cpu[*]", &a->cpu[i],
> > +                                aspeed_soc_cpu_type(sc));
> > +    }
> > +
> > +    object_initialize_child(obj, "scu", &s->scu, TYPE_ASPEED_2700_SCU);
> > +    qdev_prop_set_uint32(DEVICE(&s->scu), "silicon-rev",
> > +                         sc->silicon_rev);
> > +    object_property_add_alias(obj, "hw-strap1", OBJECT(&s->scu),
> > +                              "hw-strap1");
> > +    object_property_add_alias(obj, "hw-strap2", OBJECT(&s->scu),
> > +                              "hw-strap2");
> > +    object_property_add_alias(obj, "hw-prot-key", OBJECT(&s->scu),
> > +                              "hw-prot-key");
> > +
> > +    object_initialize_child(obj, "scuio", &s->scuio,
> TYPE_ASPEED_2700_SCUIO);
> > +    qdev_prop_set_uint32(DEVICE(&s->scuio), "silicon-rev",
> > +                         sc->silicon_rev);
> > +
> > +    snprintf(typename, sizeof(typename), "aspeed.fmc-%s", socname);
> > +    object_initialize_child(obj, "fmc", &s->fmc, typename);
> > +
> > +    for (i = 0; i < sc->spis_num; i++) {
> > +        snprintf(typename, sizeof(typename), "aspeed.spi%d-%s", i,
> socname);
> > +        object_initialize_child(obj, "spi[*]", &s->spi[i], typename);
> > +    }
> > +
> > +    snprintf(typename, sizeof(typename), "aspeed.sdmc-%s", socname);
> > +    object_initialize_child(obj, "sdmc", &s->sdmc, typename);
> > +    object_property_add_alias(obj, "ram-size", OBJECT(&s->sdmc),
> > +                              "ram-size");
> > +
> > +    for (i = 0; i < sc->wdts_num; i++) {
> > +        snprintf(typename, sizeof(typename), "aspeed.wdt-%s",
> socname);
> > +        object_initialize_child(obj, "wdt[*]", &s->wdt[i], typename);
> > +    }
> > +
> > +    for (i = 0; i < sc->macs_num; i++) {
> > +        object_initialize_child(obj, "ftgmac100[*]", &s->ftgmac100[i],
> > +                                TYPE_FTGMAC100);
> > +
> > +        object_initialize_child(obj, "mii[*]", &s->mii[i],
> TYPE_ASPEED_MII);
> > +    }
> > +
> > +    for (i = 0; i < sc->uarts_num; i++) {
> > +        object_initialize_child(obj, "uart[*]", &s->uart[i],
> TYPE_SERIAL_MM);
> > +    }
> > +
> > +    object_initialize_child(obj, "sli", &s->sli, TYPE_ASPEED_2700_SLI);
> > +    object_initialize_child(obj, "sliio", &s->sliio,
> TYPE_ASPEED_2700_SLIIO);
> > +    object_initialize_child(obj, "intc", &a->intc, TYPE_ASPEED_INTC);
> > +}
> > +
> > +/*
> > + * ASPEED ast2700 has 0x0 as cluster ID
> > + *
> > + *
> > +https://developer.arm.com/documentation/100236/0100/register-descript
> > +ions/aarch64-system-registers/multiprocessor-affinity-register--el1
> > + */
> > +static uint64_t aspeed_calc_affinity(int cpu) {
> > +    return (0x0 << ARM_AFF1_SHIFT) | cpu; }
> > +
> > +static void aspeed_soc_ast2700_gic(DeviceState *dev, Error **errp) {
> > +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
> > +    AspeedSoCState *s = ASPEED_SOC(dev);
> > +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> > +    SysBusDevice *gicbusdev;
> > +    QList *redist_region_count;
> > +    int i;
> > +
> > +    a->intc.gic = qdev_new(gicv3_class_name());
> > +    qdev_prop_set_uint32(a->intc.gic, "revision", 3);
> > +    qdev_prop_set_uint32(a->intc.gic, "num-cpu", sc->num_cpus);
> > +    qdev_prop_set_uint32(a->intc.gic, "num-irq", AST2700_MAX_IRQ);
> > +
> > +    redist_region_count = qlist_new();
> > +    qlist_append_int(redist_region_count, sc->num_cpus);
> > +    qdev_prop_set_array(a->intc.gic, "redist-region-count",
> > +                            redist_region_count);
> > +
> > +    gicbusdev = SYS_BUS_DEVICE(a->intc.gic);
> > +    sysbus_realize_and_unref(gicbusdev, errp);
> > +    sysbus_mmio_map(gicbusdev, 0, sc->memmap[ASPEED_GIC_DIST]);
> > +    sysbus_mmio_map(gicbusdev, 1, sc->memmap[ASPEED_GIC_REDIST]);
> > +
> > +    for (i = 0; i < sc->num_cpus; i++) {
> > +        DeviceState *cpudev = DEVICE(qemu_get_cpu(i));
> > +        int NUM_IRQS = 256, ARCH_GIC_MAINT_IRQ = 9,
> VIRTUAL_PMU_IRQ = 7;
> > +        int ppibase = NUM_IRQS + i * GIC_INTERNAL + GIC_NR_SGIS;
> > +
> > +        const int timer_irq[] = {
> > +            [GTIMER_PHYS] = 14,
> > +            [GTIMER_VIRT] = 11,
> > +            [GTIMER_HYP]  = 10,
> > +            [GTIMER_SEC]  = 13,
> > +        };
> > +        int j;
> > +
> > +        for (j = 0; j < ARRAY_SIZE(timer_irq); j++) {
> > +            qdev_connect_gpio_out(cpudev, j,
> > +                    qdev_get_gpio_in(a->intc.gic, ppibase +
> timer_irq[j]));
> > +        }
> > +
> > +        qemu_irq irq = qdev_get_gpio_in(a->intc.gic,
> > +                                        ppibase +
> ARCH_GIC_MAINT_IRQ);
> > +        qdev_connect_gpio_out_named(cpudev,
> "gicv3-maintenance-interrupt",
> > +                                    0, irq);
> > +        qdev_connect_gpio_out_named(cpudev, "pmu-interrupt", 0,
> > +                qdev_get_gpio_in(a->intc.gic, ppibase +
> > + VIRTUAL_PMU_IRQ));
> > +
> > +        sysbus_connect_irq(gicbusdev, i, qdev_get_gpio_in(cpudev,
> ARM_CPU_IRQ));
> > +        sysbus_connect_irq(gicbusdev, i + sc->num_cpus,
> > +                           qdev_get_gpio_in(cpudev,
> ARM_CPU_FIQ));
> > +        sysbus_connect_irq(gicbusdev, i + 2 * sc->num_cpus,
> > +                           qdev_get_gpio_in(cpudev,
> ARM_CPU_VIRQ));
> > +        sysbus_connect_irq(gicbusdev, i + 3 * sc->num_cpus,
> > +                           qdev_get_gpio_in(cpudev,
> ARM_CPU_VFIQ));
> > +    }
> > +}
> > +
> > +static void aspeed_soc_ast2700_realize(DeviceState *dev, Error
> > +**errp) {
> > +    int i;
> > +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
> > +    AspeedSoCState *s = ASPEED_SOC(dev);
> > +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> > +    g_autofree char *sram_name = NULL;
> > +
> > +    /* Default boot region (SPI memory or ROMs) */
> > +    memory_region_init(&s->spi_boot_container, OBJECT(s),
> > +                       "aspeed.spi_boot_container", 0x400000000);
> > +    memory_region_add_subregion(s->memory,
> sc->memmap[ASPEED_DEV_SPI_BOOT],
> > +                                &s->spi_boot_container);
> > +
> > +    /* CPU */
> > +    for (i = 0; i < sc->num_cpus; i++) {
> > +        object_property_set_int(OBJECT(&a->cpu[i]), "mp-affinity",
> > +                                aspeed_calc_affinity(i),
> > + &error_abort);
> > +
> > +        object_property_set_int(OBJECT(&a->cpu[i]), "cntfrq",
> 1125000000,
> > +                                &error_abort);
> > +        object_property_set_link(OBJECT(&a->cpu[i]), "memory",
> > +                                 OBJECT(s->memory),
> &error_abort);
> > +
> > +        if (!qdev_realize(DEVICE(&a->cpu[i]), NULL, errp)) {
> > +            return;
> > +        }
> > +    }
> > +
> > +    /* GIC */
> > +    aspeed_soc_ast2700_gic(dev, errp);
> 
> Hmm, we should return.
This function return type is void.
If I understand your request, do you mean do the following changes?

static bool aspeed_soc_ast2700_gic(DeviceState *dev, Error **errp) --> change return type bool
{
    ---
    gicbusdev = SYS_BUS_DEVICE(a->intc.gic); 
    if(!sysbus_realize(gicbusdev, errp)) {
        return false;
    }
    ---
    return true;
}

Then, 
/* GIC */
if(!aspeed_soc_ast2700_gic(dev, errp)) {
    return;
    }

Thanks-Jamin
> 
> > +
> > +    /* INTC */
> > +    if (!sysbus_realize(SYS_BUS_DEVICE(&a->intc), errp)) {
> > +        return;
> > +    }
> > +
> > +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&a->intc), 0,
> > +                    sc->memmap[ASPEED_DEV_INTC]);
> > +
> > +    /* GICINT orgate -> INTC -> GIC */
> > +    for (i = 0; i < ASPEED_INTC_NR_GICS; i++) {
> > +        if (!qdev_realize(DEVICE(&a->intc.gicint_orgate[i]), NULL, errp)) {
> > +            return;
> > +        }
> > +
> > +        qdev_connect_gpio_out(DEVICE(&a->intc.gicint_orgate[i]), 0,
> > +                              qdev_get_gpio_in(DEVICE(&a->intc),
> i));
> > +
> > +        sysbus_connect_irq(SYS_BUS_DEVICE(&a->intc), i,
> > +                           qdev_get_gpio_in(a->intc.gic,
> > +
> aspeed_soc_ast2700_gic_intcmap[i].irq));
> > +    }
> > +
> > +    /* SRAM */
> > +    sram_name = g_strdup_printf("aspeed.sram.%d",
> CPU(&a->cpu[0])->cpu_index);
> > +    if (!memory_region_init_ram(&s->sram, OBJECT(s), sram_name,
> sc->sram_size,
> > +                                 errp)) {
> > +        return;
> > +    }
> > +    memory_region_add_subregion(s->memory,
> > +                                sc->memmap[ASPEED_DEV_SRAM],
> > + &s->sram);
> > +
> > +    /* SCU */
> > +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scu), errp)) {
> > +        return;
> > +    }
> > +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scu), 0,
> > + sc->memmap[ASPEED_DEV_SCU]);
> > +
> > +    /* SCU1 */
> > +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scuio), errp)) {
> > +        return;
> > +    }
> > +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scuio), 0,
> > +                    sc->memmap[ASPEED_DEV_SCUIO]);
> > +
> > +    /* UART */
> > +    if (!aspeed_soc_uart_realize(s, errp)) {
> > +        return;
> > +    }
> > +
> > +    /* FMC, The number of CS is set at the board level */
> > +    object_property_set_link(OBJECT(&s->fmc), "dram",
> OBJECT(s->dram_mr),
> > +                             &error_abort);
> > +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->fmc), errp)) {
> > +        return;
> > +    }
> > +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 0,
> sc->memmap[ASPEED_DEV_FMC]);
> > +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 1,
> > +
> ASPEED_SMC_GET_CLASS(&s->fmc)->flash_window_base);
> > +    sysbus_connect_irq(SYS_BUS_DEVICE(&s->fmc), 0,
> > +                       aspeed_soc_get_irq(s, ASPEED_DEV_FMC));
> > +
> > +    /* Set up an alias on the FMC CE0 region (boot default) */
> > +    MemoryRegion *fmc0_mmio = &s->fmc.flashes[0].mmio;
> > +    memory_region_init_alias(&s->spi_boot, OBJECT(s),
> "aspeed.spi_boot",
> > +                             fmc0_mmio, 0,
> memory_region_size(fmc0_mmio));
> > +    memory_region_add_subregion(&s->spi_boot_container, 0x0,
> > + &s->spi_boot);
> > +
> > +    /* SPI */
> > +    for (i = 0; i < sc->spis_num; i++) {
> > +        object_property_set_link(OBJECT(&s->spi[i]), "dram",
> > +                                 OBJECT(s->dram_mr),
> &error_abort);
> > +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->spi[i]), errp)) {
> > +            return;
> > +        }
> > +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 0,
> > +                        sc->memmap[ASPEED_DEV_SPI0 + i]);
> > +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 1,
> > +
> ASPEED_SMC_GET_CLASS(&s->spi[i])->flash_window_base);
> > +    }
> > +
> > +    /*
> > +     * SDMC - SDRAM Memory Controller
> > +     * The SDMC controller is unlocked at SPL stage.
> > +     * At present, only supports to emulate booting
> > +     * start from u-boot stage. Set SDMC controller
> > +     * unlocked by default. It is a temporarily solution.
> > +     */
> > +    object_property_set_bool(OBJECT(&s->sdmc), "unlocked", true,
> > +                                 &error_abort);
> > +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sdmc), errp)) {
> > +        return;
> > +    }
> > +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sdmc), 0,
> > +                    sc->memmap[ASPEED_DEV_SDMC]);
> > +
> > +    /* RAM */
> > +    if (!aspeed_soc_dram_init(s, errp)) {
> > +        return;
> > +    }
> > +
> > +    for (i = 0; i < sc->macs_num; i++) {
> > +        object_property_set_bool(OBJECT(&s->ftgmac100[i]), "aspeed",
> true,
> > +                                 &error_abort);
> > +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->ftgmac100[i]), errp)) {
> > +            return;
> > +        }
> > +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
> > +                        sc->memmap[ASPEED_DEV_ETH1 + i]);
> > +        sysbus_connect_irq(SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
> > +                           aspeed_soc_get_irq(s, ASPEED_DEV_ETH1
> +
> > + i));
> > +
> > +        object_property_set_link(OBJECT(&s->mii[i]), "nic",
> > +                                 OBJECT(&s->ftgmac100[i]),
> &error_abort);
> > +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->mii[i]), errp)) {
> > +            return;
> > +        }
> > +
> > +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->mii[i]), 0,
> > +                        sc->memmap[ASPEED_DEV_MII1 + i]);
> > +    }
> > +
> > +    /* Watch dog */
> > +    for (i = 0; i < sc->wdts_num; i++) {
> > +        AspeedWDTClass *awc = ASPEED_WDT_GET_CLASS(&s->wdt[i]);
> > +        hwaddr wdt_offset = sc->memmap[ASPEED_DEV_WDT] + i *
> > + awc->iosize;
> > +
> > +        object_property_set_link(OBJECT(&s->wdt[i]), "scu",
> OBJECT(&s->scu),
> > +                                 &error_abort);
> > +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->wdt[i]), errp)) {
> > +            return;
> > +        }
> > +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->wdt[i]), 0,
> wdt_offset);
> > +    }
> > +
> > +    /* SLI */
> > +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sli), errp)) {
> > +        return;
> > +    }
> > +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sli), 0,
> > + sc->memmap[ASPEED_DEV_SLI]);
> > +
> > +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sliio), errp)) {
> > +        return;
> > +    }
> > +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sliio), 0,
> > +                    sc->memmap[ASPEED_DEV_SLIIO]);
> > +
> > +    create_unimplemented_device("ast2700.dpmcu", 0x11000000,
> 0x40000);
> > +    create_unimplemented_device("ast2700.iomem0", 0x12000000,
> 0x01000000);
> > +    create_unimplemented_device("ast2700.iomem1", 0x14000000,
> 0x01000000);
> > +    create_unimplemented_device("ast2700.ltpi", 0x30000000,
> 0x1000000);
> > +    create_unimplemented_device("ast2700.io", 0x0, 0x4000000); }
> > +
> > +static void aspeed_soc_ast2700_class_init(ObjectClass *oc, void
> > +*data) {
> > +    static const char * const valid_cpu_types[] = {
> > +        ARM_CPU_TYPE_NAME("cortex-a35"),
> > +        NULL
> > +    };
> > +    DeviceClass *dc = DEVICE_CLASS(oc);
> > +    AspeedSoCClass *sc = ASPEED_SOC_CLASS(oc);
> > +
> > +    dc->realize      = aspeed_soc_ast2700_realize;
> 
> 
> Please add :
> 
>      /* Reason: The Aspeed SoC can only be instantiated from a board */
>      dc->user_creatable = false;
> 
> 
Will add.
> Thanks,
> 
> C.
> 
> 
> > +
> > +    sc->name         = "ast2700-a0";
> > +    sc->valid_cpu_types = valid_cpu_types;
> > +    sc->silicon_rev  = AST2700_A0_SILICON_REV;
> > +    sc->sram_size    = 0x20000;
> > +    sc->spis_num     = 3;
> > +    sc->wdts_num     = 8;
> > +    sc->macs_num     = 1;
> > +    sc->uarts_num    = 13;
> > +    sc->num_cpus     = 4;
> > +    sc->uarts_base   = ASPEED_DEV_UART0;
> > +    sc->irqmap       = aspeed_soc_ast2700_irqmap;
> > +    sc->memmap       = aspeed_soc_ast2700_memmap;
> > +    sc->get_irq      = aspeed_soc_ast2700_get_irq;
> > +}
> > +
> > +static const TypeInfo aspeed_soc_ast27x0_types[] = {
> > +    {
> > +        .name           = TYPE_ASPEED27X0_SOC,
> > +        .parent         = TYPE_ASPEED_SOC,
> > +        .instance_size  = sizeof(Aspeed27x0SoCState),
> > +        .abstract       = true,
> > +    }, {
> > +        .name           = "ast2700-a0",
> > +        .parent         = TYPE_ASPEED27X0_SOC,
> > +        .instance_init  = aspeed_soc_ast2700_init,
> > +        .class_init     = aspeed_soc_ast2700_class_init,
> > +    },
> > +};
> > +
> > +DEFINE_TYPES(aspeed_soc_ast27x0_types)
> > diff --git a/hw/arm/meson.build b/hw/arm/meson.build index
> > 6808135c1f..1e3295a423 100644
> > --- a/hw/arm/meson.build
> > +++ b/hw/arm/meson.build
> > @@ -46,6 +46,7 @@ arm_ss.add(when: 'CONFIG_ASPEED_SOC', if_true:
> files(
> >     'aspeed_soc_common.c',
> >     'aspeed_ast2400.c',
> >     'aspeed_ast2600.c',
> > +  'aspeed_ast27x0.c',
> >     'aspeed_ast10x0.c',
> >     'aspeed_eeprom.c',
> >     'fby35.c'))
> > diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h
> > index c60fac900a..9f177b6037 100644
> > --- a/include/hw/arm/aspeed_soc.h
> > +++ b/include/hw/arm/aspeed_soc.h
> > @@ -15,6 +15,7 @@
> >   #include "hw/cpu/a15mpcore.h"
> >   #include "hw/arm/armv7m.h"
> >   #include "hw/intc/aspeed_vic.h"
> > +#include "hw/intc/aspeed_intc.h"
> >   #include "hw/misc/aspeed_scu.h"
> >   #include "hw/adc/aspeed_adc.h"
> >   #include "hw/misc/aspeed_sdmc.h"
> > @@ -26,6 +27,7 @@
> >   #include "hw/ssi/aspeed_smc.h"
> >   #include "hw/misc/aspeed_hace.h"
> >   #include "hw/misc/aspeed_sbc.h"
> > +#include "hw/misc/aspeed_sli.h"
> >   #include "hw/watchdog/wdt_aspeed.h"
> >   #include "hw/net/ftgmac100.h"
> >   #include "target/arm/cpu.h"
> > @@ -41,8 +43,8 @@
> >
> >   #define ASPEED_SPIS_NUM  2
> >   #define ASPEED_EHCIS_NUM 2
> > -#define ASPEED_WDTS_NUM  4
> > -#define ASPEED_CPUS_NUM  2
> > +#define ASPEED_WDTS_NUM  8
> > +#define ASPEED_CPUS_NUM  4
> >   #define ASPEED_MACS_NUM  4
> >   #define ASPEED_UARTS_NUM 13
> >   #define ASPEED_JTAG_NUM  2
> > @@ -61,6 +63,7 @@ struct AspeedSoCState {
> >       AspeedI2CState i2c;
> >       AspeedI3CState i3c;
> >       AspeedSCUState scu;
> > +    AspeedSCUState scuio;
> >       AspeedHACEState hace;
> >       AspeedXDMAState xdma;
> >       AspeedADCState adc;
> > @@ -68,6 +71,8 @@ struct AspeedSoCState {
> >       AspeedSMCState spi[ASPEED_SPIS_NUM];
> >       EHCISysBusState ehci[ASPEED_EHCIS_NUM];
> >       AspeedSBCState sbc;
> > +    AspeedSLIState sli;
> > +    AspeedSLIState sliio;
> >       MemoryRegion secsram;
> >       UnimplementedDeviceState sbc_unimplemented;
> >       AspeedSDMCState sdmc;
> > @@ -117,6 +122,16 @@ struct Aspeed2600SoCState {
> >   #define TYPE_ASPEED2600_SOC "aspeed2600-soc"
> >   OBJECT_DECLARE_SIMPLE_TYPE(Aspeed2600SoCState,
> ASPEED2600_SOC)
> >
> > +struct Aspeed27x0SoCState {
> > +    AspeedSoCState parent;
> > +
> > +    ARMCPU cpu[ASPEED_CPUS_NUM];
> > +    AspeedINTCState intc;
> > +};
> > +
> > +#define TYPE_ASPEED27X0_SOC "aspeed27x0-soc"
> > +OBJECT_DECLARE_SIMPLE_TYPE(Aspeed27x0SoCState, ASPEED27X0_SOC)
> > +
> >   struct Aspeed10x0SoCState {
> >       AspeedSoCState parent;
> >
> > @@ -168,11 +183,13 @@ enum {
> >       ASPEED_DEV_UART13,
> >       ASPEED_DEV_VUART,
> >       ASPEED_DEV_FMC,
> > +    ASPEED_DEV_SPI0,
> >       ASPEED_DEV_SPI1,
> >       ASPEED_DEV_SPI2,
> >       ASPEED_DEV_EHCI1,
> >       ASPEED_DEV_EHCI2,
> >       ASPEED_DEV_VIC,
> > +    ASPEED_DEV_INTC,
> >       ASPEED_DEV_SDMC,
> >       ASPEED_DEV_SCU,
> >       ASPEED_DEV_ADC,
> > @@ -222,6 +239,11 @@ enum {
> >       ASPEED_DEV_JTAG1,
> >       ASPEED_DEV_FSI1,
> >       ASPEED_DEV_FSI2,
> > +    ASPEED_DEV_SCUIO,
> > +    ASPEED_DEV_SLI,
> > +    ASPEED_DEV_SLIIO,
> > +    ASPEED_GIC_DIST,
> > +    ASPEED_GIC_REDIST,
> >   };
> >
> >   qemu_irq aspeed_soc_get_irq(AspeedSoCState *s, int dev);
Cédric Le Goater May 7, 2024, 1:06 p.m. UTC | #3
Peter,

Could you please look at aspeed_soc_ast2700_gic() when you have some time ?
My GIC knowledge is a bit limited and I would feel more confortable with
your feedback.

The rest looks good to me.

Thanks,

C.



On 4/16/24 11:18, Jamin Lin wrote:
> Initial definitions for a simple machine using an AST2700 SOC (Cortex-a35 CPU).
> 
> AST2700 SOC and its interrupt controller are too complex to handle
> in the common Aspeed SoC framework. We introduce a new ast2700
> class with instance_init and realize handlers.
> 
> AST2700 is a 64 bits quad core cpus and support 8 watchdog.
> Update maximum ASPEED_CPUS_NUM to 4 and ASPEED_WDTS_NUM to 8.
> In addition, update AspeedSocState to support scuio, sli, sliio and intc.
> 
> Add TYPE_ASPEED27X0_SOC machine type.
> 
> The SDMC controller is unlocked at SPL stage.
> At present, only supports to emulate booting
> start from u-boot stage. Set SDMC controller
> unlocked by default.
> 
> In INTC, each interrupt of INT 128 to INT 136 combines 32 interrupts.
> It connect GICINT IRQ GPIO-OUTPUT pins to GIC device with irq 128 to 136.
> And, if a device irq is 128 to 136, its irq GPIO-OUTPUT pin is connected to
> GICINT or-gates instead of GIC device.
> 
> Signed-off-by: Troy Lee <troy_lee@aspeedtech.com>
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> ---
>   hw/arm/aspeed_ast27x0.c     | 554 ++++++++++++++++++++++++++++++++++++
>   hw/arm/meson.build          |   1 +
>   include/hw/arm/aspeed_soc.h |  26 +-
>   3 files changed, 579 insertions(+), 2 deletions(-)
>   create mode 100644 hw/arm/aspeed_ast27x0.c
> 
> diff --git a/hw/arm/aspeed_ast27x0.c b/hw/arm/aspeed_ast27x0.c
> new file mode 100644
> index 0000000000..754c963230
> --- /dev/null
> +++ b/hw/arm/aspeed_ast27x0.c
> @@ -0,0 +1,554 @@
> +/*
> + * ASPEED SoC 27x0 family
> + *
> + * Copyright (C) 2024 ASPEED Technology Inc.
> + *
> + * This code is licensed under the GPL version 2 or later.  See
> + * the COPYING file in the top-level directory.
> + *
> + * Implementation extracted from the AST2600 and adapted for AST27x0.
> + */
> +
> +#include "qemu/osdep.h"
> +#include "qapi/error.h"
> +#include "hw/misc/unimp.h"
> +#include "hw/arm/aspeed_soc.h"
> +#include "qemu/module.h"
> +#include "qemu/error-report.h"
> +#include "hw/i2c/aspeed_i2c.h"
> +#include "net/net.h"
> +#include "sysemu/sysemu.h"
> +#include "hw/intc/arm_gicv3.h"
> +#include "qapi/qmp/qlist.h"
> +
> +static const hwaddr aspeed_soc_ast2700_memmap[] = {
> +    [ASPEED_DEV_SPI_BOOT]  =  0x400000000,
> +    [ASPEED_DEV_SRAM]      =  0x10000000,
> +    [ASPEED_DEV_SDMC]      =  0x12C00000,
> +    [ASPEED_DEV_SCU]       =  0x12C02000,
> +    [ASPEED_DEV_SCUIO]     =  0x14C02000,
> +    [ASPEED_DEV_UART0]     =  0X14C33000,
> +    [ASPEED_DEV_UART1]     =  0X14C33100,
> +    [ASPEED_DEV_UART2]     =  0X14C33200,
> +    [ASPEED_DEV_UART3]     =  0X14C33300,
> +    [ASPEED_DEV_UART4]     =  0X12C1A000,
> +    [ASPEED_DEV_UART5]     =  0X14C33400,
> +    [ASPEED_DEV_UART6]     =  0X14C33500,
> +    [ASPEED_DEV_UART7]     =  0X14C33600,
> +    [ASPEED_DEV_UART8]     =  0X14C33700,
> +    [ASPEED_DEV_UART9]     =  0X14C33800,
> +    [ASPEED_DEV_UART10]    =  0X14C33900,
> +    [ASPEED_DEV_UART11]    =  0X14C33A00,
> +    [ASPEED_DEV_UART12]    =  0X14C33B00,
> +    [ASPEED_DEV_WDT]       =  0x14C37000,
> +    [ASPEED_DEV_VUART]     =  0X14C30000,
> +    [ASPEED_DEV_FMC]       =  0x14000000,
> +    [ASPEED_DEV_SPI0]      =  0x14010000,
> +    [ASPEED_DEV_SPI1]      =  0x14020000,
> +    [ASPEED_DEV_SPI2]      =  0x14030000,
> +    [ASPEED_DEV_SDRAM]     =  0x400000000,
> +    [ASPEED_DEV_MII1]      =  0x14040000,
> +    [ASPEED_DEV_MII2]      =  0x14040008,
> +    [ASPEED_DEV_MII3]      =  0x14040010,
> +    [ASPEED_DEV_ETH1]      =  0x14050000,
> +    [ASPEED_DEV_ETH2]      =  0x14060000,
> +    [ASPEED_DEV_ETH3]      =  0x14070000,
> +    [ASPEED_DEV_EMMC]      =  0x12090000,
> +    [ASPEED_DEV_INTC]      =  0x12100000,
> +    [ASPEED_DEV_SLI]       =  0x12C17000,
> +    [ASPEED_DEV_SLIIO]     =  0x14C1E000,
> +    [ASPEED_GIC_DIST]      =  0x12200000,
> +    [ASPEED_GIC_REDIST]    =  0x12280000,
> +};
> +
> +#define AST2700_MAX_IRQ 288
> +
> +/* Shared Peripheral Interrupt values below are offset by -32 from datasheet */
> +static const int aspeed_soc_ast2700_irqmap[] = {
> +    [ASPEED_DEV_UART0]     = 132,
> +    [ASPEED_DEV_UART1]     = 132,
> +    [ASPEED_DEV_UART2]     = 132,
> +    [ASPEED_DEV_UART3]     = 132,
> +    [ASPEED_DEV_UART4]     = 8,
> +    [ASPEED_DEV_UART5]     = 132,
> +    [ASPEED_DEV_UART6]     = 132,
> +    [ASPEED_DEV_UART7]     = 132,
> +    [ASPEED_DEV_UART8]     = 132,
> +    [ASPEED_DEV_UART9]     = 132,
> +    [ASPEED_DEV_UART10]    = 132,
> +    [ASPEED_DEV_UART11]    = 132,
> +    [ASPEED_DEV_UART12]    = 132,
> +    [ASPEED_DEV_FMC]       = 131,
> +    [ASPEED_DEV_SDMC]      = 0,
> +    [ASPEED_DEV_SCU]       = 12,
> +    [ASPEED_DEV_ADC]       = 130,
> +    [ASPEED_DEV_XDMA]      = 5,
> +    [ASPEED_DEV_EMMC]      = 15,
> +    [ASPEED_DEV_GPIO]      = 11,
> +    [ASPEED_DEV_GPIO_1_8V] = 130,
> +    [ASPEED_DEV_RTC]       = 13,
> +    [ASPEED_DEV_TIMER1]    = 16,
> +    [ASPEED_DEV_TIMER2]    = 17,
> +    [ASPEED_DEV_TIMER3]    = 18,
> +    [ASPEED_DEV_TIMER4]    = 19,
> +    [ASPEED_DEV_TIMER5]    = 20,
> +    [ASPEED_DEV_TIMER6]    = 21,
> +    [ASPEED_DEV_TIMER7]    = 22,
> +    [ASPEED_DEV_TIMER8]    = 23,
> +    [ASPEED_DEV_WDT]       = 131,
> +    [ASPEED_DEV_PWM]       = 131,
> +    [ASPEED_DEV_LPC]       = 128,
> +    [ASPEED_DEV_IBT]       = 128,
> +    [ASPEED_DEV_I2C]       = 130,
> +    [ASPEED_DEV_PECI]      = 133,
> +    [ASPEED_DEV_ETH1]      = 132,
> +    [ASPEED_DEV_ETH2]      = 132,
> +    [ASPEED_DEV_ETH3]      = 132,
> +    [ASPEED_DEV_HACE]      = 4,
> +    [ASPEED_DEV_KCS]       = 128,
> +    [ASPEED_DEV_DP]        = 28,
> +    [ASPEED_DEV_I3C]       = 131,
> +};
> +
> +/* GICINT 128 */
> +static const int aspeed_soc_ast2700_gic128_intcmap[] = {
> +    [ASPEED_DEV_LPC]       = 0,
> +    [ASPEED_DEV_IBT]       = 2,
> +    [ASPEED_DEV_KCS]       = 4,
> +};
> +
> +/* GICINT 130 */
> +static const int aspeed_soc_ast2700_gic130_intcmap[] = {
> +    [ASPEED_DEV_I2C]        = 0,
> +    [ASPEED_DEV_ADC]        = 16,
> +    [ASPEED_DEV_GPIO_1_8V]  = 18,
> +};
> +
> +/* GICINT 131 */
> +static const int aspeed_soc_ast2700_gic131_intcmap[] = {
> +    [ASPEED_DEV_I3C]       = 0,
> +    [ASPEED_DEV_WDT]       = 16,
> +    [ASPEED_DEV_FMC]       = 25,
> +    [ASPEED_DEV_PWM]       = 29,
> +};
> +
> +/* GICINT 132 */
> +static const int aspeed_soc_ast2700_gic132_intcmap[] = {
> +    [ASPEED_DEV_ETH1]      = 0,
> +    [ASPEED_DEV_ETH2]      = 1,
> +    [ASPEED_DEV_ETH3]      = 2,
> +    [ASPEED_DEV_UART0]     = 7,
> +    [ASPEED_DEV_UART1]     = 8,
> +    [ASPEED_DEV_UART2]     = 9,
> +    [ASPEED_DEV_UART3]     = 10,
> +    [ASPEED_DEV_UART5]     = 11,
> +    [ASPEED_DEV_UART6]     = 12,
> +    [ASPEED_DEV_UART7]     = 13,
> +    [ASPEED_DEV_UART8]     = 14,
> +    [ASPEED_DEV_UART9]     = 15,
> +    [ASPEED_DEV_UART10]    = 16,
> +    [ASPEED_DEV_UART11]    = 17,
> +    [ASPEED_DEV_UART12]    = 18,
> +};
> +
> +/* GICINT 133 */
> +static const int aspeed_soc_ast2700_gic133_intcmap[] = {
> +    [ASPEED_DEV_PECI]      = 4,
> +};
> +
> +/* GICINT 128 ~ 136 */
> +struct gic_intc_irq_info {
> +    int irq;
> +    const int *ptr;
> +};
> +
> +static const struct gic_intc_irq_info aspeed_soc_ast2700_gic_intcmap[] = {
> +    {128,  aspeed_soc_ast2700_gic128_intcmap},
> +    {129,  NULL},
> +    {130,  aspeed_soc_ast2700_gic130_intcmap},
> +    {131,  aspeed_soc_ast2700_gic131_intcmap},
> +    {132,  aspeed_soc_ast2700_gic132_intcmap},
> +    {133,  aspeed_soc_ast2700_gic133_intcmap},
> +    {134,  NULL},
> +    {135,  NULL},
> +    {136,  NULL},
> +};
> +
> +static qemu_irq aspeed_soc_ast2700_get_irq(AspeedSoCState *s, int dev)
> +{
> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(s);
> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> +    int i;
> +
> +    for (i = 0; i < ARRAY_SIZE(aspeed_soc_ast2700_gic_intcmap); i++) {
> +        if (sc->irqmap[dev] == aspeed_soc_ast2700_gic_intcmap[i].irq) {
> +            assert(aspeed_soc_ast2700_gic_intcmap[i].ptr);
> +            return qdev_get_gpio_in(DEVICE(&a->intc.gicint_orgate[i]),
> +                aspeed_soc_ast2700_gic_intcmap[i].ptr[dev]);
> +        }
> +    }
> +
> +    return qdev_get_gpio_in(a->intc.gic, sc->irqmap[dev]);
> +}
> +
> +static void aspeed_soc_ast2700_init(Object *obj)
> +{
> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(obj);
> +    AspeedSoCState *s = ASPEED_SOC(obj);
> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> +    int i;
> +    char socname[8];
> +    char typename[64];
> +
> +    if (sscanf(sc->name, "%7s", socname) != 1) {
> +        g_assert_not_reached();
> +    }
> +
> +    for (i = 0; i < sc->num_cpus; i++) {
> +        object_initialize_child(obj, "cpu[*]", &a->cpu[i],
> +                                aspeed_soc_cpu_type(sc));
> +    }
> +
> +    object_initialize_child(obj, "scu", &s->scu, TYPE_ASPEED_2700_SCU);
> +    qdev_prop_set_uint32(DEVICE(&s->scu), "silicon-rev",
> +                         sc->silicon_rev);
> +    object_property_add_alias(obj, "hw-strap1", OBJECT(&s->scu),
> +                              "hw-strap1");
> +    object_property_add_alias(obj, "hw-strap2", OBJECT(&s->scu),
> +                              "hw-strap2");
> +    object_property_add_alias(obj, "hw-prot-key", OBJECT(&s->scu),
> +                              "hw-prot-key");
> +
> +    object_initialize_child(obj, "scuio", &s->scuio, TYPE_ASPEED_2700_SCUIO);
> +    qdev_prop_set_uint32(DEVICE(&s->scuio), "silicon-rev",
> +                         sc->silicon_rev);
> +
> +    snprintf(typename, sizeof(typename), "aspeed.fmc-%s", socname);
> +    object_initialize_child(obj, "fmc", &s->fmc, typename);
> +
> +    for (i = 0; i < sc->spis_num; i++) {
> +        snprintf(typename, sizeof(typename), "aspeed.spi%d-%s", i, socname);
> +        object_initialize_child(obj, "spi[*]", &s->spi[i], typename);
> +    }
> +
> +    snprintf(typename, sizeof(typename), "aspeed.sdmc-%s", socname);
> +    object_initialize_child(obj, "sdmc", &s->sdmc, typename);
> +    object_property_add_alias(obj, "ram-size", OBJECT(&s->sdmc),
> +                              "ram-size");
> +
> +    for (i = 0; i < sc->wdts_num; i++) {
> +        snprintf(typename, sizeof(typename), "aspeed.wdt-%s", socname);
> +        object_initialize_child(obj, "wdt[*]", &s->wdt[i], typename);
> +    }
> +
> +    for (i = 0; i < sc->macs_num; i++) {
> +        object_initialize_child(obj, "ftgmac100[*]", &s->ftgmac100[i],
> +                                TYPE_FTGMAC100);
> +
> +        object_initialize_child(obj, "mii[*]", &s->mii[i], TYPE_ASPEED_MII);
> +    }
> +
> +    for (i = 0; i < sc->uarts_num; i++) {
> +        object_initialize_child(obj, "uart[*]", &s->uart[i], TYPE_SERIAL_MM);
> +    }
> +
> +    object_initialize_child(obj, "sli", &s->sli, TYPE_ASPEED_2700_SLI);
> +    object_initialize_child(obj, "sliio", &s->sliio, TYPE_ASPEED_2700_SLIIO);
> +    object_initialize_child(obj, "intc", &a->intc, TYPE_ASPEED_INTC);
> +}
> +
> +/*
> + * ASPEED ast2700 has 0x0 as cluster ID
> + *
> + * https://developer.arm.com/documentation/100236/0100/register-descriptions/aarch64-system-registers/multiprocessor-affinity-register--el1
> + */
> +static uint64_t aspeed_calc_affinity(int cpu)
> +{
> +    return (0x0 << ARM_AFF1_SHIFT) | cpu;
> +}
> +
> +static void aspeed_soc_ast2700_gic(DeviceState *dev, Error **errp)
> +{
> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
> +    AspeedSoCState *s = ASPEED_SOC(dev);
> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> +    SysBusDevice *gicbusdev;
> +    QList *redist_region_count;
> +    int i;
> +
> +    a->intc.gic = qdev_new(gicv3_class_name());
> +    qdev_prop_set_uint32(a->intc.gic, "revision", 3);
> +    qdev_prop_set_uint32(a->intc.gic, "num-cpu", sc->num_cpus);
> +    qdev_prop_set_uint32(a->intc.gic, "num-irq", AST2700_MAX_IRQ);
> +
> +    redist_region_count = qlist_new();
> +    qlist_append_int(redist_region_count, sc->num_cpus);
> +    qdev_prop_set_array(a->intc.gic, "redist-region-count",
> +                            redist_region_count);
> +
> +    gicbusdev = SYS_BUS_DEVICE(a->intc.gic);
> +    sysbus_realize_and_unref(gicbusdev, errp);
> +    sysbus_mmio_map(gicbusdev, 0, sc->memmap[ASPEED_GIC_DIST]);
> +    sysbus_mmio_map(gicbusdev, 1, sc->memmap[ASPEED_GIC_REDIST]);
> +
> +    for (i = 0; i < sc->num_cpus; i++) {
> +        DeviceState *cpudev = DEVICE(qemu_get_cpu(i));
> +        int NUM_IRQS = 256, ARCH_GIC_MAINT_IRQ = 9, VIRTUAL_PMU_IRQ = 7;
> +        int ppibase = NUM_IRQS + i * GIC_INTERNAL + GIC_NR_SGIS;
> +
> +        const int timer_irq[] = {
> +            [GTIMER_PHYS] = 14,
> +            [GTIMER_VIRT] = 11,
> +            [GTIMER_HYP]  = 10,
> +            [GTIMER_SEC]  = 13,
> +        };
> +        int j;
> +
> +        for (j = 0; j < ARRAY_SIZE(timer_irq); j++) {
> +            qdev_connect_gpio_out(cpudev, j,
> +                    qdev_get_gpio_in(a->intc.gic, ppibase + timer_irq[j]));
> +        }
> +
> +        qemu_irq irq = qdev_get_gpio_in(a->intc.gic,
> +                                        ppibase + ARCH_GIC_MAINT_IRQ);
> +        qdev_connect_gpio_out_named(cpudev, "gicv3-maintenance-interrupt",
> +                                    0, irq);
> +        qdev_connect_gpio_out_named(cpudev, "pmu-interrupt", 0,
> +                qdev_get_gpio_in(a->intc.gic, ppibase + VIRTUAL_PMU_IRQ));
> +
> +        sysbus_connect_irq(gicbusdev, i, qdev_get_gpio_in(cpudev, ARM_CPU_IRQ));
> +        sysbus_connect_irq(gicbusdev, i + sc->num_cpus,
> +                           qdev_get_gpio_in(cpudev, ARM_CPU_FIQ));
> +        sysbus_connect_irq(gicbusdev, i + 2 * sc->num_cpus,
> +                           qdev_get_gpio_in(cpudev, ARM_CPU_VIRQ));
> +        sysbus_connect_irq(gicbusdev, i + 3 * sc->num_cpus,
> +                           qdev_get_gpio_in(cpudev, ARM_CPU_VFIQ));
> +    }
> +}
> +
> +static void aspeed_soc_ast2700_realize(DeviceState *dev, Error **errp)
> +{
> +    int i;
> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
> +    AspeedSoCState *s = ASPEED_SOC(dev);
> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> +    g_autofree char *sram_name = NULL;
> +
> +    /* Default boot region (SPI memory or ROMs) */
> +    memory_region_init(&s->spi_boot_container, OBJECT(s),
> +                       "aspeed.spi_boot_container", 0x400000000);
> +    memory_region_add_subregion(s->memory, sc->memmap[ASPEED_DEV_SPI_BOOT],
> +                                &s->spi_boot_container);
> +
> +    /* CPU */
> +    for (i = 0; i < sc->num_cpus; i++) {
> +        object_property_set_int(OBJECT(&a->cpu[i]), "mp-affinity",
> +                                aspeed_calc_affinity(i), &error_abort);
> +
> +        object_property_set_int(OBJECT(&a->cpu[i]), "cntfrq", 1125000000,
> +                                &error_abort);
> +        object_property_set_link(OBJECT(&a->cpu[i]), "memory",
> +                                 OBJECT(s->memory), &error_abort);
> +
> +        if (!qdev_realize(DEVICE(&a->cpu[i]), NULL, errp)) {
> +            return;
> +        }
> +    }
> +
> +    /* GIC */
> +    aspeed_soc_ast2700_gic(dev, errp);
> +
> +    /* INTC */
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&a->intc), errp)) {
> +        return;
> +    }
> +
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&a->intc), 0,
> +                    sc->memmap[ASPEED_DEV_INTC]);
> +
> +    /* GICINT orgate -> INTC -> GIC */
> +    for (i = 0; i < ASPEED_INTC_NR_GICS; i++) {
> +        if (!qdev_realize(DEVICE(&a->intc.gicint_orgate[i]), NULL, errp)) {
> +            return;
> +        }
> +
> +        qdev_connect_gpio_out(DEVICE(&a->intc.gicint_orgate[i]), 0,
> +                              qdev_get_gpio_in(DEVICE(&a->intc), i));
> +
> +        sysbus_connect_irq(SYS_BUS_DEVICE(&a->intc), i,
> +                           qdev_get_gpio_in(a->intc.gic,
> +                                aspeed_soc_ast2700_gic_intcmap[i].irq));
> +    }
> +
> +    /* SRAM */
> +    sram_name = g_strdup_printf("aspeed.sram.%d", CPU(&a->cpu[0])->cpu_index);
> +    if (!memory_region_init_ram(&s->sram, OBJECT(s), sram_name, sc->sram_size,
> +                                 errp)) {
> +        return;
> +    }
> +    memory_region_add_subregion(s->memory,
> +                                sc->memmap[ASPEED_DEV_SRAM], &s->sram);
> +
> +    /* SCU */
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scu), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scu), 0, sc->memmap[ASPEED_DEV_SCU]);
> +
> +    /* SCU1 */
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scuio), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scuio), 0,
> +                    sc->memmap[ASPEED_DEV_SCUIO]);
> +
> +    /* UART */
> +    if (!aspeed_soc_uart_realize(s, errp)) {
> +        return;
> +    }
> +
> +    /* FMC, The number of CS is set at the board level */
> +    object_property_set_link(OBJECT(&s->fmc), "dram", OBJECT(s->dram_mr),
> +                             &error_abort);
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->fmc), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 0, sc->memmap[ASPEED_DEV_FMC]);
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 1,
> +                    ASPEED_SMC_GET_CLASS(&s->fmc)->flash_window_base);
> +    sysbus_connect_irq(SYS_BUS_DEVICE(&s->fmc), 0,
> +                       aspeed_soc_get_irq(s, ASPEED_DEV_FMC));
> +
> +    /* Set up an alias on the FMC CE0 region (boot default) */
> +    MemoryRegion *fmc0_mmio = &s->fmc.flashes[0].mmio;
> +    memory_region_init_alias(&s->spi_boot, OBJECT(s), "aspeed.spi_boot",
> +                             fmc0_mmio, 0, memory_region_size(fmc0_mmio));
> +    memory_region_add_subregion(&s->spi_boot_container, 0x0, &s->spi_boot);
> +
> +    /* SPI */
> +    for (i = 0; i < sc->spis_num; i++) {
> +        object_property_set_link(OBJECT(&s->spi[i]), "dram",
> +                                 OBJECT(s->dram_mr), &error_abort);
> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->spi[i]), errp)) {
> +            return;
> +        }
> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 0,
> +                        sc->memmap[ASPEED_DEV_SPI0 + i]);
> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 1,
> +                        ASPEED_SMC_GET_CLASS(&s->spi[i])->flash_window_base);
> +    }
> +
> +    /*
> +     * SDMC - SDRAM Memory Controller
> +     * The SDMC controller is unlocked at SPL stage.
> +     * At present, only supports to emulate booting
> +     * start from u-boot stage. Set SDMC controller
> +     * unlocked by default. It is a temporarily solution.
> +     */
> +    object_property_set_bool(OBJECT(&s->sdmc), "unlocked", true,
> +                                 &error_abort);
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sdmc), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sdmc), 0,
> +                    sc->memmap[ASPEED_DEV_SDMC]);
> +
> +    /* RAM */
> +    if (!aspeed_soc_dram_init(s, errp)) {
> +        return;
> +    }
> +
> +    for (i = 0; i < sc->macs_num; i++) {
> +        object_property_set_bool(OBJECT(&s->ftgmac100[i]), "aspeed", true,
> +                                 &error_abort);
> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->ftgmac100[i]), errp)) {
> +            return;
> +        }
> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
> +                        sc->memmap[ASPEED_DEV_ETH1 + i]);
> +        sysbus_connect_irq(SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
> +                           aspeed_soc_get_irq(s, ASPEED_DEV_ETH1 + i));
> +
> +        object_property_set_link(OBJECT(&s->mii[i]), "nic",
> +                                 OBJECT(&s->ftgmac100[i]), &error_abort);
> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->mii[i]), errp)) {
> +            return;
> +        }
> +
> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->mii[i]), 0,
> +                        sc->memmap[ASPEED_DEV_MII1 + i]);
> +    }
> +
> +    /* Watch dog */
> +    for (i = 0; i < sc->wdts_num; i++) {
> +        AspeedWDTClass *awc = ASPEED_WDT_GET_CLASS(&s->wdt[i]);
> +        hwaddr wdt_offset = sc->memmap[ASPEED_DEV_WDT] + i * awc->iosize;
> +
> +        object_property_set_link(OBJECT(&s->wdt[i]), "scu", OBJECT(&s->scu),
> +                                 &error_abort);
> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->wdt[i]), errp)) {
> +            return;
> +        }
> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->wdt[i]), 0, wdt_offset);
> +    }
> +
> +    /* SLI */
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sli), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sli), 0, sc->memmap[ASPEED_DEV_SLI]);
> +
> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sliio), errp)) {
> +        return;
> +    }
> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sliio), 0,
> +                    sc->memmap[ASPEED_DEV_SLIIO]);
> +
> +    create_unimplemented_device("ast2700.dpmcu", 0x11000000, 0x40000);
> +    create_unimplemented_device("ast2700.iomem0", 0x12000000, 0x01000000);
> +    create_unimplemented_device("ast2700.iomem1", 0x14000000, 0x01000000);
> +    create_unimplemented_device("ast2700.ltpi", 0x30000000, 0x1000000);
> +    create_unimplemented_device("ast2700.io", 0x0, 0x4000000);
> +}
> +
> +static void aspeed_soc_ast2700_class_init(ObjectClass *oc, void *data)
> +{
> +    static const char * const valid_cpu_types[] = {
> +        ARM_CPU_TYPE_NAME("cortex-a35"),
> +        NULL
> +    };
> +    DeviceClass *dc = DEVICE_CLASS(oc);
> +    AspeedSoCClass *sc = ASPEED_SOC_CLASS(oc);
> +
> +    dc->realize      = aspeed_soc_ast2700_realize;
> +
> +    sc->name         = "ast2700-a0";
> +    sc->valid_cpu_types = valid_cpu_types;
> +    sc->silicon_rev  = AST2700_A0_SILICON_REV;
> +    sc->sram_size    = 0x20000;
> +    sc->spis_num     = 3;
> +    sc->wdts_num     = 8;
> +    sc->macs_num     = 1;
> +    sc->uarts_num    = 13;
> +    sc->num_cpus     = 4;
> +    sc->uarts_base   = ASPEED_DEV_UART0;
> +    sc->irqmap       = aspeed_soc_ast2700_irqmap;
> +    sc->memmap       = aspeed_soc_ast2700_memmap;
> +    sc->get_irq      = aspeed_soc_ast2700_get_irq;
> +}
> +
> +static const TypeInfo aspeed_soc_ast27x0_types[] = {
> +    {
> +        .name           = TYPE_ASPEED27X0_SOC,
> +        .parent         = TYPE_ASPEED_SOC,
> +        .instance_size  = sizeof(Aspeed27x0SoCState),
> +        .abstract       = true,
> +    }, {
> +        .name           = "ast2700-a0",
> +        .parent         = TYPE_ASPEED27X0_SOC,
> +        .instance_init  = aspeed_soc_ast2700_init,
> +        .class_init     = aspeed_soc_ast2700_class_init,
> +    },
> +};
> +
> +DEFINE_TYPES(aspeed_soc_ast27x0_types)
> diff --git a/hw/arm/meson.build b/hw/arm/meson.build
> index 6808135c1f..1e3295a423 100644
> --- a/hw/arm/meson.build
> +++ b/hw/arm/meson.build
> @@ -46,6 +46,7 @@ arm_ss.add(when: 'CONFIG_ASPEED_SOC', if_true: files(
>     'aspeed_soc_common.c',
>     'aspeed_ast2400.c',
>     'aspeed_ast2600.c',
> +  'aspeed_ast27x0.c',
>     'aspeed_ast10x0.c',
>     'aspeed_eeprom.c',
>     'fby35.c'))
> diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h
> index c60fac900a..9f177b6037 100644
> --- a/include/hw/arm/aspeed_soc.h
> +++ b/include/hw/arm/aspeed_soc.h
> @@ -15,6 +15,7 @@
>   #include "hw/cpu/a15mpcore.h"
>   #include "hw/arm/armv7m.h"
>   #include "hw/intc/aspeed_vic.h"
> +#include "hw/intc/aspeed_intc.h"
>   #include "hw/misc/aspeed_scu.h"
>   #include "hw/adc/aspeed_adc.h"
>   #include "hw/misc/aspeed_sdmc.h"
> @@ -26,6 +27,7 @@
>   #include "hw/ssi/aspeed_smc.h"
>   #include "hw/misc/aspeed_hace.h"
>   #include "hw/misc/aspeed_sbc.h"
> +#include "hw/misc/aspeed_sli.h"
>   #include "hw/watchdog/wdt_aspeed.h"
>   #include "hw/net/ftgmac100.h"
>   #include "target/arm/cpu.h"
> @@ -41,8 +43,8 @@
>   
>   #define ASPEED_SPIS_NUM  2
>   #define ASPEED_EHCIS_NUM 2
> -#define ASPEED_WDTS_NUM  4
> -#define ASPEED_CPUS_NUM  2
> +#define ASPEED_WDTS_NUM  8
> +#define ASPEED_CPUS_NUM  4
>   #define ASPEED_MACS_NUM  4
>   #define ASPEED_UARTS_NUM 13
>   #define ASPEED_JTAG_NUM  2
> @@ -61,6 +63,7 @@ struct AspeedSoCState {
>       AspeedI2CState i2c;
>       AspeedI3CState i3c;
>       AspeedSCUState scu;
> +    AspeedSCUState scuio;
>       AspeedHACEState hace;
>       AspeedXDMAState xdma;
>       AspeedADCState adc;
> @@ -68,6 +71,8 @@ struct AspeedSoCState {
>       AspeedSMCState spi[ASPEED_SPIS_NUM];
>       EHCISysBusState ehci[ASPEED_EHCIS_NUM];
>       AspeedSBCState sbc;
> +    AspeedSLIState sli;
> +    AspeedSLIState sliio;
>       MemoryRegion secsram;
>       UnimplementedDeviceState sbc_unimplemented;
>       AspeedSDMCState sdmc;
> @@ -117,6 +122,16 @@ struct Aspeed2600SoCState {
>   #define TYPE_ASPEED2600_SOC "aspeed2600-soc"
>   OBJECT_DECLARE_SIMPLE_TYPE(Aspeed2600SoCState, ASPEED2600_SOC)
>   
> +struct Aspeed27x0SoCState {
> +    AspeedSoCState parent;
> +
> +    ARMCPU cpu[ASPEED_CPUS_NUM];
> +    AspeedINTCState intc;
> +};
> +
> +#define TYPE_ASPEED27X0_SOC "aspeed27x0-soc"
> +OBJECT_DECLARE_SIMPLE_TYPE(Aspeed27x0SoCState, ASPEED27X0_SOC)
> +
>   struct Aspeed10x0SoCState {
>       AspeedSoCState parent;
>   
> @@ -168,11 +183,13 @@ enum {
>       ASPEED_DEV_UART13,
>       ASPEED_DEV_VUART,
>       ASPEED_DEV_FMC,
> +    ASPEED_DEV_SPI0,
>       ASPEED_DEV_SPI1,
>       ASPEED_DEV_SPI2,
>       ASPEED_DEV_EHCI1,
>       ASPEED_DEV_EHCI2,
>       ASPEED_DEV_VIC,
> +    ASPEED_DEV_INTC,
>       ASPEED_DEV_SDMC,
>       ASPEED_DEV_SCU,
>       ASPEED_DEV_ADC,
> @@ -222,6 +239,11 @@ enum {
>       ASPEED_DEV_JTAG1,
>       ASPEED_DEV_FSI1,
>       ASPEED_DEV_FSI2,
> +    ASPEED_DEV_SCUIO,
> +    ASPEED_DEV_SLI,
> +    ASPEED_DEV_SLIIO,
> +    ASPEED_GIC_DIST,
> +    ASPEED_GIC_REDIST,
>   };
>   
>   qemu_irq aspeed_soc_get_irq(AspeedSoCState *s, int dev);
Cédric Le Goater May 17, 2024, 7:58 a.m. UTC | #4
On 4/19/24 09:58, Jamin Lin wrote:
> Hi Cedric,
>> On 4/16/24 11:18, Jamin Lin wrote:
>>> Initial definitions for a simple machine using an AST2700 SOC (Cortex-a35
>> CPU).
>>>
>>> AST2700 SOC and its interrupt controller are too complex to handle in
>>> the common Aspeed SoC framework. We introduce a new ast2700 class with
>>> instance_init and realize handlers.
>>>
>>> AST2700 is a 64 bits quad core cpus and support 8 watchdog.
>>> Update maximum ASPEED_CPUS_NUM to 4 and ASPEED_WDTS_NUM to 8.
>>> In addition, update AspeedSocState to support scuio, sli, sliio and intc.
>>>
>>> Add TYPE_ASPEED27X0_SOC machine type.
>>>
>>> The SDMC controller is unlocked at SPL stage.
>>> At present, only supports to emulate booting start from u-boot stage.
>>> Set SDMC controller unlocked by default.
>>>
>>> In INTC, each interrupt of INT 128 to INT 136 combines 32 interrupts.
>>> It connect GICINT IRQ GPIO-OUTPUT pins to GIC device with irq 128 to 136.
>>> And, if a device irq is 128 to 136, its irq GPIO-OUTPUT pin is
>>> connected to GICINT or-gates instead of GIC device.
>>>
>>> Signed-off-by: Troy Lee <troy_lee@aspeedtech.com>
>>> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
>>
>> Before I forget, please see a little comment below regarding user creatable
>> devices.
>>
>> The model looks fine. The interrupt controller part is more complex than the
>> previous SoCs so I will come back to it later when I have more time.
> Thanks for your kindly support.
>>> ---
>>>    hw/arm/aspeed_ast27x0.c     | 554
>> ++++++++++++++++++++++++++++++++++++
>>>    hw/arm/meson.build          |   1 +
>>>    include/hw/arm/aspeed_soc.h |  26 +-
>>>    3 files changed, 579 insertions(+), 2 deletions(-)
>>>    create mode 100644 hw/arm/aspeed_ast27x0.c
>>>
>>> diff --git a/hw/arm/aspeed_ast27x0.c b/hw/arm/aspeed_ast27x0.c new
>>> file mode 100644 index 0000000000..754c963230
>>> --- /dev/null
>>> +++ b/hw/arm/aspeed_ast27x0.c
>>> @@ -0,0 +1,554 @@
>>> +/*
>>> + * ASPEED SoC 27x0 family
>>> + *
>>> + * Copyright (C) 2024 ASPEED Technology Inc.
>>> + *
>>> + * This code is licensed under the GPL version 2 or later.  See
>>> + * the COPYING file in the top-level directory.
>>> + *
>>> + * Implementation extracted from the AST2600 and adapted for AST27x0.
>>> + */
>>> +
>>> +#include "qemu/osdep.h"
>>> +#include "qapi/error.h"
>>> +#include "hw/misc/unimp.h"
>>> +#include "hw/arm/aspeed_soc.h"
>>> +#include "qemu/module.h"
>>> +#include "qemu/error-report.h"
>>> +#include "hw/i2c/aspeed_i2c.h"
>>> +#include "net/net.h"
>>> +#include "sysemu/sysemu.h"
>>> +#include "hw/intc/arm_gicv3.h"
>>> +#include "qapi/qmp/qlist.h"
>>> +
>>> +static const hwaddr aspeed_soc_ast2700_memmap[] = {
>>> +    [ASPEED_DEV_SPI_BOOT]  =  0x400000000,
>>> +    [ASPEED_DEV_SRAM]      =  0x10000000,
>>> +    [ASPEED_DEV_SDMC]      =  0x12C00000,
>>> +    [ASPEED_DEV_SCU]       =  0x12C02000,
>>> +    [ASPEED_DEV_SCUIO]     =  0x14C02000,
>>> +    [ASPEED_DEV_UART0]     =  0X14C33000,
>>> +    [ASPEED_DEV_UART1]     =  0X14C33100,
>>> +    [ASPEED_DEV_UART2]     =  0X14C33200,
>>> +    [ASPEED_DEV_UART3]     =  0X14C33300,
>>> +    [ASPEED_DEV_UART4]     =  0X12C1A000,
>>> +    [ASPEED_DEV_UART5]     =  0X14C33400,
>>> +    [ASPEED_DEV_UART6]     =  0X14C33500,
>>> +    [ASPEED_DEV_UART7]     =  0X14C33600,
>>> +    [ASPEED_DEV_UART8]     =  0X14C33700,
>>> +    [ASPEED_DEV_UART9]     =  0X14C33800,
>>> +    [ASPEED_DEV_UART10]    =  0X14C33900,
>>> +    [ASPEED_DEV_UART11]    =  0X14C33A00,
>>> +    [ASPEED_DEV_UART12]    =  0X14C33B00,
>>> +    [ASPEED_DEV_WDT]       =  0x14C37000,
>>> +    [ASPEED_DEV_VUART]     =  0X14C30000,
>>> +    [ASPEED_DEV_FMC]       =  0x14000000,
>>> +    [ASPEED_DEV_SPI0]      =  0x14010000,
>>> +    [ASPEED_DEV_SPI1]      =  0x14020000,
>>> +    [ASPEED_DEV_SPI2]      =  0x14030000,
>>> +    [ASPEED_DEV_SDRAM]     =  0x400000000,
>>> +    [ASPEED_DEV_MII1]      =  0x14040000,
>>> +    [ASPEED_DEV_MII2]      =  0x14040008,
>>> +    [ASPEED_DEV_MII3]      =  0x14040010,
>>> +    [ASPEED_DEV_ETH1]      =  0x14050000,
>>> +    [ASPEED_DEV_ETH2]      =  0x14060000,
>>> +    [ASPEED_DEV_ETH3]      =  0x14070000,
>>> +    [ASPEED_DEV_EMMC]      =  0x12090000,
>>> +    [ASPEED_DEV_INTC]      =  0x12100000,
>>> +    [ASPEED_DEV_SLI]       =  0x12C17000,
>>> +    [ASPEED_DEV_SLIIO]     =  0x14C1E000,
>>> +    [ASPEED_GIC_DIST]      =  0x12200000,
>>> +    [ASPEED_GIC_REDIST]    =  0x12280000,
>>> +};
>>> +
>>> +#define AST2700_MAX_IRQ 288
>>> +
>>> +/* Shared Peripheral Interrupt values below are offset by -32 from
>>> +datasheet */ static const int aspeed_soc_ast2700_irqmap[] = {
>>> +    [ASPEED_DEV_UART0]     = 132,
>>> +    [ASPEED_DEV_UART1]     = 132,
>>> +    [ASPEED_DEV_UART2]     = 132,
>>> +    [ASPEED_DEV_UART3]     = 132,
>>> +    [ASPEED_DEV_UART4]     = 8,
>>> +    [ASPEED_DEV_UART5]     = 132,
>>> +    [ASPEED_DEV_UART6]     = 132,
>>> +    [ASPEED_DEV_UART7]     = 132,
>>> +    [ASPEED_DEV_UART8]     = 132,
>>> +    [ASPEED_DEV_UART9]     = 132,
>>> +    [ASPEED_DEV_UART10]    = 132,
>>> +    [ASPEED_DEV_UART11]    = 132,
>>> +    [ASPEED_DEV_UART12]    = 132,
>>> +    [ASPEED_DEV_FMC]       = 131,
>>> +    [ASPEED_DEV_SDMC]      = 0,
>>> +    [ASPEED_DEV_SCU]       = 12,
>>> +    [ASPEED_DEV_ADC]       = 130,
>>> +    [ASPEED_DEV_XDMA]      = 5,
>>> +    [ASPEED_DEV_EMMC]      = 15,
>>> +    [ASPEED_DEV_GPIO]      = 11,
>>> +    [ASPEED_DEV_GPIO_1_8V] = 130,
>>> +    [ASPEED_DEV_RTC]       = 13,
>>> +    [ASPEED_DEV_TIMER1]    = 16,
>>> +    [ASPEED_DEV_TIMER2]    = 17,
>>> +    [ASPEED_DEV_TIMER3]    = 18,
>>> +    [ASPEED_DEV_TIMER4]    = 19,
>>> +    [ASPEED_DEV_TIMER5]    = 20,
>>> +    [ASPEED_DEV_TIMER6]    = 21,
>>> +    [ASPEED_DEV_TIMER7]    = 22,
>>> +    [ASPEED_DEV_TIMER8]    = 23,
>>> +    [ASPEED_DEV_WDT]       = 131,
>>> +    [ASPEED_DEV_PWM]       = 131,
>>> +    [ASPEED_DEV_LPC]       = 128,
>>> +    [ASPEED_DEV_IBT]       = 128,
>>> +    [ASPEED_DEV_I2C]       = 130,
>>> +    [ASPEED_DEV_PECI]      = 133,
>>> +    [ASPEED_DEV_ETH1]      = 132,
>>> +    [ASPEED_DEV_ETH2]      = 132,
>>> +    [ASPEED_DEV_ETH3]      = 132,
>>> +    [ASPEED_DEV_HACE]      = 4,
>>> +    [ASPEED_DEV_KCS]       = 128,
>>> +    [ASPEED_DEV_DP]        = 28,
>>> +    [ASPEED_DEV_I3C]       = 131,
>>> +};
>>> +
>>> +/* GICINT 128 */
>>> +static const int aspeed_soc_ast2700_gic128_intcmap[] = {
>>> +    [ASPEED_DEV_LPC]       = 0,
>>> +    [ASPEED_DEV_IBT]       = 2,
>>> +    [ASPEED_DEV_KCS]       = 4,
>>> +};
>>> +
>>> +/* GICINT 130 */
>>> +static const int aspeed_soc_ast2700_gic130_intcmap[] = {
>>> +    [ASPEED_DEV_I2C]        = 0,
>>> +    [ASPEED_DEV_ADC]        = 16,
>>> +    [ASPEED_DEV_GPIO_1_8V]  = 18,
>>> +};
>>> +
>>> +/* GICINT 131 */
>>> +static const int aspeed_soc_ast2700_gic131_intcmap[] = {
>>> +    [ASPEED_DEV_I3C]       = 0,
>>> +    [ASPEED_DEV_WDT]       = 16,
>>> +    [ASPEED_DEV_FMC]       = 25,
>>> +    [ASPEED_DEV_PWM]       = 29,
>>> +};
>>> +
>>> +/* GICINT 132 */
>>> +static const int aspeed_soc_ast2700_gic132_intcmap[] = {
>>> +    [ASPEED_DEV_ETH1]      = 0,
>>> +    [ASPEED_DEV_ETH2]      = 1,
>>> +    [ASPEED_DEV_ETH3]      = 2,
>>> +    [ASPEED_DEV_UART0]     = 7,
>>> +    [ASPEED_DEV_UART1]     = 8,
>>> +    [ASPEED_DEV_UART2]     = 9,
>>> +    [ASPEED_DEV_UART3]     = 10,
>>> +    [ASPEED_DEV_UART5]     = 11,
>>> +    [ASPEED_DEV_UART6]     = 12,
>>> +    [ASPEED_DEV_UART7]     = 13,
>>> +    [ASPEED_DEV_UART8]     = 14,
>>> +    [ASPEED_DEV_UART9]     = 15,
>>> +    [ASPEED_DEV_UART10]    = 16,
>>> +    [ASPEED_DEV_UART11]    = 17,
>>> +    [ASPEED_DEV_UART12]    = 18,
>>> +};
>>> +
>>> +/* GICINT 133 */
>>> +static const int aspeed_soc_ast2700_gic133_intcmap[] = {
>>> +    [ASPEED_DEV_PECI]      = 4,
>>> +};
>>> +
>>> +/* GICINT 128 ~ 136 */
>>> +struct gic_intc_irq_info {
>>> +    int irq;
>>> +    const int *ptr;
>>> +};
>>> +
>>> +static const struct gic_intc_irq_info aspeed_soc_ast2700_gic_intcmap[] = {
>>> +    {128,  aspeed_soc_ast2700_gic128_intcmap},
>>> +    {129,  NULL},
>>> +    {130,  aspeed_soc_ast2700_gic130_intcmap},
>>> +    {131,  aspeed_soc_ast2700_gic131_intcmap},
>>> +    {132,  aspeed_soc_ast2700_gic132_intcmap},
>>> +    {133,  aspeed_soc_ast2700_gic133_intcmap},
>>> +    {134,  NULL},
>>> +    {135,  NULL},
>>> +    {136,  NULL},
>>> +};
>>> +
>>> +static qemu_irq aspeed_soc_ast2700_get_irq(AspeedSoCState *s, int
>>> +dev) {
>>> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(s);
>>> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
>>> +    int i;
>>> +
>>> +    for (i = 0; i < ARRAY_SIZE(aspeed_soc_ast2700_gic_intcmap); i++) {
>>> +        if (sc->irqmap[dev] == aspeed_soc_ast2700_gic_intcmap[i].irq) {
>>> +            assert(aspeed_soc_ast2700_gic_intcmap[i].ptr);
>>> +            return qdev_get_gpio_in(DEVICE(&a->intc.gicint_orgate[i]),
>>> +                aspeed_soc_ast2700_gic_intcmap[i].ptr[dev]);
>>> +        }
>>> +    }
>>> +
>>> +    return qdev_get_gpio_in(a->intc.gic, sc->irqmap[dev]); }
>>> +
>>> +static void aspeed_soc_ast2700_init(Object *obj) {
>>> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(obj);
>>> +    AspeedSoCState *s = ASPEED_SOC(obj);
>>> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
>>> +    int i;
>>> +    char socname[8];
>>> +    char typename[64];
>>> +
>>> +    if (sscanf(sc->name, "%7s", socname) != 1) {
>>> +        g_assert_not_reached();
>>> +    }
>>> +
>>> +    for (i = 0; i < sc->num_cpus; i++) {
>>> +        object_initialize_child(obj, "cpu[*]", &a->cpu[i],
>>> +                                aspeed_soc_cpu_type(sc));
>>> +    }
>>> +
>>> +    object_initialize_child(obj, "scu", &s->scu, TYPE_ASPEED_2700_SCU);
>>> +    qdev_prop_set_uint32(DEVICE(&s->scu), "silicon-rev",
>>> +                         sc->silicon_rev);
>>> +    object_property_add_alias(obj, "hw-strap1", OBJECT(&s->scu),
>>> +                              "hw-strap1");
>>> +    object_property_add_alias(obj, "hw-strap2", OBJECT(&s->scu),
>>> +                              "hw-strap2");
>>> +    object_property_add_alias(obj, "hw-prot-key", OBJECT(&s->scu),
>>> +                              "hw-prot-key");
>>> +
>>> +    object_initialize_child(obj, "scuio", &s->scuio,
>> TYPE_ASPEED_2700_SCUIO);
>>> +    qdev_prop_set_uint32(DEVICE(&s->scuio), "silicon-rev",
>>> +                         sc->silicon_rev);
>>> +
>>> +    snprintf(typename, sizeof(typename), "aspeed.fmc-%s", socname);
>>> +    object_initialize_child(obj, "fmc", &s->fmc, typename);
>>> +
>>> +    for (i = 0; i < sc->spis_num; i++) {
>>> +        snprintf(typename, sizeof(typename), "aspeed.spi%d-%s", i,
>> socname);
>>> +        object_initialize_child(obj, "spi[*]", &s->spi[i], typename);
>>> +    }
>>> +
>>> +    snprintf(typename, sizeof(typename), "aspeed.sdmc-%s", socname);
>>> +    object_initialize_child(obj, "sdmc", &s->sdmc, typename);
>>> +    object_property_add_alias(obj, "ram-size", OBJECT(&s->sdmc),
>>> +                              "ram-size");
>>> +
>>> +    for (i = 0; i < sc->wdts_num; i++) {
>>> +        snprintf(typename, sizeof(typename), "aspeed.wdt-%s",
>> socname);
>>> +        object_initialize_child(obj, "wdt[*]", &s->wdt[i], typename);
>>> +    }
>>> +
>>> +    for (i = 0; i < sc->macs_num; i++) {
>>> +        object_initialize_child(obj, "ftgmac100[*]", &s->ftgmac100[i],
>>> +                                TYPE_FTGMAC100);
>>> +
>>> +        object_initialize_child(obj, "mii[*]", &s->mii[i],
>> TYPE_ASPEED_MII);
>>> +    }
>>> +
>>> +    for (i = 0; i < sc->uarts_num; i++) {
>>> +        object_initialize_child(obj, "uart[*]", &s->uart[i],
>> TYPE_SERIAL_MM);
>>> +    }
>>> +
>>> +    object_initialize_child(obj, "sli", &s->sli, TYPE_ASPEED_2700_SLI);
>>> +    object_initialize_child(obj, "sliio", &s->sliio,
>> TYPE_ASPEED_2700_SLIIO);
>>> +    object_initialize_child(obj, "intc", &a->intc, TYPE_ASPEED_INTC);
>>> +}
>>> +
>>> +/*
>>> + * ASPEED ast2700 has 0x0 as cluster ID
>>> + *
>>> + *
>>> +https://developer.arm.com/documentation/100236/0100/register-descript
>>> +ions/aarch64-system-registers/multiprocessor-affinity-register--el1
>>> + */
>>> +static uint64_t aspeed_calc_affinity(int cpu) {
>>> +    return (0x0 << ARM_AFF1_SHIFT) | cpu; }
>>> +
>>> +static void aspeed_soc_ast2700_gic(DeviceState *dev, Error **errp) {
>>> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
>>> +    AspeedSoCState *s = ASPEED_SOC(dev);
>>> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
>>> +    SysBusDevice *gicbusdev;
>>> +    QList *redist_region_count;
>>> +    int i;
>>> +
>>> +    a->intc.gic = qdev_new(gicv3_class_name());
>>> +    qdev_prop_set_uint32(a->intc.gic, "revision", 3);
>>> +    qdev_prop_set_uint32(a->intc.gic, "num-cpu", sc->num_cpus);
>>> +    qdev_prop_set_uint32(a->intc.gic, "num-irq", AST2700_MAX_IRQ);
>>> +
>>> +    redist_region_count = qlist_new();
>>> +    qlist_append_int(redist_region_count, sc->num_cpus);
>>> +    qdev_prop_set_array(a->intc.gic, "redist-region-count",
>>> +                            redist_region_count);
>>> +
>>> +    gicbusdev = SYS_BUS_DEVICE(a->intc.gic);
>>> +    sysbus_realize_and_unref(gicbusdev, errp);
>>> +    sysbus_mmio_map(gicbusdev, 0, sc->memmap[ASPEED_GIC_DIST]);
>>> +    sysbus_mmio_map(gicbusdev, 1, sc->memmap[ASPEED_GIC_REDIST]);
>>> +
>>> +    for (i = 0; i < sc->num_cpus; i++) {
>>> +        DeviceState *cpudev = DEVICE(qemu_get_cpu(i));
>>> +        int NUM_IRQS = 256, ARCH_GIC_MAINT_IRQ = 9,
>> VIRTUAL_PMU_IRQ = 7;
>>> +        int ppibase = NUM_IRQS + i * GIC_INTERNAL + GIC_NR_SGIS;
>>> +
>>> +        const int timer_irq[] = {
>>> +            [GTIMER_PHYS] = 14,
>>> +            [GTIMER_VIRT] = 11,
>>> +            [GTIMER_HYP]  = 10,
>>> +            [GTIMER_SEC]  = 13,
>>> +        };
>>> +        int j;
>>> +
>>> +        for (j = 0; j < ARRAY_SIZE(timer_irq); j++) {
>>> +            qdev_connect_gpio_out(cpudev, j,
>>> +                    qdev_get_gpio_in(a->intc.gic, ppibase +
>> timer_irq[j]));
>>> +        }
>>> +
>>> +        qemu_irq irq = qdev_get_gpio_in(a->intc.gic,
>>> +                                        ppibase +
>> ARCH_GIC_MAINT_IRQ);
>>> +        qdev_connect_gpio_out_named(cpudev,
>> "gicv3-maintenance-interrupt",
>>> +                                    0, irq);
>>> +        qdev_connect_gpio_out_named(cpudev, "pmu-interrupt", 0,
>>> +                qdev_get_gpio_in(a->intc.gic, ppibase +
>>> + VIRTUAL_PMU_IRQ));
>>> +
>>> +        sysbus_connect_irq(gicbusdev, i, qdev_get_gpio_in(cpudev,
>> ARM_CPU_IRQ));
>>> +        sysbus_connect_irq(gicbusdev, i + sc->num_cpus,
>>> +                           qdev_get_gpio_in(cpudev,
>> ARM_CPU_FIQ));
>>> +        sysbus_connect_irq(gicbusdev, i + 2 * sc->num_cpus,
>>> +                           qdev_get_gpio_in(cpudev,
>> ARM_CPU_VIRQ));
>>> +        sysbus_connect_irq(gicbusdev, i + 3 * sc->num_cpus,
>>> +                           qdev_get_gpio_in(cpudev,
>> ARM_CPU_VFIQ));
>>> +    }
>>> +}
>>> +
>>> +static void aspeed_soc_ast2700_realize(DeviceState *dev, Error
>>> +**errp) {
>>> +    int i;
>>> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
>>> +    AspeedSoCState *s = ASPEED_SOC(dev);
>>> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
>>> +    g_autofree char *sram_name = NULL;
>>> +
>>> +    /* Default boot region (SPI memory or ROMs) */
>>> +    memory_region_init(&s->spi_boot_container, OBJECT(s),
>>> +                       "aspeed.spi_boot_container", 0x400000000);
>>> +    memory_region_add_subregion(s->memory,
>> sc->memmap[ASPEED_DEV_SPI_BOOT],
>>> +                                &s->spi_boot_container);
>>> +
>>> +    /* CPU */
>>> +    for (i = 0; i < sc->num_cpus; i++) {
>>> +        object_property_set_int(OBJECT(&a->cpu[i]), "mp-affinity",
>>> +                                aspeed_calc_affinity(i),
>>> + &error_abort);
>>> +
>>> +        object_property_set_int(OBJECT(&a->cpu[i]), "cntfrq",
>> 1125000000,
>>> +                                &error_abort);
>>> +        object_property_set_link(OBJECT(&a->cpu[i]), "memory",
>>> +                                 OBJECT(s->memory),
>> &error_abort);
>>> +
>>> +        if (!qdev_realize(DEVICE(&a->cpu[i]), NULL, errp)) {
>>> +            return;
>>> +        }
>>> +    }
>>> +
>>> +    /* GIC */
>>> +    aspeed_soc_ast2700_gic(dev, errp);
>>
>> Hmm, we should return.
> This function return type is void.
> If I understand your request, do you mean do the following changes?
> 
> static bool aspeed_soc_ast2700_gic(DeviceState *dev, Error **errp) --> change return type bool
> {
>      ---
>      gicbusdev = SYS_BUS_DEVICE(a->intc.gic);
>      if(!sysbus_realize(gicbusdev, errp)) {
>          return false;
>      }
>      ---
>      return true;
> }
> 
> Then,
> /* GIC */
> if(!aspeed_soc_ast2700_gic(dev, errp)) {
>      return;
>      }


yes.


Thanks,

C.


> Thanks-Jamin
>>
>>> +
>>> +    /* INTC */
>>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&a->intc), errp)) {
>>> +        return;
>>> +    }
>>> +
>>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&a->intc), 0,
>>> +                    sc->memmap[ASPEED_DEV_INTC]);
>>> +
>>> +    /* GICINT orgate -> INTC -> GIC */
>>> +    for (i = 0; i < ASPEED_INTC_NR_GICS; i++) {
>>> +        if (!qdev_realize(DEVICE(&a->intc.gicint_orgate[i]), NULL, errp)) {
>>> +            return;
>>> +        }
>>> +
>>> +        qdev_connect_gpio_out(DEVICE(&a->intc.gicint_orgate[i]), 0,
>>> +                              qdev_get_gpio_in(DEVICE(&a->intc),
>> i));
>>> +
>>> +        sysbus_connect_irq(SYS_BUS_DEVICE(&a->intc), i,
>>> +                           qdev_get_gpio_in(a->intc.gic,
>>> +
>> aspeed_soc_ast2700_gic_intcmap[i].irq));
>>> +    }
>>> +
>>> +    /* SRAM */
>>> +    sram_name = g_strdup_printf("aspeed.sram.%d",
>> CPU(&a->cpu[0])->cpu_index);
>>> +    if (!memory_region_init_ram(&s->sram, OBJECT(s), sram_name,
>> sc->sram_size,
>>> +                                 errp)) {
>>> +        return;
>>> +    }
>>> +    memory_region_add_subregion(s->memory,
>>> +                                sc->memmap[ASPEED_DEV_SRAM],
>>> + &s->sram);
>>> +
>>> +    /* SCU */
>>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scu), errp)) {
>>> +        return;
>>> +    }
>>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scu), 0,
>>> + sc->memmap[ASPEED_DEV_SCU]);
>>> +
>>> +    /* SCU1 */
>>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scuio), errp)) {
>>> +        return;
>>> +    }
>>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scuio), 0,
>>> +                    sc->memmap[ASPEED_DEV_SCUIO]);
>>> +
>>> +    /* UART */
>>> +    if (!aspeed_soc_uart_realize(s, errp)) {
>>> +        return;
>>> +    }
>>> +
>>> +    /* FMC, The number of CS is set at the board level */
>>> +    object_property_set_link(OBJECT(&s->fmc), "dram",
>> OBJECT(s->dram_mr),
>>> +                             &error_abort);
>>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->fmc), errp)) {
>>> +        return;
>>> +    }
>>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 0,
>> sc->memmap[ASPEED_DEV_FMC]);
>>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 1,
>>> +
>> ASPEED_SMC_GET_CLASS(&s->fmc)->flash_window_base);
>>> +    sysbus_connect_irq(SYS_BUS_DEVICE(&s->fmc), 0,
>>> +                       aspeed_soc_get_irq(s, ASPEED_DEV_FMC));
>>> +
>>> +    /* Set up an alias on the FMC CE0 region (boot default) */
>>> +    MemoryRegion *fmc0_mmio = &s->fmc.flashes[0].mmio;
>>> +    memory_region_init_alias(&s->spi_boot, OBJECT(s),
>> "aspeed.spi_boot",
>>> +                             fmc0_mmio, 0,
>> memory_region_size(fmc0_mmio));
>>> +    memory_region_add_subregion(&s->spi_boot_container, 0x0,
>>> + &s->spi_boot);
>>> +
>>> +    /* SPI */
>>> +    for (i = 0; i < sc->spis_num; i++) {
>>> +        object_property_set_link(OBJECT(&s->spi[i]), "dram",
>>> +                                 OBJECT(s->dram_mr),
>> &error_abort);
>>> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->spi[i]), errp)) {
>>> +            return;
>>> +        }
>>> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 0,
>>> +                        sc->memmap[ASPEED_DEV_SPI0 + i]);
>>> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 1,
>>> +
>> ASPEED_SMC_GET_CLASS(&s->spi[i])->flash_window_base);
>>> +    }
>>> +
>>> +    /*
>>> +     * SDMC - SDRAM Memory Controller
>>> +     * The SDMC controller is unlocked at SPL stage.
>>> +     * At present, only supports to emulate booting
>>> +     * start from u-boot stage. Set SDMC controller
>>> +     * unlocked by default. It is a temporarily solution.
>>> +     */
>>> +    object_property_set_bool(OBJECT(&s->sdmc), "unlocked", true,
>>> +                                 &error_abort);
>>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sdmc), errp)) {
>>> +        return;
>>> +    }
>>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sdmc), 0,
>>> +                    sc->memmap[ASPEED_DEV_SDMC]);
>>> +
>>> +    /* RAM */
>>> +    if (!aspeed_soc_dram_init(s, errp)) {
>>> +        return;
>>> +    }
>>> +
>>> +    for (i = 0; i < sc->macs_num; i++) {
>>> +        object_property_set_bool(OBJECT(&s->ftgmac100[i]), "aspeed",
>> true,
>>> +                                 &error_abort);
>>> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->ftgmac100[i]), errp)) {
>>> +            return;
>>> +        }
>>> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
>>> +                        sc->memmap[ASPEED_DEV_ETH1 + i]);
>>> +        sysbus_connect_irq(SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
>>> +                           aspeed_soc_get_irq(s, ASPEED_DEV_ETH1
>> +
>>> + i));
>>> +
>>> +        object_property_set_link(OBJECT(&s->mii[i]), "nic",
>>> +                                 OBJECT(&s->ftgmac100[i]),
>> &error_abort);
>>> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->mii[i]), errp)) {
>>> +            return;
>>> +        }
>>> +
>>> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->mii[i]), 0,
>>> +                        sc->memmap[ASPEED_DEV_MII1 + i]);
>>> +    }
>>> +
>>> +    /* Watch dog */
>>> +    for (i = 0; i < sc->wdts_num; i++) {
>>> +        AspeedWDTClass *awc = ASPEED_WDT_GET_CLASS(&s->wdt[i]);
>>> +        hwaddr wdt_offset = sc->memmap[ASPEED_DEV_WDT] + i *
>>> + awc->iosize;
>>> +
>>> +        object_property_set_link(OBJECT(&s->wdt[i]), "scu",
>> OBJECT(&s->scu),
>>> +                                 &error_abort);
>>> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->wdt[i]), errp)) {
>>> +            return;
>>> +        }
>>> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->wdt[i]), 0,
>> wdt_offset);
>>> +    }
>>> +
>>> +    /* SLI */
>>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sli), errp)) {
>>> +        return;
>>> +    }
>>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sli), 0,
>>> + sc->memmap[ASPEED_DEV_SLI]);
>>> +
>>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sliio), errp)) {
>>> +        return;
>>> +    }
>>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sliio), 0,
>>> +                    sc->memmap[ASPEED_DEV_SLIIO]);
>>> +
>>> +    create_unimplemented_device("ast2700.dpmcu", 0x11000000,
>> 0x40000);
>>> +    create_unimplemented_device("ast2700.iomem0", 0x12000000,
>> 0x01000000);
>>> +    create_unimplemented_device("ast2700.iomem1", 0x14000000,
>> 0x01000000);
>>> +    create_unimplemented_device("ast2700.ltpi", 0x30000000,
>> 0x1000000);
>>> +    create_unimplemented_device("ast2700.io", 0x0, 0x4000000); }
>>> +
>>> +static void aspeed_soc_ast2700_class_init(ObjectClass *oc, void
>>> +*data) {
>>> +    static const char * const valid_cpu_types[] = {
>>> +        ARM_CPU_TYPE_NAME("cortex-a35"),
>>> +        NULL
>>> +    };
>>> +    DeviceClass *dc = DEVICE_CLASS(oc);
>>> +    AspeedSoCClass *sc = ASPEED_SOC_CLASS(oc);
>>> +
>>> +    dc->realize      = aspeed_soc_ast2700_realize;
>>
>>
>> Please add :
>>
>>       /* Reason: The Aspeed SoC can only be instantiated from a board */
>>       dc->user_creatable = false;
>>
>>
> Will add.
>> Thanks,
>>
>> C.
>>
>>
>>> +
>>> +    sc->name         = "ast2700-a0";
>>> +    sc->valid_cpu_types = valid_cpu_types;
>>> +    sc->silicon_rev  = AST2700_A0_SILICON_REV;
>>> +    sc->sram_size    = 0x20000;
>>> +    sc->spis_num     = 3;
>>> +    sc->wdts_num     = 8;
>>> +    sc->macs_num     = 1;
>>> +    sc->uarts_num    = 13;
>>> +    sc->num_cpus     = 4;
>>> +    sc->uarts_base   = ASPEED_DEV_UART0;
>>> +    sc->irqmap       = aspeed_soc_ast2700_irqmap;
>>> +    sc->memmap       = aspeed_soc_ast2700_memmap;
>>> +    sc->get_irq      = aspeed_soc_ast2700_get_irq;
>>> +}
>>> +
>>> +static const TypeInfo aspeed_soc_ast27x0_types[] = {
>>> +    {
>>> +        .name           = TYPE_ASPEED27X0_SOC,
>>> +        .parent         = TYPE_ASPEED_SOC,
>>> +        .instance_size  = sizeof(Aspeed27x0SoCState),
>>> +        .abstract       = true,
>>> +    }, {
>>> +        .name           = "ast2700-a0",
>>> +        .parent         = TYPE_ASPEED27X0_SOC,
>>> +        .instance_init  = aspeed_soc_ast2700_init,
>>> +        .class_init     = aspeed_soc_ast2700_class_init,
>>> +    },
>>> +};
>>> +
>>> +DEFINE_TYPES(aspeed_soc_ast27x0_types)
>>> diff --git a/hw/arm/meson.build b/hw/arm/meson.build index
>>> 6808135c1f..1e3295a423 100644
>>> --- a/hw/arm/meson.build
>>> +++ b/hw/arm/meson.build
>>> @@ -46,6 +46,7 @@ arm_ss.add(when: 'CONFIG_ASPEED_SOC', if_true:
>> files(
>>>      'aspeed_soc_common.c',
>>>      'aspeed_ast2400.c',
>>>      'aspeed_ast2600.c',
>>> +  'aspeed_ast27x0.c',
>>>      'aspeed_ast10x0.c',
>>>      'aspeed_eeprom.c',
>>>      'fby35.c'))
>>> diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h
>>> index c60fac900a..9f177b6037 100644
>>> --- a/include/hw/arm/aspeed_soc.h
>>> +++ b/include/hw/arm/aspeed_soc.h
>>> @@ -15,6 +15,7 @@
>>>    #include "hw/cpu/a15mpcore.h"
>>>    #include "hw/arm/armv7m.h"
>>>    #include "hw/intc/aspeed_vic.h"
>>> +#include "hw/intc/aspeed_intc.h"
>>>    #include "hw/misc/aspeed_scu.h"
>>>    #include "hw/adc/aspeed_adc.h"
>>>    #include "hw/misc/aspeed_sdmc.h"
>>> @@ -26,6 +27,7 @@
>>>    #include "hw/ssi/aspeed_smc.h"
>>>    #include "hw/misc/aspeed_hace.h"
>>>    #include "hw/misc/aspeed_sbc.h"
>>> +#include "hw/misc/aspeed_sli.h"
>>>    #include "hw/watchdog/wdt_aspeed.h"
>>>    #include "hw/net/ftgmac100.h"
>>>    #include "target/arm/cpu.h"
>>> @@ -41,8 +43,8 @@
>>>
>>>    #define ASPEED_SPIS_NUM  2
>>>    #define ASPEED_EHCIS_NUM 2
>>> -#define ASPEED_WDTS_NUM  4
>>> -#define ASPEED_CPUS_NUM  2
>>> +#define ASPEED_WDTS_NUM  8
>>> +#define ASPEED_CPUS_NUM  4
>>>    #define ASPEED_MACS_NUM  4
>>>    #define ASPEED_UARTS_NUM 13
>>>    #define ASPEED_JTAG_NUM  2
>>> @@ -61,6 +63,7 @@ struct AspeedSoCState {
>>>        AspeedI2CState i2c;
>>>        AspeedI3CState i3c;
>>>        AspeedSCUState scu;
>>> +    AspeedSCUState scuio;
>>>        AspeedHACEState hace;
>>>        AspeedXDMAState xdma;
>>>        AspeedADCState adc;
>>> @@ -68,6 +71,8 @@ struct AspeedSoCState {
>>>        AspeedSMCState spi[ASPEED_SPIS_NUM];
>>>        EHCISysBusState ehci[ASPEED_EHCIS_NUM];
>>>        AspeedSBCState sbc;
>>> +    AspeedSLIState sli;
>>> +    AspeedSLIState sliio;
>>>        MemoryRegion secsram;
>>>        UnimplementedDeviceState sbc_unimplemented;
>>>        AspeedSDMCState sdmc;
>>> @@ -117,6 +122,16 @@ struct Aspeed2600SoCState {
>>>    #define TYPE_ASPEED2600_SOC "aspeed2600-soc"
>>>    OBJECT_DECLARE_SIMPLE_TYPE(Aspeed2600SoCState,
>> ASPEED2600_SOC)
>>>
>>> +struct Aspeed27x0SoCState {
>>> +    AspeedSoCState parent;
>>> +
>>> +    ARMCPU cpu[ASPEED_CPUS_NUM];
>>> +    AspeedINTCState intc;
>>> +};
>>> +
>>> +#define TYPE_ASPEED27X0_SOC "aspeed27x0-soc"
>>> +OBJECT_DECLARE_SIMPLE_TYPE(Aspeed27x0SoCState, ASPEED27X0_SOC)
>>> +
>>>    struct Aspeed10x0SoCState {
>>>        AspeedSoCState parent;
>>>
>>> @@ -168,11 +183,13 @@ enum {
>>>        ASPEED_DEV_UART13,
>>>        ASPEED_DEV_VUART,
>>>        ASPEED_DEV_FMC,
>>> +    ASPEED_DEV_SPI0,
>>>        ASPEED_DEV_SPI1,
>>>        ASPEED_DEV_SPI2,
>>>        ASPEED_DEV_EHCI1,
>>>        ASPEED_DEV_EHCI2,
>>>        ASPEED_DEV_VIC,
>>> +    ASPEED_DEV_INTC,
>>>        ASPEED_DEV_SDMC,
>>>        ASPEED_DEV_SCU,
>>>        ASPEED_DEV_ADC,
>>> @@ -222,6 +239,11 @@ enum {
>>>        ASPEED_DEV_JTAG1,
>>>        ASPEED_DEV_FSI1,
>>>        ASPEED_DEV_FSI2,
>>> +    ASPEED_DEV_SCUIO,
>>> +    ASPEED_DEV_SLI,
>>> +    ASPEED_DEV_SLIIO,
>>> +    ASPEED_GIC_DIST,
>>> +    ASPEED_GIC_REDIST,
>>>    };
>>>
>>>    qemu_irq aspeed_soc_get_irq(AspeedSoCState *s, int dev);
>
Jamin Lin May 17, 2024, 9:13 a.m. UTC | #5
Hi Cerdric,

> On 4/19/24 09:58, Jamin Lin wrote:
> > Hi Cedric,
> >> On 4/16/24 11:18, Jamin Lin wrote:
> >>> Initial definitions for a simple machine using an AST2700 SOC
> >>> (Cortex-a35
> >> CPU).
> >>>
> >>> AST2700 SOC and its interrupt controller are too complex to handle
> >>> in the common Aspeed SoC framework. We introduce a new ast2700 class
> >>> with instance_init and realize handlers.
> >>>
> >>> AST2700 is a 64 bits quad core cpus and support 8 watchdog.
> >>> Update maximum ASPEED_CPUS_NUM to 4 and ASPEED_WDTS_NUM to
> 8.
> >>> In addition, update AspeedSocState to support scuio, sli, sliio and intc.
> >>>
> >>> Add TYPE_ASPEED27X0_SOC machine type.
> >>>
> >>> The SDMC controller is unlocked at SPL stage.
> >>> At present, only supports to emulate booting start from u-boot stage.
> >>> Set SDMC controller unlocked by default.
> >>>
> >>> In INTC, each interrupt of INT 128 to INT 136 combines 32 interrupts.
> >>> It connect GICINT IRQ GPIO-OUTPUT pins to GIC device with irq 128 to
> 136.
> >>> And, if a device irq is 128 to 136, its irq GPIO-OUTPUT pin is
> >>> connected to GICINT or-gates instead of GIC device.
> >>>
> >>> Signed-off-by: Troy Lee <troy_lee@aspeedtech.com>
> >>> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> >>
> >> Before I forget, please see a little comment below regarding user
> >> creatable devices.
> >>
> >> The model looks fine. The interrupt controller part is more complex
> >> than the previous SoCs so I will come back to it later when I have more
> time.
> > Thanks for your kindly support.
> >>> ---
> >>>    hw/arm/aspeed_ast27x0.c     | 554
> >> ++++++++++++++++++++++++++++++++++++
> >>>    hw/arm/meson.build          |   1 +
> >>>    include/hw/arm/aspeed_soc.h |  26 +-
> >>>    3 files changed, 579 insertions(+), 2 deletions(-)
> >>>    create mode 100644 hw/arm/aspeed_ast27x0.c
> >>>
> >>> diff --git a/hw/arm/aspeed_ast27x0.c b/hw/arm/aspeed_ast27x0.c new
> >>> file mode 100644 index 0000000000..754c963230
> >>> --- /dev/null
> >>> +++ b/hw/arm/aspeed_ast27x0.c
> >>> @@ -0,0 +1,554 @@
> >>> +/*
> >>> + * ASPEED SoC 27x0 family
> >>> + *
> >>> + * Copyright (C) 2024 ASPEED Technology Inc.
> >>> + *
> >>> + * This code is licensed under the GPL version 2 or later.  See
> >>> + * the COPYING file in the top-level directory.
> >>> + *
> >>> + * Implementation extracted from the AST2600 and adapted for AST27x0.
> >>> + */
> >>> +
> >>> +#include "qemu/osdep.h"
> >>> +#include "qapi/error.h"
> >>> +#include "hw/misc/unimp.h"
> >>> +#include "hw/arm/aspeed_soc.h"
> >>> +#include "qemu/module.h"
> >>> +#include "qemu/error-report.h"
> >>> +#include "hw/i2c/aspeed_i2c.h"
> >>> +#include "net/net.h"
> >>> +#include "sysemu/sysemu.h"
> >>> +#include "hw/intc/arm_gicv3.h"
> >>> +#include "qapi/qmp/qlist.h"
> >>> +
> >>> +static const hwaddr aspeed_soc_ast2700_memmap[] = {
> >>> +    [ASPEED_DEV_SPI_BOOT]  =  0x400000000,
> >>> +    [ASPEED_DEV_SRAM]      =  0x10000000,
> >>> +    [ASPEED_DEV_SDMC]      =  0x12C00000,
> >>> +    [ASPEED_DEV_SCU]       =  0x12C02000,
> >>> +    [ASPEED_DEV_SCUIO]     =  0x14C02000,
> >>> +    [ASPEED_DEV_UART0]     =  0X14C33000,
> >>> +    [ASPEED_DEV_UART1]     =  0X14C33100,
> >>> +    [ASPEED_DEV_UART2]     =  0X14C33200,
> >>> +    [ASPEED_DEV_UART3]     =  0X14C33300,
> >>> +    [ASPEED_DEV_UART4]     =  0X12C1A000,
> >>> +    [ASPEED_DEV_UART5]     =  0X14C33400,
> >>> +    [ASPEED_DEV_UART6]     =  0X14C33500,
> >>> +    [ASPEED_DEV_UART7]     =  0X14C33600,
> >>> +    [ASPEED_DEV_UART8]     =  0X14C33700,
> >>> +    [ASPEED_DEV_UART9]     =  0X14C33800,
> >>> +    [ASPEED_DEV_UART10]    =  0X14C33900,
> >>> +    [ASPEED_DEV_UART11]    =  0X14C33A00,
> >>> +    [ASPEED_DEV_UART12]    =  0X14C33B00,
> >>> +    [ASPEED_DEV_WDT]       =  0x14C37000,
> >>> +    [ASPEED_DEV_VUART]     =  0X14C30000,
> >>> +    [ASPEED_DEV_FMC]       =  0x14000000,
> >>> +    [ASPEED_DEV_SPI0]      =  0x14010000,
> >>> +    [ASPEED_DEV_SPI1]      =  0x14020000,
> >>> +    [ASPEED_DEV_SPI2]      =  0x14030000,
> >>> +    [ASPEED_DEV_SDRAM]     =  0x400000000,
> >>> +    [ASPEED_DEV_MII1]      =  0x14040000,
> >>> +    [ASPEED_DEV_MII2]      =  0x14040008,
> >>> +    [ASPEED_DEV_MII3]      =  0x14040010,
> >>> +    [ASPEED_DEV_ETH1]      =  0x14050000,
> >>> +    [ASPEED_DEV_ETH2]      =  0x14060000,
> >>> +    [ASPEED_DEV_ETH3]      =  0x14070000,
> >>> +    [ASPEED_DEV_EMMC]      =  0x12090000,
> >>> +    [ASPEED_DEV_INTC]      =  0x12100000,
> >>> +    [ASPEED_DEV_SLI]       =  0x12C17000,
> >>> +    [ASPEED_DEV_SLIIO]     =  0x14C1E000,
> >>> +    [ASPEED_GIC_DIST]      =  0x12200000,
> >>> +    [ASPEED_GIC_REDIST]    =  0x12280000,
> >>> +};
> >>> +
> >>> +#define AST2700_MAX_IRQ 288
> >>> +
> >>> +/* Shared Peripheral Interrupt values below are offset by -32 from
> >>> +datasheet */ static const int aspeed_soc_ast2700_irqmap[] = {
> >>> +    [ASPEED_DEV_UART0]     = 132,
> >>> +    [ASPEED_DEV_UART1]     = 132,
> >>> +    [ASPEED_DEV_UART2]     = 132,
> >>> +    [ASPEED_DEV_UART3]     = 132,
> >>> +    [ASPEED_DEV_UART4]     = 8,
> >>> +    [ASPEED_DEV_UART5]     = 132,
> >>> +    [ASPEED_DEV_UART6]     = 132,
> >>> +    [ASPEED_DEV_UART7]     = 132,
> >>> +    [ASPEED_DEV_UART8]     = 132,
> >>> +    [ASPEED_DEV_UART9]     = 132,
> >>> +    [ASPEED_DEV_UART10]    = 132,
> >>> +    [ASPEED_DEV_UART11]    = 132,
> >>> +    [ASPEED_DEV_UART12]    = 132,
> >>> +    [ASPEED_DEV_FMC]       = 131,
> >>> +    [ASPEED_DEV_SDMC]      = 0,
> >>> +    [ASPEED_DEV_SCU]       = 12,
> >>> +    [ASPEED_DEV_ADC]       = 130,
> >>> +    [ASPEED_DEV_XDMA]      = 5,
> >>> +    [ASPEED_DEV_EMMC]      = 15,
> >>> +    [ASPEED_DEV_GPIO]      = 11,
> >>> +    [ASPEED_DEV_GPIO_1_8V] = 130,
> >>> +    [ASPEED_DEV_RTC]       = 13,
> >>> +    [ASPEED_DEV_TIMER1]    = 16,
> >>> +    [ASPEED_DEV_TIMER2]    = 17,
> >>> +    [ASPEED_DEV_TIMER3]    = 18,
> >>> +    [ASPEED_DEV_TIMER4]    = 19,
> >>> +    [ASPEED_DEV_TIMER5]    = 20,
> >>> +    [ASPEED_DEV_TIMER6]    = 21,
> >>> +    [ASPEED_DEV_TIMER7]    = 22,
> >>> +    [ASPEED_DEV_TIMER8]    = 23,
> >>> +    [ASPEED_DEV_WDT]       = 131,
> >>> +    [ASPEED_DEV_PWM]       = 131,
> >>> +    [ASPEED_DEV_LPC]       = 128,
> >>> +    [ASPEED_DEV_IBT]       = 128,
> >>> +    [ASPEED_DEV_I2C]       = 130,
> >>> +    [ASPEED_DEV_PECI]      = 133,
> >>> +    [ASPEED_DEV_ETH1]      = 132,
> >>> +    [ASPEED_DEV_ETH2]      = 132,
> >>> +    [ASPEED_DEV_ETH3]      = 132,
> >>> +    [ASPEED_DEV_HACE]      = 4,
> >>> +    [ASPEED_DEV_KCS]       = 128,
> >>> +    [ASPEED_DEV_DP]        = 28,
> >>> +    [ASPEED_DEV_I3C]       = 131,
> >>> +};
> >>> +
> >>> +/* GICINT 128 */
> >>> +static const int aspeed_soc_ast2700_gic128_intcmap[] = {
> >>> +    [ASPEED_DEV_LPC]       = 0,
> >>> +    [ASPEED_DEV_IBT]       = 2,
> >>> +    [ASPEED_DEV_KCS]       = 4,
> >>> +};
> >>> +
> >>> +/* GICINT 130 */
> >>> +static const int aspeed_soc_ast2700_gic130_intcmap[] = {
> >>> +    [ASPEED_DEV_I2C]        = 0,
> >>> +    [ASPEED_DEV_ADC]        = 16,
> >>> +    [ASPEED_DEV_GPIO_1_8V]  = 18,
> >>> +};
> >>> +
> >>> +/* GICINT 131 */
> >>> +static const int aspeed_soc_ast2700_gic131_intcmap[] = {
> >>> +    [ASPEED_DEV_I3C]       = 0,
> >>> +    [ASPEED_DEV_WDT]       = 16,
> >>> +    [ASPEED_DEV_FMC]       = 25,
> >>> +    [ASPEED_DEV_PWM]       = 29,
> >>> +};
> >>> +
> >>> +/* GICINT 132 */
> >>> +static const int aspeed_soc_ast2700_gic132_intcmap[] = {
> >>> +    [ASPEED_DEV_ETH1]      = 0,
> >>> +    [ASPEED_DEV_ETH2]      = 1,
> >>> +    [ASPEED_DEV_ETH3]      = 2,
> >>> +    [ASPEED_DEV_UART0]     = 7,
> >>> +    [ASPEED_DEV_UART1]     = 8,
> >>> +    [ASPEED_DEV_UART2]     = 9,
> >>> +    [ASPEED_DEV_UART3]     = 10,
> >>> +    [ASPEED_DEV_UART5]     = 11,
> >>> +    [ASPEED_DEV_UART6]     = 12,
> >>> +    [ASPEED_DEV_UART7]     = 13,
> >>> +    [ASPEED_DEV_UART8]     = 14,
> >>> +    [ASPEED_DEV_UART9]     = 15,
> >>> +    [ASPEED_DEV_UART10]    = 16,
> >>> +    [ASPEED_DEV_UART11]    = 17,
> >>> +    [ASPEED_DEV_UART12]    = 18,
> >>> +};
> >>> +
> >>> +/* GICINT 133 */
> >>> +static const int aspeed_soc_ast2700_gic133_intcmap[] = {
> >>> +    [ASPEED_DEV_PECI]      = 4,
> >>> +};
> >>> +
> >>> +/* GICINT 128 ~ 136 */
> >>> +struct gic_intc_irq_info {
> >>> +    int irq;
> >>> +    const int *ptr;
> >>> +};
> >>> +
> >>> +static const struct gic_intc_irq_info aspeed_soc_ast2700_gic_intcmap[] =
> {
> >>> +    {128,  aspeed_soc_ast2700_gic128_intcmap},
> >>> +    {129,  NULL},
> >>> +    {130,  aspeed_soc_ast2700_gic130_intcmap},
> >>> +    {131,  aspeed_soc_ast2700_gic131_intcmap},
> >>> +    {132,  aspeed_soc_ast2700_gic132_intcmap},
> >>> +    {133,  aspeed_soc_ast2700_gic133_intcmap},
> >>> +    {134,  NULL},
> >>> +    {135,  NULL},
> >>> +    {136,  NULL},
> >>> +};
> >>> +
> >>> +static qemu_irq aspeed_soc_ast2700_get_irq(AspeedSoCState *s, int
> >>> +dev) {
> >>> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(s);
> >>> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> >>> +    int i;
> >>> +
> >>> +    for (i = 0; i < ARRAY_SIZE(aspeed_soc_ast2700_gic_intcmap); i++) {
> >>> +        if (sc->irqmap[dev] == aspeed_soc_ast2700_gic_intcmap[i].irq)
> {
> >>> +            assert(aspeed_soc_ast2700_gic_intcmap[i].ptr);
> >>> +            return
> qdev_get_gpio_in(DEVICE(&a->intc.gicint_orgate[i]),
> >>> +                aspeed_soc_ast2700_gic_intcmap[i].ptr[dev]);
> >>> +        }
> >>> +    }
> >>> +
> >>> +    return qdev_get_gpio_in(a->intc.gic, sc->irqmap[dev]); }
> >>> +
> >>> +static void aspeed_soc_ast2700_init(Object *obj) {
> >>> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(obj);
> >>> +    AspeedSoCState *s = ASPEED_SOC(obj);
> >>> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> >>> +    int i;
> >>> +    char socname[8];
> >>> +    char typename[64];
> >>> +
> >>> +    if (sscanf(sc->name, "%7s", socname) != 1) {
> >>> +        g_assert_not_reached();
> >>> +    }
> >>> +
> >>> +    for (i = 0; i < sc->num_cpus; i++) {
> >>> +        object_initialize_child(obj, "cpu[*]", &a->cpu[i],
> >>> +                                aspeed_soc_cpu_type(sc));
> >>> +    }
> >>> +
> >>> +    object_initialize_child(obj, "scu", &s->scu,
> TYPE_ASPEED_2700_SCU);
> >>> +    qdev_prop_set_uint32(DEVICE(&s->scu), "silicon-rev",
> >>> +                         sc->silicon_rev);
> >>> +    object_property_add_alias(obj, "hw-strap1", OBJECT(&s->scu),
> >>> +                              "hw-strap1");
> >>> +    object_property_add_alias(obj, "hw-strap2", OBJECT(&s->scu),
> >>> +                              "hw-strap2");
> >>> +    object_property_add_alias(obj, "hw-prot-key", OBJECT(&s->scu),
> >>> +                              "hw-prot-key");
> >>> +
> >>> +    object_initialize_child(obj, "scuio", &s->scuio,
> >> TYPE_ASPEED_2700_SCUIO);
> >>> +    qdev_prop_set_uint32(DEVICE(&s->scuio), "silicon-rev",
> >>> +                         sc->silicon_rev);
> >>> +
> >>> +    snprintf(typename, sizeof(typename), "aspeed.fmc-%s", socname);
> >>> +    object_initialize_child(obj, "fmc", &s->fmc, typename);
> >>> +
> >>> +    for (i = 0; i < sc->spis_num; i++) {
> >>> +        snprintf(typename, sizeof(typename), "aspeed.spi%d-%s", i,
> >> socname);
> >>> +        object_initialize_child(obj, "spi[*]", &s->spi[i], typename);
> >>> +    }
> >>> +
> >>> +    snprintf(typename, sizeof(typename), "aspeed.sdmc-%s", socname);
> >>> +    object_initialize_child(obj, "sdmc", &s->sdmc, typename);
> >>> +    object_property_add_alias(obj, "ram-size", OBJECT(&s->sdmc),
> >>> +                              "ram-size");
> >>> +
> >>> +    for (i = 0; i < sc->wdts_num; i++) {
> >>> +        snprintf(typename, sizeof(typename), "aspeed.wdt-%s",
> >> socname);
> >>> +        object_initialize_child(obj, "wdt[*]", &s->wdt[i], typename);
> >>> +    }
> >>> +
> >>> +    for (i = 0; i < sc->macs_num; i++) {
> >>> +        object_initialize_child(obj, "ftgmac100[*]", &s->ftgmac100[i],
> >>> +                                TYPE_FTGMAC100);
> >>> +
> >>> +        object_initialize_child(obj, "mii[*]", &s->mii[i],
> >> TYPE_ASPEED_MII);
> >>> +    }
> >>> +
> >>> +    for (i = 0; i < sc->uarts_num; i++) {
> >>> +        object_initialize_child(obj, "uart[*]", &s->uart[i],
> >> TYPE_SERIAL_MM);
> >>> +    }
> >>> +
> >>> +    object_initialize_child(obj, "sli", &s->sli, TYPE_ASPEED_2700_SLI);
> >>> +    object_initialize_child(obj, "sliio", &s->sliio,
> >> TYPE_ASPEED_2700_SLIIO);
> >>> +    object_initialize_child(obj, "intc", &a->intc,
> >>> +TYPE_ASPEED_INTC); }
> >>> +
> >>> +/*
> >>> + * ASPEED ast2700 has 0x0 as cluster ID
> >>> + *
> >>> + *
> >>> +https://developer.arm.com/documentation/100236/0100/register-descri
> >>> +pt
> >>> +ions/aarch64-system-registers/multiprocessor-affinity-register--el1
> >>> + */
> >>> +static uint64_t aspeed_calc_affinity(int cpu) {
> >>> +    return (0x0 << ARM_AFF1_SHIFT) | cpu; }
> >>> +
> >>> +static void aspeed_soc_ast2700_gic(DeviceState *dev, Error **errp) {
> >>> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
> >>> +    AspeedSoCState *s = ASPEED_SOC(dev);
> >>> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> >>> +    SysBusDevice *gicbusdev;
> >>> +    QList *redist_region_count;
> >>> +    int i;
> >>> +
> >>> +    a->intc.gic = qdev_new(gicv3_class_name());
> >>> +    qdev_prop_set_uint32(a->intc.gic, "revision", 3);
> >>> +    qdev_prop_set_uint32(a->intc.gic, "num-cpu", sc->num_cpus);
> >>> +    qdev_prop_set_uint32(a->intc.gic, "num-irq", AST2700_MAX_IRQ);
> >>> +
> >>> +    redist_region_count = qlist_new();
> >>> +    qlist_append_int(redist_region_count, sc->num_cpus);
> >>> +    qdev_prop_set_array(a->intc.gic, "redist-region-count",
> >>> +                            redist_region_count);
> >>> +
> >>> +    gicbusdev = SYS_BUS_DEVICE(a->intc.gic);
> >>> +    sysbus_realize_and_unref(gicbusdev, errp);
> >>> +    sysbus_mmio_map(gicbusdev, 0, sc->memmap[ASPEED_GIC_DIST]);
> >>> +    sysbus_mmio_map(gicbusdev, 1,
> sc->memmap[ASPEED_GIC_REDIST]);
> >>> +
> >>> +    for (i = 0; i < sc->num_cpus; i++) {
> >>> +        DeviceState *cpudev = DEVICE(qemu_get_cpu(i));
> >>> +        int NUM_IRQS = 256, ARCH_GIC_MAINT_IRQ = 9,
> >> VIRTUAL_PMU_IRQ = 7;
> >>> +        int ppibase = NUM_IRQS + i * GIC_INTERNAL + GIC_NR_SGIS;
> >>> +
> >>> +        const int timer_irq[] = {
> >>> +            [GTIMER_PHYS] = 14,
> >>> +            [GTIMER_VIRT] = 11,
> >>> +            [GTIMER_HYP]  = 10,
> >>> +            [GTIMER_SEC]  = 13,
> >>> +        };
> >>> +        int j;
> >>> +
> >>> +        for (j = 0; j < ARRAY_SIZE(timer_irq); j++) {
> >>> +            qdev_connect_gpio_out(cpudev, j,
> >>> +                    qdev_get_gpio_in(a->intc.gic, ppibase +
> >> timer_irq[j]));
> >>> +        }
> >>> +
> >>> +        qemu_irq irq = qdev_get_gpio_in(a->intc.gic,
> >>> +                                        ppibase +
> >> ARCH_GIC_MAINT_IRQ);
> >>> +        qdev_connect_gpio_out_named(cpudev,
> >> "gicv3-maintenance-interrupt",
> >>> +                                    0, irq);
> >>> +        qdev_connect_gpio_out_named(cpudev, "pmu-interrupt", 0,
> >>> +                qdev_get_gpio_in(a->intc.gic, ppibase +
> >>> + VIRTUAL_PMU_IRQ));
> >>> +
> >>> +        sysbus_connect_irq(gicbusdev, i, qdev_get_gpio_in(cpudev,
> >> ARM_CPU_IRQ));
> >>> +        sysbus_connect_irq(gicbusdev, i + sc->num_cpus,
> >>> +                           qdev_get_gpio_in(cpudev,
> >> ARM_CPU_FIQ));
> >>> +        sysbus_connect_irq(gicbusdev, i + 2 * sc->num_cpus,
> >>> +                           qdev_get_gpio_in(cpudev,
> >> ARM_CPU_VIRQ));
> >>> +        sysbus_connect_irq(gicbusdev, i + 3 * sc->num_cpus,
> >>> +                           qdev_get_gpio_in(cpudev,
> >> ARM_CPU_VFIQ));
> >>> +    }
> >>> +}
> >>> +
> >>> +static void aspeed_soc_ast2700_realize(DeviceState *dev, Error
> >>> +**errp) {
> >>> +    int i;
> >>> +    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
> >>> +    AspeedSoCState *s = ASPEED_SOC(dev);
> >>> +    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> >>> +    g_autofree char *sram_name = NULL;
> >>> +
> >>> +    /* Default boot region (SPI memory or ROMs) */
> >>> +    memory_region_init(&s->spi_boot_container, OBJECT(s),
> >>> +                       "aspeed.spi_boot_container", 0x400000000);
> >>> +    memory_region_add_subregion(s->memory,
> >> sc->memmap[ASPEED_DEV_SPI_BOOT],
> >>> +                                &s->spi_boot_container);
> >>> +
> >>> +    /* CPU */
> >>> +    for (i = 0; i < sc->num_cpus; i++) {
> >>> +        object_property_set_int(OBJECT(&a->cpu[i]), "mp-affinity",
> >>> +                                aspeed_calc_affinity(i),
> >>> + &error_abort);
> >>> +
> >>> +        object_property_set_int(OBJECT(&a->cpu[i]), "cntfrq",
> >> 1125000000,
> >>> +                                &error_abort);
> >>> +        object_property_set_link(OBJECT(&a->cpu[i]), "memory",
> >>> +                                 OBJECT(s->memory),
> >> &error_abort);
> >>> +
> >>> +        if (!qdev_realize(DEVICE(&a->cpu[i]), NULL, errp)) {
> >>> +            return;
> >>> +        }
> >>> +    }
> >>> +
> >>> +    /* GIC */
> >>> +    aspeed_soc_ast2700_gic(dev, errp);
> >>
> >> Hmm, we should return.
> > This function return type is void.
> > If I understand your request, do you mean do the following changes?
> >
> > static bool aspeed_soc_ast2700_gic(DeviceState *dev, Error **errp) -->
> > change return type bool {
> >      ---
> >      gicbusdev = SYS_BUS_DEVICE(a->intc.gic);
> >      if(!sysbus_realize(gicbusdev, errp)) {
> >          return false;
> >      }
> >      ---
> >      return true;
> > }
> >
> > Then,
> > /* GIC */
> > if(!aspeed_soc_ast2700_gic(dev, errp)) {
> >      return;
> >      }
> 
> 
> yes.
> 
> 
> Thanks,
> 
> C.
> 
Will change it.
Thanks-Jamin
> 
> > Thanks-Jamin
> >>
> >>> +
> >>> +    /* INTC */
> >>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&a->intc), errp)) {
> >>> +        return;
> >>> +    }
> >>> +
> >>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&a->intc), 0,
> >>> +                    sc->memmap[ASPEED_DEV_INTC]);
> >>> +
> >>> +    /* GICINT orgate -> INTC -> GIC */
> >>> +    for (i = 0; i < ASPEED_INTC_NR_GICS; i++) {
> >>> +        if (!qdev_realize(DEVICE(&a->intc.gicint_orgate[i]), NULL, errp))
> {
> >>> +            return;
> >>> +        }
> >>> +
> >>> +        qdev_connect_gpio_out(DEVICE(&a->intc.gicint_orgate[i]), 0,
> >>> +
> qdev_get_gpio_in(DEVICE(&a->intc),
> >> i));
> >>> +
> >>> +        sysbus_connect_irq(SYS_BUS_DEVICE(&a->intc), i,
> >>> +                           qdev_get_gpio_in(a->intc.gic,
> >>> +
> >> aspeed_soc_ast2700_gic_intcmap[i].irq));
> >>> +    }
> >>> +
> >>> +    /* SRAM */
> >>> +    sram_name = g_strdup_printf("aspeed.sram.%d",
> >> CPU(&a->cpu[0])->cpu_index);
> >>> +    if (!memory_region_init_ram(&s->sram, OBJECT(s), sram_name,
> >> sc->sram_size,
> >>> +                                 errp)) {
> >>> +        return;
> >>> +    }
> >>> +    memory_region_add_subregion(s->memory,
> >>> +
> sc->memmap[ASPEED_DEV_SRAM],
> >>> + &s->sram);
> >>> +
> >>> +    /* SCU */
> >>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scu), errp)) {
> >>> +        return;
> >>> +    }
> >>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scu), 0,
> >>> + sc->memmap[ASPEED_DEV_SCU]);
> >>> +
> >>> +    /* SCU1 */
> >>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scuio), errp)) {
> >>> +        return;
> >>> +    }
> >>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scuio), 0,
> >>> +                    sc->memmap[ASPEED_DEV_SCUIO]);
> >>> +
> >>> +    /* UART */
> >>> +    if (!aspeed_soc_uart_realize(s, errp)) {
> >>> +        return;
> >>> +    }
> >>> +
> >>> +    /* FMC, The number of CS is set at the board level */
> >>> +    object_property_set_link(OBJECT(&s->fmc), "dram",
> >> OBJECT(s->dram_mr),
> >>> +                             &error_abort);
> >>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->fmc), errp)) {
> >>> +        return;
> >>> +    }
> >>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 0,
> >> sc->memmap[ASPEED_DEV_FMC]);
> >>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 1,
> >>> +
> >> ASPEED_SMC_GET_CLASS(&s->fmc)->flash_window_base);
> >>> +    sysbus_connect_irq(SYS_BUS_DEVICE(&s->fmc), 0,
> >>> +                       aspeed_soc_get_irq(s, ASPEED_DEV_FMC));
> >>> +
> >>> +    /* Set up an alias on the FMC CE0 region (boot default) */
> >>> +    MemoryRegion *fmc0_mmio = &s->fmc.flashes[0].mmio;
> >>> +    memory_region_init_alias(&s->spi_boot, OBJECT(s),
> >> "aspeed.spi_boot",
> >>> +                             fmc0_mmio, 0,
> >> memory_region_size(fmc0_mmio));
> >>> +    memory_region_add_subregion(&s->spi_boot_container, 0x0,
> >>> + &s->spi_boot);
> >>> +
> >>> +    /* SPI */
> >>> +    for (i = 0; i < sc->spis_num; i++) {
> >>> +        object_property_set_link(OBJECT(&s->spi[i]), "dram",
> >>> +                                 OBJECT(s->dram_mr),
> >> &error_abort);
> >>> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->spi[i]), errp)) {
> >>> +            return;
> >>> +        }
> >>> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 0,
> >>> +                        sc->memmap[ASPEED_DEV_SPI0 + i]);
> >>> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 1,
> >>> +
> >> ASPEED_SMC_GET_CLASS(&s->spi[i])->flash_window_base);
> >>> +    }
> >>> +
> >>> +    /*
> >>> +     * SDMC - SDRAM Memory Controller
> >>> +     * The SDMC controller is unlocked at SPL stage.
> >>> +     * At present, only supports to emulate booting
> >>> +     * start from u-boot stage. Set SDMC controller
> >>> +     * unlocked by default. It is a temporarily solution.
> >>> +     */
> >>> +    object_property_set_bool(OBJECT(&s->sdmc), "unlocked", true,
> >>> +                                 &error_abort);
> >>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sdmc), errp)) {
> >>> +        return;
> >>> +    }
> >>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sdmc), 0,
> >>> +                    sc->memmap[ASPEED_DEV_SDMC]);
> >>> +
> >>> +    /* RAM */
> >>> +    if (!aspeed_soc_dram_init(s, errp)) {
> >>> +        return;
> >>> +    }
> >>> +
> >>> +    for (i = 0; i < sc->macs_num; i++) {
> >>> +        object_property_set_bool(OBJECT(&s->ftgmac100[i]),
> >>> + "aspeed",
> >> true,
> >>> +                                 &error_abort);
> >>> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->ftgmac100[i]), errp)) {
> >>> +            return;
> >>> +        }
> >>> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
> >>> +                        sc->memmap[ASPEED_DEV_ETH1 + i]);
> >>> +        sysbus_connect_irq(SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
> >>> +                           aspeed_soc_get_irq(s,
> ASPEED_DEV_ETH1
> >> +
> >>> + i));
> >>> +
> >>> +        object_property_set_link(OBJECT(&s->mii[i]), "nic",
> >>> +                                 OBJECT(&s->ftgmac100[i]),
> >> &error_abort);
> >>> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->mii[i]), errp)) {
> >>> +            return;
> >>> +        }
> >>> +
> >>> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->mii[i]), 0,
> >>> +                        sc->memmap[ASPEED_DEV_MII1 + i]);
> >>> +    }
> >>> +
> >>> +    /* Watch dog */
> >>> +    for (i = 0; i < sc->wdts_num; i++) {
> >>> +        AspeedWDTClass *awc =
> ASPEED_WDT_GET_CLASS(&s->wdt[i]);
> >>> +        hwaddr wdt_offset = sc->memmap[ASPEED_DEV_WDT] + i *
> >>> + awc->iosize;
> >>> +
> >>> +        object_property_set_link(OBJECT(&s->wdt[i]), "scu",
> >> OBJECT(&s->scu),
> >>> +                                 &error_abort);
> >>> +        if (!sysbus_realize(SYS_BUS_DEVICE(&s->wdt[i]), errp)) {
> >>> +            return;
> >>> +        }
> >>> +        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->wdt[i]), 0,
> >> wdt_offset);
> >>> +    }
> >>> +
> >>> +    /* SLI */
> >>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sli), errp)) {
> >>> +        return;
> >>> +    }
> >>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sli), 0,
> >>> + sc->memmap[ASPEED_DEV_SLI]);
> >>> +
> >>> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sliio), errp)) {
> >>> +        return;
> >>> +    }
> >>> +    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sliio), 0,
> >>> +                    sc->memmap[ASPEED_DEV_SLIIO]);
> >>> +
> >>> +    create_unimplemented_device("ast2700.dpmcu", 0x11000000,
> >> 0x40000);
> >>> +    create_unimplemented_device("ast2700.iomem0", 0x12000000,
> >> 0x01000000);
> >>> +    create_unimplemented_device("ast2700.iomem1", 0x14000000,
> >> 0x01000000);
> >>> +    create_unimplemented_device("ast2700.ltpi", 0x30000000,
> >> 0x1000000);
> >>> +    create_unimplemented_device("ast2700.io", 0x0, 0x4000000); }
> >>> +
> >>> +static void aspeed_soc_ast2700_class_init(ObjectClass *oc, void
> >>> +*data) {
> >>> +    static const char * const valid_cpu_types[] = {
> >>> +        ARM_CPU_TYPE_NAME("cortex-a35"),
> >>> +        NULL
> >>> +    };
> >>> +    DeviceClass *dc = DEVICE_CLASS(oc);
> >>> +    AspeedSoCClass *sc = ASPEED_SOC_CLASS(oc);
> >>> +
> >>> +    dc->realize      = aspeed_soc_ast2700_realize;
> >>
> >>
> >> Please add :
> >>
> >>       /* Reason: The Aspeed SoC can only be instantiated from a board */
> >>       dc->user_creatable = false;
> >>
> >>
> > Will add.
> >> Thanks,
> >>
> >> C.
> >>
> >>
> >>> +
> >>> +    sc->name         = "ast2700-a0";
> >>> +    sc->valid_cpu_types = valid_cpu_types;
> >>> +    sc->silicon_rev  = AST2700_A0_SILICON_REV;
> >>> +    sc->sram_size    = 0x20000;
> >>> +    sc->spis_num     = 3;
> >>> +    sc->wdts_num     = 8;
> >>> +    sc->macs_num     = 1;
> >>> +    sc->uarts_num    = 13;
> >>> +    sc->num_cpus     = 4;
> >>> +    sc->uarts_base   = ASPEED_DEV_UART0;
> >>> +    sc->irqmap       = aspeed_soc_ast2700_irqmap;
> >>> +    sc->memmap       = aspeed_soc_ast2700_memmap;
> >>> +    sc->get_irq      = aspeed_soc_ast2700_get_irq;
> >>> +}
> >>> +
> >>> +static const TypeInfo aspeed_soc_ast27x0_types[] = {
> >>> +    {
> >>> +        .name           = TYPE_ASPEED27X0_SOC,
> >>> +        .parent         = TYPE_ASPEED_SOC,
> >>> +        .instance_size  = sizeof(Aspeed27x0SoCState),
> >>> +        .abstract       = true,
> >>> +    }, {
> >>> +        .name           = "ast2700-a0",
> >>> +        .parent         = TYPE_ASPEED27X0_SOC,
> >>> +        .instance_init  = aspeed_soc_ast2700_init,
> >>> +        .class_init     = aspeed_soc_ast2700_class_init,
> >>> +    },
> >>> +};
> >>> +
> >>> +DEFINE_TYPES(aspeed_soc_ast27x0_types)
> >>> diff --git a/hw/arm/meson.build b/hw/arm/meson.build index
> >>> 6808135c1f..1e3295a423 100644
> >>> --- a/hw/arm/meson.build
> >>> +++ b/hw/arm/meson.build
> >>> @@ -46,6 +46,7 @@ arm_ss.add(when: 'CONFIG_ASPEED_SOC', if_true:
> >> files(
> >>>      'aspeed_soc_common.c',
> >>>      'aspeed_ast2400.c',
> >>>      'aspeed_ast2600.c',
> >>> +  'aspeed_ast27x0.c',
> >>>      'aspeed_ast10x0.c',
> >>>      'aspeed_eeprom.c',
> >>>      'fby35.c'))
> >>> diff --git a/include/hw/arm/aspeed_soc.h
> >>> b/include/hw/arm/aspeed_soc.h index c60fac900a..9f177b6037 100644
> >>> --- a/include/hw/arm/aspeed_soc.h
> >>> +++ b/include/hw/arm/aspeed_soc.h
> >>> @@ -15,6 +15,7 @@
> >>>    #include "hw/cpu/a15mpcore.h"
> >>>    #include "hw/arm/armv7m.h"
> >>>    #include "hw/intc/aspeed_vic.h"
> >>> +#include "hw/intc/aspeed_intc.h"
> >>>    #include "hw/misc/aspeed_scu.h"
> >>>    #include "hw/adc/aspeed_adc.h"
> >>>    #include "hw/misc/aspeed_sdmc.h"
> >>> @@ -26,6 +27,7 @@
> >>>    #include "hw/ssi/aspeed_smc.h"
> >>>    #include "hw/misc/aspeed_hace.h"
> >>>    #include "hw/misc/aspeed_sbc.h"
> >>> +#include "hw/misc/aspeed_sli.h"
> >>>    #include "hw/watchdog/wdt_aspeed.h"
> >>>    #include "hw/net/ftgmac100.h"
> >>>    #include "target/arm/cpu.h"
> >>> @@ -41,8 +43,8 @@
> >>>
> >>>    #define ASPEED_SPIS_NUM  2
> >>>    #define ASPEED_EHCIS_NUM 2
> >>> -#define ASPEED_WDTS_NUM  4
> >>> -#define ASPEED_CPUS_NUM  2
> >>> +#define ASPEED_WDTS_NUM  8
> >>> +#define ASPEED_CPUS_NUM  4
> >>>    #define ASPEED_MACS_NUM  4
> >>>    #define ASPEED_UARTS_NUM 13
> >>>    #define ASPEED_JTAG_NUM  2
> >>> @@ -61,6 +63,7 @@ struct AspeedSoCState {
> >>>        AspeedI2CState i2c;
> >>>        AspeedI3CState i3c;
> >>>        AspeedSCUState scu;
> >>> +    AspeedSCUState scuio;
> >>>        AspeedHACEState hace;
> >>>        AspeedXDMAState xdma;
> >>>        AspeedADCState adc;
> >>> @@ -68,6 +71,8 @@ struct AspeedSoCState {
> >>>        AspeedSMCState spi[ASPEED_SPIS_NUM];
> >>>        EHCISysBusState ehci[ASPEED_EHCIS_NUM];
> >>>        AspeedSBCState sbc;
> >>> +    AspeedSLIState sli;
> >>> +    AspeedSLIState sliio;
> >>>        MemoryRegion secsram;
> >>>        UnimplementedDeviceState sbc_unimplemented;
> >>>        AspeedSDMCState sdmc;
> >>> @@ -117,6 +122,16 @@ struct Aspeed2600SoCState {
> >>>    #define TYPE_ASPEED2600_SOC "aspeed2600-soc"
> >>>    OBJECT_DECLARE_SIMPLE_TYPE(Aspeed2600SoCState,
> >> ASPEED2600_SOC)
> >>>
> >>> +struct Aspeed27x0SoCState {
> >>> +    AspeedSoCState parent;
> >>> +
> >>> +    ARMCPU cpu[ASPEED_CPUS_NUM];
> >>> +    AspeedINTCState intc;
> >>> +};
> >>> +
> >>> +#define TYPE_ASPEED27X0_SOC "aspeed27x0-soc"
> >>> +OBJECT_DECLARE_SIMPLE_TYPE(Aspeed27x0SoCState,
> ASPEED27X0_SOC)
> >>> +
> >>>    struct Aspeed10x0SoCState {
> >>>        AspeedSoCState parent;
> >>>
> >>> @@ -168,11 +183,13 @@ enum {
> >>>        ASPEED_DEV_UART13,
> >>>        ASPEED_DEV_VUART,
> >>>        ASPEED_DEV_FMC,
> >>> +    ASPEED_DEV_SPI0,
> >>>        ASPEED_DEV_SPI1,
> >>>        ASPEED_DEV_SPI2,
> >>>        ASPEED_DEV_EHCI1,
> >>>        ASPEED_DEV_EHCI2,
> >>>        ASPEED_DEV_VIC,
> >>> +    ASPEED_DEV_INTC,
> >>>        ASPEED_DEV_SDMC,
> >>>        ASPEED_DEV_SCU,
> >>>        ASPEED_DEV_ADC,
> >>> @@ -222,6 +239,11 @@ enum {
> >>>        ASPEED_DEV_JTAG1,
> >>>        ASPEED_DEV_FSI1,
> >>>        ASPEED_DEV_FSI2,
> >>> +    ASPEED_DEV_SCUIO,
> >>> +    ASPEED_DEV_SLI,
> >>> +    ASPEED_DEV_SLIIO,
> >>> +    ASPEED_GIC_DIST,
> >>> +    ASPEED_GIC_REDIST,
> >>>    };
> >>>
> >>>    qemu_irq aspeed_soc_get_irq(AspeedSoCState *s, int dev);
> >
diff mbox series

Patch

diff --git a/hw/arm/aspeed_ast27x0.c b/hw/arm/aspeed_ast27x0.c
new file mode 100644
index 0000000000..754c963230
--- /dev/null
+++ b/hw/arm/aspeed_ast27x0.c
@@ -0,0 +1,554 @@ 
+/*
+ * ASPEED SoC 27x0 family
+ *
+ * Copyright (C) 2024 ASPEED Technology Inc.
+ *
+ * This code is licensed under the GPL version 2 or later.  See
+ * the COPYING file in the top-level directory.
+ *
+ * Implementation extracted from the AST2600 and adapted for AST27x0.
+ */
+
+#include "qemu/osdep.h"
+#include "qapi/error.h"
+#include "hw/misc/unimp.h"
+#include "hw/arm/aspeed_soc.h"
+#include "qemu/module.h"
+#include "qemu/error-report.h"
+#include "hw/i2c/aspeed_i2c.h"
+#include "net/net.h"
+#include "sysemu/sysemu.h"
+#include "hw/intc/arm_gicv3.h"
+#include "qapi/qmp/qlist.h"
+
+static const hwaddr aspeed_soc_ast2700_memmap[] = {
+    [ASPEED_DEV_SPI_BOOT]  =  0x400000000,
+    [ASPEED_DEV_SRAM]      =  0x10000000,
+    [ASPEED_DEV_SDMC]      =  0x12C00000,
+    [ASPEED_DEV_SCU]       =  0x12C02000,
+    [ASPEED_DEV_SCUIO]     =  0x14C02000,
+    [ASPEED_DEV_UART0]     =  0X14C33000,
+    [ASPEED_DEV_UART1]     =  0X14C33100,
+    [ASPEED_DEV_UART2]     =  0X14C33200,
+    [ASPEED_DEV_UART3]     =  0X14C33300,
+    [ASPEED_DEV_UART4]     =  0X12C1A000,
+    [ASPEED_DEV_UART5]     =  0X14C33400,
+    [ASPEED_DEV_UART6]     =  0X14C33500,
+    [ASPEED_DEV_UART7]     =  0X14C33600,
+    [ASPEED_DEV_UART8]     =  0X14C33700,
+    [ASPEED_DEV_UART9]     =  0X14C33800,
+    [ASPEED_DEV_UART10]    =  0X14C33900,
+    [ASPEED_DEV_UART11]    =  0X14C33A00,
+    [ASPEED_DEV_UART12]    =  0X14C33B00,
+    [ASPEED_DEV_WDT]       =  0x14C37000,
+    [ASPEED_DEV_VUART]     =  0X14C30000,
+    [ASPEED_DEV_FMC]       =  0x14000000,
+    [ASPEED_DEV_SPI0]      =  0x14010000,
+    [ASPEED_DEV_SPI1]      =  0x14020000,
+    [ASPEED_DEV_SPI2]      =  0x14030000,
+    [ASPEED_DEV_SDRAM]     =  0x400000000,
+    [ASPEED_DEV_MII1]      =  0x14040000,
+    [ASPEED_DEV_MII2]      =  0x14040008,
+    [ASPEED_DEV_MII3]      =  0x14040010,
+    [ASPEED_DEV_ETH1]      =  0x14050000,
+    [ASPEED_DEV_ETH2]      =  0x14060000,
+    [ASPEED_DEV_ETH3]      =  0x14070000,
+    [ASPEED_DEV_EMMC]      =  0x12090000,
+    [ASPEED_DEV_INTC]      =  0x12100000,
+    [ASPEED_DEV_SLI]       =  0x12C17000,
+    [ASPEED_DEV_SLIIO]     =  0x14C1E000,
+    [ASPEED_GIC_DIST]      =  0x12200000,
+    [ASPEED_GIC_REDIST]    =  0x12280000,
+};
+
+#define AST2700_MAX_IRQ 288
+
+/* Shared Peripheral Interrupt values below are offset by -32 from datasheet */
+static const int aspeed_soc_ast2700_irqmap[] = {
+    [ASPEED_DEV_UART0]     = 132,
+    [ASPEED_DEV_UART1]     = 132,
+    [ASPEED_DEV_UART2]     = 132,
+    [ASPEED_DEV_UART3]     = 132,
+    [ASPEED_DEV_UART4]     = 8,
+    [ASPEED_DEV_UART5]     = 132,
+    [ASPEED_DEV_UART6]     = 132,
+    [ASPEED_DEV_UART7]     = 132,
+    [ASPEED_DEV_UART8]     = 132,
+    [ASPEED_DEV_UART9]     = 132,
+    [ASPEED_DEV_UART10]    = 132,
+    [ASPEED_DEV_UART11]    = 132,
+    [ASPEED_DEV_UART12]    = 132,
+    [ASPEED_DEV_FMC]       = 131,
+    [ASPEED_DEV_SDMC]      = 0,
+    [ASPEED_DEV_SCU]       = 12,
+    [ASPEED_DEV_ADC]       = 130,
+    [ASPEED_DEV_XDMA]      = 5,
+    [ASPEED_DEV_EMMC]      = 15,
+    [ASPEED_DEV_GPIO]      = 11,
+    [ASPEED_DEV_GPIO_1_8V] = 130,
+    [ASPEED_DEV_RTC]       = 13,
+    [ASPEED_DEV_TIMER1]    = 16,
+    [ASPEED_DEV_TIMER2]    = 17,
+    [ASPEED_DEV_TIMER3]    = 18,
+    [ASPEED_DEV_TIMER4]    = 19,
+    [ASPEED_DEV_TIMER5]    = 20,
+    [ASPEED_DEV_TIMER6]    = 21,
+    [ASPEED_DEV_TIMER7]    = 22,
+    [ASPEED_DEV_TIMER8]    = 23,
+    [ASPEED_DEV_WDT]       = 131,
+    [ASPEED_DEV_PWM]       = 131,
+    [ASPEED_DEV_LPC]       = 128,
+    [ASPEED_DEV_IBT]       = 128,
+    [ASPEED_DEV_I2C]       = 130,
+    [ASPEED_DEV_PECI]      = 133,
+    [ASPEED_DEV_ETH1]      = 132,
+    [ASPEED_DEV_ETH2]      = 132,
+    [ASPEED_DEV_ETH3]      = 132,
+    [ASPEED_DEV_HACE]      = 4,
+    [ASPEED_DEV_KCS]       = 128,
+    [ASPEED_DEV_DP]        = 28,
+    [ASPEED_DEV_I3C]       = 131,
+};
+
+/* GICINT 128 */
+static const int aspeed_soc_ast2700_gic128_intcmap[] = {
+    [ASPEED_DEV_LPC]       = 0,
+    [ASPEED_DEV_IBT]       = 2,
+    [ASPEED_DEV_KCS]       = 4,
+};
+
+/* GICINT 130 */
+static const int aspeed_soc_ast2700_gic130_intcmap[] = {
+    [ASPEED_DEV_I2C]        = 0,
+    [ASPEED_DEV_ADC]        = 16,
+    [ASPEED_DEV_GPIO_1_8V]  = 18,
+};
+
+/* GICINT 131 */
+static const int aspeed_soc_ast2700_gic131_intcmap[] = {
+    [ASPEED_DEV_I3C]       = 0,
+    [ASPEED_DEV_WDT]       = 16,
+    [ASPEED_DEV_FMC]       = 25,
+    [ASPEED_DEV_PWM]       = 29,
+};
+
+/* GICINT 132 */
+static const int aspeed_soc_ast2700_gic132_intcmap[] = {
+    [ASPEED_DEV_ETH1]      = 0,
+    [ASPEED_DEV_ETH2]      = 1,
+    [ASPEED_DEV_ETH3]      = 2,
+    [ASPEED_DEV_UART0]     = 7,
+    [ASPEED_DEV_UART1]     = 8,
+    [ASPEED_DEV_UART2]     = 9,
+    [ASPEED_DEV_UART3]     = 10,
+    [ASPEED_DEV_UART5]     = 11,
+    [ASPEED_DEV_UART6]     = 12,
+    [ASPEED_DEV_UART7]     = 13,
+    [ASPEED_DEV_UART8]     = 14,
+    [ASPEED_DEV_UART9]     = 15,
+    [ASPEED_DEV_UART10]    = 16,
+    [ASPEED_DEV_UART11]    = 17,
+    [ASPEED_DEV_UART12]    = 18,
+};
+
+/* GICINT 133 */
+static const int aspeed_soc_ast2700_gic133_intcmap[] = {
+    [ASPEED_DEV_PECI]      = 4,
+};
+
+/* GICINT 128 ~ 136 */
+struct gic_intc_irq_info {
+    int irq;
+    const int *ptr;
+};
+
+static const struct gic_intc_irq_info aspeed_soc_ast2700_gic_intcmap[] = {
+    {128,  aspeed_soc_ast2700_gic128_intcmap},
+    {129,  NULL},
+    {130,  aspeed_soc_ast2700_gic130_intcmap},
+    {131,  aspeed_soc_ast2700_gic131_intcmap},
+    {132,  aspeed_soc_ast2700_gic132_intcmap},
+    {133,  aspeed_soc_ast2700_gic133_intcmap},
+    {134,  NULL},
+    {135,  NULL},
+    {136,  NULL},
+};
+
+static qemu_irq aspeed_soc_ast2700_get_irq(AspeedSoCState *s, int dev)
+{
+    Aspeed27x0SoCState *a = ASPEED27X0_SOC(s);
+    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
+    int i;
+
+    for (i = 0; i < ARRAY_SIZE(aspeed_soc_ast2700_gic_intcmap); i++) {
+        if (sc->irqmap[dev] == aspeed_soc_ast2700_gic_intcmap[i].irq) {
+            assert(aspeed_soc_ast2700_gic_intcmap[i].ptr);
+            return qdev_get_gpio_in(DEVICE(&a->intc.gicint_orgate[i]),
+                aspeed_soc_ast2700_gic_intcmap[i].ptr[dev]);
+        }
+    }
+
+    return qdev_get_gpio_in(a->intc.gic, sc->irqmap[dev]);
+}
+
+static void aspeed_soc_ast2700_init(Object *obj)
+{
+    Aspeed27x0SoCState *a = ASPEED27X0_SOC(obj);
+    AspeedSoCState *s = ASPEED_SOC(obj);
+    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
+    int i;
+    char socname[8];
+    char typename[64];
+
+    if (sscanf(sc->name, "%7s", socname) != 1) {
+        g_assert_not_reached();
+    }
+
+    for (i = 0; i < sc->num_cpus; i++) {
+        object_initialize_child(obj, "cpu[*]", &a->cpu[i],
+                                aspeed_soc_cpu_type(sc));
+    }
+
+    object_initialize_child(obj, "scu", &s->scu, TYPE_ASPEED_2700_SCU);
+    qdev_prop_set_uint32(DEVICE(&s->scu), "silicon-rev",
+                         sc->silicon_rev);
+    object_property_add_alias(obj, "hw-strap1", OBJECT(&s->scu),
+                              "hw-strap1");
+    object_property_add_alias(obj, "hw-strap2", OBJECT(&s->scu),
+                              "hw-strap2");
+    object_property_add_alias(obj, "hw-prot-key", OBJECT(&s->scu),
+                              "hw-prot-key");
+
+    object_initialize_child(obj, "scuio", &s->scuio, TYPE_ASPEED_2700_SCUIO);
+    qdev_prop_set_uint32(DEVICE(&s->scuio), "silicon-rev",
+                         sc->silicon_rev);
+
+    snprintf(typename, sizeof(typename), "aspeed.fmc-%s", socname);
+    object_initialize_child(obj, "fmc", &s->fmc, typename);
+
+    for (i = 0; i < sc->spis_num; i++) {
+        snprintf(typename, sizeof(typename), "aspeed.spi%d-%s", i, socname);
+        object_initialize_child(obj, "spi[*]", &s->spi[i], typename);
+    }
+
+    snprintf(typename, sizeof(typename), "aspeed.sdmc-%s", socname);
+    object_initialize_child(obj, "sdmc", &s->sdmc, typename);
+    object_property_add_alias(obj, "ram-size", OBJECT(&s->sdmc),
+                              "ram-size");
+
+    for (i = 0; i < sc->wdts_num; i++) {
+        snprintf(typename, sizeof(typename), "aspeed.wdt-%s", socname);
+        object_initialize_child(obj, "wdt[*]", &s->wdt[i], typename);
+    }
+
+    for (i = 0; i < sc->macs_num; i++) {
+        object_initialize_child(obj, "ftgmac100[*]", &s->ftgmac100[i],
+                                TYPE_FTGMAC100);
+
+        object_initialize_child(obj, "mii[*]", &s->mii[i], TYPE_ASPEED_MII);
+    }
+
+    for (i = 0; i < sc->uarts_num; i++) {
+        object_initialize_child(obj, "uart[*]", &s->uart[i], TYPE_SERIAL_MM);
+    }
+
+    object_initialize_child(obj, "sli", &s->sli, TYPE_ASPEED_2700_SLI);
+    object_initialize_child(obj, "sliio", &s->sliio, TYPE_ASPEED_2700_SLIIO);
+    object_initialize_child(obj, "intc", &a->intc, TYPE_ASPEED_INTC);
+}
+
+/*
+ * ASPEED ast2700 has 0x0 as cluster ID
+ *
+ * https://developer.arm.com/documentation/100236/0100/register-descriptions/aarch64-system-registers/multiprocessor-affinity-register--el1
+ */
+static uint64_t aspeed_calc_affinity(int cpu)
+{
+    return (0x0 << ARM_AFF1_SHIFT) | cpu;
+}
+
+static void aspeed_soc_ast2700_gic(DeviceState *dev, Error **errp)
+{
+    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
+    AspeedSoCState *s = ASPEED_SOC(dev);
+    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
+    SysBusDevice *gicbusdev;
+    QList *redist_region_count;
+    int i;
+
+    a->intc.gic = qdev_new(gicv3_class_name());
+    qdev_prop_set_uint32(a->intc.gic, "revision", 3);
+    qdev_prop_set_uint32(a->intc.gic, "num-cpu", sc->num_cpus);
+    qdev_prop_set_uint32(a->intc.gic, "num-irq", AST2700_MAX_IRQ);
+
+    redist_region_count = qlist_new();
+    qlist_append_int(redist_region_count, sc->num_cpus);
+    qdev_prop_set_array(a->intc.gic, "redist-region-count",
+                            redist_region_count);
+
+    gicbusdev = SYS_BUS_DEVICE(a->intc.gic);
+    sysbus_realize_and_unref(gicbusdev, errp);
+    sysbus_mmio_map(gicbusdev, 0, sc->memmap[ASPEED_GIC_DIST]);
+    sysbus_mmio_map(gicbusdev, 1, sc->memmap[ASPEED_GIC_REDIST]);
+
+    for (i = 0; i < sc->num_cpus; i++) {
+        DeviceState *cpudev = DEVICE(qemu_get_cpu(i));
+        int NUM_IRQS = 256, ARCH_GIC_MAINT_IRQ = 9, VIRTUAL_PMU_IRQ = 7;
+        int ppibase = NUM_IRQS + i * GIC_INTERNAL + GIC_NR_SGIS;
+
+        const int timer_irq[] = {
+            [GTIMER_PHYS] = 14,
+            [GTIMER_VIRT] = 11,
+            [GTIMER_HYP]  = 10,
+            [GTIMER_SEC]  = 13,
+        };
+        int j;
+
+        for (j = 0; j < ARRAY_SIZE(timer_irq); j++) {
+            qdev_connect_gpio_out(cpudev, j,
+                    qdev_get_gpio_in(a->intc.gic, ppibase + timer_irq[j]));
+        }
+
+        qemu_irq irq = qdev_get_gpio_in(a->intc.gic,
+                                        ppibase + ARCH_GIC_MAINT_IRQ);
+        qdev_connect_gpio_out_named(cpudev, "gicv3-maintenance-interrupt",
+                                    0, irq);
+        qdev_connect_gpio_out_named(cpudev, "pmu-interrupt", 0,
+                qdev_get_gpio_in(a->intc.gic, ppibase + VIRTUAL_PMU_IRQ));
+
+        sysbus_connect_irq(gicbusdev, i, qdev_get_gpio_in(cpudev, ARM_CPU_IRQ));
+        sysbus_connect_irq(gicbusdev, i + sc->num_cpus,
+                           qdev_get_gpio_in(cpudev, ARM_CPU_FIQ));
+        sysbus_connect_irq(gicbusdev, i + 2 * sc->num_cpus,
+                           qdev_get_gpio_in(cpudev, ARM_CPU_VIRQ));
+        sysbus_connect_irq(gicbusdev, i + 3 * sc->num_cpus,
+                           qdev_get_gpio_in(cpudev, ARM_CPU_VFIQ));
+    }
+}
+
+static void aspeed_soc_ast2700_realize(DeviceState *dev, Error **errp)
+{
+    int i;
+    Aspeed27x0SoCState *a = ASPEED27X0_SOC(dev);
+    AspeedSoCState *s = ASPEED_SOC(dev);
+    AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
+    g_autofree char *sram_name = NULL;
+
+    /* Default boot region (SPI memory or ROMs) */
+    memory_region_init(&s->spi_boot_container, OBJECT(s),
+                       "aspeed.spi_boot_container", 0x400000000);
+    memory_region_add_subregion(s->memory, sc->memmap[ASPEED_DEV_SPI_BOOT],
+                                &s->spi_boot_container);
+
+    /* CPU */
+    for (i = 0; i < sc->num_cpus; i++) {
+        object_property_set_int(OBJECT(&a->cpu[i]), "mp-affinity",
+                                aspeed_calc_affinity(i), &error_abort);
+
+        object_property_set_int(OBJECT(&a->cpu[i]), "cntfrq", 1125000000,
+                                &error_abort);
+        object_property_set_link(OBJECT(&a->cpu[i]), "memory",
+                                 OBJECT(s->memory), &error_abort);
+
+        if (!qdev_realize(DEVICE(&a->cpu[i]), NULL, errp)) {
+            return;
+        }
+    }
+
+    /* GIC */
+    aspeed_soc_ast2700_gic(dev, errp);
+
+    /* INTC */
+    if (!sysbus_realize(SYS_BUS_DEVICE(&a->intc), errp)) {
+        return;
+    }
+
+    aspeed_mmio_map(s, SYS_BUS_DEVICE(&a->intc), 0,
+                    sc->memmap[ASPEED_DEV_INTC]);
+
+    /* GICINT orgate -> INTC -> GIC */
+    for (i = 0; i < ASPEED_INTC_NR_GICS; i++) {
+        if (!qdev_realize(DEVICE(&a->intc.gicint_orgate[i]), NULL, errp)) {
+            return;
+        }
+
+        qdev_connect_gpio_out(DEVICE(&a->intc.gicint_orgate[i]), 0,
+                              qdev_get_gpio_in(DEVICE(&a->intc), i));
+
+        sysbus_connect_irq(SYS_BUS_DEVICE(&a->intc), i,
+                           qdev_get_gpio_in(a->intc.gic,
+                                aspeed_soc_ast2700_gic_intcmap[i].irq));
+    }
+
+    /* SRAM */
+    sram_name = g_strdup_printf("aspeed.sram.%d", CPU(&a->cpu[0])->cpu_index);
+    if (!memory_region_init_ram(&s->sram, OBJECT(s), sram_name, sc->sram_size,
+                                 errp)) {
+        return;
+    }
+    memory_region_add_subregion(s->memory,
+                                sc->memmap[ASPEED_DEV_SRAM], &s->sram);
+
+    /* SCU */
+    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scu), errp)) {
+        return;
+    }
+    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scu), 0, sc->memmap[ASPEED_DEV_SCU]);
+
+    /* SCU1 */
+    if (!sysbus_realize(SYS_BUS_DEVICE(&s->scuio), errp)) {
+        return;
+    }
+    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->scuio), 0,
+                    sc->memmap[ASPEED_DEV_SCUIO]);
+
+    /* UART */
+    if (!aspeed_soc_uart_realize(s, errp)) {
+        return;
+    }
+
+    /* FMC, The number of CS is set at the board level */
+    object_property_set_link(OBJECT(&s->fmc), "dram", OBJECT(s->dram_mr),
+                             &error_abort);
+    if (!sysbus_realize(SYS_BUS_DEVICE(&s->fmc), errp)) {
+        return;
+    }
+    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 0, sc->memmap[ASPEED_DEV_FMC]);
+    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->fmc), 1,
+                    ASPEED_SMC_GET_CLASS(&s->fmc)->flash_window_base);
+    sysbus_connect_irq(SYS_BUS_DEVICE(&s->fmc), 0,
+                       aspeed_soc_get_irq(s, ASPEED_DEV_FMC));
+
+    /* Set up an alias on the FMC CE0 region (boot default) */
+    MemoryRegion *fmc0_mmio = &s->fmc.flashes[0].mmio;
+    memory_region_init_alias(&s->spi_boot, OBJECT(s), "aspeed.spi_boot",
+                             fmc0_mmio, 0, memory_region_size(fmc0_mmio));
+    memory_region_add_subregion(&s->spi_boot_container, 0x0, &s->spi_boot);
+
+    /* SPI */
+    for (i = 0; i < sc->spis_num; i++) {
+        object_property_set_link(OBJECT(&s->spi[i]), "dram",
+                                 OBJECT(s->dram_mr), &error_abort);
+        if (!sysbus_realize(SYS_BUS_DEVICE(&s->spi[i]), errp)) {
+            return;
+        }
+        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 0,
+                        sc->memmap[ASPEED_DEV_SPI0 + i]);
+        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->spi[i]), 1,
+                        ASPEED_SMC_GET_CLASS(&s->spi[i])->flash_window_base);
+    }
+
+    /*
+     * SDMC - SDRAM Memory Controller
+     * The SDMC controller is unlocked at SPL stage.
+     * At present, only supports to emulate booting
+     * start from u-boot stage. Set SDMC controller
+     * unlocked by default. It is a temporarily solution.
+     */
+    object_property_set_bool(OBJECT(&s->sdmc), "unlocked", true,
+                                 &error_abort);
+    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sdmc), errp)) {
+        return;
+    }
+    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sdmc), 0,
+                    sc->memmap[ASPEED_DEV_SDMC]);
+
+    /* RAM */
+    if (!aspeed_soc_dram_init(s, errp)) {
+        return;
+    }
+
+    for (i = 0; i < sc->macs_num; i++) {
+        object_property_set_bool(OBJECT(&s->ftgmac100[i]), "aspeed", true,
+                                 &error_abort);
+        if (!sysbus_realize(SYS_BUS_DEVICE(&s->ftgmac100[i]), errp)) {
+            return;
+        }
+        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
+                        sc->memmap[ASPEED_DEV_ETH1 + i]);
+        sysbus_connect_irq(SYS_BUS_DEVICE(&s->ftgmac100[i]), 0,
+                           aspeed_soc_get_irq(s, ASPEED_DEV_ETH1 + i));
+
+        object_property_set_link(OBJECT(&s->mii[i]), "nic",
+                                 OBJECT(&s->ftgmac100[i]), &error_abort);
+        if (!sysbus_realize(SYS_BUS_DEVICE(&s->mii[i]), errp)) {
+            return;
+        }
+
+        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->mii[i]), 0,
+                        sc->memmap[ASPEED_DEV_MII1 + i]);
+    }
+
+    /* Watch dog */
+    for (i = 0; i < sc->wdts_num; i++) {
+        AspeedWDTClass *awc = ASPEED_WDT_GET_CLASS(&s->wdt[i]);
+        hwaddr wdt_offset = sc->memmap[ASPEED_DEV_WDT] + i * awc->iosize;
+
+        object_property_set_link(OBJECT(&s->wdt[i]), "scu", OBJECT(&s->scu),
+                                 &error_abort);
+        if (!sysbus_realize(SYS_BUS_DEVICE(&s->wdt[i]), errp)) {
+            return;
+        }
+        aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->wdt[i]), 0, wdt_offset);
+    }
+
+    /* SLI */
+    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sli), errp)) {
+        return;
+    }
+    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sli), 0, sc->memmap[ASPEED_DEV_SLI]);
+
+    if (!sysbus_realize(SYS_BUS_DEVICE(&s->sliio), errp)) {
+        return;
+    }
+    aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->sliio), 0,
+                    sc->memmap[ASPEED_DEV_SLIIO]);
+
+    create_unimplemented_device("ast2700.dpmcu", 0x11000000, 0x40000);
+    create_unimplemented_device("ast2700.iomem0", 0x12000000, 0x01000000);
+    create_unimplemented_device("ast2700.iomem1", 0x14000000, 0x01000000);
+    create_unimplemented_device("ast2700.ltpi", 0x30000000, 0x1000000);
+    create_unimplemented_device("ast2700.io", 0x0, 0x4000000);
+}
+
+static void aspeed_soc_ast2700_class_init(ObjectClass *oc, void *data)
+{
+    static const char * const valid_cpu_types[] = {
+        ARM_CPU_TYPE_NAME("cortex-a35"),
+        NULL
+    };
+    DeviceClass *dc = DEVICE_CLASS(oc);
+    AspeedSoCClass *sc = ASPEED_SOC_CLASS(oc);
+
+    dc->realize      = aspeed_soc_ast2700_realize;
+
+    sc->name         = "ast2700-a0";
+    sc->valid_cpu_types = valid_cpu_types;
+    sc->silicon_rev  = AST2700_A0_SILICON_REV;
+    sc->sram_size    = 0x20000;
+    sc->spis_num     = 3;
+    sc->wdts_num     = 8;
+    sc->macs_num     = 1;
+    sc->uarts_num    = 13;
+    sc->num_cpus     = 4;
+    sc->uarts_base   = ASPEED_DEV_UART0;
+    sc->irqmap       = aspeed_soc_ast2700_irqmap;
+    sc->memmap       = aspeed_soc_ast2700_memmap;
+    sc->get_irq      = aspeed_soc_ast2700_get_irq;
+}
+
+static const TypeInfo aspeed_soc_ast27x0_types[] = {
+    {
+        .name           = TYPE_ASPEED27X0_SOC,
+        .parent         = TYPE_ASPEED_SOC,
+        .instance_size  = sizeof(Aspeed27x0SoCState),
+        .abstract       = true,
+    }, {
+        .name           = "ast2700-a0",
+        .parent         = TYPE_ASPEED27X0_SOC,
+        .instance_init  = aspeed_soc_ast2700_init,
+        .class_init     = aspeed_soc_ast2700_class_init,
+    },
+};
+
+DEFINE_TYPES(aspeed_soc_ast27x0_types)
diff --git a/hw/arm/meson.build b/hw/arm/meson.build
index 6808135c1f..1e3295a423 100644
--- a/hw/arm/meson.build
+++ b/hw/arm/meson.build
@@ -46,6 +46,7 @@  arm_ss.add(when: 'CONFIG_ASPEED_SOC', if_true: files(
   'aspeed_soc_common.c',
   'aspeed_ast2400.c',
   'aspeed_ast2600.c',
+  'aspeed_ast27x0.c',
   'aspeed_ast10x0.c',
   'aspeed_eeprom.c',
   'fby35.c'))
diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h
index c60fac900a..9f177b6037 100644
--- a/include/hw/arm/aspeed_soc.h
+++ b/include/hw/arm/aspeed_soc.h
@@ -15,6 +15,7 @@ 
 #include "hw/cpu/a15mpcore.h"
 #include "hw/arm/armv7m.h"
 #include "hw/intc/aspeed_vic.h"
+#include "hw/intc/aspeed_intc.h"
 #include "hw/misc/aspeed_scu.h"
 #include "hw/adc/aspeed_adc.h"
 #include "hw/misc/aspeed_sdmc.h"
@@ -26,6 +27,7 @@ 
 #include "hw/ssi/aspeed_smc.h"
 #include "hw/misc/aspeed_hace.h"
 #include "hw/misc/aspeed_sbc.h"
+#include "hw/misc/aspeed_sli.h"
 #include "hw/watchdog/wdt_aspeed.h"
 #include "hw/net/ftgmac100.h"
 #include "target/arm/cpu.h"
@@ -41,8 +43,8 @@ 
 
 #define ASPEED_SPIS_NUM  2
 #define ASPEED_EHCIS_NUM 2
-#define ASPEED_WDTS_NUM  4
-#define ASPEED_CPUS_NUM  2
+#define ASPEED_WDTS_NUM  8
+#define ASPEED_CPUS_NUM  4
 #define ASPEED_MACS_NUM  4
 #define ASPEED_UARTS_NUM 13
 #define ASPEED_JTAG_NUM  2
@@ -61,6 +63,7 @@  struct AspeedSoCState {
     AspeedI2CState i2c;
     AspeedI3CState i3c;
     AspeedSCUState scu;
+    AspeedSCUState scuio;
     AspeedHACEState hace;
     AspeedXDMAState xdma;
     AspeedADCState adc;
@@ -68,6 +71,8 @@  struct AspeedSoCState {
     AspeedSMCState spi[ASPEED_SPIS_NUM];
     EHCISysBusState ehci[ASPEED_EHCIS_NUM];
     AspeedSBCState sbc;
+    AspeedSLIState sli;
+    AspeedSLIState sliio;
     MemoryRegion secsram;
     UnimplementedDeviceState sbc_unimplemented;
     AspeedSDMCState sdmc;
@@ -117,6 +122,16 @@  struct Aspeed2600SoCState {
 #define TYPE_ASPEED2600_SOC "aspeed2600-soc"
 OBJECT_DECLARE_SIMPLE_TYPE(Aspeed2600SoCState, ASPEED2600_SOC)
 
+struct Aspeed27x0SoCState {
+    AspeedSoCState parent;
+
+    ARMCPU cpu[ASPEED_CPUS_NUM];
+    AspeedINTCState intc;
+};
+
+#define TYPE_ASPEED27X0_SOC "aspeed27x0-soc"
+OBJECT_DECLARE_SIMPLE_TYPE(Aspeed27x0SoCState, ASPEED27X0_SOC)
+
 struct Aspeed10x0SoCState {
     AspeedSoCState parent;
 
@@ -168,11 +183,13 @@  enum {
     ASPEED_DEV_UART13,
     ASPEED_DEV_VUART,
     ASPEED_DEV_FMC,
+    ASPEED_DEV_SPI0,
     ASPEED_DEV_SPI1,
     ASPEED_DEV_SPI2,
     ASPEED_DEV_EHCI1,
     ASPEED_DEV_EHCI2,
     ASPEED_DEV_VIC,
+    ASPEED_DEV_INTC,
     ASPEED_DEV_SDMC,
     ASPEED_DEV_SCU,
     ASPEED_DEV_ADC,
@@ -222,6 +239,11 @@  enum {
     ASPEED_DEV_JTAG1,
     ASPEED_DEV_FSI1,
     ASPEED_DEV_FSI2,
+    ASPEED_DEV_SCUIO,
+    ASPEED_DEV_SLI,
+    ASPEED_DEV_SLIIO,
+    ASPEED_GIC_DIST,
+    ASPEED_GIC_REDIST,
 };
 
 qemu_irq aspeed_soc_get_irq(AspeedSoCState *s, int dev);