On Tue, Feb 28, 2017 at 11:47:01AM -0600, Andrew F. Davis wrote: > On 02/27/2017 09:19 AM, Tom Rini wrote: > > On Fri, Feb 24, 2017 at 06:59:45AM -0600, Andrew F. Davis wrote: > > > >> From: Vitaly Andrianov <vita...@ti.com> > >> > >> TI K2E secure devices have to be built with TI_SECURE_DEVICE, FIT, and > >> FIT_IMAGE_POST_PROCESS enabled. Add a dedicated defconfig for this. > >> > >> Signed-off-by: Vitaly Andrianov <vita...@ti.com> > >> Signed-off-by: Madan Srinivas <mad...@ti.com> > >> Signed-off-by: Andrew F. Davis <a...@ti.com> > >> --- > >> configs/k2e_hs_evm_defconfig | 51 > >> ++++++++++++++++++++++++++++++++++++++++++++ > >> 1 file changed, 51 insertions(+) > >> create mode 100644 configs/k2e_hs_evm_defconfig > >> > >> diff --git a/configs/k2e_hs_evm_defconfig b/configs/k2e_hs_evm_defconfig > >> new file mode 100644 > >> index 0000000000..d515cedaca > >> --- /dev/null > >> +++ b/configs/k2e_hs_evm_defconfig > >> @@ -0,0 +1,51 @@ > >> +CONFIG_ARM=y > >> +CONFIG_ARCH_KEYSTONE=y > >> +CONFIG_SYS_TEXT_BASE=0x0c000060 > >> +CONFIG_TARGET_K2E_EVM=y > >> +CONFIG_TI_SECURE_DEVICE=y > >> +CONFIG_DEFAULT_DEVICE_TREE="keystone-k2e-evm" > >> +CONFIG_FIT=y > >> +CONFIG_FIT_IMAGE_POST_PROCESS=y > >> +CONFIG_OF_BOARD_SETUP=y > >> +CONFIG_SYS_CONSOLE_INFO_QUIET=y > >> +CONFIG_VERSION_VARIABLE=y > >> +CONFIG_HUSH_PARSER=y > >> +CONFIG_SYS_PROMPT="K2E HS EVM # " > >> +CONFIG_CMD_BOOTZ=y > >> +# CONFIG_CMD_IMLS is not set > >> +CONFIG_CMD_ASKENV=y > >> +# CONFIG_CMD_FLASH is not set > >> +CONFIG_CMD_NAND=y > >> +CONFIG_CMD_PART=y > >> +CONFIG_CMD_SF=y > >> +CONFIG_CMD_SPI=y > >> +CONFIG_CMD_I2C=y > >> +CONFIG_CMD_USB=y > >> +# CONFIG_CMD_SETEXPR is not set > >> +CONFIG_CMD_DHCP=y > >> +CONFIG_CMD_MII=y > >> +CONFIG_CMD_PING=y > >> +CONFIG_CMD_EXT2=y > >> +CONFIG_CMD_EXT4=y > >> +CONFIG_CMD_EXT4_WRITE=y > >> +CONFIG_CMD_FAT=y > >> +CONFIG_CMD_FS_GENERIC=y > >> +CONFIG_CMD_UBI=y > >> +CONFIG_ISO_PARTITION=y > >> +CONFIG_EFI_PARTITION=y > >> +CONFIG_OF_CONTROL=y > >> +CONFIG_NET_RANDOM_ETHADDR=y > >> +CONFIG_DM=y > >> +CONFIG_TI_AEMIF=y > >> +# CONFIG_MMC is not set > >> +CONFIG_DM_SPI_FLASH=y > >> +CONFIG_SPI_FLASH=y > >> +CONFIG_SPI_FLASH_STMICRO=y > >> +CONFIG_DM_ETH=y > >> +CONFIG_DM_SERIAL=y > >> +CONFIG_SYS_NS16550=y > >> +CONFIG_DM_SPI=y > >> +CONFIG_USB=y > >> +CONFIG_USB_XHCI_HCD=y > >> +CONFIG_USB_XHCI_DWC3=y > >> +CONFIG_USB_STORAGE=y > > > > This shows a number of the will-be-problems like the AM43/AM33 devices > > have. More things need to be select'd and imply'd so that the _hs_ > > variant defconfigs do not get out of sync easily and often. > > > > I do not think selecting all these options in Kconfig files is safe > right now, at least until moving some more symbols to Kconfig is > complete. After that we can add proper dependencies to all the symbols > and some things like _CMD_ symbols could be added automatically. > > Defconfigs are easier to cleanup than Kconfig definitions. I do not want > to maintain the per-platform Kconfig select'd list before we get symbol > dependencies worked out.
Well, at the end of the day, the pain is on you on re-syncing the defconfig files, so if you want to wait on adding more logic, OK, I'll remove my objection. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/listinfo/u-boot