Hi Nabih, I will submit another patch for the LTPI controller soon, and I'll also fix this typo there. If there are no further concerns, could we leave this change as is?
Best Regards, Kane > -----Original Message----- > From: Jamin Lin <[email protected]> > Sent: Wednesday, November 5, 2025 9:11 AM > To: Nabih Estefan <[email protected]>; [email protected]; > Kane Chen <[email protected]> > Cc: [email protected]; [email protected]; Steven Lee > <[email protected]>; [email protected]; [email protected] > Subject: RE: [PATCH] hw/arm/ast27x0: Fix typo in LTPI address > > + Kane > > Hi Kane, > > Could you please help to review it? > Thanks-Jamin > > > Subject: [PATCH] hw/arm/ast27x0: Fix typo in LTPI address > > > > The address for LTPI has one more 0 that it should, bug introduced in > > commit 91064bea6b2d747a981cb3bd2904e56f443e6c67. > > > > Signed-off-by: Nabih Estefan <[email protected]> > > --- > > hw/arm/aspeed_ast27x0.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/hw/arm/aspeed_ast27x0.c b/hw/arm/aspeed_ast27x0.c index > > c484bcd4e2..1e6f469538 100644 > > --- a/hw/arm/aspeed_ast27x0.c > > +++ b/hw/arm/aspeed_ast27x0.c > > @@ -87,11 +87,11 @@ static const hwaddr > aspeed_soc_ast2700_memmap[] = > > { > > [ASPEED_DEV_UART11] = 0x14C33A00, > > [ASPEED_DEV_UART12] = 0x14C33B00, > > [ASPEED_DEV_WDT] = 0x14C37000, > > + [ASPEED_DEV_LTPI] = 0x30000000, > > [ASPEED_DEV_PCIE_MMIO0] = 0x60000000, > > [ASPEED_DEV_PCIE_MMIO1] = 0x80000000, > > [ASPEED_DEV_PCIE_MMIO2] = 0xA0000000, > > [ASPEED_DEV_SPI_BOOT] = 0x100000000, > > - [ASPEED_DEV_LTPI] = 0x300000000, > > [ASPEED_DEV_SDRAM] = 0x400000000, > > }; > > > > -- > > 2.51.2.1006.ga50a493c49-goog
