Hi,
On Fri, Mar 25, 2011 at 10:55:21AM -0600, Marc Jones wrote:
Signed-off-by: Prakash Punnoor prakash@punnoor.de
Thanks for the contribution and for the reminder.
Acked-by: Marc Jones marcj303@gmail.com
Yup, thanks! However, the patch is not quite correct, I think.
The last datasheet (from Winbond) for the "W83627DHG" doesn't mention those registers, the superiotool list was correct.
However, you likely looked at the new "W83627DHG-P/-PT" datasheet from Nuvoton (which now owns the Winbond Super I/O stuff):
http://www.nuvoton.com/NuvotonMOSS/Community/ProductInfo.aspx?tp_GUID=0516ba...
But, this is a different chip from the "W83627DHG". The original Winbond W83627DHG has the ID bytes 0xa0 / 0x2y (y is the revision). The Nuvoton W83627DHG-P/-PT has the IDs 0xb0 / 0x7y (y is the revision). Thus, these are two different chips.
Prakash, let us know if you want to work on a patch which reverts your last changes and adds a new entry for the Nuvoton W83627DHG-P/-PT instead. Otherwise I might do it when I have some spare time.
But please carefully check all registers and their default values. While many may be similar to the old W83627DHG there may also be various other differences in the W83627DHG-P/-PT. You have to check each value.
Thanks, Uwe.