Change in ...coreboot[master]: src: 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: Remove unused variables
......................................................................

Patch Set 13:

(2 comments)

https://review.coreboot.org/#/c/29917/13/src/soc/intel/fsp_broadwell_de/roms... 
File src/soc/intel/fsp_broadwell_de/romstage/romstage.c:

https://review.coreboot.org/#/c/29917/13/src/soc/intel/fsp_broadwell_de/roms... 
PS13, Line 118:
> breaks cbmem

Is this because "cbmem_was_initted" is used somewhere else, or because the call to "cbmem_recovery(0)" does some important stuff?

https://review.coreboot.org/#/c/29917/13/src/soc/intel/quark/i2c.c 
File src/soc/intel/quark/i2c.c:

https://review.coreboot.org/#/c/29917/13/src/soc/intel/quark/i2c.c@a132 
PS13, Line 132:
> that breaks the fifo cleaning code

How does it break? Is there anything that could be done to remove the variable?