>Number:         372
>Category:       port-i386
>Synopsis:       Noisy "timeout updating leds" warnings with my Gateway kbd
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    gnats-admin (GNATS administrator)
>State:          open
>Class:          sw-bug
>Submitter-Id:   mhw
>Arrival-Date:   Sun Jul 31 01:20:03 1994
>Originator:     Mark Weaver
>Organization:
--------------------------------------------------------------------
Email: Mark_Weaver@brown.edu           | Brown University
PGP Key: finger mhw@cs.brown.edu       | Dept of Computer Science
>Release:        NetBSD 1.0-BETA (27-July-1994)
>Environment:
Gateway 2000 486/DX2-66 VLB, 16mb RAM, AnyKey keyboard

System: NetBSD cis-ts3-slip4.cis.brown.edu 1.0_BETA NetBSD 1.0_BETA (EXCELSIOR) #1: Sat Jul 30 13:36:57 EDT 1994 mhw@cis-ts3-slip4.cis.brown.edu:/usr/src/sys/arch/i386/compile/EXCELSIOR i386


>Description:
	Virtually every time I boot my system, I get the following
	warnings:

kbd_cmd: input char 47 lost
pcprobe: reset error 4

	Although sometimes the second one doesn't happen.  However,
	unlike versions of the kernel several months ago, pccons
	works flawlessly with my keyboard.  However, every time
	one of the keyboard lights needs to be updated, it prints:

pc: timeout updating leds

	However, it always works perfectly.  I should note here
	that previous versions of the kernel would sometimes not
	update the leds properly.  In any case, you should consider
	silencing this warning before releasing 1.0, since it will
	be VERY annoying to some users.  I don't mind because I
	never use caps lock or num lock anyway.

>How-To-Repeat:
>Fix:
diff -c  src/sys/arch/i386/isa/pccons.c.mhw1 src/sys/arch/i386/isa/pccons.c
*** src/sys/arch/i386/isa/pccons.c.mhw1	Thu May  5 07:11:11 1994
--- src/sys/arch/i386/isa/pccons.c	Sun Jul 31 04:06:41 1994
***************
*** 269,275 ****
  		old_lock_state = lock_state;
  		if (!kbd_cmd(KBC_MODEIND, poll) ||
  		    !kbd_cmd(lock_state, poll)) {
! 			printf("pc: timeout updating leds\n");
  			(void) kbd_cmd(KBC_ENABLE, poll);
  		}
  	}
--- 269,275 ----
  		old_lock_state = lock_state;
  		if (!kbd_cmd(KBC_MODEIND, poll) ||
  		    !kbd_cmd(lock_state, poll)) {
! 		     /*	printf("pc: timeout updating leds\n"); */
  			(void) kbd_cmd(KBC_ENABLE, poll);
  		}
  	}
>Audit-Trail:
>Unformatted:


------------------------------------------------------------------------------