David Hendricks has posted comments on this change. ( https://review.coreboot.org/25381 )
Change subject: Remove address from GPLv2 headers ......................................................................
Patch Set 5: Code-Review+1
(2 comments)
minor nits, otherwise LGTM
https://review.coreboot.org/#/c/25381/5/buspirate_spi.c File buspirate_spi.c:
https://review.coreboot.org/#/c/25381/5/buspirate_spi.c@660 PS5, Line 660: } Looks like one of your whitespace fixes crept in?
https://review.coreboot.org/#/c/25381/5/coreboot_tables.h File coreboot_tables.h:
https://review.coreboot.org/#/c/25381/5/coreboot_tables.h@28 PS5, Line 28: * All of the information should be Position Independent Data. I just noticed that this long comment has a few trailing whitespaces which you may want to address in 25380.