On Thu, Nov 21, 2019 at 4:10 PM Ooi, Joyce joyce.ooi@intel.com wrote:
Add Linux kernel line parameters for Stratix10 U-Boot. The "panic=-1" indicates the reset should occur immediately (without a delay)
Signed-off-by: Thor Thayer thor.thayer@linux.intel.com Signed-off-by: Ooi, Joyce joyce.ooi@intel.com
You might want to send socfpga patches to the U-Boot socfpga maintainers as well...
include/configs/socfpga_stratix10_socdk.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/socfpga_stratix10_socdk.h b/include/configs/socfpga_stratix10_socdk.h index 8e6ecf4..09ea066 100644 --- a/include/configs/socfpga_stratix10_socdk.h +++ b/include/configs/socfpga_stratix10_socdk.h @@ -88,7 +88,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"
That looks a bit like a hack. In my opinion, bootargs should be provided by the boot configuration (e.g. included in devicetree), not hardcoded in the bootloader.
I'd rather take a patch that removes this line than adding more.
Regards, Simon
#define CONFIG_BOOTCOMMAND "run fatscript; run mmcload;run linux_qspi_enable;" \ "run mmcboot"
-- 1.9.1