> -----Original Message----- > From: Alexander Graf [mailto:ag...@suse.de] > Sent: Monday, April 14, 2014 6:01 AM > To: Yoder Stuart-B08248 > Cc: qemu-...@nongnu.org; qemu-devel@nongnu.org > Subject: Re: [PATCH 1/2] QEMU: PPC: specify PVRs for all e500 cores > > > On 14.02.14 20:22, Stuart Yoder wrote: > > From: Stuart Yoder <stuart.yo...@freescale.com> > > > > Signed-off-by: Stuart Yoder <stuart.yo...@freescale.com> > > --- > > target-ppc/cpu-models.c | 64 > +++++++++++++++++++++++++++++++++++++++++++++-- > > target-ppc/cpu-models.h | 30 ++++++++++++++++++++-- > > 2 files changed, 90 insertions(+), 4 deletions(-) > > > > diff --git a/target-ppc/cpu-models.c b/target-ppc/cpu-models.c > > index f6c9b3a..9cf603b 100644 > > --- a/target-ppc/cpu-models.c > > +++ b/target-ppc/cpu-models.c > > @@ -677,19 +677,79 @@ > > "PowerPC e500 v2.0 core") > > POWERPC_DEF("e500v2_v10", CPU_POWERPC_e500v2_v10, > e500v2, > > "PowerPC e500v2 v1.0 core") > > + POWERPC_DEF("e500v2_v11", CPU_POWERPC_e500v2_v11, > e500v2, > > + "PowerPC e500v2 v1.1 core") > > POWERPC_DEF("e500v2_v20", CPU_POWERPC_e500v2_v20, > e500v2, > > "PowerPC e500v2 v2.0 core") > > POWERPC_DEF("e500v2_v21", CPU_POWERPC_e500v2_v21, > e500v2, > > "PowerPC e500v2 v2.1 core") > > POWERPC_DEF("e500v2_v22", CPU_POWERPC_e500v2_v22, > e500v2, > > "PowerPC e500v2 v2.2 core") > > + POWERPC_DEF("e500v2_v23", CPU_POWERPC_e500v2_v23, > e500v2, > > + "PowerPC e500v2 v2.3 core") > > POWERPC_DEF("e500v2_v30", CPU_POWERPC_e500v2_v30, > e500v2, > > "PowerPC e500v2 v3.0 core") > > + POWERPC_DEF("e500v2_v31", CPU_POWERPC_e500v2_v31, > e500v2, > > + "PowerPC e500v2 v3.1 core") > > + POWERPC_DEF("e500v2_v1040", CPU_POWERPC_e500v2_v1040, > e500v2, > > + "PowerPC e500v2 v104.0 core") > > + POWERPC_DEF("e500v2_v1050", CPU_POWERPC_e500v2_v1050, > e500v2, > > + "PowerPC e500v2 v105.0 core") > > + POWERPC_DEF("e500v2_v1051", CPU_POWERPC_e500v2_v1051, > e500v2, > > + "PowerPC e500v2 v105.1 core") > > + POWERPC_DEF("e500v2_v1151", CPU_POWERPC_e500v2_v1151, > e500v2, > > + "PowerPC e500v2 v115.1 core") > > + POWERPC_DEF("e500v2_v1152", CPU_POWERPC_e500v2_v1152, > e500v2, > > + "PowerPC e500v2 v115.2 core") > > + POWERPC_DEF("e500v2_v2050", CPU_POWERPC_e500v2_v2050, > e500v2, > > + "PowerPC e500v2 v205.0 core") > > + POWERPC_DEF("e500v2_v2051", CPU_POWERPC_e500v2_v2051, > e500v2, > > + "PowerPC e500v2 v205.1 core") > > + POWERPC_DEF("e500v2_v2151", CPU_POWERPC_e500v2_v2151, > e500v2, > > + "PowerPC e500v2 v215.1 core") > > + POWERPC_DEF("e500v2_v2152", CPU_POWERPC_e500v2_v2152, > e500v2, > > + "PowerPC e500v2 v215.2 core") > > + POWERPC_DEF("e500v2_v2251", CPU_POWERPC_e500v2_v2251, > e500v2, > > + "PowerPC e500v2 v225.1 core") > > + > > + /* e500mc family */ > > POWERPC_DEF_SVR("e500mc", "e500mc", > > - CPU_POWERPC_e500mc, POWERPC_SVR_E500, > e500mc) > > + CPU_POWERPC_e500mc_v20, POWERPC_SVR_E500, > e500mc) > > How about we use aliases instead like the other CPU types? :) > > > + POWERPC_DEF_SVR("e500mc_v10", "PowerPC e500mc v1.0 core", > > + CPU_POWERPC_e500mc_v10, POWERPC_SVR_E500, > e500mc) > > + POWERPC_DEF_SVR("e500mc_v21", "PowerPC e500mc v2.1 core", > > + CPU_POWERPC_e500mc_v21, POWERPC_SVR_E500, > e500mc) > > + POWERPC_DEF_SVR("e500mc_v22", "PowerPC e500mc v2.2 core", > > + CPU_POWERPC_e500mc_v22, POWERPC_SVR_E500, > e500mc) > > + POWERPC_DEF_SVR("e500mc_v23", "PowerPC e500mc v2.3 core", > > + CPU_POWERPC_e500mc_v23, POWERPC_SVR_E500, > e500mc) > > + POWERPC_DEF_SVR("e500mc_v1030", "PowerPC e500mc v103.0 core", > > + CPU_POWERPC_e500mc_v1030, POWERPC_SVR_E500, > e500mc) > > + POWERPC_DEF_SVR("e500mc_v30", "PowerPC e500mc v3.0 core", > > + CPU_POWERPC_e500mc_v30, POWERPC_SVR_E500, > e500mc) > > + POWERPC_DEF_SVR("e500mc_v31", "PowerPC e500mc v3.1 core", > > + CPU_POWERPC_e500mc_v31, POWERPC_SVR_E500, > e500mc) > > + POWERPC_DEF_SVR("e500mc_v32", "PowerPC e500mc v3.2 core", > > + CPU_POWERPC_e500mc_v32, POWERPC_SVR_E500, > e500mc) > > + > > #ifdef TARGET_PPC64 > > + /* e5500 family */ > > POWERPC_DEF_SVR("e5500", "e5500", > > - CPU_POWERPC_e5500, POWERPC_SVR_E500, > e5500) > > Same here
Can you explain you mean? what kind of alias? Thanks, Stuart
