[coreboot-gerrit] Change in ...coreboot[master]: libpayload/i8042/keyboard: Fix return value check for keyboard_cmd

[Matt Delco (Code Review)]

Matt Delco has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33185 )

Change subject: libpayload/i8042/keyboard: Fix return value check for keyboard_cmd
......................................................................

Patch Set 2: Code-Review+1

(1 comment)

LGTM, thanks.  Nothing else in the function logs on error but I like having at least some msg there that helps make it clear when the keyboard isn't working.

https://review.coreboot.org/#/c/33185/2/payloads/libpayload/drivers/i8042/ke... 
File payloads/libpayload/drivers/i8042/keyboard.c:

https://review.coreboot.org/#/c/33185/2/payloads/libpayload/drivers/i8042/ke... 
PS2, Line 175: 0xfa
nit: using I8042_KBCMD_ACK here would help avoid the newly added #define from now being unused.