Thread (42 messages) 42 messages, 8 authors, 2018-09-18

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

From: Nicolas Pitre <hidden>
Date: 2016-02-19 17:31:07
Also in: lkml

On Fri, 19 Feb 2016, Arnd Bergmann wrote:
On Friday 19 February 2016 11:10:22 Nicolas Pitre wrote:
quoted
On Fri, 19 Feb 2016, Arnd Bergmann wrote:
quoted
On Thursday 18 February 2016 11:02:33 Nicolas Pitre wrote:
quoted
Acked-by: Nicolas Pitre <redacted>

Is there a way to provide a default for defaults?
...
quoted
quoted
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
Ah, that was my previous approach, but Russell didn't like
how it makes it easier to fall back to an incorrect address
instead of forcing a build error when the address is not
configured.
Makes sense.

Yet, the only reason for a default here is to accommodate automatic 
build tests like randconfig, right?

If so then this should be "fixed" by having the config system provide 
built-in symbols that can be tested from kconfig files.  This way you 
could terminate the above list with:

	default 0x00000000 if RANDCONFIG || ALLYESCONFIG

or the like.


Nicolas
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help