Attention is currently required from: Angel Pons, Felix Held, Michał Żygowski.
Alicja Michalska 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 6:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/80318/comment/e2e32333_586cb8d3?usp... : PS6, Line 7: util/superiotool/ite: Add extra dumps for IT8613E EC
Where did you get the register information from? Datasheet? Would be nice to specify in the commit m […]
I believe they have access to the datasheet under NDA with the vendor.
We discussed this patch during FOSDEM, when I mentioned my WIP port and issues I stumbled upon (with reading proper CPU temp value) but of course didn't see any contents of it as it was confidential.
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).