<div class="gmail_quote">On Wed, Feb 16, 2011 at 5:09 PM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Auf 17.02.2011 01:11, David Hendricks schrieb:<br>
<div class="im">> Attached is a patch to add support for the SMSC MEC1308 embedded controller.<br>
><br>
> Unfortunately, the device ID it conflicts with another SMSC Super IO chip,<br>
> the FDC37M81x. However, since superiotool does not really support the<br>
> FDC37M81x in any really useful capacity, I think it's fair to just disable<br>
> it for now.<br>
><br>
<br>
</div>Unless I'm mistaken, this patch brings the MEC1308 to the level of<br>
support the FDC37M81x had before your patch. I was just looking at the<br>
line count per chip, so maybe there is some key information I'm missing.<br>
Could you please enlighten me what "not really supported" means in this<br>
context? Thanks!<br></blockquote><div><br></div><div>Huh, odd... I actually wrote the patch over a year ago, and was probably using the obsolete branch (before it got put back into coreboot/util). Clearly the FDC37M81x has support now. Should we just leave both of them enabled?</div>

<div><br></div><div>(apologies for the hasty commit, i had an immediate use case)</div><meta http-equiv="content-type" content="text/html; charset=utf-8"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">


<br>
Regards,<br>
Carl-Daniel<br>
<font color="#888888"><br>
--<br>
<a href="http://www.hailfinger.org/" target="_blank">http://www.hailfinger.org/</a><br>
<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>