[U-Boot] [PATCH] ARM: Generic cache ops skeleton

[Mike Frysinger <vapier@gentoo.org>]

On Sunday 06 November 2011 13:07:29 Marek Vasut wrote:
> > On Sunday 06 November 2011 07:39:56 Marek Vasut wrote:
> > > > On Saturday 05 November 2011 21:16:23 Marek Vasut wrote:
> > > > > This patch should allow a CPU to register it's own cache ops. This
> > > > > shall allow multiple CPUs with different cache handlings to be
> > > > > supported.
> > > > 
> > > > sorry, what's the point ?  this would make sense if you wanted to
> > > > build one u- boot image to run on multiple SoCs, but since that
> > > > doesn't make sense, i don't see what this patch gets you.
> > > 
> > > Well that's the ultimate goal, isn't it ...
> > 
> > is it actually ?  i must have missed the memo.
> > 
> > support for multiple-SoC-support-in-a-single-image shouldn't bloat the
> > single SoC case.  this code seems to do just that.  so it sounds like you
> > should find a CONFIG name for this setup and start putting everything
> > behind that.
> 
> That's a good point indeed. The other question is, if gcc4.6's LTO won't
> fix that for us. Surely enough though, we can't rely on that.

i don't think LTO would help here
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111106/0681fbed/attachment.pgp