Re: [coreboot] Change in coreboot[master]: coreboot_table: don't add CMOS checksum twice.

[Nico Huber]

Am 16.04.2015 15:12, schrieb gerrit code review:
> From Nico Huber :
>
> Nico Huber has posted comments on this change.
>
> Change subject: coreboot_table: don't add CMOS checksum twice.
> ......................................................................
>
> Patch Set 3:
>
> Ahh, took me a day to find this commit and what is going wrong.
>
> You were right, that the checksum appeared twice in a row. But that's
> only true, if you don't read the table from the beginning: The
> checksum from the cmos_layout is nested in another tag. That makes a
> difference because libpayload parses the table, looks for the
> checksum, but doesn't understand nested tags.
>
> I hadn't the time to check whether this problem persists upstream, but
> it looks like it does.

This still persists. To be clear: This change was meant to be cosmetic
but has real impact. Upstream, libpayload is unable to read nvram
variables since this has been merged (January 2014).

The commit still seems to be a good idea. But If nobody plans to adapt
libpayload, shall we revert it?

Nico