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

From	Chris Brandt <Chris.Brandt@renesas.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values
Date	2016-02-19 15:30 +0100
Message-ID	<r3Uee-6kG-7@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 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?


Here's a question:

Can we just get rid of PHYS_OFFSET???

If it's only used at boot for XIP systems, we could:

A) pass it in via an unused register like atags and DT
 or
B) just assume that atags or DT is in RAM, so round down to the nearest section and assume that is the start of your RAM

If it is needed after initial boot, then on first boot we save what was passed in from the boot loader for later use.


Chris

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