[PATCH 2/7] [RFC] ARM: shmobile: remove unneeded hack to calculate zreladdr
Uwe Kleine-König
u.kleine-koenig at pengutronix.de
Tue Sep 14 08:23:20 EDT 2010
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.
> > -
>
> [snip]
>
> > -#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) */
> > +
>
> 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
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
More information about the linux-arm-kernel
mailing list