public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] Move preprocessor test for cmd_nand.c to Makefile
@ 2009-11-19 16:08 Robert P. J. Day
  2009-11-19 16:15 ` Peter Tyser
  0 siblings, 1 reply; 3+ messages in thread
From: Robert P. J. Day @ 2009-11-19 16:08 UTC (permalink / raw)
  To: u-boot


Move the CONFIG_CMD_NAND preprocessor test out of common/cmd_nand.c
and into the Makefile for consistency with other commands.

Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

---

  AFAICT, this doesn't affect any functionality, right?

diff --git a/common/Makefile b/common/Makefile
index 3781738..b8c596c 100644
--- a/common/Makefile
+++ b/common/Makefile
@@ -116,7 +116,7 @@ COBJS-$(CONFIG_CMD_MISC) += cmd_misc.o
 COBJS-$(CONFIG_CMD_MMC) += cmd_mmc.o
 COBJS-$(CONFIG_MP) += cmd_mp.o
 COBJS-$(CONFIG_CMD_MTDPARTS) += cmd_mtdparts.o
-COBJS-y += cmd_nand.o
+COBJS-$(CONFIG_CMD_NAND) += cmd_nand.o
 COBJS-$(CONFIG_CMD_NET) += cmd_net.o
 COBJS-$(CONFIG_CMD_ONENAND) += cmd_onenand.o
 COBJS-$(CONFIG_CMD_OTP) += cmd_otp.o
diff --git a/common/cmd_nand.c b/common/cmd_nand.c
index 158a55f..1b17159 100644
--- a/common/cmd_nand.c
+++ b/common/cmd_nand.c
@@ -19,8 +19,6 @@
 #include <common.h>
 #include <linux/mtd/mtd.h>

-#if defined(CONFIG_CMD_NAND)
-
 #include <command.h>
 #include <watchdog.h>
 #include <malloc.h>
@@ -686,4 +684,3 @@ U_BOOT_CMD(nboot, 4, 1, do_nandboot,
 	"boot from NAND device",
 	"[partition] | [[[loadAddr] dev] offset]"
 );
-#endif

========================================================================
Robert P. J. Day                               Waterloo, Ontario, CANADA

            Linux Consulting, Training and Kernel Pedantry.

Web page:                                          http://crashcourse.ca
Twitter:                                       http://twitter.com/rpjday
========================================================================

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] Move preprocessor test for cmd_nand.c to Makefile
  2009-11-19 16:08 [U-Boot] [PATCH] Move preprocessor test for cmd_nand.c to Makefile Robert P. J. Day
@ 2009-11-19 16:15 ` Peter Tyser
  2009-11-19 18:34   ` Scott Wood
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Tyser @ 2009-11-19 16:15 UTC (permalink / raw)
  To: u-boot

On Thu, 2009-11-19 at 11:08 -0500, Robert P. J. Day wrote:
> Move the CONFIG_CMD_NAND preprocessor test out of common/cmd_nand.c
> and into the Makefile for consistency with other commands.

This change has already been submitted by 2 different people and I
believe is in the u-boot-nand-flash/next tree.

Best,
Peter

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] Move preprocessor test for cmd_nand.c to Makefile
  2009-11-19 16:15 ` Peter Tyser
@ 2009-11-19 18:34   ` Scott Wood
  0 siblings, 0 replies; 3+ messages in thread
From: Scott Wood @ 2009-11-19 18:34 UTC (permalink / raw)
  To: u-boot

On Thu, Nov 19, 2009 at 10:15:44AM -0600, Peter Tyser wrote:
> On Thu, 2009-11-19 at 11:08 -0500, Robert P. J. Day wrote:
> > Move the CONFIG_CMD_NAND preprocessor test out of common/cmd_nand.c
> > and into the Makefile for consistency with other commands.
> 
> This change has already been submitted by 2 different people and I
> believe is in the u-boot-nand-flash/next tree.

It's in Wolfgang's next tree now.

-Scott

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-11-19 18:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-11-19 16:08 [U-Boot] [PATCH] Move preprocessor test for cmd_nand.c to Makefile Robert P. J. Day
2009-11-19 16:15 ` Peter Tyser
2009-11-19 18:34   ` Scott Wood

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox