diff options
| author | Johnny Lin <johnny_lin@wiwynn.com> | 2020-11-13 17:21:25 +0800 |
|---|---|---|
| committer | Marc Jones <marc@marcjonesconsulting.com> | 2020-11-20 18:58:54 +0000 |
| commit | 5b47d77047fbe42c8ab5ce63fbb26aee4899a8e5 (patch) | |
| tree | e82bfd51a521e813239b13c621832fa5a94e266d /src/drivers/intel/fsp2_0/header_display.c | |
| parent | 982f64d1b7fd17e7063608fc23dc56010c36a877 (diff) | |
| download | coreboot-5b47d77047fbe42c8ab5ce63fbb26aee4899a8e5.tar.gz coreboot-5b47d77047fbe42c8ab5ce63fbb26aee4899a8e5.tar.bz2 coreboot-5b47d77047fbe42c8ab5ce63fbb26aee4899a8e5.zip | |
intel/fsp2_0: Add soc_validate_fsp_version for FSP version check
Only need to check this once so check it at romstage where
the console is usually ready. Also define union fsp_revision
to avoid code duplication.
Change-Id: I628014e05bd567462f50af2633fbf48f3dc412bc
Signed-off-by: Johnny Lin <johnny_lin@wiwynn.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/47559
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marc Jones <marc@marcjonesconsulting.com>
Reviewed-by: Jonathan Zhang <jonzhang@fb.com>
Diffstat (limited to 'src/drivers/intel/fsp2_0/header_display.c')
| -rw-r--r-- | src/drivers/intel/fsp2_0/header_display.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/src/drivers/intel/fsp2_0/header_display.c b/src/drivers/intel/fsp2_0/header_display.c index 57e13b5354de..b45b4d8a7c02 100644 --- a/src/drivers/intel/fsp2_0/header_display.c +++ b/src/drivers/intel/fsp2_0/header_display.c @@ -5,15 +5,7 @@ void fsp_print_header_info(const struct fsp_header *hdr) { - union { - uint32_t val; - struct { - uint8_t bld_num; - uint8_t revision; - uint8_t minor; - uint8_t major; - } rev; - } revision; + union fsp_revision revision; revision.val = hdr->fsp_revision; |