coreboot-kgpe-d16/src
Barnali Sarkar a1af090002 soc/intel/skylake: Remove incorrect ME PG-status print
As per discussion with CSME team, ME is NOT using PCI Config
Space register HFSTS2 Bit 10 to update ME power-gated status.

ME goes to CM0-PG state after ME device becomes idle after
Bit 2 of MMIO register offset 0x800 (D0i3 Control - HECI1_D0I3C)
is being set.

And to retrieve the PG status of ME, one should read from the
PWRMBASE+offset 0x590 (which should give the value 0xF9) and
PWRMBASE+offset 0x594 (which should give the value 0xFF).

But, 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.

So, in coreboot, if we read the ME PG state in finalize.c, which
happens just after FSP Notify phase, where actually ME D0i3 bit
is set, we do not read the correct PG state values (i.e, 0xF9
and 0xFF).

But, once it boots to Kernel, if we read those same registers
through iotool mmio_read32 command, we get correct values.

So, removing the ME PG state prints from coreboot, since it is
actually showing wrong information, although ME Power Gating is
successful.

Change-Id: Idd31a9803b4c9db7d4bb8bbec5374583a8df0c41
Signed-off-by: Barnali Sarkar <barnali.sarkar@intel.com>
Reviewed-on: https://review.coreboot.org/20172
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
2017-07-28 16:17:26 +00:00
..
acpi src/acpi: Capitalize ACPI and SATA 2016-07-31 19:25:40 +02:00
arch Port cmos.default handling to C environment bootblock 2017-07-26 19:30:01 +00:00
commonlib Rename __attribute__((packed)) --> __packed 2017-07-13 19:45:59 +00:00
console src: add IS_ENABLED() around Kconfig symbol references 2017-07-13 23:57:07 +00:00
cpu Fix files with multiple newlines at the end. 2017-07-24 15:08:08 +00:00
device src/device: Fix checkpatch warning: no spaces at the start of a line 2017-07-25 14:26:20 +00:00
drivers siemens/nc_fpga: Add support for lowest FAN speed to FAN controller 2017-07-28 16:16:42 +00:00
ec ec/lenovo/h8/h8: Always enable tp-smapi and thermal 2017-07-18 19:23:26 +00:00
include timestamp: Apply factor to recorded stamps 2017-07-27 06:55:03 +00:00
lib timestamp: Apply factor to recorded stamps 2017-07-27 06:55:03 +00:00
mainboard google/kahlee: Add ASL for Elan touchpad 2017-07-28 16:13:40 +00:00
northbridge nb/intel/i440bx: Add final newline to raminit.c 2017-07-24 17:48:54 +00:00
soc soc/intel/skylake: Remove incorrect ME PG-status print 2017-07-28 16:17:26 +00:00
southbridge sb/intel/i82801jx: Add Interrupt pin and routing RCBA offsets macros 2017-07-25 15:15:41 +00:00
superio superio/ite/it8716f: Update init_ec 2017-07-08 19:02:13 +00:00
vboot vboot: Remove get_sw_write_protect_state callback 2017-07-18 23:24:01 +00:00
vendorcode vendorcode/siemens: Fix typo in hwilib 2017-07-28 16:16:05 +00:00
Kconfig Add support for Undefined Behavior Sanitizer 2017-06-14 19:56:59 +02:00