[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