I noticed that commit 0d618afc modified the CBFS file format. (It looks like the "checksum" field was replaced with a new file "attributes" feature.) However, the CBFS documentation in the repository (Documentation/cbfs.txt) was not updated.
It would be great if the documentation could be kept in sync with the CBFS implementation. (Or, if that's not feasible, I think it would be preferable to remove Documentation/cbfs.txt .) Having up to date specifications helps other projects (eg, SeaBIOS) keep up with changes.
Also, I didn't see any emails for the above change on the mailing list. What is the best way to keep up with notable architectural changes in coreboot?
-Kevin
Hi Kevin,
Please accept our apologies for this oversight. This should have been caught during review, but it was missed. I'll try to get a patch together to update the documentation.
Alex
On 11/17/2015 12:36 PM, Kevin O'Connor wrote:
I noticed that commit 0d618afc modified the CBFS file format. (It looks like the "checksum" field was replaced with a new file "attributes" feature.) However, the CBFS documentation in the repository (Documentation/cbfs.txt) was not updated.
It would be great if the documentation could be kept in sync with the CBFS implementation. (Or, if that's not feasible, I think it would be preferable to remove Documentation/cbfs.txt .) Having up to date specifications helps other projects (eg, SeaBIOS) keep up with changes.
Also, I didn't see any emails for the above change on the mailing list. What is the best way to keep up with notable architectural changes in coreboot?
-Kevin
-
Alex G. -
Kevin O'Connor