[PATCH 1/2] configs: socfpga: Add QSPI support for Cyclone 5

[Ley Foon Tan]

On Fri, Feb 21, 2020 at 9:25 AM Ley Foon Tan <ley.foon.tan at intel.com> 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 <ley.foon.tan at intel.com>
> ---
>  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