On Sun, 5 Mar 2023 at 12:29, Pali Rohár <p...@kernel.org> wrote: > On Sunday 05 March 2023 21:30:59 Martin Rowe wrote: > > This new clearfog_spi_defconfig file is a copy of existing > > clearfog_defconfig file modified to instruct build system to generate > > final kwbimage for SPI booting and to store the environment in SPI. > > > > Signed-off-by: Martin Rowe <martin.p.r...@gmail.com> > > --- > > configs/{clearfog_sata_defconfig => clearfog_spi_defconfig} | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > copy configs/{clearfog_sata_defconfig => clearfog_spi_defconfig} (96%) > > > > diff --git a/configs/clearfog_sata_defconfig > b/configs/clearfog_spi_defconfig > > similarity index 96% > > copy from configs/clearfog_sata_defconfig > > copy to configs/clearfog_spi_defconfig > > index b11d8746b0..d6695f31e6 100644 > > --- a/configs/clearfog_sata_defconfig > > +++ b/configs/clearfog_spi_defconfig > > @@ -10,7 +10,7 @@ CONFIG_NR_DRAM_BANKS=2 > > CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y > > CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0xff0000 > > CONFIG_TARGET_CLEARFOG=y > > -CONFIG_MVEBU_SPL_BOOT_DEVICE_SATA=y > > +CONFIG_ENV_SECT_SIZE=0x10000 > > CONFIG_DEFAULT_DEVICE_TREE="armada-388-clearfog" > > CONFIG_SPL_TEXT_BASE=0x40000030 > > CONFIG_SPL_MMC=y > > @@ -19,6 +19,7 @@ CONFIG_SPL_STACK=0x4002c000 > > CONFIG_SPL=y > > CONFIG_DEBUG_UART_BASE=0xf1012000 > > CONFIG_DEBUG_UART_CLOCK=250000000 > > +CONFIG_SPL_LIBDISK_SUPPORT=y > > I think you do not need libdisk support in SPL which loads main u-boot > from SPI. >
Makes sense, I'll remove > > CONFIG_SYS_LOAD_ADDR=0x800000 > > CONFIG_DEBUG_UART=y > > CONFIG_AHCI=y > > @@ -48,6 +49,7 @@ CONFIG_CMD_USB=y > > CONFIG_CMD_TFTPPUT=y > > CONFIG_CMD_CACHE=y > > CONFIG_CMD_TIME=y > > +CONFIG_MVEBU_SATA_BOOT=y > > Should not be default bubt location SPI for SPI booting? > Yes, I'll fix > CONFIG_ENV_OVERWRITE=y > > CONFIG_ENV_MIN_ENTRIES=128 > > CONFIG_ARP_TIMEOUT=200 > > -- > > 2.39.2 > > >