Hi Lukas Auer, Lukas <lukas.a...@aisec.fraunhofer.de> 於 2019年4月26日 週五 上午4:56寫道: > > Hi Rick, > > On Wed, 2019-04-24 at 14:33 +0800, Andes wrote: > > From: Rick Chen <r...@andestech.com> > > > > Add two defconfigs to support AE350 SMP booting from flash. > > > > Signed-off-by: Rick Chen <r...@andestech.com> > > Cc: Greentime Hu <greent...@andestech.com> > > --- > > configs/ae350_rv32_xip_defconfig | 36 ++++++++++++++++++++++++++++++++++++ > > configs/ae350_rv64_xip_defconfig | 37 +++++++++++++++++++++++++++++++++++++ > > 2 files changed, 73 insertions(+) > > create mode 100644 configs/ae350_rv32_xip_defconfig > > create mode 100644 configs/ae350_rv64_xip_defconfig > > > > diff --git a/configs/ae350_rv32_xip_defconfig > > b/configs/ae350_rv32_xip_defconfig > > new file mode 100644 > > index 0000000..7c46769 > > --- /dev/null > > +++ b/configs/ae350_rv32_xip_defconfig > > @@ -0,0 +1,36 @@ > > +CONFIG_RISCV=y > > +CONFIG_SYS_TEXT_BASE=0x80000000 > > +CONFIG_XIP=y > > +CONFIG_TARGET_AX25_AE350=y > > +CONFIG_DISTRO_DEFAULTS=y > > +CONFIG_NR_DRAM_BANKS=2 > > +CONFIG_FIT=y > > +CONFIG_BOOTDELAY=3 > > +CONFIG_BOARD_EARLY_INIT_F=y > > +CONFIG_SYS_PROMPT="RISC-V # " > > +CONFIG_CMD_IMLS=y > > +CONFIG_CMD_MMC=y > > +CONFIG_CMD_SF=y > > +CONFIG_CMD_SF_TEST=y > > +# CONFIG_CMD_SETEXPR is not set > > +CONFIG_BOOTP_PREFER_SERVERIP=y > > +CONFIG_CMD_CACHE=y > > +CONFIG_OF_BOARD=y > > +CONFIG_DEFAULT_DEVICE_TREE="ae350_32" > > +CONFIG_ENV_IS_IN_SPI_FLASH=y > > +CONFIG_NET_RANDOM_ETHADDR=y > > +CONFIG_MMC=y > > +CONFIG_FTSDC010=y > > +CONFIG_FTSDC010_SDIO=y > > +CONFIG_MTD_NOR_FLASH=y > > +CONFIG_FLASH_CFI_DRIVER=y > > +CONFIG_CFI_FLASH=y > > +CONFIG_SYS_FLASH_USE_BUFFER_WRITE=y > > +CONFIG_SYS_FLASH_CFI=y > > +CONFIG_SPI_FLASH=y > > +CONFIG_SPI_FLASH_MACRONIX=y > > +CONFIG_FTMAC100=y > > +CONFIG_BAUDRATE=38400 > > +CONFIG_SYS_NS16550=y > > +CONFIG_SPI=y > > +CONFIG_ATCSPI200_SPI=y > > diff --git a/configs/ae350_rv64_xip_defconfig > > b/configs/ae350_rv64_xip_defconfig > > new file mode 100644 > > index 0000000..67633d6 > > --- /dev/null > > +++ b/configs/ae350_rv64_xip_defconfig > > @@ -0,0 +1,37 @@ > > +CONFIG_RISCV=y > > +CONFIG_SYS_TEXT_BASE=0x80000000 > > +CONFIG_XIP=y > > +CONFIG_TARGET_AX25_AE350=y > > +CONFIG_ARCH_RV64I=y > > +CONFIG_DISTRO_DEFAULTS=y > > +CONFIG_NR_DRAM_BANKS=2 > > +CONFIG_FIT=y > > +CONFIG_BOOTDELAY=3 > > +CONFIG_BOARD_EARLY_INIT_F=y > > +CONFIG_SYS_PROMPT="RISC-V # " > > +CONFIG_CMD_IMLS=y > > +CONFIG_CMD_MMC=y > > +CONFIG_CMD_SF=y > > +CONFIG_CMD_SF_TEST=y > > +# CONFIG_CMD_SETEXPR is not set > > +CONFIG_BOOTP_PREFER_SERVERIP=y > > +CONFIG_CMD_CACHE=y > > +CONFIG_OF_BOARD=y > > +CONFIG_DEFAULT_DEVICE_TREE="ae350_64" > > +CONFIG_ENV_IS_IN_SPI_FLASH=y > > +CONFIG_NET_RANDOM_ETHADDR=y > > +CONFIG_MMC=y > > +CONFIG_FTSDC010=y > > +CONFIG_FTSDC010_SDIO=y > > +CONFIG_MTD_NOR_FLASH=y > > +CONFIG_FLASH_CFI_DRIVER=y > > +CONFIG_CFI_FLASH=y > > +CONFIG_SYS_FLASH_USE_BUFFER_WRITE=y > > +CONFIG_SYS_FLASH_CFI=y > > +CONFIG_SPI_FLASH=y > > +CONFIG_SPI_FLASH_MACRONIX=y > > +CONFIG_FTMAC100=y > > +CONFIG_BAUDRATE=38400 > > +CONFIG_SYS_NS16550=y > > +CONFIG_SPI=y > > +CONFIG_ATCSPI200_SPI=y > > The non-xip defconfigs also define CONFIG_SF_DEFAULT_MODE as 0. Is this > missing here or not needed in the xip configuration? >
Yes. I shall keep this. Thanks Rick > Looks good otherwise. > Reviewed-by: Lukas Auer <lukas.a...@aisec.fraunhofer.de> > > Thanks, > Lukas _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot