[coreboot-gerrit] [S] Change in coreboot[main]: util/superiotool/ite: Add extra dumps for IT8613E EC

[Michał Żygowski (Code Review)]

Attention is currently required from: Alicja Michalska, Angel Pons, Felix Held.

Michał Żygowski has posted comments on this change by Michał Żygowski. ( https://review.coreboot.org/c/coreboot/+/80318?usp=email )

Change subject: util/superiotool/ite: Add extra dumps for IT8613E EC
......................................................................

Patch Set 8:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/80318/comment/68e4aefb_75f3cc34?usp... :
PS6, Line 7: util/superiotool/ite: Add extra dumps for IT8613E EC
> I can test it next week to see if there's any improvement compared to previous patchset (as temperature reporting is the last issue preventing me from having my port being merged in upstream, it's been stable for a while now).

It will show you the relevant registers, but you won't know which one is which. I would like to follow-up later with changes to ITE EC driver in coreboot that programs the registers for FAN and HW monitoring on this chip. I don;t think any of the Protectli devices have an option for attaching a fan, but ODROID has. When I find some free time to play with it more, I can draft some change.