[coreboot-gerrit] Change in ...coreboot[master]: nb/intel/sandybridge: Remove variable set but not used

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32938 )

Change subject: nb/intel/sandybridge: Remove variable set but not used
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/32938/1/src/northbridge/intel/sandybridge/ra... 
File src/northbridge/intel/sandybridge/raminit_common.c:

https://review.coreboot.org/#/c/32938/1/src/northbridge/intel/sandybridge/ra... 
PS1, Line 1455:
> Those are all MMIO reads, where it may be important to push the result of the read to the stack. […]

See CB:32948 commentary.

It's certainly important for the MMIO read to happen, but I don't see how storing the value on the stack (cachelines) could have any importance.