[SeaBIOS] [PATCH] bootmenu: Skip wait for keypress with less than one item

Matt DeVillier matt.devillier at gmail.com
Tue Jun 24 22:52:37 CEST 2014


Hey Paul,

On 6/24/2014 3:48 PM, Paul Menzel wrote:
> Dear Matt,
>
>
> Am Dienstag, den 24.06.2014, 11:48 -0500 schrieb Matt DeVillier:
>> Here's how I did it - this will skip the 'Press XXX for boot menu.' as
>> well, so that there is no delay in booting, and no confusion on the
>> user's part as to why the boot menu key isn't working :)
> yes, much better than my patch! Thanks!
>
> Could you please send the patch formatted with `git format-patch -s` so
> your Signed-off-by line is added too?
can do
>
>> requires the file /etc/boot-auto to exist in the CBFS.
>>
>>  diff --git a/src/boot.c b/src/boot.c
>> index 133e206..bb5fd1f 100644
>> --- a/src/boot.c
>> +++ b/src/boot.c
>> @@ -455,6 +455,12 @@ interactive_bootmenu(void)
>>      if (! CONFIG_BOOTMENU || !romfile_loadint("etc/show-boot-menu", 1))
>>          return;
>>  
>> +    //skip menu if only one boot device
>> +    if ( NULL == BootList.first->next  && romfile_loadint("etc/boot-auto", 0) ) {
> The logic seems to be reversed. The file content should be `1`,
> shouldn’t it?
>
>         … && romfile_loadint("etc/boot-auto", 1)
no, the 0 is the default value if the file cannot be loaded.  So logically, fail if file doesn't exist.
>
>> +       printf("\n");
>> +       return;
>> +    }
>> +
>>      while (get_keystroke(0) >= 0)
>>          ;
> Also I wonder how I could achieve this without having to put that into
> CBFS? I’d make that the default with
>
>         !romfile_loadint("etc/boot-auto", 0)
>
> or also add a Kconfig option `AUTO_BOOT` as is done for the boot menu.
I think a Kconfig option would probably be better, but I'm not set on any one solution.  The CBFS file was Kevin's suggestion/addition, I'm running it as the default here without any override.
>
>
> Thanks,
>
> Paul
cheers,
Matt
>
>
> _______________________________________________
> SeaBIOS mailing list
> SeaBIOS at seabios.org
> http://www.seabios.org/mailman/listinfo/seabios

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.seabios.org/pipermail/seabios/attachments/20140624/de66a370/attachment.html>


More information about the SeaBIOS mailing list