diff options
author | Simon Glass <sjg@chromium.org> | 2015-10-18 21:17:20 -0600 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2015-11-19 20:13:40 -0700 |
commit | 4f087bace2dab1ceb42077a044c918e22691a172 (patch) | |
tree | eff13b8b418c7b7707128282868bd7d6d7babe3a | |
parent | 31d38ee66dfad4284866267ff0651fa0614dad3c (diff) | |
download | u-boot-imx-4f087bace2dab1ceb42077a044c918e22691a172.zip u-boot-imx-4f087bace2dab1ceb42077a044c918e22691a172.tar.gz u-boot-imx-4f087bace2dab1ceb42077a044c918e22691a172.tar.bz2 |
i8042: Adjust kbd_reset() to collect all failures
Rather than lots of 'return' statements, use goto to a single return.
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
-rw-r--r-- | drivers/input/i8042.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/drivers/input/i8042.c b/drivers/input/i8042.c index dbd4b00..4c887f4 100644 --- a/drivers/input/i8042.c +++ b/drivers/input/i8042.c @@ -485,30 +485,33 @@ static int kbd_reset(void) /* controller self test */ if (kbd_cmd_read(CMD_SELF_TEST) != KBC_TEST_OK) - return -1; + goto err; /* keyboard reset */ if (kbd_write(I8042_DATA_REG, CMD_RESET_KBD) || kbd_read(I8042_DATA_REG) != KBD_ACK || kbd_read(I8042_DATA_REG) != KBD_POR) - return -1; + goto err; /* set AT translation and disable irq */ config = kbd_cmd_read(CMD_RD_CONFIG); if (config == -1) - return -1; + goto err; config |= CONFIG_AT_TRANS; config &= ~(CONFIG_KIRQ_EN | CONFIG_MIRQ_EN); if (kbd_cmd_write(CMD_WR_CONFIG, config)) - return -1; + goto err; /* enable keyboard */ if (kbd_write(I8042_CMD_REG, CMD_KBD_EN) || !kbd_input_empty()) - return -1; + goto err; return 0; +err: + debug("%s: Keyboard failure\n", __func__); + return -1; } static int kbd_controller_present(void) |