[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 22/27] target/ppc: cpu_init: Rename register_ne_601_sprs
From: |
David Gibson |
Subject: |
Re: [PATCH 22/27] target/ppc: cpu_init: Rename register_ne_601_sprs |
Date: |
Wed, 16 Feb 2022 13:59:15 +1100 |
On Tue, Feb 15, 2022 at 06:41:43PM -0300, Fabiano Rosas wrote:
> The important part of this function is that it applies to non-embedded
> CPUs, not that it also applies to the 601. We removed support for the
> 601 anyway, so rename this function.
>
> Signed-off-by: Fabiano Rosas <farosas@linux.ibm.com>
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
Although, I wonder if "books_common" or something might be a better
name, though. Admittedly, I don't think the "BookS" terminology
existed at the time of most of these earlier CPUs. However, these
days the majority of 7xx chips are probably in embedded applications,
even if they weren't designed for an embedded chip line.
> ---
> target/ppc/cpu_init.c | 41 ++++++++++++++++++++---------------------
> 1 file changed, 20 insertions(+), 21 deletions(-)
>
> diff --git a/target/ppc/cpu_init.c b/target/ppc/cpu_init.c
> index 3e687d0301..621812d079 100644
> --- a/target/ppc/cpu_init.c
> +++ b/target/ppc/cpu_init.c
> @@ -241,8 +241,7 @@ static void register_generic_sprs(PowerPCCPU *cpu)
> 0x00000000);
> }
>
> -/* SPR common to all non-embedded PowerPC, including 601 */
> -static void register_ne_601_sprs(CPUPPCState *env)
> +static void register_non_embedded_sprs(CPUPPCState *env)
> {
> /* Exception processing */
> spr_register_kvm(env, SPR_DSISR, "DSISR",
> @@ -3022,7 +3021,7 @@ POWERPC_FAMILY(MPC8xx)(ObjectClass *oc, void *data)
>
> static void init_proc_G2(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_G2_sprs(env);
>
> @@ -3732,7 +3731,7 @@ POWERPC_FAMILY(e6500)(ObjectClass *oc, void *data)
> /* Non-embedded PowerPC
> */
> static void init_proc_603(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_603_sprs(env);
>
> @@ -3872,7 +3871,7 @@ POWERPC_FAMILY(e300)(ObjectClass *oc, void *data)
>
> static void init_proc_604(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_604_sprs(env);
>
> @@ -3971,7 +3970,7 @@ POWERPC_FAMILY(604E)(ObjectClass *oc, void *data)
>
> static void init_proc_740(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_7xx_sprs(env);
> /* Thermal management */
> @@ -4027,7 +4026,7 @@ POWERPC_FAMILY(740)(ObjectClass *oc, void *data)
>
> static void init_proc_750(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_7xx_sprs(env);
>
> @@ -4092,7 +4091,7 @@ POWERPC_FAMILY(750)(ObjectClass *oc, void *data)
>
> static void init_proc_750cl(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_7xx_sprs(env);
>
> @@ -4277,7 +4276,7 @@ POWERPC_FAMILY(750cl)(ObjectClass *oc, void *data)
>
> static void init_proc_750cx(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_7xx_sprs(env);
>
> @@ -4345,7 +4344,7 @@ POWERPC_FAMILY(750cx)(ObjectClass *oc, void *data)
>
> static void init_proc_750fx(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_7xx_sprs(env);
>
> @@ -4417,7 +4416,7 @@ POWERPC_FAMILY(750fx)(ObjectClass *oc, void *data)
>
> static void init_proc_750gx(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_7xx_sprs(env);
>
> @@ -4489,7 +4488,7 @@ POWERPC_FAMILY(750gx)(ObjectClass *oc, void *data)
>
> static void init_proc_745(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_7xx_sprs(env);
> register_755_sprs(env);
> @@ -4593,7 +4592,7 @@ POWERPC_FAMILY(755)(ObjectClass *oc, void *data)
>
> static void init_proc_7400(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_74xx_sprs(env);
> vscr_init(env, 0x00010000);
> @@ -4665,7 +4664,7 @@ POWERPC_FAMILY(7400)(ObjectClass *oc, void *data)
>
> static void init_proc_7410(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_74xx_sprs(env);
> vscr_init(env, 0x00010000);
> @@ -4744,7 +4743,7 @@ POWERPC_FAMILY(7410)(ObjectClass *oc, void *data)
>
> static void init_proc_7440(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_74xx_sprs(env);
> vscr_init(env, 0x00010000);
> @@ -4844,7 +4843,7 @@ POWERPC_FAMILY(7440)(ObjectClass *oc, void *data)
>
> static void init_proc_7450(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_74xx_sprs(env);
> vscr_init(env, 0x00010000);
> @@ -4966,7 +4965,7 @@ POWERPC_FAMILY(7450)(ObjectClass *oc, void *data)
>
> static void init_proc_7445(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_74xx_sprs(env);
> vscr_init(env, 0x00010000);
> @@ -5095,7 +5094,7 @@ POWERPC_FAMILY(7445)(ObjectClass *oc, void *data)
>
> static void init_proc_7455(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_74xx_sprs(env);
> vscr_init(env, 0x00010000);
> @@ -5226,7 +5225,7 @@ POWERPC_FAMILY(7455)(ObjectClass *oc, void *data)
>
> static void init_proc_7457(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_74xx_sprs(env);
> vscr_init(env, 0x00010000);
> @@ -5377,7 +5376,7 @@ POWERPC_FAMILY(7457)(ObjectClass *oc, void *data)
>
> static void init_proc_e600(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_sdr1_sprs(env);
> register_74xx_sprs(env);
> vscr_init(env, 0x00010000);
> @@ -6200,7 +6199,7 @@ static void init_tcg_pmu_power8(CPUPPCState *env)
>
> static void init_proc_book3s_common(CPUPPCState *env)
> {
> - register_ne_601_sprs(env);
> + register_non_embedded_sprs(env);
> register_usprg3_sprs(env);
> register_book3s_altivec_sprs(env);
> register_book3s_pmu_sup_sprs(env);
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- Re: [PATCH 15/27] target/ppc: cpu_init: Move 755 L2 cache SPRs into a function, (continued)
- [PATCH 21/27] target/ppc: cpu_init: Rename software TLB function, Fabiano Rosas, 2022/02/15
- [PATCH 25/27] target/ppc: cpu_init: Move SPR registration macros to a header, Fabiano Rosas, 2022/02/15
- [PATCH 23/27] target/ppc: cpu_init: Remove register_usprg3_sprs, Fabiano Rosas, 2022/02/15
- [PATCH 19/27] target/ppc: cpu_init: Reuse init_proc_604 for the 604e, Fabiano Rosas, 2022/02/15
- [PATCH 22/27] target/ppc: cpu_init: Rename register_ne_601_sprs, Fabiano Rosas, 2022/02/15
- Re: [PATCH 22/27] target/ppc: cpu_init: Rename register_ne_601_sprs,
David Gibson <=
- [PATCH 24/27] target/ppc: cpu_init: Expose some SPR registration helpers, Fabiano Rosas, 2022/02/15
- [PATCH 27/27] target/ppc: Move common SPR functions out of cpu_init, Fabiano Rosas, 2022/02/15
- [PATCH 20/27] target/ppc: cpu_init: Reuse init_proc_745 for the 755, Fabiano Rosas, 2022/02/15
- [PATCH 26/27] target/ppc: cpu_init: Move check_pow and QOM macros to a header, Fabiano Rosas, 2022/02/15