Change in ...coreboot[master]: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 75 de...
Sumeet R Pawnikar has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/33130
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 75 degree C ......................................................................
mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 75 degree C
PMC logic shuts down the PCH thermal sensor when CPU is in a C-state and DTS Temp <= Low Temp Threshold (LTT) in case of Dynamic Thermal shutdown when S0ix is enabled.
BUG=None BRANCH=None TEST=Verified Thermal Device(B0: D18: F0) TSPM offset 0x1c [LTT (8:0)] value is 0xFA.
Change-Id: I1915b974b10638b0f6ab97c6fb9b7a58d2cabc59 Signed-off-by: Sumeet Pawnikar sumeet.r.pawnikar@intel.com --- M src/mainboard/google/sarien/variants/arcada/devicetree.cb 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/30/33130/1
diff --git a/src/mainboard/google/sarien/variants/arcada/devicetree.cb b/src/mainboard/google/sarien/variants/arcada/devicetree.cb index 767df1f..b6aa041 100644 --- a/src/mainboard/google/sarien/variants/arcada/devicetree.cb +++ b/src/mainboard/google/sarien/variants/arcada/devicetree.cb @@ -164,6 +164,9 @@
register "tcc_offset" = "1"
+ # PCH Trip Temperature in degree C + register "pch_trip_temp" = "75" + register "common_soc_config" = "{ .chipset_lockdown = CHIPSET_LOCKDOWN_COREBOOT, .i2c[0] = {
Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33130 )
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 75 degree C ......................................................................
Patch Set 1:
This CL depends on https://review.coreboot.org/c/coreboot/+/33129
Hello Aaron Durbin, Duncan Laurie, build bot (Jenkins), Furquan Shaikh, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/33130
to look at the new patch set (#3).
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C
PMC logic shuts down the PCH thermal sensor when CPU is in a C-state and DTS Temp <= Low Temp Threshold (LTT) in case of Dynamic Thermal shutdown when S0ix is enabled.
BUG=None BRANCH=None TEST=Verified Thermal Device(B0: D18: F0) TSPM offset 0x1c [LTT (8:0)] value is 0xFA.
Change-Id: I1915b974b10638b0f6ab97c6fb9b7a58d2cabc59 Signed-off-by: Sumeet Pawnikar sumeet.r.pawnikar@intel.com --- M src/mainboard/google/sarien/variants/arcada/devicetree.cb 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/30/33130/3
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33130 )
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/33130/3/src/mainboard/google/sarien... File src/mainboard/google/sarien/variants/arcada/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/33130/3/src/mainboard/google/sarien... PS3, Line 224: device pci 12.0 on end # Thermal Subsystem place under thermal device
Hello Aaron Durbin, Duncan Laurie, build bot (Jenkins), Furquan Shaikh, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/33130
to look at the new patch set (#4).
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C
PMC logic shuts down the PCH thermal sensor when CPU is in a C-state and DTS Temp <= Low Temp Threshold (LTT) in case of Dynamic Thermal shutdown when S0ix is enabled.
BUG=None BRANCH=None TEST=Verified Thermal Device(B0: D18: F0) TSPM offset 0x1c [LTT (8:0)] value is 0xFE on Arcada.
Change-Id: I1915b974b10638b0f6ab97c6fb9b7a58d2cabc59 Signed-off-by: Sumeet Pawnikar sumeet.r.pawnikar@intel.com --- M src/mainboard/google/sarien/variants/arcada/devicetree.cb 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/30/33130/4
Hello Aaron Durbin, Duncan Laurie, build bot (Jenkins), Furquan Shaikh, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/33130
to look at the new patch set (#5).
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C
PMC logic shuts down the PCH thermal sensor when CPU is in a C-state and DTS Temp <= Low Temp Threshold (LTT) in case of Dynamic Thermal shutdown when S0ix is enabled.
BUG=None BRANCH=None TEST=Verified Thermal Device(B0: D18: F0) TSPM offset 0x1c [LTT (8:0)] value is 0xFE on Arcada.
Change-Id: I1915b974b10638b0f6ab97c6fb9b7a58d2cabc59 Signed-off-by: Sumeet Pawnikar sumeet.r.pawnikar@intel.com --- M src/mainboard/google/sarien/variants/arcada/devicetree.cb 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/30/33130/5
Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33130 )
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
Patch Set 5:
(1 comment)
https://review.coreboot.org/c/coreboot/+/33130/3/src/mainboard/google/sarien... File src/mainboard/google/sarien/variants/arcada/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/33130/3/src/mainboard/google/sarien... PS3, Line 224: device pci 12.0 on end # Thermal Subsystem
place under thermal device
Ok
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33130 )
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
Patch Set 5: Code-Review+1
Change looks good to me. I will let Duncan +2 this.
Hello Aaron Durbin, Duncan Laurie, build bot (Jenkins), Furquan Shaikh, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/33130
to look at the new patch set (#6).
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C
PMC logic shuts down the PCH thermal sensor when CPU is in a C-state and DTS Temp <= Low Temp Threshold (LTT) in case of Dynamic Thermal shutdown when S0ix is enabled.
BUG=None BRANCH=None TEST=Verified Thermal Device(B0: D18: F0) TSPM offset 0x1c [LTT (8:0)] value is 0xFE on Arcada.
Change-Id: I1915b974b10638b0f6ab97c6fb9b7a58d2cabc59 Signed-off-by: Sumeet Pawnikar sumeet.r.pawnikar@intel.com --- M src/mainboard/google/sarien/variants/arcada/devicetree.cb 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/30/33130/6
Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33130 )
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
Patch Set 6:
Patch Set 5: Code-Review+1
Change looks good to me. I will let Duncan +2 this.
I have uploaded new patch set 6 as per recent common/block/thermal patches merged based on pch thermal trip added under common_soc_config.
Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33130 )
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
Patch Set 6:
Request to review this. Thanks.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33130 )
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
Patch Set 6: Code-Review+2
Martin Roth has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/33130 )
Change subject: mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C ......................................................................
mb/google/sarien/variants/arcada: Set PCH Thermal Trip point to 77 degree C
PMC logic shuts down the PCH thermal sensor when CPU is in a C-state and DTS Temp <= Low Temp Threshold (LTT) in case of Dynamic Thermal shutdown when S0ix is enabled.
BUG=None BRANCH=None TEST=Verified Thermal Device(B0: D18: F0) TSPM offset 0x1c [LTT (8:0)] value is 0xFE on Arcada.
Change-Id: I1915b974b10638b0f6ab97c6fb9b7a58d2cabc59 Signed-off-by: Sumeet Pawnikar sumeet.r.pawnikar@intel.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/33130 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Furquan Shaikh furquan@google.com --- M src/mainboard/google/sarien/variants/arcada/devicetree.cb 1 file changed, 3 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Furquan Shaikh: Looks good to me, approved
diff --git a/src/mainboard/google/sarien/variants/arcada/devicetree.cb b/src/mainboard/google/sarien/variants/arcada/devicetree.cb index 1799127..ebcf140 100644 --- a/src/mainboard/google/sarien/variants/arcada/devicetree.cb +++ b/src/mainboard/google/sarien/variants/arcada/devicetree.cb @@ -164,6 +164,9 @@
register "tcc_offset" = "1"
+ # PCH Thermal Trip Temperature in deg C + register "common_soc_config.pch_thermal_trip" = "77" + register "common_soc_config" = "{ .chipset_lockdown = CHIPSET_LOCKDOWN_COREBOOT, .i2c[0] = {
-
Furquan Shaikh (Code Review) -
Martin Roth (Code Review)
-
Patrick Rudolph (Code Review)
-
Sumeet R Pawnikar (Code Review)