Hi Cédric,

> Subject: Re: [PATCH v1 00/10] Support ASPEED AST2700 A2
> 
> Jamin,
> 
> On 2/6/26 06:33, Jamin Lin wrote:
> > v1
> >    1. Add AST2700 A2 SoC support
> >    2. Add AST2700 A2 EVB machine
> >    3. Alias ast2700-evb to ast2700a2-evb
> >    4. Switch AST2700 FC machine to use the A2 SoC
> >    5. Update functional tests for both AST2700 A1 and A2
> >    6. Fix I2C Fix Out-of-Bounds access issue
> >
> > Jamin Lin (10):
> >    hw/i2c/aspeed_i2c: Fix Out-of-Bounds access by using dynamic register
> >      array
> >    hw/i2c/aspeed_i2c: Increase I2C device register size to 0xA0
> >    hw/misc/aspeed_scu: Remove unused SoC silicon revision definitions
> >    hw/misc/aspeed_scu: Add AST2700 A2 silicon revisions
> >    hw/arm/aspeed_ast27x0: Add AST2700 A2 SoC support
> >    hw/arm/aspeed_ast27x0_evb: Add AST2700 A2 EVB machine
> >    tests/qtest/ast2700-hace-test: Use ast2700-evb alias for AST2700 HACE
> >      tests
> >    tests/functional/aarch64/test_aspeed_ast2700: Add AST2700 A2 EVB
> >      functional tests
> >    hw/arm/aspeed_ast27x0-fc: Switch AST2700 FC machine to A2 SoC
> >    tests/functional/aarch64/test_aspeed_ast2700fc: Use AST2700 A2 SDK
> >      image for FC tests
> >
> >   include/hw/i2c/aspeed_i2c.h                   |  4 +-
> >   include/hw/misc/aspeed_scu.h                  | 11 +---
> >   hw/arm/aspeed_ast27x0-fc.c                    | 15 +++---
> >   hw/arm/aspeed_ast27x0.c                       | 36
> +++++++++++++
> >   hw/arm/aspeed_ast27x0_evb.c                   | 29 +++++++++-
> >   hw/i2c/aspeed_i2c.c                           | 54
> +++++++++++--------
> >   hw/misc/aspeed_scu.c                          | 11 +---
> >   tests/qtest/ast2700-hace-test.c               | 22 ++++----
> >   .../functional/aarch64/test_aspeed_ast2700.py | 44 +++++++++++++++
> >   .../aarch64/test_aspeed_ast2700fc.py          | 16 +++---
> >   10 files changed, 169 insertions(+), 73 deletions(-)
> >
> 
> 
> It would nice if you could provide a summary of the changes introduced in the
> A2 revision compared to the A1. That in the cover letter and in the commit log
> of the patch adding support. We're curious !
> 
Thanks for the review and the suggestion.

I have added a summary of the differences between AST2700 A1 and AST2700 A2 to the commit log of the patch that adds A2 support.
The same information has also been added to the cover letter.

For clarity, the differences are:
V2:
 1. AST2700 A2 is functionally identical to AST2700 A1. There are no changes to the IRQ layout, memory map, or peripheral configuration; the only difference is the silicon revision.
 2. The ast2700a2-evb machine is largely identical to ast2700a1-evb. The only difference is the default DRAM size, which is increased to 2 GB.
 3. The I2C out-of-bounds fix has been separated into a new patch series.

Thanks,
Jamin

> Thanks,
> 
> C.