[LinuxBIOS] [PATCH] flashrom: Fix wrong VT8235 flash enable failed warning.
Uwe Hermann
uwe at hermann-uwe.de
Fri Mar 2 15:15:28 CET 2007
Hi,
On Fri, Mar 02, 2007 at 02:22:37PM +0100, Luc Verhaegen wrote:
> This fixes the warning that's present for all VT8235 southbridges:
>
> ...
> Enabling flash write on VT8235...tried to set 0x4 to 0x14 on VT8235 failed (WARNING ONLY)
> ...
>
> It's bourne from misinterpreting the return value of pci_write_byte,
> which peculiarly returns 1 for success.
>
> Luc Verhaegen.
> flashrom: Fix wrong VT8235 flash enable failed warning.
>
> * Fix harmless but worrying warning where the return value of
> pci_write_byte is misinterpreted.
> * Hash together VT8231 and VT8235 code into VT823x. VT8231 is the better
> implementation, but lacked the write protect disable code that's
> apparently needed for VT8235.
>
> Signed-off-by: Luc Verhaegen <libv at skynet.be>
Looks reasonable to me, but I cannot test this.
Have you tested the patch on both vt8235 and vt8231 (actual hardware)?
Any complaints from anybody? Otherwise I guess we can commit this.
Uwe.
--
http://www.hermann-uwe.de | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20070302/57b418f0/attachment.sig>
More information about the coreboot
mailing list