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

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32948 )

Change subject: nb/intel/x4x/rcven.c: Remove variable set but not used
......................................................................

Patch Set 1: Code-Review-1

(1 comment)

The resulting assembly is definitely not the same... You'd want to test this on hardware

https://review.coreboot.org/#/c/32948/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/32948/1//COMMIT_MSG@7 
PS1, Line 7: Remove variable set but not used
The whole purpose of this variable is to sample the strobe so it is normal that it does not get used, hence the use of the volatile keyword. Also the resulting assembly is different with this change.