On 01/09/08 21:29 +0200, Patrick Georgi wrote: > Uwe Hermann schrieb: > > On Mon, Sep 01, 2008 at 05:53:54PM +0200, Stefan Reinauer wrote: > > > >> fix keyboard driver bugs. > >> > > > > Jordan is probably better-suited to review this, but can you elaborate > > a bit more in the commit message? What bug, what's wrong, what effects does > > the bug have, how is it fixed, etc? > > > > The |0x80 codes are "break codes", that means, codes that are emitted > when the key transitions from pressed to non-pressed, so the modifier > was always in the wrong state, as soon as you pressed shift for the > first time. > > I have no idea why that didn't occur to me when I looked at it earlier > today :-/ > > Acked-by: Patrick Georgi <[EMAIL PROTECTED]>
Thats right - I hosed up that state machine and never noticed it because we have never had any payloads that used special keys to any degree. Also acked by me. Jordan -- coreboot mailing list [email protected] http://www.coreboot.org/mailman/listinfo/coreboot

