[coreboot-gerrit] Change in coreboot[master]: soc/intel/skylake: Remove print wrongly showing ME PG-state

[Barnali Sarkar (Code Review)]

Barnali Sarkar has posted comments on this change. ( https://review.coreboot.org/20172 )

Change subject: soc/intel/skylake: Remove print wrongly showing ME PG-state
......................................................................


Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/20172/1//COMMIT_MSG
Commit Message:

https://review.coreboot.org/#/c/20172/1//COMMIT_MSG@19
PS1, Line 19:  also it needs some time for the ME FW to go to idle state
            : and reflect these values in PWRMBASE registers after D0i3 bit
            : is being set. This does not happen instantly.
> Question asked to ME team -
>>> I cannot say when CSME goes into idle state since it’s up to OS and applications activities which are running on CSME. 

Here, ME OS is meant.



-- 
To view, visit https://review.coreboot.org/20172
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idd31a9803b4c9db7d4bb8bbec5374583a8df0c41
Gerrit-Change-Number: 20172
Gerrit-PatchSet: 1
Gerrit-Owner: Barnali Sarkar <barnali.sarkar at intel.com>
Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan at intel.com>
Gerrit-Reviewer: Barnali Sarkar <barnali.sarkar at intel.com>
Gerrit-Reviewer: Pratikkumar V Prajapati <pratikkumar.v.prajapati at intel.com>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi at intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik at intel.com>
Gerrit-Reviewer: dhaval v sharma <dhaval.v.sharma at intel.com>
Gerrit-Comment-Date: Wed, 14 Jun 2017 11:41:29 +0000
Gerrit-HasComments: Yes