[flashrom] [PATCH]add AMD hudson support

Uwe Hermann uwe at hermann-uwe.de
Thu Aug 25 23:22:08 CEST 2011


Hi,

On Thu, Aug 25, 2011 at 11:06:12PM +0800, QingPei Wang wrote:
> The HUDSON has different vendor & device id than SBx00. The ids are got
> by lspci.
> Signed-off-by: Wang Qing Pei <wangqingpei at gmail.com>

Thanks for the patch, we'll merge it soonish, but please see below for
some questions.

 
> Index: chipset_enable.c
> ===================================================================
> --- chipset_enable.c    (revision 1419)
> +++ chipset_enable.c    (working copy)
> @@ -1052,6 +1052,7 @@

> +    {0x1002, 0x780e, OK, "AMD", "HUDSON",        enable_flash_sb600},

Shouldn't this be 0x1022 (AMD instead of ATI ID) here?

Also, did you test the chipset-enable on real hardware? Otherwise we
should probably commit this as "NT" (not tested) instead of "OK".

Please post a log of "./flashrom -V" with the patch applied if you own
some test hardware, thanks!


Uwe.
-- 
http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org




More information about the flashrom mailing list