[U-Boot] [PATCH] common: add new boot media kconfig entry

[Peng Fan <van.freenix@gmail.com>]

Hello Heiko,
On Wed, Jun 15, 2016 at 05:34:49PM +0200, Heiko Schocher wrote:
>Hello Peng,
>
>Am 15.06.2016 um 08:15 schrieb Peng Fan:
>>Add CONFIG_{SD|NOR|NAND|ONENAND|SPI|QSPI|SATA}_BOOT kconfig entries.
>>
>>SoCs supports loading U-Boot from different medias to DRAM, such as
>>i.MX6/7 supports loading U-Boot to DRAM from sd/emmc/nand/qspi/spi/sata
>>and etc. For i.MX, imximage will generate different IVT headers according
>>to boot medias.
>>
>>Signed-off-by: Peng Fan <peng.fan@nxp.com>
>>Cc: Simon Glass <sjg@chromium.org>
>>Cc: Heiko Schocher <hs@denx.de>
>>Cc: Joe Hershberger <joe.hershberger@ni.com>
>>Cc: Bin Meng <bmeng.cn@gmail.com>
>>Cc: Christophe Ricard <christophe-h.ricard@st.com>
>>Cc: Nikita Kiryanov <nikita@compulab.co.il>
>>Cc: Francois Retief <fgretief@spaceteq.co.za>
>>Cc: Tom Rini <trini@konsulko.com>
>>---
>>  common/Kconfig | 46 ++++++++++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 46 insertions(+)
>
>Hmm... are there no places where the introduced config options
>already used in config files ?
>
>>diff --git a/common/Kconfig b/common/Kconfig
>>index 4d17b10..a41b670 100644
>>--- a/common/Kconfig
>>+++ b/common/Kconfig
>>@@ -97,6 +97,52 @@ config BOOTSTAGE_STASH_SIZE
>>
>>  endmenu
>>
>>+menu "Boot media"
>>+
>>+config SD_BOOT
>>+	bool "Boot from SD"
>>+	default n
>>+	help
>>+	  U-Boot image is stored in SD card.
>>+
>>+config NOR_BOOT
>>+	bool "Boot from NOR"
>>+	default n
>>+	help
>>+	  U-Boot image is stored in NOR flash.
>
>For example this is already defined in
>
>./board/ti/am335x/Kconfig

My bad.

>
>So I miss at least the lines, which remove this part in your patch,
>else I think, this would lead in a Kconfig error, if we have the
>same symbol in two places ... or?

I did not check other Kconfig files. The idea is to introduce config
entries to support booting uboot from nand/nor/sd/spi and etc.

I'll take care in V2.

>
>>+
>>+config NAND_BOOT
>>+	bool "Boot from NAND"
>>+	default n
>>+	help
>>+	  U-Boot image is stored in NAND flash.
>>+
>>+config ONENAND_BOOT
>>+	bool "Boot from ONENAND"
>>+	default n
>>+	help
>>+	  U-Boot image is stored in ONENAND flash.
>>+
>>+config SPI_BOOT
>>+	bool "Boot from SPI"
>>+	default n
>>+	help
>>+	  U-Boot image is stored in SPI flash.
>>+
>>+config QSPI_BOOT
>>+	bool "Boot from QSPI"
>>+	default n
>>+	help
>>+	  U-Boot image is stored in QSPI flash.
>>+
>>+config SATA_BOOT
>>+	bool "Boot from SATA"
>>+	default n
>>+	help
>>+	  U-Boot image is stored in SATA.
>>+
>>+endmenu
>>+
>>  config BOOTDELAY
>>  	int "delay in seconds before automatically booting"
>>  	default 0
>>
>
>may you want to use my brand new tbot testcase [1] for testing, if your
>patch not breaks boards?

I'll try this.

Thanks,
Peng.

>
>bye,
>Heiko
>[1] http://lists.denx.de/pipermail/u-boot/2016-June/258119.html
>-- 
>DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
>HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany