Attention is currently required from: Martin Roth. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52181 )
Change subject: mb/google/guybrush/var/guybrush: Add FPMCU configration ......................................................................
Patch Set 1:
(4 comments)
File src/mainboard/google/guybrush/mainboard.c:
https://review.coreboot.org/c/coreboot/+/52181/comment/dc2787e7_0377b7d8 PS1, Line 121: /* Default weak implementation */ code indent should use tabs where possible
File src/mainboard/google/guybrush/variants/guybrush/variant.c:
https://review.coreboot.org/c/coreboot/+/52181/comment/60d9519e_e5052f8c PS1, Line 14: gpio_set(GPIO_32, 1); please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/52181/comment/03edf64b_5cb22cef PS1, Line 15: mdelay(3); please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/52181/comment/4a174074_fdf2d9a4 PS1, Line 16: gpio_set(GPIO_11, 1); please, no spaces at the start of a line