Jacob Garber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32428 )
Change subject: device/dram/ddr3: Remove unused variables ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/32428/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32428/1//COMMIT_MSG@7 PS1, Line 7: variables In this case it's just a single variable
https://review.coreboot.org/#/c/32428/1//COMMIT_MSG@8 PS1, Line 8: This variable is unused because printram only expands to something in debug builds (not default ones) - it might be nice to explain this, since otherwise it's not immediately clear why this variable is unused.