[LinuxBIOS] [PATCH] enable Rev. F CPU support for m57sli-s4

[Ward Vandewege]

Subject says it all...

Thanks,
Ward.

[Peter Stuge]

On Mon, May 21, 2007 at 04:29:31PM -0400, Ward Vandewege wrote:
> The Gigabyte m57sli-s4 board supports Rev. F CPUs.
>
> Signed-off-by: Ward Vandewege

r2683

[Ward Vandewege]

On Mon, May 21, 2007 at 10:51:05PM +0200, Peter Stuge wrote:
> On Mon, May 21, 2007 at 04:29:31PM -0400, Ward Vandewege wrote:
> > The Gigabyte m57sli-s4 board supports Rev. F CPUs.
> >
> > Signed-off-by: Ward Vandewege
>
> r2683

This should be reverted. Should I send in another patch for that?

Thanks,
Ward.

[Stefan Reinauer]

* Ward Vandewege  [070522 16:56]:
> On Mon, May 21, 2007 at 10:51:05PM +0200, Peter Stuge wrote:
> > On Mon, May 21, 2007 at 04:29:31PM -0400, Ward Vandewege wrote:
> > > The Gigabyte m57sli-s4 board supports Rev. F CPUs.
> > >
> > > Signed-off-by: Ward Vandewege
> >
> > r2683
>
> This should be reverted. Should I send in another patch for that?

Not necessary. svn diff -r 2683:2682

Stefan

[Peter Stuge]

On Tue, May 22, 2007 at 05:05:59PM +0200, Stefan Reinauer wrote:
> > This should be reverted. Should I send in another patch for that?
>
> Not necessary. svn diff -r 2683:2682

Thanks. I should have backed this out but you got to it before me.

//Peter

[yhlu]

NAK.

Already defined src/cpu/amd/socket_AM2/Config.lb.

and your MB Config.lb will include that with

chip cpu/amd/socket_AM2

YH

On 5/21/07, Ward Vandewege  wrote:
> Subject says it all...
>
> Thanks,
> Ward.

[Stefan Reinauer]

* yhlu  [070521 23:12]:
> NAK.
>
> Already defined src/cpu/amd/socket_AM2/Config.lb.
>
> and your MB Config.lb will include that with
>
> chip cpu/amd/socket_AM2

Cool. That's the right way to do it, but why does Sven's cpuname patch
work with Ward's patch?

[Ward Vandewege]

On Mon, May 21, 2007 at 02:12:52PM -0700, yhlu wrote:
> NAK.
>
> Already defined src/cpu/amd/socket_AM2/Config.lb.
>
> and your MB Config.lb will include that with
>
> chip cpu/amd/socket_AM2

Yes, you're quite right, my patch is not necessary.

Thanks,
Ward.