On Fri, Jun 9, 2017 at 12:17 AM, Babu Moger <babu.moger@xxxxxxxxxx> wrote:
Display warning if CPU_BIG_ENDIAN is not defined on big endian architecture and also warn if it defined on little endian architectures. We have seen some generic code(for example code include/asm-generic/qrwlock.h) uses CONFIG_CPU_BIG_ENDIAN to decide the endianess.
That example is IMHO the least harmful, as qrwlock must be selected explicitly by the architecture. The uses in drivers/of/base.c drivers/of/fdt.c drivers/tty/serial/earlycon.c drivers/tty/serial/serial_core.c are more dangerous, and may have bitten people already. In addition, people may have worked around them in DT, so this series may actually introduce regressions.
Here is the original discussion http://www.spinics.net/lists/devicetree/msg178101.html Signed-off-by: Babu Moger <babu.moger@xxxxxxxxxx> Suggested-by: Arnd Bergmann <arnd@xxxxxxxx>
Hmm, the link above refers to a mail from me? ;-) Acked-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-m68k" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html