summaryrefslogtreecommitdiffstats
path: root/common/usb_kbd.c
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2014-07-23 06:54:58 -0600
committerSimon Glass <sjg@chromium.org>2014-07-23 14:07:23 +0100
commit91d0be1dd845913ba276e041dc11d1297390de11 (patch)
treeff4358ffe721beb4f134f307cc37ee9d004db20d /common/usb_kbd.c
parenteb9ef5fee7243d43d6fa29652c8ffa987f71b834 (diff)
downloadtalos-obmc-uboot-91d0be1dd845913ba276e041dc11d1297390de11.tar.gz
talos-obmc-uboot-91d0be1dd845913ba276e041dc11d1297390de11.zip
stdio: Remove redundant code around stdio_register() calls
There is no point in setting a structure's memory to NULL when it has already been zeroed with memset(). Also, there is no need to create a stub function for stdio to call - if the function is NULL it will not be called. This is a clean-up, with no change in functionality. Signed-off-by: Simon Glass <sjg@chromium.org> Acked-by: Marek Vasut <marex@denx.de>
Diffstat (limited to 'common/usb_kbd.c')
-rw-r--r--common/usb_kbd.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/common/usb_kbd.c b/common/usb_kbd.c
index 0b77c16c5f..371e5bc144 100644
--- a/common/usb_kbd.c
+++ b/common/usb_kbd.c
@@ -522,8 +522,6 @@ int drv_usb_kbd_init(void)
memset(&usb_kbd_dev, 0, sizeof(struct stdio_dev));
strcpy(usb_kbd_dev.name, DEVNAME);
usb_kbd_dev.flags = DEV_FLAGS_INPUT | DEV_FLAGS_SYSTEM;
- usb_kbd_dev.putc = NULL;
- usb_kbd_dev.puts = NULL;
usb_kbd_dev.getc = usb_kbd_getc;
usb_kbd_dev.tstc = usb_kbd_testc;
usb_kbd_dev.priv = (void *)dev;
OpenPOWER on IntegriCloud