build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62080 )
Change subject: sb/amd/sb700: Strip the code just to be buildable ......................................................................
Patch Set 1:
(2 comments)
File src/southbridge/amd/sb700/fadt.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-141257): https://review.coreboot.org/c/coreboot/+/62080/comment/7e6f6323_0f06886f PS1, Line 29: void acpi_fill_fadt(acpi_fadt_t * fadt) "foo * bar" should be "foo *bar"
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-141257): https://review.coreboot.org/c/coreboot/+/62080/comment/ad05b449_5da2a642 PS1, Line 142: // amd_powernow_update_fadt(fadt); please, no space before tabs