From mboxrd@z Thu Jan  1 00:00:00 1970
From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?=
Date: Tue, 14 Sep 2010 12:23:20 +0000
Subject: Re: [PATCH 2/7] [RFC] ARM: shmobile: remove unneeded hack to
Message-Id: <20100914122320.GB16960@pengutronix.de>
List-Id: <linux-sh.vger.kernel.org>
References: <20100903095504.GC13643@pengutronix.de>
	<1283507818-1573-2-git-send-email-u.kleine-koenig@pengutronix.de>
	<20100914080829.GI7336@linux-sh.org>
In-Reply-To: <20100914080829.GI7336@linux-sh.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
To: linux-arm-kernel@lists.infradead.org

Hello Paul,

On Tue, Sep 14, 2010 at 05:08:29PM +0900, Paul Mundt wrote:
> On Fri, Sep 03, 2010 at 11:56:53AM +0200, Uwe Kleine-K??nig wrote:
> > -config MEMORY_START
> > -	hex "Physical memory start address"
> > -	default "0x50000000" if MACH_G3EVM
> > -	default "0x40000000" if MACH_G4EVM
> > -	default "0x40000000" if MACH_AP4EVB
> > -	default "0x00000000"
> > -	---help---
> > -	  Tweak this only when porting to a new machine which does not
> > -	  already have a defconfig. Changing it from the known correct
> > -	  value on any of the known systems will only lead to disaster.
> > -
>=20
> [snip]
>=20
> > -#define PHYS_OFFSET	UL(CONFIG_MEMORY_START)
> > +#define SH7367_PHYS_OFFSET UL(0x50000000)
> > +#define SH7372_PHYS_OFFSET UL(0x40000000)
> > +#define SH7377_PHYS_OFFSET UL(0x40000000)
> > +
> > +#if !defined(CONFIG_RUNTIME_PHYS_OFFSET)
> > +# if defined(CONFIG_ARCH_SH7367)
> > +#  define PHYS_OFFSET	SH7367_PHYS_OFFSET
> > +# elif defined(CONFIG_ARCH_SH7372)
> > +#  define PHYS_OFFSET	SH7372_PHYS_OFFSET
> > +# elif defined(CONFIG_ARCH_SH7377)
> > +#  define PHYS_OFFSET	SH7377_PHYS_OFFSET
> > +# endif
> > +#endif /* if !defined(CONFIG_RUNTIME_PHYS_OFFSET) */
> > +
>=20
> You've taken a board-specific property and made it CPU-specific. There is
> nothing about these CPUs that mandates memory starting here, it just
> happens that for the moment we have 1 board per CPU, but that's not an
> assumption we can really bank on.
I choosed to do so as CONFIG_ZRELADDR (which is now reverted) used the
ARCH_... symbols, too, to select the needed values.  And that's my I
marked the patch as RFC.  So would you consider this patch acceptible if
I respin it using the MACH_... symbols?

Best regards
Uwe

--=20
Pengutronix e.K.                           | Uwe Kleine-K=F6nig            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |