public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] OMAP3: Introduce CONFIG option for power code
Date: Thu, 7 May 2009 22:46:17 +0200	[thread overview]
Message-ID: <20090507204617.GE3302@game.jcrosoft.org> (raw)
In-Reply-To: <1241623288-7105-1-git-send-email-dirk.behme@googlemail.com>

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.

  reply	other threads:[~2009-05-07 20:46 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-06 15:21 [U-Boot] [PATCH] OMAP3: Introduce CONFIG option for power code Dirk Behme
2009-05-07 20:46 ` Jean-Christophe PLAGNIOL-VILLARD [this message]
2009-05-08 15:27   ` Dirk Behme
2009-05-10 15:16     ` Jean-Christophe PLAGNIOL-VILLARD
2009-05-10 17:03       ` Dirk Behme
2009-05-12  1:13         ` Jean-Christophe PLAGNIOL-VILLARD
2009-05-12 17:41           ` Dirk Behme
2009-05-12 22:34             ` Jean-Christophe PLAGNIOL-VILLARD

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090507204617.GE3302@game.jcrosoft.org \
    --to=plagnioj@jcrosoft.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox