<div dir="ltr">On Thu, Oct 9, 2008 at 10:59 AM, Uwe Hermann <span dir="ltr"><<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="Wj3C7c">On Thu, Oct 09, 2008 at 08:26:09AM -0400, Josh wrote:<br>
> According to IT8718F_V0 3_(for C version).pdf, it looks like<br>
> 0xf0 should be 0x08.<br>
><br>
> Signed-off-by: Josh Profitt <<a href="mailto:zorn169@gmail.com">zorn169@gmail.com</a>><br>
<br>
</div></div>> Index: util/superiotool/ite.c<br>
> ===================================================================<br>
> --- util/superiotool/ite.c    (revision 3642)<br>
> +++ util/superiotool/ite.c    (working copy)<br>
> @@ -295,7 +295,7 @@<br>
>                        0x00,NANA,NANA,EOT}},<br>
>               {0x5, "Keyboard",<br>
>                       {0x30,0x60,0x61,0x62,0x63,0x70,0x71,0xf0,EOT},<br>
> -                     {0x01,0x00,0x60,0x00,0x64,0x01,0x02,0x00,EOT}},<br>
> +                     {0x01,0x00,0x60,0x00,0x64,0x01,0x02,0x08,EOT}},<br>
<br>
Maybe, maybe not. Page 56 says 0x08, but on page 32 it says 0x00. The<br>
datasheet contradicts itself here, and we don't know which one is<br>
actually the correct default. We should probably document this in the<br>
code, though.<br>
<br>
The difference is that KBC clock is either 8 or 12 MHz, which is<br>
board-dependent, unclear what could be the default for the chip.</blockquote><div><br>So shouldn't we make this configurable via mainboard Config.lb, and use the value dumped by superiotool?<br><br>-Corey <br></div></div>
<br></div>