CBFS file format changed but documentation not updated

[Kevin O'Connor]

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

CBFS file format changed but documentation not updated

[Alex G.]

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