On Sun, Oct 31, 2010 at 22:32, Andreas Schwab <schwab@xxxxxxxxxxxxxx> wrote: > Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> writes: > >> Which one is preferable? >> >> ------------------------------------------------------------------------------- >> Since commit 31c911329e048b715a1dfeaaf617be9430fd7f4e ("mm: check the argument >> of kunmap on architectures without highmem"), we get lots of warnings like >> >> arch/m68k/kernel/sys_m68k.c:508: warning: passing argument 1 of âkunmapâ from incompatible pointer type >> >> As m68k doesn't support highmem anyway, open code the calls to kmap() and >> kunmap() (the latter is a no-op) to kill the warnings. > > I prefer this one, it matches all architectures without CONFIG_HIGHPTE. Thx, applied for 2.6.38, with updated commit message. 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, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom policy in Canada: sign http://dissolvethecrtc.ca/ Don't email: <a href