----- Original Message ----
> From: Chris Lingard <[EMAIL PROTECTED]>
> To: Coreboot <[email protected]>
> Sent: Wednesday, November 12, 2008 11:35:18 AM
> Subject: Re: [coreboot] AMD Athlon(tm) dual core processor, (X2 AM2 ,,,,)
> 

> > We need an additional entry in the table. See table9 in the K8 revision 
> > guide.
> > 
> http://www.amd.com/us-en/assets/content_type/white_papers_and_tech_docs/33610.pdf
> > 
> > I attached a patch for you to try. If it works please ack it and I will 
> > commit 
> it.
> 
> 
> Ack
> 
> Works fine.

Thanks Chris but to do a proper ack you need to do it like this.
Example: Acked-by: John Doe <[EMAIL PROTECTED]>

See the guidelines here:
http://www.coreboot.org/Development_Guidelines#Reviews

Thanks,
Marc

--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to