[coreboot] [PATCH] superiotool: fix wrong IT8718F keyboard value

Uwe Hermann uwe at hermann-uwe.de
Fri Oct 10 00:41:26 CEST 2008


On Thu, Oct 09, 2008 at 02:08:32PM -0400, Corey Osgood wrote:
> > > Index: util/superiotool/ite.c
> > > ===================================================================
> > > --- util/superiotool/ite.c    (revision 3642)
> > > +++ util/superiotool/ite.c    (working copy)
> > > @@ -295,7 +295,7 @@
> > >                        0x00,NANA,NANA,EOT}},
> > >               {0x5, "Keyboard",
> > >                       {0x30,0x60,0x61,0x62,0x63,0x70,0x71,0xf0,EOT},
> > > -                     {0x01,0x00,0x60,0x00,0x64,0x01,0x02,0x00,EOT}},
> > > +                     {0x01,0x00,0x60,0x00,0x64,0x01,0x02,0x08,EOT}},
> >
> > Maybe, maybe not. Page 56 says 0x08, but on page 32 it says 0x00. The
> > datasheet contradicts itself here, and we don't know which one is
> > actually the correct default. We should probably document this in the
> > code, though.
> >
> > The difference is that KBC clock is either 8 or 12 MHz, which is
> > board-dependent, unclear what could be the default for the chip.
> 
> So shouldn't we make this configurable via mainboard Config.lb, and use the
> value dumped by superiotool?

If the keyboard doesn't work for some board, this could be one reason,
yes. Keyboard does work for most boards out of the box though. And yes,
I think you can already set this via Config.lb. However, I don't
think we actually have a board with IT8718F so far, so this is not yet
relevant :)


Uwe.
-- 
http://www.hermann-uwe.de  | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org




More information about the coreboot mailing list