[U-Boot] [PATCH] arm: omap3: Add cmdline editing feature

This has been missing from beagleboard for quite some time. Add this now, as its very convenient for the user.
Additionally the "1" is removed after CONFIG_AUTO_COMPLETE. An empty define is preferred for such compile options.
Signed-off-by: Stefan Roese sr@denx.de Cc: Tom Rini trini@ti.com --- include/configs/omap3_beagle.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h index 1b566c0..3dad16a 100644 --- a/include/configs/omap3_beagle.h +++ b/include/configs/omap3_beagle.h @@ -309,7 +309,9 @@ "run mmcbootz; " \ "fi; " \
-#define CONFIG_AUTO_COMPLETE 1 +#define CONFIG_AUTO_COMPLETE +#define CONFIG_CMDLINE_EDITING + /* * Miscellaneous configurable options */

On Tue, Dec 17, 2013 at 02:14:18PM +0100, Stefan Roese wrote:
This has been missing from beagleboard for quite some time. Add this now, as its very convenient for the user.
Additionally the "1" is removed after CONFIG_AUTO_COMPLETE. An empty define is preferred for such compile options.
Signed-off-by: Stefan Roese sr@denx.de Cc: Tom Rini trini@ti.com
include/configs/omap3_beagle.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h index 1b566c0..3dad16a 100644 --- a/include/configs/omap3_beagle.h +++ b/include/configs/omap3_beagle.h @@ -309,7 +309,9 @@ "run mmcbootz; " \ "fi; " \
-#define CONFIG_AUTO_COMPLETE 1 +#define CONFIG_AUTO_COMPLETE +#define CONFIG_CMDLINE_EDITING
/*
- Miscellaneous configurable options
*/
I'm fine with this concept, but it shouldn't be needed with Enric's series to move omap3 to ti_armv7_common.h
participants (2)
-
Stefan Roese
-
Tom Rini