Hello Rajneesh Bhardwaj, Rushikesh S Kadam, build bot (Jenkins), Furquan Shaikh,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/22237
to look at the new patch set (#2).
Change subject: soc/intel/skylake: Turnoff XTAL Disqualification for XTAL Shutdown
......................................................................
soc/intel/skylake: Turnoff XTAL Disqualification for XTAL Shutdown
24MHz Crystal Shutdown Qualification bit (0x31Ch[22]) need to be '0'
if system can enter S0ix with XTAL Shutdown.
Note: This patch should be applyied togather with below two patches
so that system can enter S0ix with XTAL Shutdown.
1. https://review.coreboot.org/c/22236
2. https://review.coreboot.org/c/22240
BUG=None
BRANCH=None
TEST=Build for kabylake board with XTAL Disqualification off in pmc.h
And read 24MHz Crystal Shutdown Qualification bit (0x31Ch[22])
iotools mmio_read32 0xFE00031c should return '0' on BIT[22]
Change-Id: I8abc6e9b88b3bc25cf865824aa14b292ac0524ef
Signed-off-by: Roy Mingi Park <roy.mingi.park(a)intel.com>
---
M src/soc/intel/skylake/include/soc/pmc.h
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/37/22237/2
--
To view, visit https://review.coreboot.org/22237
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8abc6e9b88b3bc25cf865824aa14b292ac0524ef
Gerrit-Change-Number: 22237
Gerrit-PatchSet: 2
Gerrit-Owner: Roy Mingi Park <roy.mingi.park(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Rajneesh Bhardwaj <rajneesh.bhardwaj(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Rushikesh S Kadam <rushikesh.s.kadam(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
rushikesh s kadam has posted comments on this change. ( https://review.coreboot.org/22237 )
Change subject: soc/intel/skylake: Turnoff XTAL Disqualification for XTAL Shutdown
......................................................................
Patch Set 1:
Roy,
If we turn off XTAL disqualification, the SoC will not enter S0ix for scenarios such as Wake-On-Voice.
We could enable / disable in ASL based on WoV status, or we could just leave it on.
Btw I did not see you reset the bit in this patch?
--
To view, visit https://review.coreboot.org/22237
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8abc6e9b88b3bc25cf865824aa14b292ac0524ef
Gerrit-Change-Number: 22237
Gerrit-PatchSet: 1
Gerrit-Owner: Roy Mingi Park <roy.mingi.park(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Rajneesh Bhardwaj <rajneesh.bhardwaj(a)intel.corp-partner.google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: rushikesh s kadam <rushikesh.s.kadam(a)intel.com>
Gerrit-Comment-Date: Tue, 31 Oct 2017 03:20:13 +0000
Gerrit-HasComments: No
Roy Mingi Park has removed Rajneesh Bhardwaj from this change. ( https://review.coreboot.org/22236 )
Change subject: soc/intelskylake: enable 8254 Static Clock Gating for XTAL shutdown
......................................................................
Removed reviewer Rajneesh Bhardwaj.
--
To view, visit https://review.coreboot.org/22236
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: deleteReviewer
Gerrit-Change-Id: I87fb6fa86f10be6e3c888dd3590d0e77a2b886e8
Gerrit-Change-Number: 22236
Gerrit-PatchSet: 2
Gerrit-Owner: Roy Mingi Park <roy.mingi.park(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Rajat Jain <rajatja(a)google.com>
Gerrit-Reviewer: Rajneesh Bhardwaj <rajneesh.bhardwaj(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Roy Mingi Park <roy.mingi.park(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: rushikesh s kadam <rushikesh.s.kadam(a)intel.com>