Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34289 )
Change subject: soc/intel/cannonlake: Split the "internal PME" wake-up into more detail ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/34289/1/src/soc/intel/cannonlake/el... File src/soc/intel/cannonlake/elog.c:
https://review.coreboot.org/c/coreboot/+/34289/1/src/soc/intel/cannonlake/el... PS1, Line 57: struct device *dev We don't seem to use this parameter inside this function. Do we need it?