Duncan Laurie has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31658
Change subject: libpayload: i8042: Only test PS/2 AUX port when enabled ......................................................................
libpayload: i8042: Only test PS/2 AUX port when enabled
If a PS/2 AUX device is not present then the AUX test command during i8042_probe() will time out and add ~500ms to the boot time.
In order to avoid this only test the PS/2 AUX port if CONFIG_LP_PC_MOUSE is enabled.
BUG=b:126633269 TEST=boot on device without AUX port and check that this command does not get executed, saving ~500ms at boot.
Change-Id: I2ebdecc66933bd33d320b17aa4608caf4aaf54aa Signed-off-by: Duncan Laurie dlaurie@google.com --- M payloads/libpayload/drivers/i8042/i8042.c 1 file changed, 4 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/58/31658/1
diff --git a/payloads/libpayload/drivers/i8042/i8042.c b/payloads/libpayload/drivers/i8042/i8042.c index 69740d9..4cdd0be 100644 --- a/payloads/libpayload/drivers/i8042/i8042.c +++ b/payloads/libpayload/drivers/i8042/i8042.c @@ -229,8 +229,10 @@ }
/* Test secondary port */ - if (i8042_cmd_with_response(I8042_CMD_AUX_TEST) == 0) - aux_fifo = fifo_init(4 * 32); + if (IS_ENABLED(CONFIG_LP_PC_MOUSE)) { + if (i8042_cmd_with_response(I8042_CMD_AUX_TEST) == 0) + aux_fifo = fifo_init(4 * 32); + }
/* Test first PS/2 port */ if (i8042_cmd_with_response(I8042_CMD_KB_TEST) == 0)