[PATCH v2 17/17] configs: starfive: add starfive_visionfive2_defconfig

[Heinrich Schuchardt]

On 1/18/23 09:11, Yanhong Wang wrote:
> This is the initial basic config for StarFive VisionFive v2 board. It
> includes consol, Norflash, sdio, ddr etc.
> 
> Signed-off-by: Yanhong Wang <yanhong.wang at starfivetech.com>
> ---
>   configs/starfive_visionfive2_defconfig | 72 ++++++++++++++++++++++++++
>   1 file changed, 72 insertions(+)
>   create mode 100644 configs/starfive_visionfive2_defconfig
> 
> diff --git a/configs/starfive_visionfive2_defconfig b/configs/starfive_visionfive2_defconfig
> new file mode 100644
> index 0000000000..54ae6b2a43
> --- /dev/null
> +++ b/configs/starfive_visionfive2_defconfig
> @@ -0,0 +1,72 @@
> +CONFIG_RISCV=y
> +CONFIG_SYS_MALLOC_LEN=0x800000
> +CONFIG_SYS_MALLOC_F_LEN=0x8000
> +CONFIG_NR_DRAM_BANKS=1
> +CONFIG_SPL_DM_SPI=y
> +CONFIG_DEFAULT_DEVICE_TREE="starfive_visionfive2"
> +CONFIG_SPL_TEXT_BASE=0x8000000
> +CONFIG_SYS_PROMPT="StarFive #"
> +CONFIG_SPL_MMC=y
> +CONFIG_SPL=y
> +CONFIG_SPL_SPI_FLASH_SUPPORT=y
> +CONFIG_SPL_SPI=y
> +CONFIG_SYS_LOAD_ADDR=0x82000000
> +CONFIG_TARGET_STARFIVE_VISIONFIVE2=y
> +CONFIG_SPL_OPENSBI_LOAD_ADDR=0x40000000
> +CONFIG_ARCH_RV64I=y
> +CONFIG_CMODEL_MEDANY=y
> +CONFIG_RISCV_SMODE=y
> +CONFIG_DISTRO_DEFAULTS=y
> +CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
> +CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80000000
> +CONFIG_FIT=y
> +CONFIG_USE_BOOTARGS=y
> +CONFIG_BOOTARGS="console=ttyS0,115200 debug rootwait earlycon=sbi"
> +CONFIG_USE_PREBOOT=y
> +CONFIG_PREBOOT="setenv fdt_addr ${fdtcontroladdr};fdt addr ${fdtcontroladdr};"
> +CONFIG_DEFAULT_FDT_FILE="starfive/starfive_visionfive2.dtb"
> +CONFIG_DISPLAY_CPUINFO=y
> +CONFIG_DISPLAY_BOARDINFO=y
> +CONFIG_SPL_MAX_SIZE=0x40000
> +CONFIG_SPL_PAD_TO=0x0
> +CONFIG_SPL_BSS_START_ADDR=0x8040000
> +CONFIG_SPL_BSS_MAX_SIZE=0x10000
> +# CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
> +CONFIG_SPL_STACK=0x8180000
> +CONFIG_SYS_SPL_MALLOC=y
> +CONFIG_HAS_CUSTOM_SPL_MALLOC_START=y
> +CONFIG_CUSTOM_SYS_SPL_MALLOC_ADDR=0x80000000
> +CONFIG_SYS_SPL_MALLOC_SIZE=0x400000
> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_PARTITION=y

With this patch we get CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_PARTITION=1 
which means that partition 1 on the SD-card is used for main U-Boot. 
This conflicts with the usage of partition number 1 in the Ubuntu 
installer images.

The vendor U-Boot uses CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_PARTITION=2. 
This is not flexible either. But at least it would be compatible with 
the vendor U-Boot.

I think we should extend mmc_load_image_raw_partition() to allow 
specifying a partition type GUID identifying the partition harboring 
main U-Boot. I will send a patch for this.

This partition type GUID must not be 
2E54B353-1271-4842-806F-E436D6AF6985 as that GUID is used to identify SPL.

Best regards

Heinrich


> +CONFIG_SPL_DM_SPI_FLASH=y
> +CONFIG_SPL_DM_RESET=y
> +CONFIG_SPL_SPI_LOAD=y
> +CONFIG_SYS_CBSIZE=256
> +CONFIG_SYS_PBSIZE=276
> +CONFIG_SYS_BOOTM_LEN=0x4000000
> +CONFIG_CMD_MEMINFO=y
> +CONFIG_CMD_TFTPPUT=y
> +CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> +CONFIG_SPL_CLK_COMPOSITE_CCF=y
> +CONFIG_CLK_CCF=y
> +CONFIG_CLK_COMPOSITE_CCF=y
> +CONFIG_SPL_CLK_JH7110=y
> +# CONFIG_I2C is not set
> +CONFIG_MMC_HS400_SUPPORT=y
> +CONFIG_SPL_MMC_HS400_SUPPORT=y
> +CONFIG_MMC_DW=y
> +CONFIG_MMC_DW_SNPS=y
> +CONFIG_SF_DEFAULT_SPEED=100000000
> +CONFIG_SPI_FLASH_EON=y
> +CONFIG_SPI_FLASH_GIGADEVICE=y
> +CONFIG_SPI_FLASH_ISSI=y
> +CONFIG_SPI_FLASH_MACRONIX=y
> +CONFIG_PINCTRL=y
> +CONFIG_PINCONF=y
> +CONFIG_PINCTRL_STARFIVE=y
> +# CONFIG_RAM_SIFIVE is not set
> +CONFIG_DM_RESET=y
> +CONFIG_SYS_NS16550=y
> +CONFIG_CADENCE_QSPI=y
> +CONFIG_TIMER_EARLY=y
> +CONFIG_OF_LIBFDT_OVERLAY=y