diff options
author | Johnny Lin <johnny_lin@wiwynn.com> | 2023-07-06 13:15:24 +0800 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2023-07-07 13:56:02 +0000 |
commit | 908be4f6edd27ef84ceb904912b92656258a88c0 (patch) | |
tree | 3825607b917643590fc3a327674ca4461cdb1aa3 /src | |
parent | 222c44796cf7d496864d16cbab3d844d23068250 (diff) |
soc/intel/xeon_sp/spr: Fix upd_display.c build error
Fix below build error after DISPLAY_UPD_DATA is selected:
src/soc/intel/xeon_sp/spr/upd_display.c:131:29: error: variable 'old' set but not used [-Werror=unused-but-set-variable]
131 | const FSP_S_CONFIG *old;
| ^~~
src/soc/intel/xeon_sp/spr/upd_display.c:130:29: error: variable 'new' set but not used [-Werror=unused-but-set-variable]
130 | const FSP_S_CONFIG *new;
Change-Id: I43ed5fadab58e0d4dc824457c7a1bdf48511198e
Signed-off-by: Johnny Lin <johnny_lin@wiwynn.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/76342
Reviewed-by: Sridhar Siricilla <sridhar.siricilla@intel.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src')
-rw-r--r-- | src/soc/intel/xeon_sp/spr/upd_display.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/src/soc/intel/xeon_sp/spr/upd_display.c b/src/soc/intel/xeon_sp/spr/upd_display.c index 078624db15..c69a9a0562 100644 --- a/src/soc/intel/xeon_sp/spr/upd_display.c +++ b/src/soc/intel/xeon_sp/spr/upd_display.c @@ -127,13 +127,8 @@ void soc_display_fspm_upd_params(const FSPM_UPD *fspm_old_upd, const FSPM_UPD *f /* Display the UPD parameters for SiliconInit */ void soc_display_fsps_upd_params(const FSPS_UPD *fsps_old_upd, const FSPS_UPD *fsps_new_upd) { - const FSP_S_CONFIG *new; - const FSP_S_CONFIG *old; - - old = &fsps_old_upd->FspsConfig; - new = &fsps_new_upd->FspsConfig; - - printk(BIOS_DEBUG, "UPD values for SiliconInit:\n"); - + printk(BIOS_DEBUG, "Dump of original UPD values for SiliconInit:\n"); + hexdump(fsps_old_upd, sizeof(*fsps_old_upd)); + printk(BIOS_DEBUG, "Dump of updated UPD values for SiliconInit:\n"); hexdump(fsps_new_upd, sizeof(*fsps_new_upd)); } |