public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH 0/17] Version 0 of Kconfig for U-Boot
Date: Mon, 24 Mar 2014 08:30:56 +0100	[thread overview]
Message-ID: <20140324073056.C1FC338042D@gemini.denx.de> (raw)
In-Reply-To: <20140324145814.B35F.AA925319@jp.panasonic.com>

Dear Masahiro,

In message <20140324145814.B35F.AA925319@jp.panasonic.com> you wrote:
> 
> You are suggesting a better idea below.
> We should not treat SPL as a special case.
> 
> In my opinion, CONFIG_SPL_*  should be discontinued.
> 
> For example, we can merge CONFIG_SPL_TEXT_BASE
> to CONFIG_SYS_TEXT_BASE.

Are you sure this is always possible?

> #ifdef CONFIG_SPL_BUILD
> #  define CONFIG_SYS_TEXT_BASE    0x00000000
> #else
> #  define CONFIG_SYS_TEXT_BASE    0x10000000
> #endif
> 
> rather than
> #define CONFIG_SPL_TEXT_BASE    0x00000000
> #define CONFIG_SYS_TEXT_BASE    0x10000000

Are there really no cases where for example the SPL needs to know
CONFIG_SYS_TEXT_BASE, for example when loading U-Boot to RAM?
I think that the "normal" U-Boot does not need to know about the SPL,
but is this also always true the other way round?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Our business is run on trust.  We trust you will pay in advance.

  reply	other threads:[~2014-03-24  7:30 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-17  8:52 [U-Boot] [RFC PATCH 0/17] Version 0 of Kconfig for U-Boot Masahiro Yamada
2014-03-17  8:52 ` [U-Boot] [RFC PATCH 01/17] kconfig: import Kconfig files from Linux v3.13 tag Masahiro Yamada
2014-03-17  8:52 ` [U-Boot] [RFC PATCH 02/17] kconfig: add basic Kconfig files Masahiro Yamada
2014-03-17  8:52 ` [U-Boot] [RFC PATCH 03/17] Do not apply: tools: add gendefconfigs Masahiro Yamada
2014-03-17  8:52 ` [U-Boot] [RFC PATCH 04/17] kconfig: add defconfig files for all boards Masahiro Yamada
2014-03-18  3:08   ` Stephen Warren
2014-03-19  3:16     ` Masahiro Yamada
2014-03-19  3:39       ` Stephen Warren
2014-03-19  4:50         ` Masahiro Yamada
2014-03-19  9:56           ` Wolfgang Denk
2014-03-19 10:51             ` Masahiro Yamada
2014-03-19 12:37               ` Daniel Schwierzeck
2014-03-19 14:20                 ` Wolfgang Denk
2014-03-19 16:06                   ` Tom Rini
2014-03-19 23:48                     ` Masahiro Yamada
2014-03-21 13:41                       ` Tom Rini
2014-03-20  0:11                 ` Masahiro Yamada
2014-03-20 13:17                   ` Daniel Schwierzeck
2014-03-21 18:05                     ` Tom Rini
2014-03-22 17:14                       ` Daniel Schwierzeck
2014-03-24  6:35                         ` Masahiro Yamada
2014-03-24 20:35                           ` Daniel Schwierzeck
2014-03-28  2:25                             ` Masahiro Yamada
2014-03-28 20:35                               ` Daniel Schwierzeck
2014-03-31  8:56                                 ` Masahiro Yamada
2014-03-19 19:58           ` Stephen Warren
2014-03-19 12:54   ` Tom Rini
2014-03-19 22:58     ` Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 05/17] include: define CONFIG_SPL and CONFIG_TPL as 1 Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 06/17] m68k: define processor family CONFIGs " Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 07/17] kconfig: switch over to Kconfig Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 08/17] MAKEALL: adjust for Kconfig Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 09/17] buildman: " Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 10/17] kconfig: delete redundant CONFIG_${ARCH} definition Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 11/17] sh: remove redundant CPU family definition Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 12/17] sparc: " Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 13/17] m68k: " Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 14/17] powerpc: " Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 15/17] kbuild: remove CONFIG_SPL/CONFIG_TPL definition in config headers Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 16/17] kconfig: remove old script Masahiro Yamada
2014-03-17  8:53 ` [U-Boot] [RFC PATCH 17/17] kconfig: add CONFIG_CROSS_COMPILE Masahiro Yamada
2014-03-21  2:15 ` [U-Boot] [RFC PATCH 0/17] Version 0 of Kconfig for U-Boot Simon Glass
2014-03-24  5:58   ` Masahiro Yamada
2014-03-24  7:30     ` Wolfgang Denk [this message]
2014-03-24  7:52       ` Masahiro Yamada

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=20140324073056.C1FC338042D@gemini.denx.de \
    --to=wd@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox