[coreboot] Outstanding patches

Jordan Crouse jordan.crouse at amd.com
Tue May 6 23:21:04 CEST 2008


On 06/05/08 15:17 -0600, Myles Watson wrote:
> 
> 
> > -----Original Message-----
> > From: Jordan Crouse [mailto:jordan.crouse at amd.com]
> > Sent: Tuesday, May 06, 2008 3:05 PM
> > To: Myles Watson
> > Cc: coreboot at coreboot.org
> > Subject: Re: Outstanding patches
> > 
> > On 06/05/08 14:59 -0600, Myles Watson wrote:
> > > On Tue, May 6, 2008 at 2:49 PM, Myles Watson <mylesgw at gmail.com> wrote:
> > > > > I have a number of outstanding patches that need to be ACKed or
> > > >  > NAKed.  Please review if you have the time:
> > > >  >
> > > >  > coreinfo submenu patch:
> > > >  > http://www.coreboot.org/pipermail/coreboot/2008-April/033837.html
> > > >  > coreinfo:  Add "a submenu"
> > > >
> > > >  Here's a start for this one.
> > >
> > > I forgot to say  that if you configure it without a couple of menus,
> > > it refuses to build because of unused variable warnings.  I'm not sure
> > > that -Werror is the right thing to do when it's in buildrom, even
> > > though I don't like warnings.
> > 
> > Unused functions are bad though - can you point me to the config you used?
> 
> I blew it away to test the lar patch...
> 
> I will try to recreate it, but it was a rdtsc function that wasn't being
> used, if that helps.

Ah, yes - somebody else talked about that too.  I'll fix that.  Thanks.

Jordan

-- 
Jordan Crouse
Systems Software Development Engineer 
Advanced Micro Devices, Inc.





More information about the coreboot mailing list