From 0859991c4bafa00db675ca587f90cebfcffe35f1 Mon Sep 17 00:00:00 2001 From: Nico Huber Date: Mon, 21 Sep 2015 20:11:47 +0200 Subject: [PATCH] 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 Change-Id: I6cf6b2a1523d771bb52f3d5720b1b16ed4b348db Reviewed-on: http://review.coreboot.org/11696 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer Reviewed-by: Patrick Georgi --- src/lib/coreboot_table.c | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) 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 #include #include +#if CONFIG_USE_OPTION_TABLE +#include +#endif #if CONFIG_CHROMEOS #if CONFIG_HAVE_ACPI_TABLES #include @@ -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"); }