Hi Andrew, On 09:54-20230803, Andrew Davis wrote: > K3 devices have runtime type board detection. Make the default defconfig > include the secure configuration. Then remove the HS specific config. > > Non-HS devices will continue to boot due to runtime device type detection. > > Signed-off-by: Andrew Davis <a...@ti.com> > --- > MAINTAINERS | 2 - > configs/am65x_evm_a53_defconfig | 3 +- > configs/am65x_evm_r5_defconfig | 1 + > configs/am65x_evm_r5_usbdfu_defconfig | 1 + > configs/am65x_evm_r5_usbmsc_defconfig | 1 + > configs/am65x_hs_evm_a53_defconfig | 158 -------------------------- > configs/am65x_hs_evm_r5_defconfig | 135 ---------------------- > 7 files changed, 5 insertions(+), 296 deletions(-) > delete mode 100644 configs/am65x_hs_evm_a53_defconfig > delete mode 100644 configs/am65x_hs_evm_r5_defconfig > > diff --git a/MAINTAINERS b/MAINTAINERS > index 47581cf6fbb..d3b9f4cc6c4 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1506,8 +1506,6 @@ F: configs/k2hk_hs_evm_defconfig > F: configs/k2e_hs_evm_defconfig > F: configs/k2g_hs_evm_defconfig > F: configs/k2l_hs_evm_defconfig > -F: configs/am65x_hs_evm_r5_defconfig > -F: configs/am65x_hs_evm_a53_defconfig
While merging these defconfigs for other boards I had faced an issue where the multi DTB detection wasn't working for HS devices and we were hitting a firewall exception while trying to access the EEPROM Scratchpad before the firewalls were opened. I see that AM65x still uses the old Scratchpad address in the Kconfig which had triggered exceptions for other K3 devices. config SYS_K3_MCU_SCRATCHPAD_BASE hex default 0x40280000 if SOC_K3_AM654 default 0x41cff9fc if SOC_K3_J721S2 default 0x41cff9fc if SOC_K3_J721E Can you double check if this also needs a change along with the merged defconfigs? Regards, Manorit > > TPM DRIVERS > M: Ilias Apalodimas <ilias.apalodi...@linaro.org> > diff --git a/configs/am65x_evm_a53_defconfig b/configs/am65x_evm_a53_defconfig > index 4301553af8d..55c1b23c8db 100644 > --- a/configs/am65x_evm_a53_defconfig > +++ b/configs/am65x_evm_a53_defconfig > @@ -1,6 +1,7 @@ > CONFIG_ARM=y > CONFIG_SKIP_LOWLEVEL_INIT=y > CONFIG_ARCH_K3=y > +CONFIG_TI_SECURE_DEVICE=y > CONFIG_SYS_MALLOC_LEN=0x2000000 > CONFIG_SYS_MALLOC_F_LEN=0x8000 > CONFIG_SPL_LIBCOMMON_SUPPORT=y > @@ -34,7 +35,7 @@ CONFIG_SPL_LOAD_FIT=y > CONFIG_SPL_LOAD_FIT_ADDRESS=0x81000000 > CONFIG_OF_SYSTEM_SETUP=y > CONFIG_DISTRO_DEFAULTS=y > -CONFIG_BOOTCOMMAND="run findfdt; run distro_bootcmd; run init_${boot}; run > boot_rprocs; run get_kern_${boot}; run get_fdt_${boot}; run > get_overlay_${boot}; run run_kern" > +CONFIG_BOOTCOMMAND="run findfdt; run distro_bootcmd; run init_${boot}; run > boot_rprocs; if test ${boot_fit} -eq 1; then run get_fit_${boot}; run > get_overlaystring; run run_fit; else; run get_kern_${boot}; run > get_fdt_${boot}; run get_overlay_${boot}; run run_kern; fi;" > CONFIG_LOGLEVEL=7 > CONFIG_CONSOLE_MUX=y > CONFIG_SPL_MAX_SIZE=0x58000 > diff --git a/configs/am65x_evm_r5_defconfig b/configs/am65x_evm_r5_defconfig > index d5196c2024e..b9b69b921be 100644 > --- a/configs/am65x_evm_r5_defconfig > +++ b/configs/am65x_evm_r5_defconfig > @@ -1,5 +1,6 @@ > CONFIG_ARM=y > CONFIG_ARCH_K3=y > +CONFIG_TI_SECURE_DEVICE=y > CONFIG_SYS_MALLOC_LEN=0x2000000 > CONFIG_SYS_MALLOC_F_LEN=0x55000 > CONFIG_SPL_GPIO=y > diff --git a/configs/am65x_evm_r5_usbdfu_defconfig > b/configs/am65x_evm_r5_usbdfu_defconfig > index 88f68aa70e4..fb93024bda9 100644 > --- a/configs/am65x_evm_r5_usbdfu_defconfig > +++ b/configs/am65x_evm_r5_usbdfu_defconfig > @@ -1,5 +1,6 @@ > CONFIG_ARM=y > CONFIG_ARCH_K3=y > +CONFIG_TI_SECURE_DEVICE=y > CONFIG_SYS_MALLOC_LEN=0x2000000 > CONFIG_SYS_MALLOC_F_LEN=0x55000 > CONFIG_SPL_GPIO=y > diff --git a/configs/am65x_evm_r5_usbmsc_defconfig > b/configs/am65x_evm_r5_usbmsc_defconfig > index 8da49c78c82..931b756ab52 100644 > --- a/configs/am65x_evm_r5_usbmsc_defconfig > +++ b/configs/am65x_evm_r5_usbmsc_defconfig > @@ -1,5 +1,6 @@ > CONFIG_ARM=y > CONFIG_ARCH_K3=y > +CONFIG_TI_SECURE_DEVICE=y > CONFIG_SYS_MALLOC_LEN=0x2000000 > CONFIG_SYS_MALLOC_F_LEN=0x55000 > CONFIG_SPL_GPIO=y > diff --git a/configs/am65x_hs_evm_a53_defconfig > b/configs/am65x_hs_evm_a53_defconfig > deleted file mode 100644 > index 3f9626e4550..00000000000 > --- a/configs/am65x_hs_evm_a53_defconfig > +++ /dev/null > @@ -1,158 +0,0 @@ > -CONFIG_ARM=y > -CONFIG_SKIP_LOWLEVEL_INIT=y > -CONFIG_ARCH_K3=y > -CONFIG_TI_SECURE_DEVICE=y > -CONFIG_SYS_MALLOC_LEN=0x2000000 > -CONFIG_SYS_MALLOC_F_LEN=0x8000 > -CONFIG_SPL_LIBCOMMON_SUPPORT=y > -CONFIG_SPL_LIBGENERIC_SUPPORT=y > -CONFIG_NR_DRAM_BANKS=2 > -CONFIG_SOC_K3_AM654=y > -CONFIG_TARGET_AM654_A53_EVM=y > -CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y > -CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80480000 > -CONFIG_ENV_SIZE=0x20000 > -CONFIG_ENV_OFFSET=0x680000 > -CONFIG_DM_GPIO=y > -CONFIG_SPL_DM_SPI=y > -CONFIG_DEFAULT_DEVICE_TREE="k3-am654-base-board" > -CONFIG_SPL_TEXT_BASE=0x80080000 > -CONFIG_OF_LIBFDT_OVERLAY=y > -CONFIG_DM_RESET=y > -CONFIG_SPL_MMC=y > -CONFIG_SPL_SERIAL=y > -CONFIG_SPL_DRIVERS_MISC=y > -CONFIG_SPL_STACK_R_ADDR=0x82000000 > -CONFIG_ENV_OFFSET_REDUND=0x6A0000 > -CONFIG_SPL_FS_FAT=y > -CONFIG_SPL_LIBDISK_SUPPORT=y > -CONFIG_SPL_SPI_FLASH_SUPPORT=y > -CONFIG_SPL_SPI=y > -# CONFIG_PSCI_RESET is not set > -CONFIG_PCI=y > -# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set > -CONFIG_SPL_LOAD_FIT=y > -CONFIG_OF_SYSTEM_SETUP=y > -CONFIG_DISTRO_DEFAULTS=y > -CONFIG_BOOTCOMMAND="run findfdt; run envboot; run init_${boot}; run > get_fit_${boot}; run get_overlaystring; run run_fit" > -CONFIG_LOGLEVEL=7 > -CONFIG_CONSOLE_MUX=y > -CONFIG_SPL_MAX_SIZE=0x58000 > -CONFIG_SPL_HAS_BSS_LINKER_SECTION=y > -CONFIG_SPL_BSS_START_ADDR=0x80a00000 > -CONFIG_SPL_BSS_MAX_SIZE=0x80000 > -CONFIG_SPL_SYS_MALLOC_SIMPLE=y > -CONFIG_SPL_STACK_R=y > -CONFIG_SYS_SPL_MALLOC=y > -CONFIG_SYS_SPL_MALLOC_SIZE=0x800000 > -CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y > -CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x1400 > -CONFIG_SPL_DMA=y > -CONFIG_SPL_FS_LOAD_PAYLOAD_NAME="u-boot.img" > -CONFIG_SPL_I2C=y > -CONFIG_SPL_DM_MAILBOX=y > -CONFIG_SPL_MTD_SUPPORT=y > -CONFIG_SPL_DM_SPI_FLASH=y > -CONFIG_SPL_DM_RESET=y > -CONFIG_SPL_POWER_DOMAIN=y > -# CONFIG_SPL_SPI_FLASH_TINY is not set > -CONFIG_SPL_SPI_FLASH_SFDP_SUPPORT=y > -CONFIG_SPL_SPI_LOAD=y > -CONFIG_SYS_SPI_U_BOOT_OFFS=0x280000 > -CONFIG_SPL_YMODEM_SUPPORT=y > -CONFIG_SYS_MAXARGS=64 > -CONFIG_CMD_ASKENV=y > -CONFIG_CMD_DFU=y > -CONFIG_CMD_GPT=y > -CONFIG_CMD_I2C=y > -CONFIG_CMD_MMC=y > -CONFIG_CMD_PCI=y > -CONFIG_CMD_REMOTEPROC=y > -CONFIG_CMD_USB=y > -CONFIG_CMD_TIME=y > -CONFIG_MTDIDS_DEFAULT="nor0=47040000.spi.0" > -CONFIG_MTDPARTS_DEFAULT="mtdparts=47040000.spi.0:512k(ospi.tiboot3),2m(ospi.tispl),4m(ospi.u-boot),128k(ospi.env),128k(ospi.env.backup),1m(ospi.sysfw),-@8m(ospi.rootfs)" > -CONFIG_CMD_UBI=y > -# CONFIG_ISO_PARTITION is not set > -CONFIG_OF_CONTROL=y > -CONFIG_SPL_OF_CONTROL=y > -CONFIG_SPL_MULTI_DTB_FIT=y > -CONFIG_SPL_MULTI_DTB_FIT_NO_COMPRESSION=y > -CONFIG_ENV_OVERWRITE=y > -CONFIG_ENV_IS_IN_MMC=y > -CONFIG_SYS_REDUNDAND_ENVIRONMENT=y > -CONFIG_SYS_RELOC_GD_ENV_ADDR=y > -CONFIG_SYS_MMC_ENV_PART=1 > -CONFIG_NET_RANDOM_ETHADDR=y > -CONFIG_SPL_DM=y > -CONFIG_SPL_DM_SEQ_ALIAS=y > -CONFIG_SPL_REGMAP=y > -CONFIG_SPL_OF_TRANSLATE=y > -CONFIG_CLK=y > -CONFIG_SPL_CLK=y > -CONFIG_CLK_TI_SCI=y > -CONFIG_DFU_MMC=y > -CONFIG_DFU_RAM=y > -CONFIG_DFU_SF=y > -CONFIG_SYS_DFU_DATA_BUF_SIZE=0x20000 > -CONFIG_SYS_DFU_MAX_FILE_SIZE=0x800000 > -CONFIG_DMA_CHANNELS=y > -CONFIG_TI_K3_NAVSS_UDMA=y > -CONFIG_TI_SCI_PROTOCOL=y > -CONFIG_DM_PCA953X=y > -CONFIG_DM_I2C=y > -CONFIG_I2C_SET_DEFAULT_BUS_NUM=y > -CONFIG_SYS_I2C_OMAP24XX=y > -CONFIG_DM_MAILBOX=y > -CONFIG_K3_SEC_PROXY=y > -CONFIG_MMC_SDHCI=y > -CONFIG_MMC_SDHCI_ADMA=y > -CONFIG_SPL_MMC_SDHCI_ADMA=y > -CONFIG_MMC_SDHCI_AM654=y > -CONFIG_MTD=y > -CONFIG_DM_MTD=y > -CONFIG_DM_SPI_FLASH=y > -CONFIG_SPI_FLASH_SFDP_SUPPORT=y > -CONFIG_SPI_FLASH_STMICRO=y > -# CONFIG_SPI_FLASH_USE_4K_SECTORS is not set > -CONFIG_SPI_FLASH_MTD=y > -CONFIG_PHY_TI_DP83867=y > -CONFIG_PHY_FIXED=y > -CONFIG_E1000=y > -CONFIG_CMD_E1000=y > -CONFIG_TI_AM65_CPSW_NUSS=y > -CONFIG_PCI_KEYSTONE=y > -CONFIG_PHY=y > -CONFIG_AM654_PHY=y > -CONFIG_OMAP_USB2_PHY=y > -CONFIG_PINCTRL=y > -# CONFIG_PINCTRL_GENERIC is not set > -CONFIG_SPL_PINCTRL=y > -# CONFIG_SPL_PINCTRL_GENERIC is not set > -CONFIG_PINCTRL_SINGLE=y > -CONFIG_POWER_DOMAIN=y > -CONFIG_TI_SCI_POWER_DOMAIN=y > -CONFIG_REMOTEPROC_TI_K3_R5F=y > -CONFIG_RESET_TI_SCI=y > -CONFIG_DM_SERIAL=y > -CONFIG_SOC_DEVICE=y > -CONFIG_SOC_DEVICE_TI_K3=y > -CONFIG_SOC_TI=y > -CONFIG_SPI=y > -CONFIG_DM_SPI=y > -CONFIG_CADENCE_QSPI=y > -CONFIG_SYSRESET=y > -CONFIG_SPL_SYSRESET=y > -CONFIG_SYSRESET_TI_SCI=y > -CONFIG_USB=y > -CONFIG_DM_USB_GADGET=y > -CONFIG_USB_XHCI_HCD=y > -CONFIG_USB_XHCI_DWC3=y > -CONFIG_USB_DWC3=y > -CONFIG_USB_DWC3_GENERIC=y > -CONFIG_USB_KEYBOARD=y > -CONFIG_USB_GADGET=y > -CONFIG_USB_GADGET_MANUFACTURER="Texas Instruments" > -CONFIG_USB_GADGET_VENDOR_NUM=0x0451 > -CONFIG_USB_GADGET_PRODUCT_NUM=0x6162 > -CONFIG_USB_GADGET_DOWNLOAD=y > diff --git a/configs/am65x_hs_evm_r5_defconfig > b/configs/am65x_hs_evm_r5_defconfig > deleted file mode 100644 > index 0c7b53ca2c0..00000000000 > --- a/configs/am65x_hs_evm_r5_defconfig > +++ /dev/null > @@ -1,135 +0,0 @@ > -CONFIG_ARM=y > -CONFIG_ARCH_K3=y > -CONFIG_TI_SECURE_DEVICE=y > -CONFIG_SYS_MALLOC_LEN=0x2000000 > -CONFIG_SYS_MALLOC_F_LEN=0x55000 > -CONFIG_SPL_GPIO=y > -CONFIG_SPL_LIBCOMMON_SUPPORT=y > -CONFIG_SPL_LIBGENERIC_SUPPORT=y > -CONFIG_NR_DRAM_BANKS=2 > -CONFIG_SOC_K3_AM654=y > -CONFIG_K3_EARLY_CONS=y > -CONFIG_TARGET_AM654_R5_EVM=y > -CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y > -CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x41c7effc > -CONFIG_ENV_SIZE=0x20000 > -CONFIG_DM_GPIO=y > -CONFIG_SPL_DM_SPI=y > -CONFIG_DEFAULT_DEVICE_TREE="k3-am654-r5-base-board" > -CONFIG_SPL_TEXT_BASE=0x41c00000 > -CONFIG_DM_RESET=y > -CONFIG_SPL_MMC=y > -CONFIG_SPL_SERIAL=y > -CONFIG_SPL_DRIVERS_MISC=y > -CONFIG_SPL_STACK_R_ADDR=0x82000000 > -CONFIG_SPL_FS_FAT=y > -CONFIG_SPL_LIBDISK_SUPPORT=y > -CONFIG_SPL_SPI_FLASH_SUPPORT=y > -CONFIG_SPL_SPI=y > -# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set > -CONFIG_SPL_LOAD_FIT=y > -CONFIG_USE_BOOTCOMMAND=y > -# CONFIG_DISPLAY_CPUINFO is not set > -CONFIG_SPL_MAX_SIZE=0x58000 > -CONFIG_SPL_HAS_BSS_LINKER_SECTION=y > -CONFIG_SPL_BSS_START_ADDR=0x41c7effc > -CONFIG_SPL_BSS_MAX_SIZE=0xc00 > -CONFIG_SPL_STACK_R=y > -CONFIG_SPL_SEPARATE_BSS=y > -CONFIG_SYS_SPL_MALLOC=y > -CONFIG_HAS_CUSTOM_SPL_MALLOC_START=y > -CONFIG_CUSTOM_SYS_SPL_MALLOC_ADDR=0x84000000 > -CONFIG_SYS_SPL_MALLOC_SIZE=0x1000000 > -CONFIG_SPL_EARLY_BSS=y > -CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y > -CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x400 > -CONFIG_SPL_DMA=y > -CONFIG_SPL_I2C=y > -CONFIG_SPL_DM_MAILBOX=y > -CONFIG_SPL_DM_SPI_FLASH=y > -CONFIG_SPL_DM_RESET=y > -CONFIG_SPL_POWER_DOMAIN=y > -CONFIG_SPL_RAM_SUPPORT=y > -CONFIG_SPL_RAM_DEVICE=y > -CONFIG_SPL_REMOTEPROC=y > -# CONFIG_SPL_SPI_FLASH_TINY is not set > -CONFIG_SPL_SPI_FLASH_SFDP_SUPPORT=y > -CONFIG_SPL_SPI_LOAD=y > -CONFIG_SYS_SPI_U_BOOT_OFFS=0x80000 > -CONFIG_SPL_YMODEM_SUPPORT=y > -CONFIG_HUSH_PARSER=y > -CONFIG_SYS_MAXARGS=64 > -CONFIG_CMD_BOOTZ=y > -CONFIG_SYS_BOOTM_LEN=0x4000000 > -CONFIG_CMD_ASKENV=y > -CONFIG_CMD_GPT=y > -CONFIG_CMD_I2C=y > -CONFIG_CMD_MMC=y > -CONFIG_CMD_REMOTEPROC=y > -# CONFIG_CMD_SETEXPR is not set > -CONFIG_CMD_TIME=y > -CONFIG_CMD_FAT=y > -CONFIG_OF_CONTROL=y > -CONFIG_SPL_OF_CONTROL=y > -CONFIG_SPL_MULTI_DTB_FIT=y > -CONFIG_SPL_MULTI_DTB_FIT_NO_COMPRESSION=y > -CONFIG_ENV_OVERWRITE=y > -CONFIG_ENV_IS_IN_FAT=y > -CONFIG_ENV_FAT_DEVICE_AND_PART="1:1" > -CONFIG_SYS_RELOC_GD_ENV_ADDR=y > -CONFIG_SPL_DM=y > -CONFIG_SPL_DM_SEQ_ALIAS=y > -CONFIG_REGMAP=y > -CONFIG_SPL_REGMAP=y > -CONFIG_SPL_OF_TRANSLATE=y > -CONFIG_CLK=y > -CONFIG_SPL_CLK=y > -CONFIG_CLK_TI_SCI=y > -CONFIG_DMA_CHANNELS=y > -CONFIG_TI_K3_NAVSS_UDMA=y > -CONFIG_TI_SCI_PROTOCOL=y > -CONFIG_DA8XX_GPIO=y > -CONFIG_DM_I2C=y > -CONFIG_I2C_SET_DEFAULT_BUS_NUM=y > -CONFIG_SYS_I2C_OMAP24XX=y > -CONFIG_DM_MAILBOX=y > -CONFIG_K3_SEC_PROXY=y > -CONFIG_K3_AVS0=y > -CONFIG_MMC_SDHCI=y > -CONFIG_SPL_MMC_SDHCI_ADMA=y > -CONFIG_MMC_SDHCI_AM654=y > -CONFIG_DM_SPI_FLASH=y > -CONFIG_SPI_FLASH_SFDP_SUPPORT=y > -CONFIG_SPI_FLASH_STMICRO=y > -# CONFIG_SPI_FLASH_USE_4K_SECTORS is not set > -CONFIG_PINCTRL=y > -# CONFIG_PINCTRL_GENERIC is not set > -CONFIG_SPL_PINCTRL=y > -# CONFIG_SPL_PINCTRL_GENERIC is not set > -CONFIG_PINCTRL_SINGLE=y > -CONFIG_POWER_DOMAIN=y > -CONFIG_TI_SCI_POWER_DOMAIN=y > -CONFIG_DM_REGULATOR=y > -CONFIG_SPL_DM_REGULATOR=y > -CONFIG_DM_REGULATOR_GPIO=y > -CONFIG_SPL_DM_REGULATOR_GPIO=y > -CONFIG_DM_REGULATOR_TPS62360=y > -CONFIG_RAM=y > -CONFIG_SPL_RAM=y > -CONFIG_K3_SYSTEM_CONTROLLER=y > -CONFIG_REMOTEPROC_TI_K3_ARM64=y > -CONFIG_RESET_TI_SCI=y > -CONFIG_DM_SERIAL=y > -CONFIG_SOC_DEVICE=y > -CONFIG_SOC_DEVICE_TI_K3=y > -CONFIG_SOC_TI=y > -CONFIG_SPI=y > -CONFIG_DM_SPI=y > -CONFIG_CADENCE_QSPI=y > -CONFIG_SYSRESET=y > -CONFIG_SPL_SYSRESET=y > -CONFIG_SYSRESET_TI_SCI=y > -CONFIG_TIMER=y > -CONFIG_SPL_TIMER=y > -CONFIG_OMAP_TIMER=y > -CONFIG_FS_FAT_MAX_CLUSTSIZE=16384 > -- > 2.39.2 >