On Thursday 27 January 2005 11:47, Michael Gernoth wrote: > Hi, > > since the introduction of libps2 in the mainline 2.6 kernel I had the > issue that my keyboard[1] was no longer recognized. > The cause of this is that my "keyboard" responds to all commands with > an acknowledgement (0xFA), even if the command is not implemented. One > of those not implemented commands is 0xF2 (ATKBD_GETID_CMD). > > In drivers/input/keyboard/atkbd.c ATKBD_GETID_CMD is used to probe > for the keyboard, and if this fails, another method of detecting > the keyboard is used. It seems that in 2.6.10 atkbd_command > indicated that my keyboard did not successfully execute the command, > but in the current bk-version ps2_command is used, which indicates > a successfull execution, leaving behind invalid keyboard-ids. > This leads to the kernel ignoring my keyboard. >
Hi, Thanks for noticing this. The following patch should fix timeout handling in libps2 and restore previous behavior: ===== drivers/input/serio/libps2.c 1.4 vs edited ===== --- 1.4/drivers/input/serio/libps2.c 2005-01-27 02:13:43 -05:00 +++ edited/drivers/input/serio/libps2.c 2005-01-27 22:52:36 -05:00 @@ -115,8 +115,8 @@ */ timeout = msecs_to_jiffies(command == PS2_CMD_RESET_BAT ? 4000 : 500); - wait_event_interruptible_timeout(ps2dev->wait, - !(ps2dev->flags & PS2_FLAG_CMD1), timeout); + timeout = wait_event_interruptible_timeout(ps2dev->wait, + !(ps2dev->flags & PS2_FLAG_CMD1), timeout); if (ps2dev->cmdcnt && timeout > 0) { -- Dmitry - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/