Tan, Ley Foon <ley.foon....@intel.com> schrieb am Fr., 7. Aug. 2020, 09:25:
> > > Ooi, Joyce wrote: > > From: Joyce Ooi <joyce....@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....@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 >
