From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Date: Thu, 7 May 2009 22:46:17 +0200
Subject: [U-Boot] [PATCH] OMAP3: Introduce CONFIG option for power code
In-Reply-To: <1241623288-7105-1-git-send-email-dirk.behme@googlemail.com>
References: <1241623288-7105-1-git-send-email-dirk.behme@googlemail.com>
Message-ID: <20090507204617.GE3302@game.jcrosoft.org>
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

On 17:21 Wed 06 May     , Dirk Behme wrote:
> Some OMAP3 boards need control for external power companion
> chips. Introduce a CONFIG option for this, to avoid Makefile
> changes for each board.

please also move it to cpu/arm_cortexa8/omap3/

omap3 and davinci is not a vendor
so common code will need to go to drivers or cpu/<arch>/<soc>
for the davinci it's already start to move to it

Best Regards,
J.
> 
> Signed-off-by: Dirk Behme <dirk.behme@googlemail.com>
> ---
> 
> Compile tested with ./MAKEALL ARM_CORTEX_A8, boot tested on
> OMAP3 based BeagleBoard.
> 
> Note: This patch trys to fix the first comment of
> 
> http://lists.denx.de/pipermail/u-boot/2009-April/051383.html
> 
>  board/omap3/common/Makefile     |    5 +----
>  doc/README.omap3                |   11 +++++++++++
>  include/configs/omap3_beagle.h  |    1 +
>  include/configs/omap3_overo.h   |    1 +
>  include/configs/omap3_pandora.h |    1 +
>  include/configs/omap3_zoom1.h   |    1 +
>  6 files changed, 16 insertions(+), 4 deletions(-)
> 
> Index: u-boot-main/doc/README.omap3
> ===================================================================
> --- u-boot-main.orig/doc/README.omap3
> +++ u-boot-main/doc/README.omap3
> @@ -77,6 +77,17 @@ For all other commands see
>  
>  help
>  
> +Custom configs
> +==============
> +
> +CONFIG_OMAP3_POWER
> +
> +Some OMAP3 boards use external power companion chips to be configured. Enable
> +this CONFIG option in your board specific configuration file if your board
> +uses such a companion chip.
could you name it ine the config and the file name

Best Regards,
J.