Attention is currently required from: Patrick Rudolph. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49356 )
Change subject: soc/intel/broadwell: acpi_wake_source() ......................................................................
Patch Set 1:
(4 comments)
File src/soc/intel/baytrail/ramstage.c:
https://review.coreboot.org/c/coreboot/+/49356/comment/a5eaef5f_2cc22dad PS1, Line 126: if (!ps || !gnvs) spaces required around that '||' (ctx:WxV)
https://review.coreboot.org/c/coreboot/+/49356/comment/7fd40946_bb6545b1 PS1, Line 126: if (!ps || !gnvs) space required before that '!' (ctx:VxV)
File src/soc/intel/broadwell/ramstage.c:
https://review.coreboot.org/c/coreboot/+/49356/comment/2391e946_83357cd5 PS1, Line 22: if (!ps || !gnvs) spaces required around that '||' (ctx:WxV)
https://review.coreboot.org/c/coreboot/+/49356/comment/32b3a15e_05e858ea PS1, Line 22: if (!ps || !gnvs) space required before that '!' (ctx:VxV)