From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrey Danin Date: Tue, 17 Mar 2015 10:58:57 +0300 Subject: [U-Boot] [RFC 4/4] paz00: enable bootmenu In-Reply-To: <55071D85.50909@wwwdotorg.org> References: <1426286965-18117-1-git-send-email-danindrey@mail.ru> <1426286965-18117-5-git-send-email-danindrey@mail.ru> <55071D85.50909@wwwdotorg.org> Message-ID: <5507DEC1.30707@mail.ru> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 16.03.2015 21:14, Stephen Warren wrote: > On 03/13/2015 04:49 PM, Andrey Danin wrote: >> Signed-off-by: Andrey Danin > > Some explanation of what these new options do might be nice; the > features/benefits they enable. > > Are any of the options generally useful? If so, should they be enabled > in tegra-common*.h? > > You also didn't Cc Tom Warren (Tegra maintainer) so he likely won't see > the emails and hence won't apply this patch. Thanks for the reply Stephen! I will add Tom Warren to Cc next time. Bootmenu allows to create user friendly menu to choose different boot options. For example next 3 lines in boot config --- setenv bootmenu_0 "Boot kernel = setenv bootargs ..." setenv bootmenu_1 "Boot recovery = setenv bootargs ..." bootmenu 20 --- will generate bootmenu with 3 options: --- *** U-Boot Boot Menu *** Boot kernel Boot recovery U-Boot console Press UP/DOWN to move, ENTER to select --- More information about bootmenu is available at doc/README.bootmenu I was focused on console part for this RFC. This patch is for testing purposes. I will use tegra config instead of paz00 next time to allow other users of tegra devices to try changes.