So; r2972 introduced two patches with this comment:

==========================================================================

r2972 | hailfinger | 2007-11-14 10:09:30 -0500 (Wed, 14 Nov 2007) | 17 lines

Autodetect presence of serial flash and set up the board accordingly.
This enables us to have only one configuration and one set of code for
all revisions of the Gigabyte GA-M57SLI-S4.
Flash is now setup correctly for both SPI and LPC flash.

Detection of SPI flash in flashrom on rev. 2.x boards now hangs
instead of failing. However, that is just an effect of the combination
of incomplete initialization of the SPI controller and paranoid checks
in the flashrom SPI code.
If anyone wants to work on that, he needs a logic analyzer or creative
imagination. Hint: LPC-to-SPI read passthrough, clock signal.

Remaining issues for the M57SLI: Fan/environment control.

Signed-off-by: Carl-Daniel Hailfinger <[EMAIL PROTECTED]>
Acked-by: Harald Gutmann <[EMAIL PROTECTED]>

==========================================================================

One of those patches breaks flashrom on rev 1 and 1.1 of the m57sli (that's
the plcc version of the board):

Index: src/mainboard/gigabyte/m57sli/Config.lb
===================================================================
--- src/mainboard/gigabyte/m57sli/Config.lb     (revision 2955)
+++ src/mainboard/gigabyte/m57sli/Config.lb     (revision 2972)
@@ -310,7 +310,7 @@
                                        # SIO pin set 1 input mode
                                                                #irq 0xc8 = 0x0
                                        # SIO pin set 2 mixed input/output
                                        # mode
-                                                               irq 0xc9 = 0x0
+                                                               irq 0xc9 = 0x40
                                        # SIO pin set 4 input mode
                                                                #irq 0xcb = 0x0
                                        # Generate SMI# on EC IRQ

Reversing this hunk fixes flashrom on the plcc version of our board.

If it turns out that this value needs to be different for the various
revisions of the m57sli, any thoughts on how we are going to handle that? 

Carl-Daniel - this hunk seems somewhat unrelated to the log message and the
other part of r2972; do you remember what exactly it was for?

Thanks,
Ward.

-- 
Ward Vandewege <[EMAIL PROTECTED]>
Free Software Foundation - Senior System Administrator

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

Reply via email to