[flashrom] [commit] r887 - trunk

[Luc Verhaegen]

On Thu, Feb 04, 2010 at 03:43:21AM +0100, Uwe Hermann wrote:
> 00:00.0 Host bridge [0600]: VIA Technologies, Inc. VT82C693A/694x [Apollo PRO133x] [1106:0691] (rev 01)

> 00:01.0 PCI bridge [0604]: VIA Technologies, Inc. VT82C598/694x [Apollo MVP3/Pro133x AGP] [1106:8598] (prog-if 00 [Normal decode])

> 00:07.0 ISA bridge [0601]: VIA Technologies, Inc. VT82C596 ISA [Mobile South] [1106:0596] (rev 06)
> 	Subsystem: VIA Technologies, Inc. VT82C596/A/B PCI to ISA Bridge [1106:0000]

> 00:07.1 IDE interface [0101]: VIA Technologies, Inc. VT82C586A/B/VT82C686/A/B/VT823x/A/C PIPC Bus Master IDE [1106:0571] (rev 06) (prog-if 8a [Master SecP PriP])

> 00:07.2 USB Controller [0c03]: VIA Technologies, Inc. VT82xxxxx UHCI USB 1.1 Controller [1106:3038] (rev 02) (prog-if 00 [UHCI])
> 	Subsystem: First International Computer, Inc. VA-502 Mainboard [0925:1234]

This is a subsystem id which was submitted to pciids.sf.net, and is then per definition
not unique.

> 00:07.3 Host bridge [0600]: VIA Technologies, Inc. VT82C596 Power Management [1106:3050]

Our match is not unique.

> Found Winbond W83877AF (id=0x0b) at 0x3f0
> Register dump:
> idx 00 01 02 03 04 05 06 07  08 09 0a 0b 0c 0d 0e 0f  10 11 12 13 14 15 16 17  1e 20 21 22 23 24 25 26  27 28 29 2a 2b 2c 2d
> val 0c 00 02 50 44 00 00 0b  00 8b 1f 0c 28 ab 00 ff  00 00 00 00 00 00 07 00  00 fc 00 01 de fe be 23  25 43 60 00 00 10 2d
> def 00 00 00 30 00 00 00 00  00 0a 1f 0c 28 a3 RR RR  00 00 00 00 00 00 0e 00  MM MM MM MM MM MM MM MM  MM MM MM MM MM MM MM
> Hardware monitor (0x0004)

Aha, need to add detection for this one too in my "if i ever get to finishing that"
superio code :)

> DMI string 0: "VIA Technologies, Inc."
> DMI string 1: "VT82C691"
> DMI string 2: " "
> DMI string 3: "Tekram Technology Co., Ltd."
> DMI string 4: "P6PROA5"
> DMI string 5: "Rev. 1.0"

We need to add a DMI string, and luckily, today, we can :)

Please add "^P6PROA5$" in the dmi field to make this match unique.

Trivial change, please test, then you have:

Acked-by: Luc Verhaegen <libv at skynet.be>

Luc Verhaegen.