uboot-imx should have hush shell support enabled by default

Bug #510585 reported by Oliver Grawert
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
uboot-imx (Ubuntu)
Invalid
Wishlist
Unassigned

Bug Description

on our images we currently use boot.scr scripts in uboot based systems to make it easy to set cmdline options for the kernel...
we currently use the patch below for our uboot-imx package, it would be nice to have these two options set by default upstream instead:

--- uboot-imx-2009.01/include/configs/imx51.h 2010-01-18 13:38:50.473949230 +0000
+++ uboot-imx-2009.01.new/include/configs/imx51.h 2010-01-18 13:41:42.713979851 +0000
@@ -180,6 +181,8 @@
  * Miscellaneous configurable options
  */
 #define CONFIG_SYS_LONGHELP /* undef to save memory */
+#define CONFIG_SYS_HUSH_PARSER /* use "hush" command parser */
+#define CONFIG_SYS_PROMPT_HUSH_PS2 "BBG U-Boot > "
 #define CONFIG_SYS_PROMPT "BBG U-Boot > "
 #define CONFIG_AUTO_COMPLETE
 #define CONFIG_SYS_CBSIZE 256 /* Console I/O Buffer Size */

Tags: armel
Oliver Grawert (ogra)
tags: added: armel
Revision history for this message
Alexander Sack (asac) wrote :

subscribed terry.

terry, can you check if you could turn that on by default in future code drops?

Changed in uboot-imx (Ubuntu):
status: New → Triaged
importance: Undecided → Wishlist
assignee: nobody → Terry Lv (r65388)
Revision history for this message
Steve Langasek (vorlon) wrote :

This package is no longer in Ubuntu (as of natty).

Should this bug be retargeted to u-boot-linaro? Or do you want it fixed in SRU for previous releases?

Revision history for this message
Loïc Minier (lool) wrote :

Upstream u-boot and hence u-boot-linaro have CONFIG_SYS_HUSH_PARSER set and CONFIG_SYS_PROMPT_HUSH_PS2 defined to "> " for mx51evk

Revision history for this message
Oliver Grawert (ogra) wrote :

uboot-imx was never used in ubuntu, in fact it was not even in main. With regard to SRUs, feel free to close this bug.

Loïc Minier (lool)
Changed in uboot-imx (Ubuntu):
status: Triaged → Invalid
assignee: Terry Lv (r65388) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.