Hi Stefan,
Stefan Roese wrote:
After this patch is now applied to the main u-boot repo, I get the following warning when compiling my new board with OneNAND & JFFS2 support:
[stefan@kubuntu u-boot (vcth-onenand-ubi-ver3)]$ make -s -j5 jffs2_1pass.c:1411:1: warning: "min_t" redefined In file included from /home/stefan/git/u-boot/u-boot/include/linux/mtd/mtd-abi.h:11, from /home/stefan/git/u-boot/u-boot/include/linux/mtd/mtd.h:11, from jffs2_1pass.c:275: /home/stefan/git/u-boot/u-boot/include/linux/mtd/compat.h:34:1: warning: this is the location of the previous definition
As it seems you added the "min_t" define to this file (see below). Why was this needed? I can safely remove it from my version. Do you see a problem with removing it on your test platform?
Thanks for reporting. That's obviously because we already have min_t definition in <linux/mtd/compat.h> and OneNAND support includes this header from somewhere while NOR support doesn't. I've changed jffs2_1pass.c to directly include <linux/mtd/compat.h> directly instead of defining own min_t. Please see patch in the follow up message.
Regards, Ilya.