[U-Boot-Users] [Patch] Make MPC8266ADS compile again

Make MPC8266ADS board compile again. Signed-off-by: Runet Torgersen runet@innovsys.com
diff --git a/include/configs/MPC8266ADS.h b/include/configs/MPC8266ADS.h index 14b041e..d392b98 100644 --- a/include/configs/MPC8266ADS.h +++ b/include/configs/MPC8266ADS.h @@ -168,6 +168,7 @@ #undef CONFIG_CMD_MFSL #undef CONFIG_CMD_MMC #undef CONFIG_CMD_NAND +#undef CONFIG_CMD_ONENAND #undef CONFIG_CMD_PCMCIA #undef CONFIG_CMD_REISER #undef CONFIG_CMD_SCSI @@ -177,6 +178,7 @@ #undef CONFIG_CMD_UNIVERSE #undef CONFIG_CMD_USB #undef CONFIG_CMD_XIMG +#undef CONFIG_CMD_AT91_SPIMUX
/* Define a command string that is automatically executed when no character * is read on the console interface withing "Boot Delay" after reset.

runet@innovsys.com wrote:
Make MPC8266ADS board compile again. Signed-off-by: Runet Torgersen runet@innovsys.com
diff --git a/include/configs/MPC8266ADS.h b/include/configs/MPC8266ADS.h index 14b041e..d392b98 100644 --- a/include/configs/MPC8266ADS.h +++ b/include/configs/MPC8266ADS.h @@ -168,6 +168,7 @@ #undef CONFIG_CMD_MFSL #undef CONFIG_CMD_MMC #undef CONFIG_CMD_NAND +#undef CONFIG_CMD_ONENAND #undef CONFIG_CMD_PCMCIA #undef CONFIG_CMD_REISER #undef CONFIG_CMD_SCSI @@ -177,6 +178,7 @@ #undef CONFIG_CMD_UNIVERSE #undef CONFIG_CMD_USB #undef CONFIG_CMD_XIMG +#undef CONFIG_CMD_AT91_SPIMUX
/* Define a command string that is automatically executed when no character
- is read on the console interface withing "Boot Delay" after reset.
The *proper* fix is to delete the #include <config_cmd_all.h> and #define only the commands that actually are desired.
The simplest (and probably most proper) way to do this is actually to #include <config_cmd_default.h> and then add non-default commands that may be useful on the MPC8266ADS board.
The current configuration is legacy from the old bit-mapped configuration ways plus a very poor choice to do the "all-subtractive" method of selecting commands.
With the "all-subtractive" method, new commands will *FOREVER* be breaking the board, resulting in patches and/or bit rot forever. Please fix it once, right, and we will all thank you forever.
TIA, gvb

In message 471518AF.8080509@ge.com you wrote:
The *proper* fix is to delete the #include <config_cmd_all.h> and #define only the commands that actually are desired.
Hm...
The simplest (and probably most proper) way to do this is actually to #include <config_cmd_default.h> and then add non-default commands that may be useful on the MPC8266ADS board.
Indeed, that's the preferred way for 99% of all situations.
The current configuration is legacy from the old bit-mapped configuration ways plus a very poor choice to do the "all-subtractive" method of selecting commands.
With the "all-subtractive" method, new commands will *FOREVER* be breaking the board, resulting in patches and/or bit rot forever. Please fix it once, right, and we will all thank you forever.
Agreed.
Nevertheless, the patch fixes a real build problem now, so I'll apply it.
Best regards,
Wolfgang Denk

In message 47151610.47FrR7hZ3a0AxzYl%runet@innovsys.com you wrote:
Make MPC8266ADS board compile again. Signed-off-by: Runet Torgersen runet@innovsys.com
Applied, but please see JVB's remarks!
Best regards,
Wolfgang Denk
participants (3)
-
Jerry Van Baren
-
runet@innovsys.com
-
Wolfgang Denk