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

[Paul Menzel (Code Review)]

Paul Menzel 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 3:

(1 comment)

https://review.coreboot.org/#/c/33185/3//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/33185/3//COMMIT_MSG@21 
PS3, Line 21: anymore.
Could you please clarify, if only the log message was the problem, or if the keyboard also did not work. In QEMU, there was just the log message, but the keyboard worked in QEMU and on the ASRock E350M1 – in contrast to before the commit, where it did not work.

On the coreboot mailing list, Martin reported [1], that with the commit, nothing worked for him anymore.

[1]: https://mail.coreboot.org/hyperkitty/list/[e-mail]/thread/QQTZ3...