From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1428144AbcBSQYP (ORCPT <rfc822;w@1wt.eu>);
	Fri, 19 Feb 2016 11:24:15 -0500
Received: from mout.kundenserver.de ([212.227.17.10]:56403 "EHLO
	mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1422656AbcBSQYN (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Fri, 19 Feb 2016 11:24:13 -0500
From: Arnd Bergmann <arnd@arndb.de>
To: Nicolas Pitre <nicolas.pitre@linaro.org>
Cc: Russell King <linux@arm.linux.org.uk>,
        linux-arm-kernel@lists.infradead.org,
        Ard Biesheuvel <ard.biesheuvel@linaro.org>,
        Jon Medhurst <tixy@linaro.org>, Marc Zyngier <marc.zyngier@arm.com>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH 4/9] ARM: add CONFIG_PHYS_OFFSET default values
Date: Fri, 19 Feb 2016 17:23:44 +0100
Message-ID: <38246773.SnToSslHuj@wuerfel>
User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; )
In-Reply-To: <alpine.LFD.2.20.1602191106020.13632@knanqh.ubzr>
References: <1455804123-2526139-1-git-send-email-arnd@arndb.de> <3974581.yknKb3tq7o@wuerfel> <alpine.LFD.2.20.1602191106020.13632@knanqh.ubzr>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
X-Provags-ID: V03:K0:Ew0zs+zDl47kSeSy4VTQM3JA5D9VhqXHpff6XKPO6IXGYE+X+Xg
 Z1W9OfAqmSssX4AD1SoJLRrCOYYMpP38ZT34UtPm9Hgaw/tAt9z5xSKMyS3AhG/n+MdZZLF
 GZSK8E3pxyHcB4bpA+MfaGrC6pkeZDM21mdY3/PCykVKqng51nz1dh95Aa/P8TDf8a3JqQo
 l7Z4ISL0mMx5bZRYTHYlw==
X-UI-Out-Filterresults: notjunk:1;V01:K0:NjJy8+Kr6As=:VjtXHKvRDAV3O+cWkTB0iR
 PZMF0rYTMi8qC4DWV5yCHvAWANmLJvyWE82+waaqccu1TKuX/bnx+QX1y98InYdecrMvB/W6L
 9rK5XmD4CeH31R4MEQ0HIOErBvZdHAEMYSbcoO32hnKtheqRCodu4MtsAhNI4Wui5G++WQfD4
 VasCzouUUt/AxSYE8X1BKF0e/agT8vGQLfKBLR+d4m27MGP//8UN+9FNTwikUnJZwgdDfrziS
 OTUDVj+sw5uweq/Lse+CiqDOVDB5OiVwS76Xm5U0D4FXTCg6riM3CRFN9/d5J43fFHQZczFmA
 UzJYuMghChjdaVErIj9vdx6PhQeMcSvbqvZVSzxeDy/Lt7xUvMjxfOm0pMkTVA2zWMDOMLhFS
 S5NPcnTRAP6NT2XTOJHHYigRz4Pk8e1O5trpJpHPUXuquNcTtHp37bXliA8AlHmXZAQxFJOQl
 GrQ0QnSvThukE4wHS4PjefMlBdFQNne4KrU0DhDUloLHCzGrQoPORNYC+3ZdM0s5zhS2T3bRV
 F0wOH5D9Z5Ol5puzWz9P8sGJ4k4+Ow+7MT+CvU41dsHKZqdG7T9Bpw+n4knxZCX4MOYsV9VXG
 iz3c9NOyGvcXOzp+X4E0qVEIDi4xHCPPAZhK7K1Om111ygFji2WrPs+3v/kAFbaa2S6QP1moe
 LreCs9cf9WeD5CSds9di/stCIY2yF2Njk75y0x1wrAy9WCE0rHbwjbSP74l9HCy2gejY0XAAO
 O/io82Le/ZdEZ+Oh
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Friday 19 February 2016 11:10:22 Nicolas Pitre wrote:
> 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?
...
> > 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.

	Arnd