]> git.karo-electronics.de Git - karo-tx-linux.git/commit
blackfin: fix ifdef fustercluck in mach-bf538/boards/ezkit.c
authorPaul Gortmaker <paul.gortmaker@windriver.com>
Thu, 19 Apr 2012 03:31:28 +0000 (23:31 -0400)
committerPaul Gortmaker <paul.gortmaker@windriver.com>
Mon, 23 Apr 2012 15:39:19 +0000 (11:39 -0400)
commit3c5c69b8baef178c1f8a414d5d5c9dc93eaceafb
tree860076c80b6651bd0451c904f0c41120d4b6e702
parente5f0870c8b046eefd65a7de6e73c8abc4d3f4fe5
blackfin: fix ifdef fustercluck in mach-bf538/boards/ezkit.c

This file has lots and lots of ifdef, around structure decls
and structure usages.  The failure issue was that we would
build the BF538-EZKIT_defconfig and get:

 arch/blackfin/mach-bf538/boards/ezkit.c:924:3: error: 'bfin_lq035q1_device'
 undeclared here (not in a function)

even though the same ifdef _appeared_ to enable both the struct
declaration and the code that used it.  Yet cpp was telling us we
didn't have the struct, but we still had the usage of it.

However, _appeared_ is the operative word.  After marking all the
anonymous #endif with their parent #ifdef config options, it was
_then_ clear that there was a misplaced #endif that was hiding
the struct declaration.

The real guts of the patch boils down to this:

-#endif
+#endif /* CONFIG_MTD_M25P80 */
+#endif /* CONFIG_SPI_BFIN5XX */
 [...]
-#endif  /* spi master and devices */

but since I had to tag the #endif with their respective #ifdef
options to find this misplaced SPI endif, it would be silly to
then go and delete them all.  So they stay.

Cc: Sonic Zhang <sonic.zhang@analog.com>
Cc: Bob Liu <lliubbo@gmail.com>
Acked-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
arch/blackfin/mach-bf538/boards/ezkit.c