[coreboot-gerrit] New patch to review for coreboot: libpayload: Add VPD address into lib_sysinfo

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Thu Jan 12 16:40:34 CET 2017


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18124

-gerrit

commit 8bd27e69e01cb849431fce324ecd7cc5ff686f38
Author: Kan Yan <kyan at google.com>
Date:   Thu Jan 5 19:42:53 2017 -0800

    libpayload: Add VPD address into lib_sysinfo
    
    BUG=chrome-os-partner:56947
    TEST=Verifed country code can be parsed from VPD in depthcharge.
    BRANCH=None
    
    Change-Id: I2fbbd4a784c50538331747e1ef78c33c6b8a679b
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: acea6e2a200e8bd78fd458255ac7fad307406989
    Original-Change-Id: I4616fefc6a377d7830397cdadb493927358e25cc
    Original-Signed-off-by: Kan Yan <kyan at google.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/425819
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
---
 payloads/libpayload/include/coreboot_tables.h | 1 +
 payloads/libpayload/include/sysinfo.h         | 1 +
 payloads/libpayload/libc/coreboot.c           | 9 +++++++++
 3 files changed, 11 insertions(+)

diff --git a/payloads/libpayload/include/coreboot_tables.h b/payloads/libpayload/include/coreboot_tables.h
index d77242a..9e00c0e 100644
--- a/payloads/libpayload/include/coreboot_tables.h
+++ b/payloads/libpayload/include/coreboot_tables.h
@@ -206,6 +206,7 @@ struct cb_gpios {
 #define CB_TAG_DMA		0x0022
 #define CB_TAG_RAM_OOPS		0x0023
 #define CB_TAG_MTC		0x002b
+#define CB_TAG_VPD		0x002c
 struct lb_range {
 	uint32_t tag;
 	uint32_t size;
diff --git a/payloads/libpayload/include/sysinfo.h b/payloads/libpayload/include/sysinfo.h
index 2110d88..2e5a837 100644
--- a/payloads/libpayload/include/sysinfo.h
+++ b/payloads/libpayload/include/sysinfo.h
@@ -124,6 +124,7 @@ struct sysinfo_t {
 	uint64_t boot_media_size;
 	uint64_t mtc_start;
 	uint32_t mtc_size;
+	void	*chromeos_vpd;
 };
 
 extern struct sysinfo_t lib_sysinfo;
diff --git a/payloads/libpayload/libc/coreboot.c b/payloads/libpayload/libc/coreboot.c
index 9fc6a69..10b801c 100644
--- a/payloads/libpayload/libc/coreboot.c
+++ b/payloads/libpayload/libc/coreboot.c
@@ -228,6 +228,12 @@ static void cb_parse_boot_media_params(unsigned char *ptr,
 	info->boot_media_size = bmp->boot_media_size;
 }
 
+static void cb_parse_vpd(void *ptr, struct sysinfo_t *info)
+{
+	struct cb_cbmem_tab *const cbmem = (struct cb_cbmem_tab *)ptr;
+	info->chromeos_vpd = phys_to_virt(cbmem->cbmem_tab);
+}
+
 #if IS_ENABLED(CONFIG_LP_TIMER_RDTSC)
 static void cb_parse_tsc_info(void *ptr, struct sysinfo_t *info)
 {
@@ -398,6 +404,9 @@ int cb_parse_header(void *addr, int len, struct sysinfo_t *info)
 			cb_parse_tsc_info(ptr, info);
 			break;
 #endif
+		case CB_TAG_VPD:
+			cb_parse_vpd(ptr, info);
+			break;
 		default:
 			cb_parse_arch_specific(rec, info);
 			break;



More information about the coreboot-gerrit mailing list