On Fri, Feb 21, 2020 at 9:25 AM Ley Foon Tan <[email protected]> wrote: > > Add QSPI boot support to boot target devices list. > Platform can provide their own boot settings through SOCFPGA_BOOT_SETTINGS > macro if needed. > > Add SOCFPGA_BOOT_SETTINGS for Cyclone 5. > > Signed-off-by: Ley Foon Tan <[email protected]> > --- > include/configs/socfpga_common.h | 18 ++++++++++++++++++ > include/configs/socfpga_cyclone5_socdk.h | 18 ++++++++++++++++++ > 2 files changed, 36 insertions(+) > > diff --git a/include/configs/socfpga_common.h > b/include/configs/socfpga_common.h > index 8d10469e7c..f3ddfca289 100644 > --- a/include/configs/socfpga_common.h > +++ b/include/configs/socfpga_common.h > @@ -228,11 +228,28 @@ unsigned int cm_get_qspi_controller_clk_hz(void); > #define BOOT_TARGET_DEVICES_MMC(func) > #endif > > +#ifdef CONFIG_CMD_SF > +#define BOOT_TARGET_DEVICES_QSPI(func) func(QSPI, qspi, na) > +#else > +#define BOOT_TARGET_DEVICES_QSPI(func) > +#endif > + > +#define BOOTENV_DEV_QSPI(devtypeu, devtypel, instance) \ > + "bootcmd_qspi=run qspiload; run qspiboot\0" > + > +#define BOOTENV_DEV_NAME_QSPI(devtypeu, devtypel, instance) \ > + "qspi " > + > #define BOOT_TARGET_DEVICES(func) \ > BOOT_TARGET_DEVICES_MMC(func) \ > + BOOT_TARGET_DEVICES_QSPI(func) \ > BOOT_TARGET_DEVICES_PXE(func) \ > BOOT_TARGET_DEVICES_DHCP(func) > > +#ifndef SOCFPGA_BOOT_SETTINGS > +#define SOCFPGA_BOOT_SETTINGS > +#endif > + > #include <config_distro_bootcmd.h> > > #ifndef CONFIG_EXTRA_ENV_SETTINGS > @@ -245,6 +262,7 @@ unsigned int cm_get_qspi_controller_clk_hz(void); > "pxefile_addr_r=0x02200000\0" \ > "ramdisk_addr_r=0x02300000\0" \ > "socfpga_legacy_reset_compat=1\0" \ > + SOCFPGA_BOOT_SETTINGS \ > BOOTENV > > #endif > diff --git a/include/configs/socfpga_cyclone5_socdk.h > b/include/configs/socfpga_cyclone5_socdk.h > index 028db2a09e..62ad001c4b 100644 > --- a/include/configs/socfpga_cyclone5_socdk.h > +++ b/include/configs/socfpga_cyclone5_socdk.h > @@ -14,6 +14,24 @@ > #define CONFIG_LOADADDR 0x01000000 > #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR > > +/* QSPI boot */ > +#define FDT_SIZE __stringify(0x00010000) > +#define KERNEL_SIZE __stringify(0x005d0000) > +#define QSPI_FDT_ADDR __stringify(0x00220000) > +#define QSPI_KERNEL_ADDR __stringify(0x00230000) > + > +#define SOCFPGA_BOOT_SETTINGS \ > + "fdt_size=" FDT_SIZE "\0" \ > + "kernel_size=" KERNEL_SIZE "\0" \ > + "qspi_fdt_addr=" QSPI_FDT_ADDR "\0" \ > + "qspi_kernel_addr=" QSPI_KERNEL_ADDR "\0" \ > + "qspiboot=setenv bootargs earlycon " \ > + "root=/dev/mtdblock1 rw rootfstype=jffs2; " \ > + "bootz ${kernel_addr_r} - ${fdt_addr_r}\0" \ > + "qspiload=sf probe; " \ > + "sf read ${kernel_addr_r} ${qspi_kernel_addr} ${kernel_size}; > " \ > + "sf read ${fdt_addr_r} ${qspi_fdt_addr} ${fdt_size}\0" > + > /* The rest of the configuration is shared */ > #include <configs/socfpga_common.h> > Any comment on this patch?
Regards Ley Foon

