From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Makefile: remove tx25 exception
Date: Tue, 30 Aug 2011 11:30:16 +0200 [thread overview]
Message-ID: <4E5CADA8.1030006@denx.de> (raw)
In-Reply-To: <20110829195941.63A4918C46F9@gemini.denx.de>
On 08/29/2011 09:59 PM, Wolfgang Denk wrote:
> Dear Stefano Babic,
>
> In message <1314629830-32588-1-git-send-email-sbabic@denx.de> you wrote:
>> An entry in the main Makefile for the tx25 board is not
>> necessary, and the board is added to boards.cfg as all
>> other targets.
>
> In theorey you are right, but this patch appears to be wrong:
>
>> -tx25_config : unconfig
>> - @echo "CONFIG_NAND_U_BOOT = y" >> $(obj)include/config.mk
>> - @$(MKCONFIG) $@ arm arm926ejs tx25 karo mx25
>
> The Makefile defined CONFIG_NAND_U_BOOT
In principle you are right, too, but in include/configs/tx25.h:
/* NAND BOOT is the only boot method */
#define CONFIG_NAND_U_BOOT
CONFIG_NAND_U_BOOT is always defined for this board. No idea because
there is a special rule in Makefile.
>
>> +tx25 arm arm926ejs tx25 karo mx25
>
> This new entry fails to do that. Please make sure to add NAND_U_BOOT
> in the "Options" column.
I do not think we need it, because it is already defined in the config.h.
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
=====================================================================
next prev parent reply other threads:[~2011-08-30 9:30 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-29 14:57 [U-Boot] [PATCH] Makefile: remove tx25 exception Stefano Babic
2011-08-29 19:59 ` Wolfgang Denk
2011-08-30 9:30 ` Stefano Babic [this message]
2011-08-30 9:38 ` Wolfgang Denk
2011-10-05 18:45 ` Wolfgang Denk
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=4E5CADA8.1030006@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.