diff options
| author | Nico Huber <nico.h@gmx.de> | 2015-09-21 20:11:47 +0200 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2015-10-20 16:35:12 +0200 |
| commit | 0859991c4bafa00db675ca587f90cebfcffe35f1 (patch) | |
| tree | 35f7b5a2ec45cb8506462cc86027eb84e3a7abd7 /src/lib | |
| parent | 821bcd64562b6ab7098503afc0a97d7fd11c1291 (diff) | |
| download | coreboot-0859991c4bafa00db675ca587f90cebfcffe35f1.tar.xz | |
Revert "coreboot_table: don't add CMOS checksum twice."
This reverts commit e6606518243d9beda31693d40493b5f7a1a3e2e0.
After some discussion on IRC we decided to revert it as libpayload can
only read the copy that was removed (and other users like nvramtool can
only read the other copy). So we need both copies at this time.
Signed-off-by: Nico Huber <nico.h@gmx.de>
Change-Id: I6cf6b2a1523d771bb52f3d5720b1b16ed4b348db
Reviewed-on: http://review.coreboot.org/11696
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'src/lib')
| -rw-r--r-- | src/lib/coreboot_table.c | 25 |
1 files changed, 25 insertions, 0 deletions
diff --git a/src/lib/coreboot_table.c b/src/lib/coreboot_table.c index 6859bf2d91..35341abee1 100644 --- a/src/lib/coreboot_table.c +++ b/src/lib/coreboot_table.c @@ -34,6 +34,9 @@ #include <cbmem.h> #include <bootmem.h> #include <spi_flash.h> +#if CONFIG_USE_OPTION_TABLE +#include <option_table.h> +#endif #if CONFIG_CHROMEOS #if CONFIG_HAVE_ACPI_TABLES #include <arch/acpi.h> @@ -333,6 +336,26 @@ static struct lb_mainboard *lb_mainboard(struct lb_header *header) return mainboard; } +#if CONFIG_USE_OPTION_TABLE +static struct cmos_checksum *lb_cmos_checksum(struct lb_header *header) +{ + struct lb_record *rec; + struct cmos_checksum *cmos_checksum; + rec = lb_new_record(header); + cmos_checksum = (struct cmos_checksum *)rec; + cmos_checksum->tag = LB_TAG_OPTION_CHECKSUM; + + cmos_checksum->size = (sizeof(*cmos_checksum)); + + cmos_checksum->range_start = LB_CKS_RANGE_START * 8; + cmos_checksum->range_end = ( LB_CKS_RANGE_END * 8 ) + 7; + cmos_checksum->location = LB_CKS_LOC * 8; + cmos_checksum->type = CHECKSUM_PCBIOS; + + return cmos_checksum; +} +#endif + static void lb_strings(struct lb_header *header) { static const struct { @@ -435,6 +458,8 @@ unsigned long write_coreboot_table( struct lb_record *rec_dest = lb_new_record(head); /* Copy the option config table, it's already a lb_record... */ memcpy(rec_dest, option_table, option_table->size); + /* Create cmos checksum entry in coreboot table */ + lb_cmos_checksum(head); } else { printk(BIOS_ERR, "cmos_layout.bin could not be found!\n"); } |