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

[Marek Vasut]

On 3/2/20 10:34 AM, Ley Foon Tan wrote:
> 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?

Same as A10 very much.