Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35531 )
Change subject: Makefile: Create the build directory before bootblock.bin ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35531/3/Makefile.inc File Makefile.inc:
https://review.coreboot.org/c/coreboot/+/35531/3/Makefile.inc@507 PS3, Line 507: build-dirs $(objcbfs) $(objgenerated):
I added the actual directory variables because it was a pain trying to search to see what random pho […]
I think the phony target was created originally to decouple the dependencies of files to their directories: if you have foo/bar.o depend on foo/, any change to foo/* will update foo's timestamp. This in turn means that make considers foo/bar.o to be out of date, regenerating it.
Might be worth checking if we're still needing and achieving that decoupling or if we're cargo-culting some stuff along that outlived its usefulness a long time ago.