Mike Banon has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33914 )
Change subject: mb/asus/am1i-a/buildOpts.c: detailed paths to the include files ......................................................................
Patch Set 7:
(3 comments)
https://review.coreboot.org/c/coreboot/+/33914/7//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/33914/7//COMMIT_MSG@7 PS7, Line 7: detailed paths to the include files
Use fully-qualified paths on includes
Done.
https://review.coreboot.org/c/coreboot/+/33914/7//COMMIT_MSG@9 PS7, Line 9: Make it clear which include files are being used, by using the detailed paths.
This makes it easier to know which files are being included.
Done.
https://review.coreboot.org/c/coreboot/+/33914/7/src/mainboard/asus/am1i-a/b... File src/mainboard/asus/am1i-a/buildOpts.c:
https://review.coreboot.org/c/coreboot/+/33914/7/src/mainboard/asus/am1i-a/b... PS7, Line 20: /* Include the files that instantiate the configuration definitions. */
Do these files depend on the #defines that originally preceded them?
Just checked, luckily there aren't such dependencies.