[flashrom] [PATCH] board enable for DFI AD77
Uwe Hermann
uwe at hermann-uwe.de
Thu Jun 23 17:53:24 CEST 2011
Hi,
On Mon, May 23, 2011 at 07:12:40AM +0200, Stefan Tauner wrote:
> On Sun, 22 May 2011 19:27:49 +0300
> Tadas Slotkus <devtadas at gmail.com> wrote:
>
> > Could someone review?
>
> hey
>
> the url should link to the manufacturer's site if possible e.g.
> http://dfics.dfi.com.tw/Product/xx_product_spec_details_r_us.jsp?PRODUCT_ID=1010&CATEGORY_TYPE=MB&SITE=US
>
> you have used the pciids of the generic VT8377 host bridge and the
> audio controller. the audio controller is the only device with a DFI
> subsystem id, but it can probably be disabled in the bios and then it
> does not match...
> since we can (and you patch does) use dmi matching anyway i think we
> should use (for example) the isa bridge instead of the audio controller.
> also we could mark it as tested if we match ^AD77$ instead of ^AD77.
A few more comments:
Tadas, could you please re-run your lspci as "lspci -xxxnnnvvv" as root,
the current one is missing the full info.
Also, did you try a write/erase operation on the board/chip? Without a
successful write operation we cannot mark the board/chip as supported,
reading alone is not enough unfortunately.
Please post write logs (-w -VV) and the lspci mentioned above, if possible.
Thanks, Uwe.
--
http://hermann-uwe.de | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org
More information about the flashrom
mailing list