[U-Boot] [PATCH] ARMV7: Fix build for non-OMAP3 boards
Premi, Sanjeev
premi at ti.com
Mon Oct 25 16:40:33 CEST 2010
> -----Original Message-----
> From: Steve Sakoman [mailto:steve at sakoman.com]
> Sent: Monday, October 25, 2010 6:52 PM
> To: Premi, Sanjeev
> Cc: u-boot at lists.denx.de
> Subject: RE: [U-Boot] [PATCH] ARMV7: Fix build for non-OMAP3 boards
>
> On Mon, 2010-10-25 at 15:28 +0530, Premi, Sanjeev wrote:
> > > -----Original Message-----
> > > From: u-boot-bounces at lists.denx.de
> > > [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Steve Sakoman
> > > Sent: Saturday, October 23, 2010 2:20 AM
> > > To: u-boot at lists.denx.de
> > > Subject: [U-Boot] [PATCH] ARMV7: Fix build for non-OMAP3 boards
> > >
> > > Commit c3d3a54 uses CONFIG_ARMV7 to determine whether to call the
> > > v7_flush_cache_all function. This breaks the build for
> all non-OMAP3
> > > boards (like Panda and OMAP4430SDP) since there is only a
> > > v7_flush_cache_all
> > > implementation for OMAP3.
> > >
> > > This patch uses CONFIG_OMAP3XXX instead of CONFIG_ARMV7 so
> > > that only boards
> > > with a v7_flush_cache_all will make the call.
> >
> > [sp] Is this call board specific?
>
> No, it is not.
[sp] I asked because I can see omap3/cache.S but not under omap-common/
(as I was expecting) - neither in omap4/
Doesn't this patch works-around the problem by using CONFIG_OMAP34XX?
Wouldn't change in cache.S (or Makefile) be better solution/ or
workaround. At least workaround will be visible to more eyes.
~sanjeev
>
> Steve
>
>
>
>
More information about the U-Boot
mailing list