Re: [PATCH 3/5] aspeed: Introduce AST27x0 SoC with Cortex-M4 support

[Cédric Le Goater]

Hello Steven,

[ ... ]

> > > +struct Aspeed27x0CM4SoCState {
> > > +    AspeedSoCState parent;
> > > +
> > > +    ARMv7MState armv7m;
> >
> > Why not use Aspeed10x0SoCState instead ?
>
> The AST2700's CM4 is not part of the AST10x0 series SoC.

ok. These are different HW designs.

> However, since the structure is identical, I can switch to 
> using Aspeed10x0SoCState if reusing it is deemed more appropriate.

That's fine. Let's keep it this way.

Thanks,

C.