[U-Boot] [PATCH 1/1] configs: x86: allow to override CONFIG_BOOTCOMMAND
Allow to override CONFIG_BOOTCOMMAND in .config. Signed-off-by: Heinrich Schuchardt --- include/configs/x86-common.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/configs/x86-common.h b/include/configs/x86-common.h index 064c546403..3d536dc9d5 100644 --- a/include/configs/x86-common.h +++ b/include/configs/x86-common.h @@ -64,8 +64,10 @@ * Command line configuration. */ +#ifndef CONFIG_BOOTCOMMAND #define CONFIG_BOOTCOMMAND \ "ext2load scsi 0:3 0100 /boot/vmlinuz; zboot 0100" +#endif #if defined(CONFIG_CMD_KGDB) #define CONFIG_KGDB_BAUDRATE 115200 -- 2.14.2 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 1/1] configs: x86: allow to override CONFIG_BOOTCOMMAND
On Wed, Jan 3, 2018 at 11:23 PM, Heinrich Schuchardt wrote: > Allow to override CONFIG_BOOTCOMMAND in .config. > > Signed-off-by: Heinrich Schuchardt > --- > include/configs/x86-common.h | 2 ++ > 1 file changed, 2 insertions(+) > Reviewed-by: Bin Meng ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 1/1] configs: x86: allow to override CONFIG_BOOTCOMMAND
Hi Heinrich, On 3 January 2018 at 08:23, Heinrich Schuchardt wrote: > Allow to override CONFIG_BOOTCOMMAND in .config. > > Signed-off-by: Heinrich Schuchardt > --- > include/configs/x86-common.h | 2 ++ > 1 file changed, 2 insertions(+) This is a Chrome OS boot line. I think you should consider whether it should move into x86-chromebook.h or similar? Then you can just remove it from the common file. Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 1/1] configs: x86: allow to override CONFIG_BOOTCOMMAND
On 01/08/2018 05:50 AM, Simon Glass wrote: Hi Heinrich, On 3 January 2018 at 08:23, Heinrich Schuchardt wrote: Allow to override CONFIG_BOOTCOMMAND in .config. Signed-off-by: Heinrich Schuchardt --- include/configs/x86-common.h | 2 ++ 1 file changed, 2 insertions(+) This is a Chrome OS boot line. I think you should consider whether it should move into x86-chromebook.h or similar? Then you can just remove it from the common file. I understand Chromebooks uses this boot line. theadorable-x86-common.h undefines CONFIG_BOOTCOMMAND anyway. For qemu I need to override the value. What about all the other x86 boards? They seem to have bootargs matching the CONFIG_BOOTCOMMAND. As they are not using distro boot, wouldn't removing the CONFIG_BOOTCOMMAND default value create havoc? As a downstream user I would not cherish such a change. I think it is safer to leave the default value where it is. Would you accept the patch as is? Best regards Heinrich ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 1/1] configs: x86: allow to override CONFIG_BOOTCOMMAND
Hi Heinrich, On 8 January 2018 at 13:57, Heinrich Schuchardt wrote: > On 01/08/2018 05:50 AM, Simon Glass wrote: >> >> Hi Heinrich, >> >> On 3 January 2018 at 08:23, Heinrich Schuchardt >> wrote: >>> >>> Allow to override CONFIG_BOOTCOMMAND in .config. >>> >>> Signed-off-by: Heinrich Schuchardt >>> --- >>> include/configs/x86-common.h | 2 ++ >>> 1 file changed, 2 insertions(+) >> >> >> This is a Chrome OS boot line. I think you should consider whether it >> should move into x86-chromebook.h or similar? Then you can just remove >> it from the common file. > > > I understand Chromebooks uses this boot line. > > theadorable-x86-common.h undefines CONFIG_BOOTCOMMAND anyway. > For qemu I need to override the value. > > What about all the other x86 boards? > They seem to have bootargs matching the CONFIG_BOOTCOMMAND. > > As they are not using distro boot, wouldn't removing the CONFIG_BOOTCOMMAND > default value create havoc? > As a downstream user I would not cherish such a change. > > I think it is safer to leave the default value where it is. > > Would you accept the patch as is? Bin seems happy and I have no objection. Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 1/1] configs: x86: allow to override CONFIG_BOOTCOMMAND
On Mon, Jan 8, 2018 at 11:10 AM, Bin Meng wrote: > On Wed, Jan 3, 2018 at 11:23 PM, Heinrich Schuchardt > wrote: >> Allow to override CONFIG_BOOTCOMMAND in .config. >> >> Signed-off-by: Heinrich Schuchardt >> --- >> include/configs/x86-common.h | 2 ++ >> 1 file changed, 2 insertions(+) >> > > Reviewed-by: Bin Meng applied to u-boot-x86, thanks! ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot