From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Wed, 15 Jun 2016 17:34:49 +0200
Subject: [U-Boot] [PATCH] common: add new boot media kconfig entry
In-Reply-To: <1465971344-22119-1-git-send-email-peng.fan@nxp.com>
References: <1465971344-22119-1-git-send-email-peng.fan@nxp.com>
Message-ID: <57617599.1070801@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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

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?

> +
> +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?

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