Re: [U-Boot] [PATCH v2 12/28] i8042: Adjust keyboard init to assume success

2015-10-30 Thread Simon Glass
Applied to u-boot-dm.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2 12/28] i8042: Adjust keyboard init to assume success

2015-10-18 Thread Simon Glass
Modify i8042_kbd_init() so that the normal pass is sucessful init and
failure exits early. This will make the code easier to extend and is easier
to read.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
---

Changes in v2: None

 drivers/input/i8042.c | 19 +--
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/drivers/input/i8042.c b/drivers/input/i8042.c
index 4c887f4..b1ada86 100644
--- a/drivers/input/i8042.c
+++ b/drivers/input/i8042.c
@@ -585,18 +585,17 @@ int i8042_kbd_init(void)
keymap = KBD_GER;
}
 
-   for (try = 0; try < KBD_RESET_TRIES; try++) {
-   if (kbd_reset() == 0) {
-   kbd_mapping = keymap;
-   kbd_flags   = NORMAL;
-   kbd_state   = 0;
-   kbd_led_set();
-
-   return 0;
-   }
+   for (try = 0; kbd_reset() != 0; try++) {
+   if (try >= KBD_RESET_TRIES)
+   return -1;
}
 
-   return -1;
+   kbd_mapping = keymap;
+   kbd_flags   = NORMAL;
+   kbd_state   = 0;
+   kbd_led_set();
+
+   return 0;
 }
 
 /*
-- 
2.6.0.rc2.230.g3dd15c0

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot