Furquan Shaikh 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 89: b/136673612 This has no meaning for upstream coreboot since b/ is accessible only to Googlers. It is better to just post a comment referencing doc# or have a generic comment that explains the context.