[coreboot-gerrit] Change in coreboot[master]: libpayload/i8042: Use 'INFO' instead of 'ERROR' when probing failed

[Nico Huber (Code Review)]

Attention is currently required from: Tim Wawrzynczak, Nick Vaccaro, Zhuohao Lee.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60129 )

Change subject: libpayload/i8042: Use 'INFO' instead of 'ERROR' when probing failed
......................................................................

Patch Set 2:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/60129/comment/8e0e8d7a_495f19f4 
PS2, Line 13: when doing the autotest.
> Sort of. […]

Hmmm, if it's critical pretty much depends on the hardware :)
When somebody chose the driver on purpose, failed probing would be
an error. If it's just enabled to work in case there is a controller
but not necessarily (e.g. one payload for different machines), there
would be nothing to complain about, actually.

coreboot would know if we have to expect a controller, but I guess
it doesn't tell the payload?

A better commit message would be nice. Currently it suggests to me
that the code does something wrong. However, the error seems actually
expected (depending on the hardware), so where is the test specifi-
cation and how can it be fixed? ;)