On 04/09/2014 11:28 PM, Jonas Gorski wrote:
+#ifdef CONFIG_CPU_LITTLE_ENDIAN + /* This needs little endian fixup */ + if (config_enabled(CPU_MIPSR1)) {Hm, looking at arch/mips/Kconfig, this will falsely identify CPU_BMIPS as R2 as it does not select CPU_MIPSR1. How about "if (cpu_has_mips_r1)"? There are some targets that have both R1 and R2 cores (e.g. bcm47xx), and even if we built the kernel for R1, it does not prevent us from emitting R2 instructions because we do this at runtime when we can check for it.
Sounds about right. Let me have a look -- markos
