[coreboot] [commit] r5901 - in trunk/src/cpu/intel: model_106cxmodel_6ex model_6fx

Myles Watson mylesgw at gmail.com
Fri Oct 1 23:39:43 CEST 2010


> On Fri, Oct 01, 2010 at 11:55:49AM -0600, Myles Watson wrote:
> > > Modified:
> > >    trunk/src/cpu/intel/model_106cx/cache_as_ram.inc
> > >    trunk/src/cpu/intel/model_6ex/cache_as_ram.inc
> > >    trunk/src/cpu/intel/model_6fx/cache_as_ram.inc
> > It looks like src/cpu/intel/car/cache_as_ram.inc could use the same
> love.
> 
> Hm, I think I already cleaned up that one.
Sorry about that.  I was trying to diff it with model_6ex/cache_as_ram.inc,
and lots of the comments, indenting, etc. didn't match yet.

> > I'd be interested to see how much uglier it made these files to combine
> > them.  There's very little difference between the 6ex & 6fx code.
> 
> Indeed, I'll post a patch to unify 6ex, 6fx, and 106cx at the very
> least, their differences are minimal.
> 
> The other ones are a bit different, but I'll post a few more patches to
> factor out common stuff and further simplify the code, which might
> allow us to factor out more generic code maybe, we'll see.

Thanks for doing it!
Myles






More information about the coreboot mailing list