diff options
| author | Deng-Cheng Zhu <dengcheng.zhu@imgtec.com> | 2013-04-01 14:14:28 -0400 |
|---|---|---|
| committer | Ralf Baechle <ralf@linux-mips.org> | 2013-04-05 09:10:45 -0400 |
| commit | adb3789264c4e8567113a0e764ad30ce6e8737f3 (patch) | |
| tree | 8cca117e544ee9a6a6586e45ab18b15069bf213e | |
| parent | ed1197f9317c960a199f491779e056c572506dd3 (diff) | |
MIPS: Fix ISA level which causes secondary cache init bypassing and more
The commit a96102be70 introduced set_isa() where compatible ISA info is
also set aside from the one gets passed in. It means, for example, 1004K
will have MIPS_CPU_ISA_M32R2/M32R1/II/I flags. This leads to things like
the following inappropriate:
if (c->isa_level == MIPS_CPU_ISA_M32R1 ||
c->isa_level == MIPS_CPU_ISA_M32R2 ||
c->isa_level == MIPS_CPU_ISA_M64R1 ||
c->isa_level == MIPS_CPU_ISA_M64R2)
This patch fixes it.
Signed-off-by: Deng-Cheng Zhu <dengcheng.zhu@imgtec.com>
Cc: Steven J. Hill <Steven.Hill@imgtec.com>
Cc: linux-mips@linux-mips.org
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
| -rw-r--r-- | arch/mips/kernel/cpu-probe.c | 6 | ||||
| -rw-r--r-- | arch/mips/kernel/traps.c | 2 | ||||
| -rw-r--r-- | arch/mips/mm/c-r4k.c | 6 | ||||
| -rw-r--r-- | arch/mips/mm/sc-mips.c | 6 |
4 files changed, 7 insertions, 13 deletions
diff --git a/arch/mips/kernel/cpu-probe.c b/arch/mips/kernel/cpu-probe.c index 6bfccc227a95..ed80c3844345 100644 --- a/arch/mips/kernel/cpu-probe.c +++ b/arch/mips/kernel/cpu-probe.c | |||
| @@ -1226,10 +1226,8 @@ __cpuinit void cpu_probe(void) | |||
| 1226 | if (c->options & MIPS_CPU_FPU) { | 1226 | if (c->options & MIPS_CPU_FPU) { |
| 1227 | c->fpu_id = cpu_get_fpu_id(); | 1227 | c->fpu_id = cpu_get_fpu_id(); |
| 1228 | 1228 | ||
| 1229 | if (c->isa_level == MIPS_CPU_ISA_M32R1 || | 1229 | if (c->isa_level & (MIPS_CPU_ISA_M32R1 | MIPS_CPU_ISA_M32R2 | |
| 1230 | c->isa_level == MIPS_CPU_ISA_M32R2 || | 1230 | MIPS_CPU_ISA_M64R1 | MIPS_CPU_ISA_M64R2)) { |
| 1231 | c->isa_level == MIPS_CPU_ISA_M64R1 || | ||
| 1232 | c->isa_level == MIPS_CPU_ISA_M64R2) { | ||
| 1233 | if (c->fpu_id & MIPS_FPIR_3D) | 1231 | if (c->fpu_id & MIPS_FPIR_3D) |
| 1234 | c->ases |= MIPS_ASE_MIPS3D; | 1232 | c->ases |= MIPS_ASE_MIPS3D; |
| 1235 | } | 1233 | } |
diff --git a/arch/mips/kernel/traps.c b/arch/mips/kernel/traps.c index a200b5bdbb87..c3abb88170fc 100644 --- a/arch/mips/kernel/traps.c +++ b/arch/mips/kernel/traps.c | |||
| @@ -1571,7 +1571,7 @@ void __cpuinit per_cpu_trap_init(bool is_boot_cpu) | |||
| 1571 | #ifdef CONFIG_64BIT | 1571 | #ifdef CONFIG_64BIT |
| 1572 | status_set |= ST0_FR|ST0_KX|ST0_SX|ST0_UX; | 1572 | status_set |= ST0_FR|ST0_KX|ST0_SX|ST0_UX; |
| 1573 | #endif | 1573 | #endif |
| 1574 | if (current_cpu_data.isa_level == MIPS_CPU_ISA_IV) | 1574 | if (current_cpu_data.isa_level & MIPS_CPU_ISA_IV) |
| 1575 | status_set |= ST0_XX; | 1575 | status_set |= ST0_XX; |
| 1576 | if (cpu_has_dsp) | 1576 | if (cpu_has_dsp) |
| 1577 | status_set |= ST0_MX; | 1577 | status_set |= ST0_MX; |
diff --git a/arch/mips/mm/c-r4k.c b/arch/mips/mm/c-r4k.c index ecca559b8d7b..2078915eacb9 100644 --- a/arch/mips/mm/c-r4k.c +++ b/arch/mips/mm/c-r4k.c | |||
| @@ -1247,10 +1247,8 @@ static void __cpuinit setup_scache(void) | |||
| 1247 | return; | 1247 | return; |
| 1248 | 1248 | ||
| 1249 | default: | 1249 | default: |
| 1250 | if (c->isa_level == MIPS_CPU_ISA_M32R1 || | 1250 | if (c->isa_level & (MIPS_CPU_ISA_M32R1 | MIPS_CPU_ISA_M32R2 | |
| 1251 | c->isa_level == MIPS_CPU_ISA_M32R2 || | 1251 | MIPS_CPU_ISA_M64R1 | MIPS_CPU_ISA_M64R2)) { |
| 1252 | c->isa_level == MIPS_CPU_ISA_M64R1 || | ||
| 1253 | c->isa_level == MIPS_CPU_ISA_M64R2) { | ||
| 1254 | #ifdef CONFIG_MIPS_CPU_SCACHE | 1252 | #ifdef CONFIG_MIPS_CPU_SCACHE |
| 1255 | if (mips_sc_init ()) { | 1253 | if (mips_sc_init ()) { |
| 1256 | scache_size = c->scache.ways * c->scache.sets * c->scache.linesz; | 1254 | scache_size = c->scache.ways * c->scache.sets * c->scache.linesz; |
diff --git a/arch/mips/mm/sc-mips.c b/arch/mips/mm/sc-mips.c index 93d937b4b1ba..df96da7e939b 100644 --- a/arch/mips/mm/sc-mips.c +++ b/arch/mips/mm/sc-mips.c | |||
| @@ -98,10 +98,8 @@ static inline int __init mips_sc_probe(void) | |||
| 98 | c->scache.flags |= MIPS_CACHE_NOT_PRESENT; | 98 | c->scache.flags |= MIPS_CACHE_NOT_PRESENT; |
| 99 | 99 | ||
| 100 | /* Ignore anything but MIPSxx processors */ | 100 | /* Ignore anything but MIPSxx processors */ |
| 101 | if (c->isa_level != MIPS_CPU_ISA_M32R1 && | 101 | if (!(c->isa_level & (MIPS_CPU_ISA_M32R1 | MIPS_CPU_ISA_M32R2 | |
| 102 | c->isa_level != MIPS_CPU_ISA_M32R2 && | 102 | MIPS_CPU_ISA_M64R1 | MIPS_CPU_ISA_M64R2))) |
| 103 | c->isa_level != MIPS_CPU_ISA_M64R1 && | ||
| 104 | c->isa_level != MIPS_CPU_ISA_M64R2) | ||
| 105 | return 0; | 103 | return 0; |
| 106 | 104 | ||
| 107 | /* Does this MIPS32/MIPS64 CPU have a config2 register? */ | 105 | /* Does this MIPS32/MIPS64 CPU have a config2 register? */ |
