[coreboot-gerrit] Change in coreboot[master]: soc/intel/skylake/elog: fix BUG: pch_log_rp_wake_source requests hidden

[Michael Niewöhner (Code Review)]

Hello Kyösti Mälkki, Patrick Rudolph, Angel Pons, build bot (Jenkins), Nico Huber, Patrick Georgi,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/39089

to look at the new patch set (#2).

Change subject: soc/intel/skylake/elog: fix BUG: pch_log_rp_wake_source requests hidden
......................................................................

soc/intel/skylake/elog: fix BUG: pch_log_rp_wake_source requests hidden

The current elog implemetation searches for an active PME status bit by
iterating the PCI devices. On disabled or hidden devices a BUG gets
triggered: BUG: pch_log_rp_wake_source requests hidden ...

This is caused by the use of the PCH_DEV_* macros which resolve to
_PCH_DEV and finally call pcidev_path_on_root_debug.

Disabled devices are skipped already so we can safely use the DEVFNs
instead, circumventing the BUG.

Change-Id: Id126e2c51aec84a4af9354b39754ee74687cefc8
Signed-off-by: Michael Niewöhner 
---
M src/soc/intel/skylake/elog.c
1 file changed, 39 insertions(+), 58 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/39089/2