<div class="gmail_quote">On Tue, Nov 2, 2010 at 12:45 PM, Uwe Hermann <span dir="ltr"><<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Hi,<br>
<div class="im"><br>
On Mon, Nov 01, 2010 at 01:02:53PM -0700, David Hendricks wrote:<br>
> The patch (attached) was tested by a user on IRC who had the F71889FG. I<br>
> wrote it using documentation from Fintek's website available here:<br>
> <a href="http://www.fintek.com.tw/files/productfiles/F71889_V0.28P.pdf" target="_blank">http://www.fintek.com.tw/files/productfiles/F71889_V0.28P.pdf</a><br>
><br>
> This patch also seems to work for the F71889ED, which uses 0x09 and 0x09 for<br>
<br>
</div>Both times 0x09? Or is this a typo?<br></blockquote><div><br></div><div>FWIW, lm-sensors has these two entries:</div><div><div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">       </font></span><font class="Apple-style-span" face="'courier new', monospace">}, {</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">             </font></span><font class="Apple-style-span" face="'courier new', monospace">name => "Fintek F71889FG Super IO Sensors",</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">             </font></span><font class="Apple-style-span" face="'courier new', monospace">driver => "f71882fg",</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">             </font></span><font class="Apple-style-span" face="'courier new', monospace">devid => 0x0723,</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">             </font></span><font class="Apple-style-span" face="'courier new', monospace">logdev => 0x04,</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">             </font></span><font class="Apple-style-span" face="'courier new', monospace">features => FEAT_IN | FEAT_FAN | FEAT_TEMP,</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">     </font></span><font class="Apple-style-span" face="'courier new', monospace">}, {</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">             </font></span><font class="Apple-style-span" face="'courier new', monospace">name => "Fintek F71889E Super IO Sensors",</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">             </font></span><font class="Apple-style-span" face="'courier new', monospace">driver => "to-be-written",</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">             </font></span><font class="Apple-style-span" face="'courier new', monospace">devid => 0x0909,</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">             </font></span><font class="Apple-style-span" face="'courier new', monospace">logdev => 0x04,</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">             </font></span><font class="Apple-style-span" face="'courier new', monospace">features => FEAT_IN | FEAT_FAN | FEAT_TEMP,</font></div>

<div><span class="Apple-tab-span" style="white-space:pre"><font class="Apple-style-span" face="'courier new', monospace">     </font></span><font class="Apple-style-span" face="'courier new', monospace">}, {</font></div>

</div><div><br></div><div><meta http-equiv="content-type" content="text/html; charset=utf-8">I am not certain what the difference is between the two chips, if any areĀ discernibleĀ from superiotool's perspective. The F71889FG datasheet seems to be the public one -- the 0x23 and 0x07 chip IDs match the documented values. However, the chip I tested with has F71889ED printed on it and has 0x09 for the two chip ID bytes.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
<br>
> chip ID bytes 1 & 2. However, I have not been able to find documentation to<br>
> verify that the two chips are identical from superiotool's perspective.<br>
><br>
> The F71889 seems popular on current generation platforms with AMD chipsets,<br>
> in case there are folks looking to try it on a presently unsupported<br>
> board...<br>
><br>
> Signed-off-by: David Hendricks <<a href="mailto:dhendrix@google.com">dhendrix@google.com</a>><br>
<br>
</div>Are you sure this is the correct patch? It doesn't seem to match the<br>
datasheet in a number of places, e.g. 0x20 and 0x21 (IDs) are incorrect<br>
in NOLDN (as well as most other values in NOLDN), some registers are missing<br>
completely, some LDNs are missing completely etc.<br>
<br>
Is this for another Super I/O, or remainders of copying another table?<br></blockquote><div><br></div><div>I took a glance at the patch again and agree that there are several places where the code does not match the doc. Probably sloppy copy + paste on my part. Let's hold off on this patch until NTU, myself, or someone has time to check the accuracy.</div>

<div><br></div><div>If someone happens to have a F71889<b>ED</b> datasheet with the chip ID bytes both listed as 0x09, that would help.</div></div><br>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>