[PATCH 0/2] UBI(FS): fixing IS_ENABLED() usage + backport trees

Brian Norris computersforpeace at gmail.com
Mon Jun 18 19:31:21 EDT 2012


Hi Artem,

It looks like your ubifs-v3.3.git tree does not compile successfully. I
think there are two issues:

 1) IS_ENABLED() should be used with the CONFIG_ prefix. i.e.,
 IS_ENABLED(CONFIG_DEBUG_FS) instead of IS_ENABLED(DEBUG_FS)

 2) IS_ENABLED() has changed some between v3.3 and when this commit was
 originally integrated in mainline, so it masks a compile error in the
 mainline version

This patch series addresses issue 1. I suggest that this be backported to
the appropriate UBIFS tree to solve some build failures. But I think issue
2 may still be a problem, so perhaps the mainline IS_ENABLED() fix should
be backported as well?

BTW, there may be more than one backport tree that fails; I just tested
v3.3.

UBIFS commit (mainline) that breaks when ported to v3.3:
5e0c43e74d16db5710caebe8169995c9a13807f9

Upstream commit that fixes IS_ENABLED:
69349c2dc01c489eccaa4c472542c08e370c6d7e

Build failure in ubifs-v3.3.git:

  CC      drivers/mtd/ubi/debug.o
drivers/mtd/ubi/debug.c: In function ‘ubi_debugfs_init’:
drivers/mtd/ubi/debug.c:267: error: ‘__enabled_DEBUG_FS’ undeclared (first use in this function)
drivers/mtd/ubi/debug.c:267: error: (Each undeclared identifier is reported only once
drivers/mtd/ubi/debug.c:267: error: for each function it appears in.)
drivers/mtd/ubi/debug.c:267: error: ‘__enabled_DEBUG_FS_MODULE’ undeclared (first use in this function)
drivers/mtd/ubi/debug.c: In function ‘ubi_debugfs_exit’:
drivers/mtd/ubi/debug.c:287: error: ‘__enabled_DEBUG_FS’ undeclared (first use in this function)
drivers/mtd/ubi/debug.c:287: error: ‘__enabled_DEBUG_FS_MODULE’ undeclared (first use in this function)
drivers/mtd/ubi/debug.c: In function ‘ubi_debugfs_init_dev’:
drivers/mtd/ubi/debug.c:418: error: ‘__enabled_DEBUG_FS’ undeclared (first use in this function)
drivers/mtd/ubi/debug.c:418: error: ‘__enabled_DEBUG_FS_MODULE’ undeclared (first use in this function)
drivers/mtd/ubi/debug.c: In function ‘ubi_debugfs_exit_dev’:
drivers/mtd/ubi/debug.c:488: error: ‘__enabled_DEBUG_FS’ undeclared (first use in this function)
drivers/mtd/ubi/debug.c:488: error: ‘__enabled_DEBUG_FS_MODULE’ undeclared (first use in this function)
make[3]: *** [drivers/mtd/ubi/debug.o] Error 1
make[2]: *** [drivers/mtd/ubi] Error 2
make[1]: *** [drivers/mtd] Error 2
make[1]: *** Waiting for unfinished jobs....

Regards,
Brian

Brian Norris (2):
  UBIFS: correct usage of IS_ENABLED()
  UBI: correct usage of IS_ENABLED()

 drivers/mtd/ubi/debug.c |    8 ++++----
 fs/ubifs/debug.c        |    8 ++++----
 2 files changed, 8 insertions(+), 8 deletions(-)




More information about the linux-mtd mailing list