mirror of https://github.com/xemu-project/xemu.git
ppc/ppc405: Restore TCR and STR write handlers
The 405 timers were broken when booke support was added. Assumption
was made that the register numbers were the same but it's not :
SPR_BOOKE_TSR (0x150)
SPR_BOOKE_TCR (0x154)
SPR_40x_TSR (0x3D8)
SPR_40x_TCR (0x3DA)
Cc: Christophe Leroy <christophe.leroy@c-s.fr>
Fixes: ddd1055b07
("PPC: booke timers")
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
Message-Id: <20211222064025.1541490-5-clg@kaod.org>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
Message-Id: <20220103063441.3424853-6-clg@kaod.org>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
This commit is contained in:
parent
56964585a0
commit
cbd8f17d16
25
hw/ppc/ppc.c
25
hw/ppc/ppc.c
|
@ -1300,6 +1300,31 @@ target_ulong load_40x_pit (CPUPPCState *env)
|
|||
return cpu_ppc_load_decr(env);
|
||||
}
|
||||
|
||||
void store_40x_tsr(CPUPPCState *env, target_ulong val)
|
||||
{
|
||||
PowerPCCPU *cpu = env_archcpu(env);
|
||||
|
||||
trace_ppc40x_store_tcr(val);
|
||||
|
||||
env->spr[SPR_40x_TSR] &= ~(val & 0xFC000000);
|
||||
if (val & 0x80000000) {
|
||||
ppc_set_irq(cpu, PPC_INTERRUPT_PIT, 0);
|
||||
}
|
||||
}
|
||||
|
||||
void store_40x_tcr(CPUPPCState *env, target_ulong val)
|
||||
{
|
||||
PowerPCCPU *cpu = env_archcpu(env);
|
||||
ppc_tb_t *tb_env;
|
||||
|
||||
trace_ppc40x_store_tsr(val);
|
||||
|
||||
tb_env = env->tb_env;
|
||||
env->spr[SPR_40x_TCR] = val & 0xFFC00000;
|
||||
start_stop_pit(env, tb_env, 1);
|
||||
cpu_4xx_wdt_cb(cpu);
|
||||
}
|
||||
|
||||
static void ppc_40x_set_tb_clk (void *opaque, uint32_t freq)
|
||||
{
|
||||
CPUPPCState *env = opaque;
|
||||
|
|
|
@ -110,6 +110,8 @@ ppc4xx_pit_start(uint64_t reload) "PIT 0x%016" PRIx64
|
|||
ppc4xx_pit(uint32_t ar, uint32_t ir, uint64_t tcr, uint64_t tsr, uint64_t reload) "ar %d ir %d TCR 0x%" PRIx64 " TSR 0x%" PRIx64 " PIT 0x%016" PRIx64
|
||||
ppc4xx_wdt(uint64_t tcr, uint64_t tsr) "TCR 0x%" PRIx64 " TSR 0x%" PRIx64
|
||||
ppc40x_store_pit(uint64_t value) "val 0x%" PRIx64
|
||||
ppc40x_store_tcr(uint64_t value) "val 0x%" PRIx64
|
||||
ppc40x_store_tsr(uint64_t value) "val 0x%" PRIx64
|
||||
ppc40x_set_tb_clk(uint32_t value) "new frequency %" PRIu32
|
||||
ppc40x_timers_init(uint32_t value) "frequency %" PRIu32
|
||||
|
||||
|
|
|
@ -1399,6 +1399,8 @@ target_ulong load_40x_pit(CPUPPCState *env);
|
|||
void store_40x_pit(CPUPPCState *env, target_ulong val);
|
||||
void store_40x_dbcr0(CPUPPCState *env, uint32_t val);
|
||||
void store_40x_sler(CPUPPCState *env, uint32_t val);
|
||||
void store_40x_tcr(CPUPPCState *env, target_ulong val);
|
||||
void store_40x_tsr(CPUPPCState *env, target_ulong val);
|
||||
void store_booke_tcr(CPUPPCState *env, target_ulong val);
|
||||
void store_booke_tsr(CPUPPCState *env, target_ulong val);
|
||||
void ppc_tlb_invalidate_all(CPUPPCState *env);
|
||||
|
|
|
@ -1440,11 +1440,11 @@ static void register_40x_sprs(CPUPPCState *env)
|
|||
0x00000000);
|
||||
spr_register(env, SPR_40x_TCR, "TCR",
|
||||
SPR_NOACCESS, SPR_NOACCESS,
|
||||
&spr_read_generic, &spr_write_booke_tcr,
|
||||
&spr_read_generic, &spr_write_40x_tcr,
|
||||
0x00000000);
|
||||
spr_register(env, SPR_40x_TSR, "TSR",
|
||||
SPR_NOACCESS, SPR_NOACCESS,
|
||||
&spr_read_generic, &spr_write_booke_tsr,
|
||||
&spr_read_generic, &spr_write_40x_tsr,
|
||||
0x00000000);
|
||||
}
|
||||
|
||||
|
|
|
@ -706,6 +706,8 @@ DEF_HELPER_2(store_hid0_601, void, env, tl)
|
|||
DEF_HELPER_3(store_403_pbr, void, env, i32, tl)
|
||||
DEF_HELPER_FLAGS_1(load_40x_pit, TCG_CALL_NO_RWG, tl, env)
|
||||
DEF_HELPER_FLAGS_2(store_40x_pit, TCG_CALL_NO_RWG, void, env, tl)
|
||||
DEF_HELPER_FLAGS_2(store_40x_tcr, TCG_CALL_NO_RWG, void, env, tl)
|
||||
DEF_HELPER_FLAGS_2(store_40x_tsr, TCG_CALL_NO_RWG, void, env, tl)
|
||||
DEF_HELPER_2(store_40x_dbcr0, void, env, tl)
|
||||
DEF_HELPER_2(store_40x_sler, void, env, tl)
|
||||
DEF_HELPER_FLAGS_2(store_booke_tcr, TCG_CALL_NO_RWG, void, env, tl)
|
||||
|
|
|
@ -87,6 +87,8 @@ void spr_read_40x_pit(DisasContext *ctx, int gprn, int sprn);
|
|||
void spr_write_40x_pit(DisasContext *ctx, int sprn, int gprn);
|
||||
void spr_write_40x_dbcr0(DisasContext *ctx, int sprn, int gprn);
|
||||
void spr_write_40x_sler(DisasContext *ctx, int sprn, int gprn);
|
||||
void spr_write_40x_tcr(DisasContext *ctx, int sprn, int gprn);
|
||||
void spr_write_40x_tsr(DisasContext *ctx, int sprn, int gprn);
|
||||
void spr_write_booke_tcr(DisasContext *ctx, int sprn, int gprn);
|
||||
void spr_write_booke_tsr(DisasContext *ctx, int sprn, int gprn);
|
||||
void spr_read_403_pbr(DisasContext *ctx, int gprn, int sprn);
|
||||
|
|
|
@ -144,6 +144,16 @@ void helper_store_40x_pit(CPUPPCState *env, target_ulong val)
|
|||
store_40x_pit(env, val);
|
||||
}
|
||||
|
||||
void helper_store_40x_tcr(CPUPPCState *env, target_ulong val)
|
||||
{
|
||||
store_40x_tcr(env, val);
|
||||
}
|
||||
|
||||
void helper_store_40x_tsr(CPUPPCState *env, target_ulong val)
|
||||
{
|
||||
store_40x_tsr(env, val);
|
||||
}
|
||||
|
||||
void helper_store_booke_tcr(CPUPPCState *env, target_ulong val)
|
||||
{
|
||||
store_booke_tcr(env, val);
|
||||
|
|
|
@ -878,6 +878,18 @@ void spr_write_40x_sler(DisasContext *ctx, int sprn, int gprn)
|
|||
gen_helper_store_40x_sler(cpu_env, cpu_gpr[gprn]);
|
||||
}
|
||||
|
||||
void spr_write_40x_tcr(DisasContext *ctx, int sprn, int gprn)
|
||||
{
|
||||
gen_icount_io_start(ctx);
|
||||
gen_helper_store_40x_tcr(cpu_env, cpu_gpr[gprn]);
|
||||
}
|
||||
|
||||
void spr_write_40x_tsr(DisasContext *ctx, int sprn, int gprn)
|
||||
{
|
||||
gen_icount_io_start(ctx);
|
||||
gen_helper_store_40x_tsr(cpu_env, cpu_gpr[gprn]);
|
||||
}
|
||||
|
||||
void spr_write_booke_tcr(DisasContext *ctx, int sprn, int gprn)
|
||||
{
|
||||
gen_icount_io_start(ctx);
|
||||
|
|
Loading…
Reference in New Issue