Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1338324

Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values

From Nicolas Pitre <nicolas.pitre@linaro.org>
Newsgroups linux.kernel
Subject Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values
Date 2016-02-19 17:20 +0100
Message-ID <r3VWH-7Ih-43@gated-at.bofh.it> (permalink)
References <r3xrj-6Oj-3@gated-at.bofh.it> <r3xrj-6Oj-1@gated-at.bofh.it> <r3zjs-84l-7@gated-at.bofh.it> <r3OLx-2j4-19@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

On Fri, 19 Feb 2016, Arnd Bergmann wrote:

> On Thursday 18 February 2016 11:02:33 Nicolas Pitre wrote:
> > 
> > Acked-by: Nicolas Pitre <nico@linaro.org>
> > 
> > Is there a way to provide a default for defaults?
> 
> We could have something like
> 
> config PHYS_OFFSET_0
> 	bool
> 
> config PHYS_OFFSET_1
> 	bool
> 
> config PHYS_OFFSET_2
> 	bool
> 
> ... (we need 8 of the 16 possible addresses)
> 
> 
> config PHYS_OFFSET
> 	hex "Physical address of main memory" if MMU
> 	default DRAM_BASE if !MMU
> 	default 0x00000000 if PHYS_OFFSET_0
> 	default 0x10000000 if PHYS_OFFSET_1
> 	default 0x20000000 if PHYS_OFFSET_2
> 	default 0x30000000 if PHYS_OFFSET_3
> 	default 0x70000000 if PHYS_OFFSET_7
> 	default 0x80000000 if PHYS_OFFSET_8
> 	default 0xa0000000 if PHYS_OFFSET_A
> 	default 0xc0000000 if PHYS_OFFSET_C
> 
> 
> and then select one of the bool symbols from each platform.
> Would that address your question?

Yes, but the ugliness factor isn't worth it IMHO.

I was wondering if something like this was possible:

config PHYS_OFFSET
        hex "Physical address of main memory" if MMU
        default DRAM_BASE if !MMU
        default 0x10000000 if FOO
        default 0x20000000 if BAR
        default 0x30000000 if BAZ
        default 0x00000000


Nicolas

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Arnd Bergmann <arnd@arndb.de> - 2016-02-18 15:10 +0100
  Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-02-18 17:10 +0100
    Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Arnd Bergmann <arnd@arndb.de> - 2016-02-19 09:40 +0100
      RE: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Chris Brandt <Chris.Brandt@renesas.com> - 2016-02-19 15:30 +0100
        Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Arnd Bergmann <arnd@arndb.de> - 2016-02-19 16:40 +0100
          Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Russell King - ARM Linux <linux@arm.linux.org.uk> - 2016-02-19 17:50 +0100
            RE: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Chris Brandt <Chris.Brandt@renesas.com> - 2016-02-19 18:20 +0100
            Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-02-19 19:00 +0100
      Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-02-19 17:20 +0100
        Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Arnd Bergmann <arnd@arndb.de> - 2016-02-19 17:30 +0100
          Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-02-19 18:40 +0100
            Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Russell King - ARM Linux <linux@arm.linux.org.uk> - 2016-02-19 19:10 +0100
              Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values Arnd Bergmann <arnd@arndb.de> - 2016-02-19 22:20 +0100

csiph-web