All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Yuslaimi, Alif Zakuan" <alif.zakuan.yuslaimi@altera.com>
To: "Chee, Tien Fong" <tien.fong.chee@altera.com>,
	"u-boot@lists.denx.de" <u-boot@lists.denx.de>
Cc: Tom Rini <trini@konsulko.com>, Marek Vasut <marex@denx.de>,
	Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com>,
	Lukasz Majewski <lukma@denx.de>, Peng Fan <peng.fan@nxp.com>,
	Jaehoon Chung <jh80.chung@samsung.com>,
	Simon Glass <sjg@chromium.org>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Kory Maincent <kory.maincent@bootlin.com>, Yao Zi <me@ziyao.cc>
Subject: Re: [PATCH v2 2/9] configs: stratix10: Combine defconfig for all boot flashes
Date: Thu, 14 May 2026 13:42:17 +0800	[thread overview]
Message-ID: <5677e494-21aa-4532-a47d-9d45a6376f44@altera.com> (raw)
In-Reply-To: <c569f03b-1383-4771-8169-99eb611d3f5d@altera.com>

Hi Tien Fong,

On 7/5/2026 5:31 pm, Chee, Tien Fong wrote:
> Hi Alif,
> 
> 
> On 28/4/2026 11:48 am, alif.zakuan.yuslaimi@altera.com wrote:
>> From: Alif Zakuan Yuslaimi <alif.zakuan.yuslaimi@altera.com>
>>
>> Combine all MMC and QSPI configs into single defconfig which is named
>> as "socfpga_stratix10_defconfig". This will be the default defconfig to
>> use.
>>
>> This will support booting from all three flashes using ARM Trusted Firmware
>> (ATF) as the secure runtime monitor.
>>
>> Signed-off-by: Alif Zakuan Yuslaimi <alif.zakuan.yuslaimi@altera.com>
>> ---
>>    configs/socfpga_stratix10_atf_defconfig | 90 ----------------------
>>    configs/socfpga_stratix10_defconfig     | 99 +++++++++++++++----------
>>    2 files changed, 58 insertions(+), 131 deletions(-)
>>    delete mode 100644 configs/socfpga_stratix10_atf_defconfig
>>
>> diff --git a/configs/socfpga_stratix10_atf_defconfig b/configs/socfpga_stratix10_atf_defconfig
>> deleted file mode 100644
>> index 206343885d9..00000000000
>> --- a/configs/socfpga_stratix10_atf_defconfig
>> +++ /dev/null
>> @@ -1,90 +0,0 @@
>> -CONFIG_ARM=y
>> -CONFIG_COUNTER_FREQUENCY=400000000
>> -CONFIG_ARCH_SOCFPGA=y
>> -CONFIG_TEXT_BASE=0x200000
>> -CONFIG_SYS_MALLOC_LEN=0x500000
>> -CONFIG_NR_DRAM_BANKS=2
>> -CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
>> -CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x300000
>> -CONFIG_SF_DEFAULT_MODE=0x2003
>> -CONFIG_ENV_SIZE=0x1000
>> -CONFIG_ENV_OFFSET=0x200
>> -CONFIG_DM_GPIO=y
>> -CONFIG_DEFAULT_DEVICE_TREE="socfpga_stratix10_socdk"
>> -CONFIG_DM_RESET=y
>> -CONFIG_SPL_STACK=0xffe3f000
>> -CONFIG_SPL_TEXT_BASE=0xFFE00000
>> -CONFIG_SPL_HAS_BSS_LINKER_SECTION=y
>> -CONFIG_SPL_BSS_START_ADDR=0x3ff00000
>> -CONFIG_SPL_BSS_MAX_SIZE=0x100000
>> -CONFIG_SYS_BOOTM_LEN=0x2000000
>> -CONFIG_SYS_LOAD_ADDR=0x02000000
>> -CONFIG_TARGET_SOCFPGA_STRATIX10_SOCDK=y
>> -CONFIG_IDENT_STRING="socfpga_stratix10"
>> -CONFIG_SPL_FS_FAT=y
>> -CONFIG_REMAKE_ELF=y
>> -CONFIG_FIT=y
>> -CONFIG_SPL_FIT_SIGNATURE=y
>> -CONFIG_SPL_LOAD_FIT=y
>> -CONFIG_SPL_LOAD_FIT_ADDRESS=0x02000000
>> -CONFIG_BOOTDELAY=5
>> -CONFIG_USE_BOOTARGS=y
>> -CONFIG_BOOTARGS="earlycon"
>> -CONFIG_USE_BOOTCOMMAND=y
>> -CONFIG_BOOTCOMMAND="run fatscript; run mmcfitload; run mmcfitboot"
>> -CONFIG_SYS_PBSIZE=2085
>> -CONFIG_SPL_MAX_SIZE=0x40000
>> -# CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
>> -CONFIG_SPL_HAVE_INIT_STACK=y
>> -CONFIG_SPL_SYS_MALLOC=y
>> -CONFIG_SPL_HAS_CUSTOM_MALLOC_START=y
>> -CONFIG_SPL_CUSTOM_SYS_MALLOC_ADDR=0x3fa00000
>> -CONFIG_SPL_SYS_MALLOC_SIZE=0x500000
>> -CONFIG_SPL_SPI_LOAD=y
>> -CONFIG_SYS_SPI_U_BOOT_OFFS=0x02000000
>> -CONFIG_SPL_ATF=y
>> -CONFIG_SPL_ATF_NO_PLATFORM_PARAM=y
>> -CONFIG_SPL_TARGET="spl/u-boot-spl-dtb.hex"
>> -CONFIG_HUSH_PARSER=y
>> -CONFIG_SYS_PROMPT="SOCFPGA_STRATIX10 # "
>> -CONFIG_CMD_MEMTEST=y
>> -CONFIG_CMD_GPIO=y
>> -CONFIG_CMD_I2C=y
>> -CONFIG_CMD_MMC=y
>> -CONFIG_CMD_SPI=y
>> -CONFIG_CMD_USB=y
>> -CONFIG_CMD_DHCP=y
>> -CONFIG_CMD_MII=y
>> -CONFIG_CMD_PING=y
>> -CONFIG_CMD_CACHE=y
>> -CONFIG_CMD_EXT4=y
>> -CONFIG_CMD_FAT=y
>> -CONFIG_CMD_FS_GENERIC=y
>> -CONFIG_ENV_IS_IN_MMC=y
>> -CONFIG_ENV_RELOC_GD_ENV_ADDR=y
>> -CONFIG_NET_RANDOM_ETHADDR=y
>> -CONFIG_SPL_DM_SEQ_ALIAS=y
>> -CONFIG_SPL_ALTERA_SDRAM=y
>> -CONFIG_DWAPB_GPIO=y
>> -CONFIG_DM_I2C=y
>> -CONFIG_SYS_I2C_DW=y
>> -CONFIG_SYS_MMC_MAX_BLK_COUNT=256
>> -CONFIG_MMC_DW=y
>> -CONFIG_SPI_FLASH_SPANSION=y
>> -CONFIG_SPI_FLASH_STMICRO=y
>> -CONFIG_PHY_MICREL=y
>> -CONFIG_PHY_MICREL_KSZ90X1=y
>> -CONFIG_ETH_DESIGNWARE=y
>> -CONFIG_MII=y
>> -CONFIG_SYS_NS16550_MEM32=y
>> -CONFIG_SPI=y
>> -CONFIG_CADENCE_QSPI=y
>> -CONFIG_DESIGNWARE_SPI=y
>> -CONFIG_USB=y
>> -CONFIG_USB_DWC2=y
>> -CONFIG_USB_STORAGE=y
>> -CONFIG_DESIGNWARE_WATCHDOG=y
>> -CONFIG_WDT=y
>> -# CONFIG_SPL_USE_TINY_PRINTF is not set
>> -CONFIG_PANIC_HANG=y
>> -CONFIG_SPL_CRC32=y
>> diff --git a/configs/socfpga_stratix10_defconfig b/configs/socfpga_stratix10_defconfig
>> index ef21dc92493..02a1875a72e 100644
>> --- a/configs/socfpga_stratix10_defconfig
>> +++ b/configs/socfpga_stratix10_defconfig
>> @@ -1,93 +1,110 @@
>>    CONFIG_ARM=y
>>    CONFIG_COUNTER_FREQUENCY=400000000
>> +CONFIG_SPL_LDSCRIPT="arch/arm/mach-socfpga/u-boot-spl-soc64.lds"
>>    CONFIG_ARCH_SOCFPGA=y
>> -CONFIG_TEXT_BASE=0x1000
>> -CONFIG_SYS_MALLOC_LEN=0x500000
>> +CONFIG_TEXT_BASE=0x200000
>> +CONFIG_SYS_MALLOC_F_LEN=0x2000
>>    CONFIG_NR_DRAM_BANKS=2
>> -CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
>> -CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x101000
>> -CONFIG_SF_DEFAULT_MODE=0x2003
>> -CONFIG_ENV_SIZE=0x1000
>> -CONFIG_ENV_OFFSET=0x200
>> +CONFIG_ENV_SIZE=0x2000
>> +CONFIG_ENV_OFFSET=0x04100000
>> +CONFIG_ENV_SECT_SIZE=0x20000
>> +CONFIG_SYS_SPI_U_BOOT_OFFS=0x04000000
>>    CONFIG_DM_GPIO=y
>>    CONFIG_DEFAULT_DEVICE_TREE="altera/socfpga_stratix10_socdk"
>>    CONFIG_OF_UPSTREAM=y
>> -CONFIG_DM_RESET=y
>> -CONFIG_SPL_STACK=0xffe3f000
>>    CONFIG_SPL_TEXT_BASE=0xFFE00000
>> -CONFIG_SPL_HAS_BSS_LINKER_SECTION=y
>> -CONFIG_SPL_BSS_START_ADDR=0x3ff00000
>> -CONFIG_SPL_BSS_MAX_SIZE=0x100000
>> -CONFIG_SYS_BOOTM_LEN=0x2000000
>> -CONFIG_SYS_LOAD_ADDR=0x02000000
>>    CONFIG_TARGET_SOCFPGA_STRATIX10_SOCDK=y
>>    CONFIG_IDENT_STRING="socfpga_stratix10"
>>    CONFIG_SPL_FS_FAT=y
>> -# CONFIG_PSCI_RESET is not set
>> -CONFIG_SYS_MEMTEST_START=0x00000000
>> -CONFIG_SYS_MEMTEST_END=0x3fe00000
>> -CONFIG_OPTIMIZE_INLINING=y
>> -CONFIG_SPL_OPTIMIZE_INLINING=y
>> -CONFIG_REMAKE_ELF=y
>> +CONFIG_DISTRO_DEFAULTS=y
>> +CONFIG_FIT=y
>> +CONFIG_SPL_FIT_SIGNATURE=y
>> +CONFIG_SPL_LOAD_FIT=y
>> +CONFIG_SPL_LOAD_FIT_ADDRESS=0x02000000
>> +# CONFIG_USE_SPL_FIT_GENERATOR is not set
>> +CONFIG_QSPI_BOOT=y
>>    CONFIG_BOOTDELAY=5
>>    CONFIG_USE_BOOTARGS=y
>> -CONFIG_BOOTARGS="earlycon"
>> -CONFIG_USE_BOOTCOMMAND=y
>> -CONFIG_BOOTCOMMAND="run fatscript; run mmcload; run linux_qspi_enable; run mmcboot"
>> -CONFIG_SYS_PBSIZE=2085
>> +# CONFIG_SPL_RAW_IMAGE_SUPPORT is not set
>> +CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
>> +CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x300000
>>    CONFIG_SPL_MAX_SIZE=0x40000
>> +CONFIG_SPL_HAS_BSS_LINKER_SECTION=y
>> +CONFIG_SPL_BSS_START_ADDR=0x3ff00000
>>    # CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
>>    CONFIG_SPL_HAVE_INIT_STACK=y
>> +CONFIG_SPL_STACK=0xffe3f000
>>    CONFIG_SPL_SYS_MALLOC=y
>>    CONFIG_SPL_HAS_CUSTOM_MALLOC_START=y
>>    CONFIG_SPL_CUSTOM_SYS_MALLOC_ADDR=0x3fa00000
>>    CONFIG_SPL_SYS_MALLOC_SIZE=0x500000
>> +CONFIG_SPL_BSS_MAX_SIZE=0x100000
>> +CONFIG_BOOTARGS="earlycon panic=-1"
>> +# CONFIG_SPL_RAW_IMAGE_SUPPORT is not set
> 
> 
> Remove one of the two identical # CONFIG_SPL_RAW_IMAGE_SUPPORT lines.
> Regenerate defconfig with make savedefconfig
> 
> 
Noted, I will clean this up with savedefconfig for v3.

>> +CONFIG_SPL_CRC32=y
>> +CONFIG_SPL_MTD=y
>> +CONFIG_SPL_MTD_SUPPORT=y
>>    CONFIG_SPL_SPI_LOAD=y
>> -CONFIG_SYS_SPI_U_BOOT_OFFS=0x3C00000
>> -CONFIG_SPL_TARGET="spl/u-boot-spl-dtb.hex"
>> -CONFIG_HUSH_PARSER=y
>> +CONFIG_SPL_ATF=y
>> +CONFIG_SPL_ATF_NO_PLATFORM_PARAM=y
>>    CONFIG_SYS_PROMPT="SOCFPGA_STRATIX10 # "
>> +CONFIG_CMD_NVEDIT_SELECT=y
>>    CONFIG_CMD_MEMTEST=y
>> +# CONFIG_CMD_FLASH is not set
>>    CONFIG_CMD_GPIO=y
>>    CONFIG_CMD_I2C=y
>>    CONFIG_CMD_MMC=y
>> +CONFIG_CMD_MTD=y
>>    CONFIG_CMD_SPI=y
>>    CONFIG_CMD_USB=y
>> -CONFIG_CMD_WDT=y
>> -CONFIG_CMD_DHCP=y
>> -CONFIG_CMD_MII=y
>> -CONFIG_CMD_PING=y
>>    CONFIG_CMD_CACHE=y
>> -CONFIG_CMD_EXT4=y
>> -CONFIG_CMD_FAT=y
>> -CONFIG_CMD_FS_GENERIC=y
>> -CONFIG_ENV_IS_IN_MMC=y
>> -CONFIG_ENV_RELOC_GD_ENV_ADDR=y
>> -CONFIG_USE_BOOTFILE=y
>> -CONFIG_BOOTFILE="Image"
>> +CONFIG_SPL_SPI_FLASH_MTD=y
>> +CONFIG_SPI_FLASH_MTD=y
>> +# CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
>> +CONFIG_CMD_UBI=y
>> +CONFIG_CMD_UBIFS=y
>> +CONFIG_MTD_UBI=y
>> +CONFIG_MTD_UBI_WL_THRESHOLD=4096
>> +CONFIG_MTD_UBI_BEB_LIMIT=20
>> +# CONFIG_ISO_PARTITION is not set
>> +# CONFIG_EFI_PARTITION is not set
>> +CONFIG_OF_LIST=""
>> +CONFIG_ENV_IS_IN_FAT=y
>> +CONFIG_ENV_IS_IN_UBI=y
>> +CONFIG_ENV_FAT_DEVICE_AND_PART="0:1"
>> +CONFIG_ENV_UBI_PART="root"
>> +CONFIG_ENV_UBI_VOLUME="env"
>> +CONFIG_SYS_RELOC_GD_ENV_ADDR=y
>>    CONFIG_NET_RANDOM_ETHADDR=y
>>    CONFIG_SPL_DM_SEQ_ALIAS=y
>>    CONFIG_SPL_ALTERA_SDRAM=y
>> +CONFIG_FPGA_INTEL_PR=y
>>    CONFIG_DWAPB_GPIO=y
>>    CONFIG_DM_I2C=y
>>    CONFIG_SYS_I2C_DW=y
>> -CONFIG_SYS_MMC_MAX_BLK_COUNT=256
>> +CONFIG_MISC=y
>>    CONFIG_MMC_DW=y
>> +CONFIG_SYS_MMC_MAX_BLK_COUNT=256
>> +CONFIG_MTD=y
>> +CONFIG_DM_MTD=y
>> +CONFIG_SF_DEFAULT_MODE=0x2003
>>    CONFIG_SPI_FLASH_SPANSION=y
>>    CONFIG_SPI_FLASH_STMICRO=y
>>    CONFIG_PHY_MICREL=y
>>    CONFIG_PHY_MICREL_KSZ90X1=y
>> +CONFIG_DM_ETH=y
>>    CONFIG_ETH_DESIGNWARE=y
>>    CONFIG_MII=y
>> +CONFIG_DM_RESET=y
>>    CONFIG_SYS_NS16550_MEM32=y
>>    CONFIG_SPI=y
>>    CONFIG_CADENCE_QSPI=y
>>    CONFIG_DESIGNWARE_SPI=y
>>    CONFIG_USB=y
>> +CONFIG_DM_USB=y
>>    CONFIG_USB_DWC2=y
>> -CONFIG_USB_STORAGE=y
> 
> 
> missing vs ATF defconfig, why dropping this?
> 
> 
> Best regards,
> 
> Tien Fong
> 
This is not needed to be enabled as CONFIG_DISTRO_DEFAULTS is enabled in 
this defconfig.

CONFIG_DISTRO_DEFAULTS -> BOOT_DEFAULTS_FEATURES -> USB_STORAGE in 
boot/Kconfig

Thanks,
Alif




  reply	other threads:[~2026-05-14  5:42 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-28  3:48 [PATCH v2 0/9] SoCFPGA: Update Boot Support for Stratix10 in U-Boot alif.zakuan.yuslaimi
2026-04-28  3:48 ` [PATCH v2 1/9] arch: arm: dts: stratix10: Switch to using upstream Linux DT config alif.zakuan.yuslaimi
2026-05-07  8:37   ` Chee, Tien Fong
2026-05-14  5:35     ` Yuslaimi, Alif Zakuan
2026-05-14  6:45       ` Yuslaimi, Alif Zakuan
2026-04-28  3:48 ` [PATCH v2 2/9] configs: stratix10: Combine defconfig for all boot flashes alif.zakuan.yuslaimi
2026-05-07  9:31   ` Chee, Tien Fong
2026-05-14  5:42     ` Yuslaimi, Alif Zakuan [this message]
2026-04-28  3:48 ` [PATCH v2 3/9] arm: socfpga: Move firmware register settings from source code to device tree alif.zakuan.yuslaimi
2026-04-28  3:48 ` [PATCH v2 4/9] arm: socfpga: Update Stratix10 SPL data save and restore implementation alif.zakuan.yuslaimi
2026-04-28  3:48 ` [PATCH v2 5/9] arm: socfpga: s10: Enable system manager driver for Stratix10 alif.zakuan.yuslaimi
2026-05-08  4:18   ` Chee, Tien Fong
2026-05-14  5:43     ` Yuslaimi, Alif Zakuan
2026-04-28  3:48 ` [PATCH v2 6/9] ddr: altera: soc64: Add secure region support for ATF flow alif.zakuan.yuslaimi
2026-04-28  3:48 ` [PATCH v2 7/9] clk: s10: Refactor S10 clock driver alif.zakuan.yuslaimi
2026-05-08  5:48   ` Chee, Tien Fong
2026-05-14  5:47     ` Yuslaimi, Alif Zakuan
2026-04-28  3:48 ` [PATCH v2 8/9] mmc: socfpga_dw_mmc: Exclude S10 from legacy clkmgr address retrieval alif.zakuan.yuslaimi
2026-04-28  3:48 ` [PATCH v2 9/9] spl: s10: Enhance watchdog support in SPL for Stratix 10 alif.zakuan.yuslaimi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5677e494-21aa-4532-a47d-9d45a6376f44@altera.com \
    --to=alif.zakuan.yuslaimi@altera.com \
    --cc=jh80.chung@samsung.com \
    --cc=kory.maincent@bootlin.com \
    --cc=lukma@denx.de \
    --cc=marex@denx.de \
    --cc=me@ziyao.cc \
    --cc=neil.armstrong@linaro.org \
    --cc=peng.fan@nxp.com \
    --cc=simon.k.r.goldschmidt@gmail.com \
    --cc=sjg@chromium.org \
    --cc=tien.fong.chee@altera.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.