[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/cannonlake: Add support to log XHCI wake events

[Paul Fagerburg (Code Review)]

Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33843 )

Change subject: soc/intel/cannonlake: Add support to log XHCI wake events
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/#/c/33843/2/src/soc/intel/cannonlake/elog.c 
File src/soc/intel/cannonlake/elog.c:

https://review.coreboot.org/#/c/33843/2/src/soc/intel/cannonlake/elog.c@70 
PS2, Line 70: if (ps->gpe0_sts[GPE_STD] & PME_B0_STS)
            : 		pch_xhci_update_wake_event(&usb_info);
> With the current change, you would lose information about PME events other than XHCI. […]

Crap, you're right. Thank you for the pointer to skylake source. I'll get another CL started.