[coreboot-gerrit] Change in ...coreboot[master]: src: Fix - Remove unused variables

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29917 )

Change subject: src: Fix - Remove unused variables
......................................................................

Patch Set 23:

(1 comment)

https://review.coreboot.org/#/c/29917/23/src/soc/intel/fsp_baytrail/romstage... 
File src/soc/intel/fsp_baytrail/romstage/romstage.c:

https://review.coreboot.org/#/c/29917/23/src/soc/intel/fsp_baytrail/romstage... 
PS23, Line 253: 	if (!cbmem_was_initted)
              : 		printk(BIOS_ERR, "cbmem init failed\n");
> unrelated to description

If the variable is not used, can't the cbmem_recovery function be called without saving the result to a variable? I feel adding this printf is a hack to make the variable used somewhere, which does not seem wise to me.