Attention is currently required from: HAOUAS Elyes. Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50286 )
Change subject: Documentation: Add chain-include list of headers ......................................................................
Patch Set 4:
(2 comments)
File Documentation/chain-include.md:
https://review.coreboot.org/c/coreboot/+/50286/comment/fa9a956c_d19b3d69 PS4, Line 5: boot You're generally inconsistent what you consider as toplevel directory. If you want to write `include/cbfs.h`, you also have to write `include/boot/coreboot_tables.h` here.
I would recommend just always using the <cbfs.h> / <commonlib/cbfs.h> notation and write all files the same way they would appear in an actual #include directive.
https://review.coreboot.org/c/coreboot/+/50286/comment/7ecca028_29977553 PS4, Line 43: metadata_hash include/ ?