16 Sep
2019
16 Sep
'19
11:08 a.m.
Maxim Polyakov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35368 )
Change subject: mb/asrock/h110m: fix SB-TSI address for SuperIO HWM ......................................................................
Patch Set 1:
Patch Set 1:
Thanks for the review
wasn't SB-TSI something specific to AMD FCHs? haven't looked in the SIO datasheet if that is also used for something else on Intel platforms
Yes you are right. Temperature Sensor Interface is used only with AMD processors. Probably the AMI firmware configures all possible interfaces in the SuperIO. This is confusing for me
--
To view, visit https://review.coreboot.org/c/coreboot/+/35368
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3aeb75bcf124a65dfebf3a8caa6ca150de617431
Gerrit-Change-Number: 35368
Gerrit-PatchSet: 1
Gerrit-Owner: Maxim Polyakov max.senia.poliak@gmail.com
Gerrit-Reviewer: Maxim Polyakov max.senia.poliak@gmail.com
Gerrit-Reviewer: Patrick Rudolph
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: Felix Held felix-coreboot@felixheld.de
Gerrit-Comment-Date: Mon, 16 Sep 2019 09:08:07 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment