Change in ...coreboot[master]: ec/lenovo/h8: Add function to query sense state

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29204 )

Change subject: ec/lenovo/h8: Add function to query sense state
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/#/c/29204/2/src/ec/lenovo/h8/sense.c 
File src/ec/lenovo/h8/sense.c:

https://review.coreboot.org/#/c/29204/2/src/ec/lenovo/h8/sense.c@25 
PS2, Line 25:  * Unlikely that all register will be zero after booting has
            :  * finished.
> So far I understood, the EC reboots when powering the laptop on.

Patrick, I still wonder what this sentence refers to,
coreboot boot process? the EC's?

If it's the EC, the statement may be true, but it
doesn't work the other way. i.e. doesn't imply that
the EC finished populating these registers, when we
see the first bit set.