Hello Subrata Banik, build bot (Jenkins), Patrick Georgi, Furquan Shaikh,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/37088
to review the following change.
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch"
This reverts commit 0bc35af93326ec3232ec73c9b1334241b85f0252.
Reason for revert: This change breaks runtime s0ix.
Change-Id: Ida80f55b56de7129ed629eb29bd14f2ef300126f --- M src/mainboard/google/hatch/variants/baseboard/devicetree.cb 1 file changed, 0 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/88/37088/1
diff --git a/src/mainboard/google/hatch/variants/baseboard/devicetree.cb b/src/mainboard/google/hatch/variants/baseboard/devicetree.cb index a2831e1..7382209 100644 --- a/src/mainboard/google/hatch/variants/baseboard/devicetree.cb +++ b/src/mainboard/google/hatch/variants/baseboard/devicetree.cb @@ -58,8 +58,6 @@
register "PmTimerDisabled" = "1"
- register "PchPmSlpS0Vm075VSupport" = "1" - # VR Settings Configuration for 4 Domains #+----------------+-------+-------+-------+-------+ #| Domain/Setting | SA | IA | GTUS | GTS |
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37088 )
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/37088/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/37088/1//COMMIT_MSG@11 PS1, Line 11: This change breaks runtime s0ix. Is there a bug # for this?
Hello Subrata Banik, Tim Wawrzynczak, build bot (Jenkins), Furquan Shaikh, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/37088
to look at the new patch set (#2).
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch"
This reverts commit 0bc35af93326ec3232ec73c9b1334241b85f0252.
Reason for revert: This change breaks runtime s0ix.
BUG=b:134092071 TEST=Check slp_s0 residency increased when system is idle.
Change-Id: Ida80f55b56de7129ed629eb29bd14f2ef300126f Signed-off-by: Kane Chen kane.chen@intel.com --- M src/mainboard/google/hatch/variants/baseboard/devicetree.cb 1 file changed, 0 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/88/37088/2
Kane Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37088 )
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/37088/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/37088/1//COMMIT_MSG@11 PS1, Line 11: This change breaks runtime s0ix.
Is there a bug # for this?
hi Furquan, i'm going to use the orig bug to check b:141831197 thanks.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37088 )
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Patch Set 2: Code-Review+2
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37088 )
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/37088/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/37088/2//COMMIT_MSG@12 PS2, Line 12: BRANCH=hatch
Hello Subrata Banik, Tim Wawrzynczak, Shelley Chen, build bot (Jenkins), Furquan Shaikh, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/37088
to look at the new patch set (#3).
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch"
This reverts commit 0bc35af93326ec3232ec73c9b1334241b85f0252.
Reason for revert: This change breaks runtime s0ix.
BRANCH=hatch BUG=b:134092071 TEST=Check slp_s0 residency increased when system is idle.
Change-Id: Ida80f55b56de7129ed629eb29bd14f2ef300126f Signed-off-by: Kane Chen kane.chen@intel.com --- M src/mainboard/google/hatch/variants/baseboard/devicetree.cb 1 file changed, 0 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/88/37088/3
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37088 )
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Patch Set 3:
I'm a little confused, I don't see any discussion in the bug listed as to where this setting is affecting runtime S0ix.
Hello Subrata Banik, Tim Wawrzynczak, Shelley Chen, build bot (Jenkins), Furquan Shaikh, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/37088
to look at the new patch set (#4).
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch"
This reverts commit 0bc35af93326ec3232ec73c9b1334241b85f0252.
Reason for revert: This change breaks runtime s0ix.
BRANCH=hatch BUG=b:141831197 TEST=Check slp_s0 residency increased when system is idle.
Change-Id: Ida80f55b56de7129ed629eb29bd14f2ef300126f Signed-off-by: Kane Chen kane.chen@intel.com --- M src/mainboard/google/hatch/variants/baseboard/devicetree.cb 1 file changed, 0 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/88/37088/4
Kane Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37088 )
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Patch Set 4:
Patch Set 3:
I'm a little confused, I don't see any discussion in the bug listed as to where this setting is affecting runtime S0ix.
sorry, i posted wrong issue number. i just fixed it
Kane Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37088 )
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Patch Set 4:
(2 comments)
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/37088/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/37088/1//COMMIT_MSG@11 PS1, Line 11: This change breaks runtime s0ix.
hi Furquan, […]
Done
https://review.coreboot.org/c/coreboot/+/37088/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/37088/2//COMMIT_MSG@12 PS2, Line 12:
BRANCH=hatch
Done
Kane Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37088 )
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Patch Set 4:
could we merge this? thanks
Furquan Shaikh has submitted this change. ( https://review.coreboot.org/c/coreboot/+/37088 )
Change subject: Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch" ......................................................................
Revert "mb/google/hatch: Enable PchPmSlpS0Vm075VSupport for hatch"
This reverts commit 0bc35af93326ec3232ec73c9b1334241b85f0252.
Reason for revert: This change breaks runtime s0ix.
BRANCH=hatch BUG=b:141831197 TEST=Check slp_s0 residency increased when system is idle.
Change-Id: Ida80f55b56de7129ed629eb29bd14f2ef300126f Signed-off-by: Kane Chen kane.chen@intel.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/37088 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Furquan Shaikh furquan@google.com --- M src/mainboard/google/hatch/variants/baseboard/devicetree.cb 1 file changed, 0 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Furquan Shaikh: Looks good to me, approved
diff --git a/src/mainboard/google/hatch/variants/baseboard/devicetree.cb b/src/mainboard/google/hatch/variants/baseboard/devicetree.cb index a2831e1..7382209 100644 --- a/src/mainboard/google/hatch/variants/baseboard/devicetree.cb +++ b/src/mainboard/google/hatch/variants/baseboard/devicetree.cb @@ -58,8 +58,6 @@
register "PmTimerDisabled" = "1"
- register "PchPmSlpS0Vm075VSupport" = "1" - # VR Settings Configuration for 4 Domains #+----------------+-------+-------+-------+-------+ #| Domain/Setting | SA | IA | GTUS | GTS |