Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33913 )
Change subject: mb/asus/am1i-a/buildOpts.c: reorder lines for comparison convenience ......................................................................
Patch Set 7: Code-Review+1
(3 comments)
https://review.coreboot.org/c/coreboot/+/33913/7//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/33913/7//COMMIT_MSG@9 PS7, Line 9: i.e. Remove, it isn't grammatically correct
It also makes the line short enough to fit in 72 chars
https://review.coreboot.org/c/coreboot/+/33913/7//COMMIT_MSG@10 PS7, Line 10: This is needed to improve the convenience of comparison during the debugging. This improves diff results, which is convenient for debugging.
https://review.coreboot.org/c/coreboot/+/33913/7//COMMIT_MSG@11 PS7, Line 11: Maybe test this with BUILD_TIMELESS=1 to make sure it is just a cosmetic change:
make BUILD_TIMELESS=1 -j$(nproc)
Build and compare (diff -s -q) the coreboot.rom that results before and after applying this change, they should be identical.