From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michal Simek <michal.simek@xilinx.com>
Date: Wed, 6 Aug 2014 08:39:47 +0200
Subject: [U-Boot] [PATCH v2 02/15] zynq: kconfig: move board select menu
	and commonsettings
In-Reply-To: <1407295083-31995-3-git-send-email-yamada.m@jp.panasonic.com>
References: <1407295083-31995-1-git-send-email-yamada.m@jp.panasonic.com>
	<1407295083-31995-3-git-send-email-yamada.m@jp.panasonic.com>
Message-ID: <3d0d4822-8a55-4c91-9fb3-54d8e0e37b24@BL2FFO11FD031.protection.gbl>
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

Hi Masahiro,

On 08/06/2014 05:17 AM, Masahiro Yamada wrote:
> Becuase the board select menu in arch/arm/Kconfig is too big,
> move the Zynq board select menu to zynq/Kconfig.
> 
> Consolidate also common settings (CONFIG_SYS_CPU="armv7" and
> CONFIG_SYS_SOC="zynq").
> 
> Refactor board/xilinx/zynq/MAINTAINERS too.
> 
> Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> Cc: Michal Simek <michal.simek@xilinx.com>
> ---
> 
> Changes in v2: None
> 
>  arch/arm/Kconfig                   | 15 ++----
>  arch/arm/cpu/armv7/zynq/Kconfig    | 43 +++++++++++++++++
>  board/xilinx/zynq/Kconfig          | 95 --------------------------------------
>  board/xilinx/zynq/MAINTAINERS      | 12 +----
>  configs/zynq_microzed_defconfig    |  1 +
>  configs/zynq_zc70x_defconfig       |  1 +
>  configs/zynq_zc770_xm010_defconfig |  1 +
>  configs/zynq_zc770_xm012_defconfig |  1 +
>  configs/zynq_zc770_xm013_defconfig |  1 +
>  configs/zynq_zed_defconfig         |  1 +
>  include/configs/zynq-common.h      |  1 -
>  11 files changed, 54 insertions(+), 118 deletions(-)
>  create mode 100644 arch/arm/cpu/armv7/zynq/Kconfig
>  delete mode 100644 board/xilinx/zynq/Kconfig
One
One thing I have noticed was that when I run
[u-boot]$ make zynq_zc70x_defconfig
...

there is incorrect CONFIG_DEFCONFIG_LIST setup

[u-boot]$ head .config
#
# Automatically generated file; DO NOT EDIT.
# U-Boot 2014.07 Configuration
#
CONFIG_DEFCONFIG_LIST="configs/sandbox_defconfig"

#
# General setup
#
CONFIG_SPL=y



BTW: I would keep that options sorted
         ? ?             (X) Xilinx Zynq Platform                      ? ?
         ? ?             ( ) NVIDIA Tegra                              | |

The rest for zynq is fine.

Tested-by: Michal Simek <michal.simek@xilinx.com>

Thanks,
Michal