Hi Cedric, Andrew > -----Original Message----- > From: Jamin Lin <[email protected]> > Sent: Thursday, February 6, 2025 5:53 PM > To: Cédric Le Goater <[email protected]>; Peter Maydell > <[email protected]>; Steven Lee <[email protected]>; Troy > Lee <[email protected]>; Andrew Jeffery <[email protected]>; > Joel Stanley <[email protected]>; open list:ASPEED BMCs > <[email protected]>; open list:All patches CC here > <[email protected]> > Cc: Jamin Lin <[email protected]>; Troy Lee > <[email protected]>; Yunlin Tang <[email protected]> > Subject: [PATCH v2 0/6] INTC model cleanup > > v2: > To streamline the review process, split the following patch series into > three parts. > > https://patchwork.kernel.org/project/qemu-devel/cover/20250121070424.246 > [email protected]/ > This patch series focuses on cleaning up the INTC model to > facilitate future support for the INTC_IO model. > > Jamin Lin (6): > hw/intc/aspeed: Support setting different memory and register size > hw/intc/aspeed: Introduce helper functions for enable and status > registers > hw/intc/aspeed: Add object type name to trace events for better > debugging > hw/arm/aspeed: Rename IRQ table and machine name for AST2700 A0 > hw/arm/aspeed_ast27x0: Sort the IRQ table by IRQ number > hw/intc/aspeed: Support different memory region ops > > hw/arm/aspeed.c | 8 +- > hw/arm/aspeed_ast27x0.c | 58 ++++----- > hw/intc/aspeed_intc.c | 227 ++++++++++++++++++++-------------- > hw/intc/trace-events | 24 ++-- > include/hw/intc/aspeed_intc.h | 5 + > 5 files changed, 183 insertions(+), 139 deletions(-) >
Please ignore the v2 patch. I will resend the v3 patch, as I am retaining the INTC naming and introducing a new INTC_IO model to support the AST2700 A1. So, I think I don't need to split 3 part patches to support AST2700 A1. Sorry for your inconvenience. Do you have any suggestion or concern? Jamin > -- > 2.34.1
