Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36135 )
Change subject: nb/intel/nehalem: Fix 'dead assignment' ......................................................................
Patch Set 4:
(2 comments)
With the proposed changes, this should end up being a mere scope reduction.
https://review.coreboot.org/c/coreboot/+/36135/4/src/northbridge/intel/nehal... File src/northbridge/intel/nehalem/raminit.c:
https://review.coreboot.org/c/coreboot/+/36135/4/src/northbridge/intel/nehal... PS4, Line 823: int some_delay_3_halfcycles; remove this
https://review.coreboot.org/c/coreboot/+/36135/4/src/northbridge/intel/nehal... PS4, Line 876: some_delay_3_halfcycles const int some_delay_3_halfcycles