All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] MX25: tx25: Pass NAND_U_BOOT option
Date: Wed, 14 Sep 2011 09:45:04 +0200	[thread overview]
Message-ID: <4E705B80.1050402@denx.de> (raw)
In-Reply-To: <1315952781-2100-1-git-send-email-festevam@gmail.com>

On 09/14/2011 12:26 AM, Fabio Estevam wrote:
> commit e5226e63 (Makefile: remove tx25 exception) missed to pass NAND_U_BOOT 
> option when moving tx25 target to boards.cfg.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  boards.cfg |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 

Hi Fabio,

> diff --git a/boards.cfg b/boards.cfg
> index 1e2ff43..58f677f 100644
> --- a/boards.cfg
> +++ b/boards.cfg
> @@ -148,7 +148,7 @@ sheevaplug                   arm         arm926ejs   -                   Marvell
>  dockstar                     arm         arm926ejs   -                   Seagate        kirkwood
>  jadecpu                      arm         arm926ejs   jadecpu             syteco         mb86r0x
>  mx25pdk                      arm         arm926ejs   mx25pdk             freescale      mx25		mx25pdk:IMX_CONFIG=board/freescale/mx25pdk/imximage.cfg
> -tx25                         arm         arm926ejs   tx25                karo           mx25
> +tx25                         arm         arm926ejs   tx25                karo           mx25		tx25:NAND_U_BOOT
>  zmx25                        arm         arm926ejs   zmx25               syteco         mx25
>  imx27lite                    arm         arm926ejs   imx27lite           logicpd        mx27
>  magnesium                    arm         arm926ejs   imx27lite           logicpd        mx27

I was too fast...I reread the whole thread
(http://patchwork.ozlabs.org/patch/112080/)
 CONFIG_NAND_U_BOOT is defined in tx25.h, and there is no need to define
it again. I think the actual patch is ok. Have you seen any problem ?

Best regards,
Stefano Babic


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

  parent reply	other threads:[~2011-09-14  7:45 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-13 22:26 [U-Boot] [PATCH] MX25: tx25: Pass NAND_U_BOOT option Fabio Estevam
2011-09-14  7:42 ` Stefano Babic
2011-09-14  7:45 ` Stefano Babic [this message]
2011-09-14 11:26   ` Fabio Estevam

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=4E705B80.1050402@denx.de \
    --to=sbabic@denx.de \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.