Attention is currently required from: Felix Singer, Maxim, Michał Żygowski, Nicholas Chin, Paul Menzel.
Alicja Michalska has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80853?usp=email )
Change subject: mb/erying: Add Erying Polestar G613 Pro (TGL-H) ......................................................................
Patch Set 7:
(1 comment)
File src/mainboard/erying/tgl/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/80853/comment/4b529f60_f1c3c32c : PS7, Line 144: register "TMPIN1.mode" = "THERMAL_DIODE"
Are these settings taken from the dump of SuperIO registers with vendor firmware? […]
Hello! Yes, it usually should be the case, but that's not how vendor firmware had it configured.
I spoke with Michal about it at FOSDEM, they never tested Environment Controller on IT8613E. Fan control just happened to work, but reading CPU temperature doesn't - no matter how I configure it (we tried configuring it as PECI, but no dice).
Since IT8613E datasheet is under NDA, he wrote this patch to grab extra dumps, but it will require more work: https://review.coreboot.org/c/coreboot/+/80318
If you would like to confirm this, you can find register dump from stock firmware here: https://github.com/ellyq/erying-logs/blob/main/tgl/sio-rset.log