On Sun, 22 May 2011 19:27:49 +0300
Tadas Slotkus <[email protected]> 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.

any comments?
-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner

_______________________________________________
flashrom mailing list
[email protected]
http://www.flashrom.org/mailman/listinfo/flashrom

Reply via email to