[U-Boot] [PATCH 1/2] USB: Rework USB keyboard driver

Wolfgang Denk wd at denx.de
Sun Oct 9 20:54:07 CEST 2011


Dear Marek Vasut,

In message <1317990657-17214-2-git-send-email-marek.vasut at gmail.com> you wrote:
> Also, fix usb drivers which use extern new.
> 
> Signed-off-by: Marek Vasut <marek.vasut at gmail.com>
> Cc: Ajay Kumar Gupta <ajay.gupta at ti.com>
> Cc: Bryan Wu <bryan.wu at analog.com>
> Cc: Cliff Cai <cliff.cai at analog.com>
> Cc: Mike Frysinger <vapier at gentoo.org>
> Cc: Remy Bohmer <linux at bohmer.net>
> Cc: Wolfgang Denk <wd at denx.de>
> ---
>  common/usb_kbd.c            |  644 ++++++++++++++++++++++++-------------------
>  drivers/usb/host/ehci-hcd.c |   20 +--
>  drivers/usb/musb/musb_hcd.c |   19 +--
>  include/usb.h               |    1 +
>  4 files changed, 369 insertions(+), 315 deletions(-)
> 
> NOTE: Tested on EfikaSB device, but please test on some more hardware first!
> 
> diff --git a/common/usb_kbd.c b/common/usb_kbd.c
> index 503d175..d19551b 100644
> --- a/common/usb_kbd.c
> +++ b/common/usb_kbd.c
> @@ -25,402 +25,490 @@
>   *
>   */
>  #include <common.h>
> +#include <malloc.h>
>  #include <stdio_dev.h>
>  #include <asm/byteorder.h>
>  
>  #include <usb.h>
>  
> -#undef USB_KBD_DEBUG
> +/* USB Keyboard Debug */
> +#undef	USB_KBD_DEBUG

Please do not undef what is not defined [or what a user might want to
define on the command line].


> +/* Keyboard sampling rate */
> +#define	REPEAT_RATE	(40 / 4)	/* 40msec -> 25cps */
> +#define	REPEAT_DELAY	10		/* 10 x REAPEAT_RATE = 400msec */

Please fix the typo while you are at it.

And please split code changes and pure cosmetic ones into separate
commits.  Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I'm what passes for a Unix guru in my office. This is  a  frightening
concept. - Lee Ann Goldstein, in <3k55ba$c43 at butch.lmsc.lockheed.com>


More information about the U-Boot mailing list