The ISA exception bit selects whether exceptions are taken in classic or microMIPS mode. This bit is Config3.ISAOnExc and was improperly defined as bits 16 and 17 instead of just bit 16. A new function was added so that platforms could set this bit when running a kernel compiled with only microMIPS instructions. Signed-off-by: Steven J. Hill <Steven.Hill@xxxxxxxxxx> Acked-by: David Daney <david.daney@xxxxxxxxxx> --- Changes in v5: * Call 'set_micromips_exception_mode' from 'traps_init' instead of being in platform-specific code. * Update 'set_micromips_exception_mode' function to set or clear the Config3.ISAOnExc depending on the binary type of the kernel. arch/mips/include/asm/mipsregs.h | 20 +++++++++++++++++++- arch/mips/kernel/cpu-probe.c | 3 --- arch/mips/kernel/traps.c | 5 +++++ 3 files changed, 24 insertions(+), 4 deletions(-) diff --git a/arch/mips/include/asm/mipsregs.h b/arch/mips/include/asm/mipsregs.h index 87e6207..52ade03 100644 --- a/arch/mips/include/asm/mipsregs.h +++ b/arch/mips/include/asm/mipsregs.h @@ -596,7 +596,7 @@ #define MIPS_CONF3_RXI (_ULCAST_(1) << 12) #define MIPS_CONF3_ULRI (_ULCAST_(1) << 13) #define MIPS_CONF3_ISA (_ULCAST_(3) << 14) -#define MIPS_CONF3_ISA_OE (_ULCAST_(3) << 16) +#define MIPS_CONF3_ISA_OE (_ULCAST_(1) << 16) #define MIPS_CONF3_VZ (_ULCAST_(1) << 23) #define MIPS_CONF4_MMUSIZEEXT (_ULCAST_(255) << 0) @@ -1161,6 +1161,24 @@ do { \ #define write_c0_brcm_sleepcount(val) __write_32bit_c0_register($22, 7, val) /* + * Set exceptions to be taken in microMIPS mode only, otherwise + * set for classic exceptions. + */ +#ifdef CONFIG_CPU_MICROMIPS +static inline void set_micromips_exception_mode(void) +{ + unsigned int config3 = read_c0_config3(); + + if (config3 & MIPS_CONF3_ISA) + write_c0_config3(config3 | MIPS_CONF3_ISA_OE); + else + write_c0_config3(config3 & ~MIPS_CONF3_ISA_OE); +} +#else +static inline void set_micromips_exception_mode(void) {} +#endif + +/* * Macros to access the floating point coprocessor control registers */ #define read_32bit_cp1_register(source) \ diff --git a/arch/mips/kernel/cpu-probe.c b/arch/mips/kernel/cpu-probe.c index c6568bf..b0d04a2 100644 --- a/arch/mips/kernel/cpu-probe.c +++ b/arch/mips/kernel/cpu-probe.c @@ -272,9 +272,6 @@ static inline unsigned int decode_config3(struct cpuinfo_mips *c) c->options |= MIPS_CPU_ULRI; if (config3 & MIPS_CONF3_ISA) c->options |= MIPS_CPU_MICROMIPS; -#ifdef CONFIG_CPU_MICROMIPS - write_c0_config3(read_c0_config3() | MIPS_CONF3_ISA_OE); -#endif if (config3 & MIPS_CONF3_VZ) c->ases |= MIPS_ASE_VZ; diff --git a/arch/mips/kernel/traps.c b/arch/mips/kernel/traps.c index a75ae40..151ed59 100644 --- a/arch/mips/kernel/traps.c +++ b/arch/mips/kernel/traps.c @@ -1837,6 +1837,11 @@ void __init trap_init(void) ebase += (read_c0_ebase() & 0x3ffff000); } + /* + * Set microMIPS exceptions for platforms that support it. + */ + set_micromips_exception_mode(); + if (board_ebase_setup) board_ebase_setup(); per_cpu_trap_init(true); -- 1.7.2.5