Re: [coreboot] AMD Athlon(tm) dual core processor, (X2 AM2 ,,,,)

[Marc Jones]

----- Original Message ----
> From: Chris Lingard 
> To: Coreboot 
> 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/3361...
> > 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

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

Thanks,
Marc