Am Dienstag, den 18.01.2011, 15:22 +0100 schrieb Mathias Krause:
> Well, but after the build the CBFS could still be modified (intentional
> or not) in such a way that it would no longer contain a file named
> "cmos_layout.bin". So at least to aid debugging it would be nice to see
> a warning here instead of wondering why the CMOS layout is missing in
> the LB table.
I added some reporting in r6269.

> I guess you assume it'll return either 1 (good case) or 0 (bad case) not
> -1 - which is the case btw. But you're right. I mixed up the size and
> count parameters of the fwrite() call. So you check is fine the way it is.
r6270 makes this more explicit for all calls (so return -E* bails out,
too).


Patrick

Attachment: signature.asc
Description: This is a digitally signed message part

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to