[coreboot-gerrit] Change in coreboot[master]: libpayload/drivers/i8042: add error message of i8042_probe

[EricR Lai (Code Review)]

EricR Lai has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37689 )

Change subject: libpayload/drivers/i8042: add error message of i8042_probe
......................................................................

libpayload/drivers/i8042: add error message of i8042_probe

Print error message before error return for better debugging.

Signed-off-by: Eric Lai 
Change-Id: I52039dcab72c6295dfb6b887a7000a6d2bd050ee
---
M payloads/libpayload/drivers/i8042/i8042.c
1 file changed, 9 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/37689/1

diff --git a/payloads/libpayload/drivers/i8042/i8042.c b/payloads/libpayload/drivers/i8042/i8042.c
index e97fab8..b9bafa7 100644
--- a/payloads/libpayload/drivers/i8042/i8042.c
+++ b/payloads/libpayload/drivers/i8042/i8042.c
@@ -197,23 +197,29 @@

/* If 0x64 returns 0xff, then we have no keyboard
     * controller */
-	if (read_status() == 0xFF)
+	if (read_status() == 0xFF) {
+		printf("%s:No keyboard controller exist!\n", __func__);
    	return 0;
+	}

-	if (!i8042_wait_cmd_rdy())
+	if (!i8042_wait_cmd_rdy()) {
+		printf("ERROR: i8042_wait_cmd_rdy failed!\n");
    	return 0;
+	}

kbc_init = 1;

/* Disable first device */
    if (i8042_cmd(I8042_CMD_DIS_KB) != 0) {
    	kbc_init = 0;
+		printf("ERROR: i8042_cmd I8042_CMD_DIS_KB failed!\n");
    	return 0;
    }

/* Disable second device */
    if (i8042_cmd(I8042_CMD_DIS_AUX) != 0) {
    	kbc_init = 0;
+		printf("ERROR: i8042_cmd I8042_CMD_DIS_AUX failed!\n");
    	return 0;
    }

@@ -225,6 +231,7 @@
    if (i8042_cmd_with_response(I8042_CMD_SELF_TEST)
        != I8042_SELF_TEST_RSP) {
    	kbc_init = 0;
+		printf("ERROR: i8042_cmd I8042_CMD_SELF_TEST failed!\n");
    	return 0;
    }