Wire up the PCIe Root Complex in the AST2600 SoC model.
According to the AST2600 firmware driver, only the RC_H controller is
supported. RC_H uses PCIe PHY1 at 0x1e6ed200 and the PCIe config (H2X)
register block at 0x1e770000. The RC_H MMIO window is mapped at
0x70000000–0x80000000. RC_L is not modeled. The RC_H interrupt is
wired to IRQ 168. Only RC_H is realized and connected to the SoC
interrupt controller.
The SoC integration initializes PCIe PHY1, instantiates a single RC
instance, wires its MMIO regions, and connects its interrupt. An alias
region is added to map the RC MMIO space into the guest physical address
space.
This provides enough functionality for firmware and guest drivers to
discover and use the AST2600 RC_H Root Complex while leaving RC_L
unimplemented.
Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
---
hw/arm/aspeed_ast2600.c | 69 +++++++++++++++++++++++++++++++++++++++--
1 file changed, 66 insertions(+), 3 deletions(-)
diff --git a/hw/arm/aspeed_ast2600.c b/hw/arm/aspeed_ast2600.c
index d12707f0ab..d521a1b4f0 100644
--- a/hw/arm/aspeed_ast2600.c
+++ b/hw/arm/aspeed_ast2600.c
@@ -48,11 +48,13 @@ static const hwaddr aspeed_soc_ast2600_memmap[] = {
[ASPEED_DEV_XDMA] = 0x1E6E7000,
[ASPEED_DEV_ADC] = 0x1E6E9000,
[ASPEED_DEV_DP] = 0x1E6EB000,
+ [ASPEED_DEV_PCIE_PHY1] = 0x1E6ED200,
[ASPEED_DEV_SBC] = 0x1E6F2000,
[ASPEED_DEV_EMMC_BC] = 0x1E6f5000,
[ASPEED_DEV_VIDEO] = 0x1E700000,
[ASPEED_DEV_SDHCI] = 0x1E740000,
[ASPEED_DEV_EMMC] = 0x1E750000,
+ [ASPEED_DEV_PCIE0] = 0x1E770000,
[ASPEED_DEV_GPIO] = 0x1E780000,
[ASPEED_DEV_GPIO_1_8V] = 0x1E780800,
[ASPEED_DEV_RTC] = 0x1E781000,
@@ -79,6 +81,7 @@ static const hwaddr aspeed_soc_ast2600_memmap[] = {
[ASPEED_DEV_FSI1] = 0x1E79B000,
[ASPEED_DEV_FSI2] = 0x1E79B100,
[ASPEED_DEV_I3C] = 0x1E7A0000,
+ [ASPEED_DEV_PCIE_MMIO1] = 0x70000000,
[ASPEED_DEV_SDRAM] = 0x80000000,
};
@@ -127,6 +130,7 @@ static const int aspeed_soc_ast2600_irqmap[] = {
[ASPEED_DEV_LPC] = 35,
[ASPEED_DEV_IBT] = 143,
[ASPEED_DEV_I2C] = 110, /* 110 -> 125 */
+ [ASPEED_DEV_PCIE0] = 168,
[ASPEED_DEV_PECI] = 38,
[ASPEED_DEV_ETH1] = 2,
[ASPEED_DEV_ETH2] = 3,
@@ -191,6 +195,10 @@ static void aspeed_soc_ast2600_init(Object *obj)
snprintf(typename, sizeof(typename), "aspeed.i2c-%s", socname);
object_initialize_child(obj, "i2c", &s->i2c, typename);
+ object_initialize_child(obj, "pcie-cfg", &s->pcie[0], TYPE_ASPEED_PCIE_CFG);
+ object_initialize_child(obj, "pcie-phy[*]", &s->pcie_phy[0],
+ TYPE_ASPEED_PCIE_PHY);
+
object_initialize_child(obj, "peci", &s->peci, TYPE_ASPEED_PECI);
snprintf(typename, sizeof(typename), "aspeed.fmc-%s", socname);
@@ -292,7 +300,9 @@ static void aspeed_soc_ast2600_realize(DeviceState *dev, Error **errp)
AspeedSoCState *s = ASPEED_SOC(dev);
AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
qemu_irq irq;
- g_autofree char *sram_name = NULL;
+ g_autofree char *name = NULL;
+ MemoryRegion *mmio_alias;
+ MemoryRegion *mmio_mr;
/* Default boot region (SPI memory or ROMs) */
memory_region_init(&s->spi_boot_container, OBJECT(s),
@@ -362,8 +372,8 @@ static void aspeed_soc_ast2600_realize(DeviceState *dev, Error **errp)
}
/* 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,
+ name = g_strdup_printf("aspeed.sram.%d", CPU(&a->cpu[0])->cpu_index);
+ if (!memory_region_init_ram(&s->sram, OBJECT(s), name, sc->sram_size,
errp)) {
return;
}
@@ -438,6 +448,59 @@ static void aspeed_soc_ast2600_realize(DeviceState *dev, Error **errp)
sysbus_connect_irq(SYS_BUS_DEVICE(&s->peci), 0,
aspeed_soc_get_irq(s, ASPEED_DEV_PECI));
+ /*
+ * PCIe Root Complex (RC)
+ *
+ * H2X register space (single block 0x00-0xFF):
+ * 0x00-0x7F : shared by RC_L (PCIe0) and RC_H (PCIe1)
+ * 0x80-0xBF : RC_L only
+ * 0xC0-0xFF : RC_H only
+ *
+ * Model scope / limitations:
+ * - Firmware supports RC_H only; this QEMU model does not support RC_L.
+ * - RC_H uses PHY1 and the MMIO window [0x70000000, 0x80000000]
+ * (aka MMIO1).
+ *
+ * Indexing convention (this model):
+ * - Expose a single logical instance at index 0.
+ * - pcie[0] -> hardware RC_H (PCIe1)
+ * - phy[0] -> hardware PHY1
+ * - mmio.0 -> guest address range MMIO1: 0x70000000-0x80000000
+ * - RC_L / PCIe0 is not created and mapped.
+ */
+ if (!sysbus_realize(SYS_BUS_DEVICE(&s->pcie_phy[0]), errp)) {
+ return;
+ }
+ aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->pcie_phy[0]), 0,
+ sc->memmap[ASPEED_DEV_PCIE_PHY1]);
+
+ object_property_set_int(OBJECT(&s->pcie[0]), "dram-base",
+ sc->memmap[ASPEED_DEV_SDRAM],
+ &error_abort);
+ object_property_set_link(OBJECT(&s->pcie[0]), "dram", OBJECT(s->dram_mr),
+ &error_abort);
+ if (!sysbus_realize(SYS_BUS_DEVICE(&s->pcie[0]), errp)) {
+ return;
+ }
+ aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->pcie[0]), 0,
+ sc->memmap[ASPEED_DEV_PCIE0]);
+
+ irq = qdev_get_gpio_in(DEVICE(&a->a7mpcore),
+ sc->irqmap[ASPEED_DEV_PCIE0]);
+ sysbus_connect_irq(SYS_BUS_DEVICE(&s->pcie[0].rc), 0, irq);
+
+ name = g_strdup_printf("aspeed.pcie-mmio.0");
+
+ mmio_alias = g_new0(MemoryRegion, 1);
+ mmio_mr = sysbus_mmio_get_region(SYS_BUS_DEVICE(&s->pcie[0].rc), 1);
+
+ memory_region_init_alias(mmio_alias, OBJECT(&s->pcie[0].rc), name,
+ mmio_mr, sc->memmap[ASPEED_DEV_PCIE_MMIO1],
+ 0x10000000);
+ memory_region_add_subregion(s->memory,
+ sc->memmap[ASPEED_DEV_PCIE_MMIO1],
+ mmio_alias);
+
/* 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);
--
2.43.0
On 9/11/25 09:24, Jamin Lin wrote:
> Wire up the PCIe Root Complex in the AST2600 SoC model.
>
> According to the AST2600 firmware driver, only the RC_H controller is
> supported. RC_H uses PCIe PHY1 at 0x1e6ed200 and the PCIe config (H2X)
> register block at 0x1e770000. The RC_H MMIO window is mapped at
> 0x70000000–0x80000000. RC_L is not modeled. The RC_H interrupt is
> wired to IRQ 168. Only RC_H is realized and connected to the SoC
> interrupt controller.
>
> The SoC integration initializes PCIe PHY1, instantiates a single RC
> instance, wires its MMIO regions, and connects its interrupt. An alias
> region is added to map the RC MMIO space into the guest physical address
> space.
>
> This provides enough functionality for firmware and guest drivers to
> discover and use the AST2600 RC_H Root Complex while leaving RC_L
> unimplemented.
>
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> ---
> hw/arm/aspeed_ast2600.c | 69 +++++++++++++++++++++++++++++++++++++++--
> 1 file changed, 66 insertions(+), 3 deletions(-)
>
> diff --git a/hw/arm/aspeed_ast2600.c b/hw/arm/aspeed_ast2600.c
> index d12707f0ab..d521a1b4f0 100644
> --- a/hw/arm/aspeed_ast2600.c
> +++ b/hw/arm/aspeed_ast2600.c
> @@ -48,11 +48,13 @@ static const hwaddr aspeed_soc_ast2600_memmap[] = {
> [ASPEED_DEV_XDMA] = 0x1E6E7000,
> [ASPEED_DEV_ADC] = 0x1E6E9000,
> [ASPEED_DEV_DP] = 0x1E6EB000,
> + [ASPEED_DEV_PCIE_PHY1] = 0x1E6ED200,
> [ASPEED_DEV_SBC] = 0x1E6F2000,
> [ASPEED_DEV_EMMC_BC] = 0x1E6f5000,
> [ASPEED_DEV_VIDEO] = 0x1E700000,
> [ASPEED_DEV_SDHCI] = 0x1E740000,
> [ASPEED_DEV_EMMC] = 0x1E750000,
> + [ASPEED_DEV_PCIE0] = 0x1E770000,
> [ASPEED_DEV_GPIO] = 0x1E780000,
> [ASPEED_DEV_GPIO_1_8V] = 0x1E780800,
> [ASPEED_DEV_RTC] = 0x1E781000,
> @@ -79,6 +81,7 @@ static const hwaddr aspeed_soc_ast2600_memmap[] = {
> [ASPEED_DEV_FSI1] = 0x1E79B000,
> [ASPEED_DEV_FSI2] = 0x1E79B100,
> [ASPEED_DEV_I3C] = 0x1E7A0000,
> + [ASPEED_DEV_PCIE_MMIO1] = 0x70000000,
> [ASPEED_DEV_SDRAM] = 0x80000000,
> };
>
> @@ -127,6 +130,7 @@ static const int aspeed_soc_ast2600_irqmap[] = {
> [ASPEED_DEV_LPC] = 35,
> [ASPEED_DEV_IBT] = 143,
> [ASPEED_DEV_I2C] = 110, /* 110 -> 125 */
> + [ASPEED_DEV_PCIE0] = 168,
> [ASPEED_DEV_PECI] = 38,
> [ASPEED_DEV_ETH1] = 2,
> [ASPEED_DEV_ETH2] = 3,
> @@ -191,6 +195,10 @@ static void aspeed_soc_ast2600_init(Object *obj)
> snprintf(typename, sizeof(typename), "aspeed.i2c-%s", socname);
> object_initialize_child(obj, "i2c", &s->i2c, typename);
>
> + object_initialize_child(obj, "pcie-cfg", &s->pcie[0], TYPE_ASPEED_PCIE_CFG);
> + object_initialize_child(obj, "pcie-phy[*]", &s->pcie_phy[0],
> + TYPE_ASPEED_PCIE_PHY);
> +
> object_initialize_child(obj, "peci", &s->peci, TYPE_ASPEED_PECI);
>
> snprintf(typename, sizeof(typename), "aspeed.fmc-%s", socname);
> @@ -292,7 +300,9 @@ static void aspeed_soc_ast2600_realize(DeviceState *dev, Error **errp)
> AspeedSoCState *s = ASPEED_SOC(dev);
> AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> qemu_irq irq;
> - g_autofree char *sram_name = NULL;
> + g_autofree char *name = NULL;
> + MemoryRegion *mmio_alias;
> + MemoryRegion *mmio_mr;
>
> /* Default boot region (SPI memory or ROMs) */
> memory_region_init(&s->spi_boot_container, OBJECT(s),
> @@ -362,8 +372,8 @@ static void aspeed_soc_ast2600_realize(DeviceState *dev, Error **errp)
> }
>
> /* 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,
> + name = g_strdup_printf("aspeed.sram.%d", CPU(&a->cpu[0])->cpu_index);
> + if (!memory_region_init_ram(&s->sram, OBJECT(s), name, sc->sram_size,
> errp)) {
> return;
> }
> @@ -438,6 +448,59 @@ static void aspeed_soc_ast2600_realize(DeviceState *dev, Error **errp)
> sysbus_connect_irq(SYS_BUS_DEVICE(&s->peci), 0,
> aspeed_soc_get_irq(s, ASPEED_DEV_PECI));
>
> + /*
> + * PCIe Root Complex (RC)
> + *
> + * H2X register space (single block 0x00-0xFF):
> + * 0x00-0x7F : shared by RC_L (PCIe0) and RC_H (PCIe1)
> + * 0x80-0xBF : RC_L only
> + * 0xC0-0xFF : RC_H only
> + *
> + * Model scope / limitations:
> + * - Firmware supports RC_H only; this QEMU model does not support RC_L.
> + * - RC_H uses PHY1 and the MMIO window [0x70000000, 0x80000000]
> + * (aka MMIO1).
> + *
> + * Indexing convention (this model):
> + * - Expose a single logical instance at index 0.
> + * - pcie[0] -> hardware RC_H (PCIe1)
> + * - phy[0] -> hardware PHY1
> + * - mmio.0 -> guest address range MMIO1: 0x70000000-0x80000000
> + * - RC_L / PCIe0 is not created and mapped.
> + */
> + if (!sysbus_realize(SYS_BUS_DEVICE(&s->pcie_phy[0]), errp)) {
> + return;
> + }
> + aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->pcie_phy[0]), 0,
> + sc->memmap[ASPEED_DEV_PCIE_PHY1]);
> +
> + object_property_set_int(OBJECT(&s->pcie[0]), "dram-base",
> + sc->memmap[ASPEED_DEV_SDRAM],
> + &error_abort);
> + object_property_set_link(OBJECT(&s->pcie[0]), "dram", OBJECT(s->dram_mr),
> + &error_abort);
> + if (!sysbus_realize(SYS_BUS_DEVICE(&s->pcie[0]), errp)) {
> + return;
> + }
> + aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->pcie[0]), 0,
> + sc->memmap[ASPEED_DEV_PCIE0]);
> +
> + irq = qdev_get_gpio_in(DEVICE(&a->a7mpcore),
> + sc->irqmap[ASPEED_DEV_PCIE0]);
> + sysbus_connect_irq(SYS_BUS_DEVICE(&s->pcie[0].rc), 0, irq);
> +
> + name = g_strdup_printf("aspeed.pcie-mmio.0");
> +
> + mmio_alias = g_new0(MemoryRegion, 1);
> + mmio_mr = sysbus_mmio_get_region(SYS_BUS_DEVICE(&s->pcie[0].rc), 1);
> +
> + memory_region_init_alias(mmio_alias, OBJECT(&s->pcie[0].rc), name,
> + mmio_mr, sc->memmap[ASPEED_DEV_PCIE_MMIO1],
> + 0x10000000);
> + memory_region_add_subregion(s->memory,
> + sc->memmap[ASPEED_DEV_PCIE_MMIO1],
> + mmio_alias);
> +
Could you please introduce a sub routine to realize the PCIe RC model ?
name it aspeed_soc_ast2600_pcie_realize() maybe.
Thanks,
C.
> /* 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);
Hi Cédric
> Subject: Re: [SPAM] [PATCH v2 08/14] hw/arm/aspeed_ast2600: Add PCIe RC
> support (RC_H only)
>
> On 9/11/25 09:24, Jamin Lin wrote:
> > Wire up the PCIe Root Complex in the AST2600 SoC model.
> >
> > According to the AST2600 firmware driver, only the RC_H controller is
> > supported. RC_H uses PCIe PHY1 at 0x1e6ed200 and the PCIe config (H2X)
> > register block at 0x1e770000. The RC_H MMIO window is mapped at
> > 0x70000000–0x80000000. RC_L is not modeled. The RC_H interrupt is
> > wired to IRQ 168. Only RC_H is realized and connected to the SoC
> > interrupt controller.
> >
> > The SoC integration initializes PCIe PHY1, instantiates a single RC
> > instance, wires its MMIO regions, and connects its interrupt. An alias
> > region is added to map the RC MMIO space into the guest physical
> > address space.
> >
> > This provides enough functionality for firmware and guest drivers to
> > discover and use the AST2600 RC_H Root Complex while leaving RC_L
> > unimplemented.
> >
> > Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> > ---
> > hw/arm/aspeed_ast2600.c | 69
> +++++++++++++++++++++++++++++++++++++++--
> > 1 file changed, 66 insertions(+), 3 deletions(-)
> >
> > diff --git a/hw/arm/aspeed_ast2600.c b/hw/arm/aspeed_ast2600.c index
> > d12707f0ab..d521a1b4f0 100644
> > --- a/hw/arm/aspeed_ast2600.c
> > +++ b/hw/arm/aspeed_ast2600.c
> > @@ -48,11 +48,13 @@ static const hwaddr aspeed_soc_ast2600_memmap[]
> = {
> > [ASPEED_DEV_XDMA] = 0x1E6E7000,
> > [ASPEED_DEV_ADC] = 0x1E6E9000,
> > [ASPEED_DEV_DP] = 0x1E6EB000,
> > + [ASPEED_DEV_PCIE_PHY1] = 0x1E6ED200,
> > [ASPEED_DEV_SBC] = 0x1E6F2000,
> > [ASPEED_DEV_EMMC_BC] = 0x1E6f5000,
> > [ASPEED_DEV_VIDEO] = 0x1E700000,
> > [ASPEED_DEV_SDHCI] = 0x1E740000,
> > [ASPEED_DEV_EMMC] = 0x1E750000,
> > + [ASPEED_DEV_PCIE0] = 0x1E770000,
> > [ASPEED_DEV_GPIO] = 0x1E780000,
> > [ASPEED_DEV_GPIO_1_8V] = 0x1E780800,
> > [ASPEED_DEV_RTC] = 0x1E781000,
> > @@ -79,6 +81,7 @@ static const hwaddr aspeed_soc_ast2600_memmap[] =
> {
> > [ASPEED_DEV_FSI1] = 0x1E79B000,
> > [ASPEED_DEV_FSI2] = 0x1E79B100,
> > [ASPEED_DEV_I3C] = 0x1E7A0000,
> > + [ASPEED_DEV_PCIE_MMIO1] = 0x70000000,
> > [ASPEED_DEV_SDRAM] = 0x80000000,
> > };
> >
> > @@ -127,6 +130,7 @@ static const int aspeed_soc_ast2600_irqmap[] = {
> > [ASPEED_DEV_LPC] = 35,
> > [ASPEED_DEV_IBT] = 143,
> > [ASPEED_DEV_I2C] = 110, /* 110 -> 125 */
> > + [ASPEED_DEV_PCIE0] = 168,
> > [ASPEED_DEV_PECI] = 38,
> > [ASPEED_DEV_ETH1] = 2,
> > [ASPEED_DEV_ETH2] = 3,
> > @@ -191,6 +195,10 @@ static void aspeed_soc_ast2600_init(Object *obj)
> > snprintf(typename, sizeof(typename), "aspeed.i2c-%s", socname);
> > object_initialize_child(obj, "i2c", &s->i2c, typename);
> >
> > + object_initialize_child(obj, "pcie-cfg", &s->pcie[0],
> TYPE_ASPEED_PCIE_CFG);
> > + object_initialize_child(obj, "pcie-phy[*]", &s->pcie_phy[0],
> > + TYPE_ASPEED_PCIE_PHY);
> > +
> > object_initialize_child(obj, "peci", &s->peci,
> > TYPE_ASPEED_PECI);
> >
> > snprintf(typename, sizeof(typename), "aspeed.fmc-%s", socname);
> > @@ -292,7 +300,9 @@ static void aspeed_soc_ast2600_realize(DeviceState
> *dev, Error **errp)
> > AspeedSoCState *s = ASPEED_SOC(dev);
> > AspeedSoCClass *sc = ASPEED_SOC_GET_CLASS(s);
> > qemu_irq irq;
> > - g_autofree char *sram_name = NULL;
> > + g_autofree char *name = NULL;
> > + MemoryRegion *mmio_alias;
> > + MemoryRegion *mmio_mr;
> >
> > /* Default boot region (SPI memory or ROMs) */
> > memory_region_init(&s->spi_boot_container, OBJECT(s), @@ -362,8
> > +372,8 @@ static void aspeed_soc_ast2600_realize(DeviceState *dev, Error
> **errp)
> > }
> >
> > /* 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,
> > + name = g_strdup_printf("aspeed.sram.%d",
> CPU(&a->cpu[0])->cpu_index);
> > + if (!memory_region_init_ram(&s->sram, OBJECT(s), name,
> > + sc->sram_size,
> > errp)) {
> > return;
> > }
> > @@ -438,6 +448,59 @@ static void
> aspeed_soc_ast2600_realize(DeviceState *dev, Error **errp)
> > sysbus_connect_irq(SYS_BUS_DEVICE(&s->peci), 0,
> > aspeed_soc_get_irq(s, ASPEED_DEV_PECI));
> >
> > + /*
> > + * PCIe Root Complex (RC)
> > + *
> > + * H2X register space (single block 0x00-0xFF):
> > + * 0x00-0x7F : shared by RC_L (PCIe0) and RC_H (PCIe1)
> > + * 0x80-0xBF : RC_L only
> > + * 0xC0-0xFF : RC_H only
> > + *
> > + * Model scope / limitations:
> > + * - Firmware supports RC_H only; this QEMU model does not
> support RC_L.
> > + * - RC_H uses PHY1 and the MMIO window [0x70000000,
> 0x80000000]
> > + * (aka MMIO1).
> > + *
> > + * Indexing convention (this model):
> > + * - Expose a single logical instance at index 0.
> > + * - pcie[0] -> hardware RC_H (PCIe1)
> > + * - phy[0] -> hardware PHY1
> > + * - mmio.0 -> guest address range MMIO1:
> 0x70000000-0x80000000
> > + * - RC_L / PCIe0 is not created and mapped.
> > + */
> > + if (!sysbus_realize(SYS_BUS_DEVICE(&s->pcie_phy[0]), errp)) {
> > + return;
> > + }
> > + aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->pcie_phy[0]), 0,
> > + sc->memmap[ASPEED_DEV_PCIE_PHY1]);
> > +
> > + object_property_set_int(OBJECT(&s->pcie[0]), "dram-base",
> > + sc->memmap[ASPEED_DEV_SDRAM],
> > + &error_abort);
> > + object_property_set_link(OBJECT(&s->pcie[0]), "dram",
> OBJECT(s->dram_mr),
> > + &error_abort);
> > + if (!sysbus_realize(SYS_BUS_DEVICE(&s->pcie[0]), errp)) {
> > + return;
> > + }
> > + aspeed_mmio_map(s, SYS_BUS_DEVICE(&s->pcie[0]), 0,
> > + sc->memmap[ASPEED_DEV_PCIE0]);
> > +
> > + irq = qdev_get_gpio_in(DEVICE(&a->a7mpcore),
> > + sc->irqmap[ASPEED_DEV_PCIE0]);
> > + sysbus_connect_irq(SYS_BUS_DEVICE(&s->pcie[0].rc), 0, irq);
> > +
> > + name = g_strdup_printf("aspeed.pcie-mmio.0");
> > +
> > + mmio_alias = g_new0(MemoryRegion, 1);
> > + mmio_mr =
> sysbus_mmio_get_region(SYS_BUS_DEVICE(&s->pcie[0].rc),
> > + 1);
> > +
> > + memory_region_init_alias(mmio_alias, OBJECT(&s->pcie[0].rc), name,
> > + mmio_mr,
> sc->memmap[ASPEED_DEV_PCIE_MMIO1],
> > + 0x10000000);
> > + memory_region_add_subregion(s->memory,
> > +
> sc->memmap[ASPEED_DEV_PCIE_MMIO1],
> > + mmio_alias);
> > +
>
> Could you please introduce a sub routine to realize the PCIe RC model ?
> name it aspeed_soc_ast2600_pcie_realize() maybe.
>
Thanks for your review and suggestion.
Will do.
Jamin
>
> Thanks,
>
> C.
>
>
>
>
> > /* 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);
© 2016 - 2025 Red Hat, Inc.