diff options
author | Subrata Banik <subratabanik@google.com> | 2023-08-17 15:44:52 +0000 |
---|---|---|
committer | Subrata Banik <subratabanik@google.com> | 2023-08-18 20:20:03 +0000 |
commit | 88512b00ad2786795889a71b7835efe1cffba458 (patch) | |
tree | fbe0ab4b6ddc79db5597562459c0f43b7dc109ad /src | |
parent | c8a041757449d34bb347ee3f5974bbffceececf2 (diff) |
{driver, soc/intel/cmn/cse}: Refactor ISH FW Version implementation
This patch uses the CSE firmware specific data to store Intel
ISH firmware related information. Sending an ISH partition version
information command on every boot cycle would impact the overall boot
performance.
This information is used by the auto-test framework to ensure the ISH
firmware update is proper for in-field devices.
BUG=b:285405031
TEST=Able to build and boot google/rex. Verified ISH FW version is
getting displayed across warm resets without impacting the boot time.
Change-Id: I0242c26dd90d834815799f54740d8147ff9d45b7
Signed-off-by: Subrata Banik <subratabanik@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/77176
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Diffstat (limited to 'src')
-rw-r--r-- | src/drivers/intel/ish/ish.c | 12 | ||||
-rw-r--r-- | src/soc/intel/common/block/cse/cse_lite.c | 12 |
2 files changed, 14 insertions, 10 deletions
diff --git a/src/drivers/intel/ish/ish.c b/src/drivers/intel/ish/ish.c index a84be5b120..d1b85f570e 100644 --- a/src/drivers/intel/ish/ish.c +++ b/src/drivers/intel/ish/ish.c @@ -50,15 +50,15 @@ static void intel_ish_enable(struct device *dev) static void intel_ish_get_version(void) { - struct cse_fw_partition_info *version = cbmem_find(CBMEM_ID_CSE_PARTITION_VERSION); - if (version == NULL) + struct cse_specific_info *info = cbmem_find(CBMEM_ID_CSE_INFO); + if (info == NULL) return; printk(BIOS_DEBUG, "ISH version: %d.%d.%d.%d\n", - version->ish_partition_info.cur_ish_fw_version.major, - version->ish_partition_info.cur_ish_fw_version.minor, - version->ish_partition_info.cur_ish_fw_version.hotfix, - version->ish_partition_info.cur_ish_fw_version.build); + info->cse_fwp_version.ish_partition_info.cur_ish_fw_version.major, + info->cse_fwp_version.ish_partition_info.cur_ish_fw_version.minor, + info->cse_fwp_version.ish_partition_info.cur_ish_fw_version.hotfix, + info->cse_fwp_version.ish_partition_info.cur_ish_fw_version.build); } static void intel_ish_final(struct device *dev) diff --git a/src/soc/intel/common/block/cse/cse_lite.c b/src/soc/intel/common/block/cse/cse_lite.c index 908b5de1fe..937f1be3e2 100644 --- a/src/soc/intel/common/block/cse/cse_lite.c +++ b/src/soc/intel/common/block/cse/cse_lite.c @@ -1333,12 +1333,13 @@ static void store_ish_version(void) if (vboot_recovery_mode_enabled()) return; - struct cse_fw_partition_info *version; - size_t size = sizeof(struct fw_version); - version = cbmem_find(CBMEM_ID_CSE_PARTITION_VERSION); - if (version == NULL) + struct cse_specific_info *info = cbmem_find(CBMEM_ID_CSE_INFO); + if (info == NULL) return; + struct cse_fw_partition_info *version = &(info->cse_fwp_version); + size_t size = sizeof(struct fw_version); + /* * Compare if stored cse version (from the previous boot) is same as current * running cse version. @@ -1360,6 +1361,9 @@ static void store_ish_version(void) /* Since cse version has been updated, ish version needs to be updated. */ memcpy(&(version->ish_partition_info.cur_ish_fw_version), &(resp.manifest_data.version), size); + + /* Update the CRC */ + cbmem_store_cse_info_crc(info); } } } |