Attention is currently required from: Michał Kopeć, Felix Held. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63927 )
Change subject: superio/nuvoton/nct6687d: Add ramstage driver and ACPI ......................................................................
Patch Set 8: Verified-1
(2 comments)
File src/superio/nuvoton/nct6687d/superio.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150382): https://review.coreboot.org/c/coreboot/+/63927/comment/fcd5d499_a0e303ba PS8, Line 42: printk(BIOS_INFO, "set power %s after power fail\n", power_status ? "on" : "off"); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-150382): https://review.coreboot.org/c/coreboot/+/63927/comment/46a5915e_a72533c3 PS8, Line 60: { NULL, NCT6687D_KBC, PNP_IO0 | PNP_IO1 | PNP_IRQ0 | PNP_IRQ1 | PNP_MSC0, trailing whitespace