[PATCH v2] configs: socfpga: add kernel parameter for immediate reboot

Simon Goldschmidt simon.k.r.goldschmidt at gmail.com
Fri Aug 7 09:47:16 CEST 2020


Tan, Ley Foon <ley.foon.tan at intel.com> schrieb am Fr., 7. Aug. 2020, 09:25:

>
>
> Ooi, Joyce wrote:
> > From: Joyce Ooi <joyce.ooi at intel.com>
> >
> > Add Linux kernel line parameters for socfpga soc64 U-Boot. The "panic=-1"
> > indicates the reset should occur immediately (without a delay).
> >
> > Signed-off-by: Joyce Ooi <joyce.ooi at intel.com>
> > ---
> > v2: fix typo in commit message
> > ---
> >  include/configs/socfpga_soc64_common.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/include/configs/socfpga_soc64_common.h
> > b/include/configs/socfpga_soc64_common.h
> > index 7237ec95e34..d51a532a929 100644
> > --- a/include/configs/socfpga_soc64_common.h
> > +++ b/include/configs/socfpga_soc64_common.h
> > @@ -79,7 +79,7 @@ unsigned int cm_get_qspi_controller_clk_hz(void);
> >   * CONFIG_BOOTARGS goes into the environment value "bootargs".
> >   * Do note the value will override also the chosen node in FDT blob.
> >   */
> > -#define CONFIG_BOOTARGS "earlycon"
> > +#define CONFIG_BOOTARGS "earlycon panic=-1"
> >  #define CONFIG_BOOTCOMMAND "run fatscript; run mmcload;run
> > linux_qspi_enable;" \
> >                          "run mmcboot"
> Hi Joyce
>
> Just noticed CONFIG_BOOTARGS is supported in Kconfig now.
> Can you move this #define to Stratix10 and Agilex *_defconfig?
>

Of course _defconfig is a much better place so that people can easily
override it...
But why do we need this in the defconfig at all? I don't know how other
platforms handle this, but to me it seems like a user setting, not a thing
that should be changed by general config?

Regards,
Simon


> Regards
> Ley Foon
>


More information about the U-Boot mailing list