target/openrisc: Remove duplicate features property
The features property has stored the exact same thing as the cpucfgr spr. Remove the feature enum and property as it is not needed. In order to preserve the behavior or keeping features accross reset this patch moves cpucfgr into the non reset region of the state struct. Since the cpucfgr is read only this means we only need to sset cpucfgr once during class init. Signed-off-by: Stafford Horne <shorne@gmail.com>
This commit is contained in:
parent
acf57591c0
commit
48a1b62baa
|
@ -52,7 +52,6 @@ static void openrisc_cpu_reset(CPUState *s)
|
|||
s->exception_index = -1;
|
||||
|
||||
cpu->env.upr = UPR_UP | UPR_DMP | UPR_IMP | UPR_PICP | UPR_TTP;
|
||||
cpu->env.cpucfgr = CPUCFGR_OB32S | CPUCFGR_OF32S | CPUCFGR_NSGF;
|
||||
cpu->env.dmmucfgr = (DMMUCFGR_NTW & (0 << 2)) | (DMMUCFGR_NTS & (6 << 2));
|
||||
cpu->env.immucfgr = (IMMUCFGR_NTW & (0 << 2)) | (IMMUCFGR_NTS & (6 << 2));
|
||||
|
||||
|
@ -65,12 +64,6 @@ static void openrisc_cpu_reset(CPUState *s)
|
|||
#endif
|
||||
}
|
||||
|
||||
static inline void set_feature(OpenRISCCPU *cpu, int feature)
|
||||
{
|
||||
cpu->feature |= feature;
|
||||
cpu->env.cpucfgr = cpu->feature;
|
||||
}
|
||||
|
||||
static void openrisc_cpu_realizefn(DeviceState *dev, Error **errp)
|
||||
{
|
||||
CPUState *cs = CPU(dev);
|
||||
|
@ -132,19 +125,15 @@ static void or1200_initfn(Object *obj)
|
|||
{
|
||||
OpenRISCCPU *cpu = OPENRISC_CPU(obj);
|
||||
|
||||
set_feature(cpu, OPENRISC_FEATURE_NSGF);
|
||||
set_feature(cpu, OPENRISC_FEATURE_OB32S);
|
||||
set_feature(cpu, OPENRISC_FEATURE_OF32S);
|
||||
set_feature(cpu, OPENRISC_FEATURE_EVBAR);
|
||||
cpu->env.cpucfgr = CPUCFGR_NSGF | CPUCFGR_OB32S | CPUCFGR_OF32S |
|
||||
CPUCFGR_EVBARP;
|
||||
}
|
||||
|
||||
static void openrisc_any_initfn(Object *obj)
|
||||
{
|
||||
OpenRISCCPU *cpu = OPENRISC_CPU(obj);
|
||||
|
||||
set_feature(cpu, OPENRISC_FEATURE_NSGF);
|
||||
set_feature(cpu, OPENRISC_FEATURE_OB32S);
|
||||
set_feature(cpu, OPENRISC_FEATURE_EVBAR);
|
||||
cpu->env.cpucfgr = CPUCFGR_NSGF | CPUCFGR_OB32S | CPUCFGR_EVBARP;
|
||||
}
|
||||
|
||||
typedef struct OpenRISCCPUInfo {
|
||||
|
|
|
@ -196,18 +196,6 @@ enum {
|
|||
SR_SCE = (1 << 17),
|
||||
};
|
||||
|
||||
/* OpenRISC Hardware Capabilities */
|
||||
enum {
|
||||
OPENRISC_FEATURE_NSGF = (15 << 0),
|
||||
OPENRISC_FEATURE_CGF = (1 << 4),
|
||||
OPENRISC_FEATURE_OB32S = (1 << 5),
|
||||
OPENRISC_FEATURE_OB64S = (1 << 6),
|
||||
OPENRISC_FEATURE_OF32S = (1 << 7),
|
||||
OPENRISC_FEATURE_OF64S = (1 << 8),
|
||||
OPENRISC_FEATURE_OV64S = (1 << 9),
|
||||
OPENRISC_FEATURE_EVBAR = (1 << 12),
|
||||
};
|
||||
|
||||
/* Tick Timer Mode Register */
|
||||
enum {
|
||||
TTMR_TP = (0xfffffff),
|
||||
|
@ -292,7 +280,6 @@ typedef struct CPUOpenRISCState {
|
|||
uint32_t sr; /* Supervisor register, without SR_{F,CY,OV} */
|
||||
uint32_t vr; /* Version register */
|
||||
uint32_t upr; /* Unit presence register */
|
||||
uint32_t cpucfgr; /* CPU configure register */
|
||||
uint32_t dmmucfgr; /* DMMU configure register */
|
||||
uint32_t immucfgr; /* IMMU configure register */
|
||||
uint32_t esr; /* Exception supervisor register */
|
||||
|
@ -311,6 +298,8 @@ typedef struct CPUOpenRISCState {
|
|||
CPU_COMMON
|
||||
|
||||
/* Fields from here on are preserved across CPU reset. */
|
||||
uint32_t cpucfgr; /* CPU configure register */
|
||||
|
||||
#ifndef CONFIG_USER_ONLY
|
||||
CPUOpenRISCTLBContext * tlb;
|
||||
|
||||
|
@ -337,7 +326,6 @@ typedef struct OpenRISCCPU {
|
|||
|
||||
CPUOpenRISCState env;
|
||||
|
||||
uint32_t feature; /* CPU Capabilities */
|
||||
} OpenRISCCPU;
|
||||
|
||||
static inline OpenRISCCPU *openrisc_env_get_cpu(CPUOpenRISCState *env)
|
||||
|
|
Loading…
Reference in a new issue