sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ChromeOS: Drop filling ECFW_RW/RO state in CNVS

Browse Source 
This field was never meant to be filled out by coreboot, because it
can't know what the right value for this will be by the time the OS
is running, so anything coreboot could fill in here is premature.

This field is only read by the chromeos-specific `crossystem` utility,
not by kernel code, so if one does not run through depthcharge there'll
be many more broken assumptions in CNVS anyway.

Change-Id: Ia56b3a3fc82f1b8247a6ee512fe960e9d3d87585
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63290
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
This commit is contained in:
Kyösti Mälkki
2022-03-29 00:44:51 +03:00
parent e50bb8fc9e
commit 740eee5eec
6 changed files with 0 additions and 38 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/vendorcode/google/chromeos/gnvs.c
View File

@ -32,8 +32,6 @@ static size_t chromeos_vpd_region(const char *region, uintptr_t *base)
return region_device_sz(&vpd);
}
__weak bool mainboard_ec_running_ro(void) { return true; }
void chromeos_init_chromeos_acpi(void)
{
size_t vpd_size;
@ -58,17 +56,6 @@ void chromeos_init_chromeos_acpi(void)
chromeos_acpi->vpd_rw_base = vpd_base;
chromeos_acpi->vpd_rw_size = vpd_size;
}
/* EC can override to ECFW_RW. */
chromeos_acpi->vbt2 = ACTIVE_ECFW_RO;
if (CONFIG(EC_GOOGLE_CHROMEEC)) {
if (!google_ec_running_ro())
chromeos_acpi->vbt2 = ACTIVE_ECFW_RW;
} else {
if (!mainboard_ec_running_ro())
chromeos_acpi->vbt2 = ACTIVE_ECFW_RW;
}
}
void chromeos_set_me_hash(u32 *hash, int len)