Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33407 )
Change subject: nb/intel/pineview: Remove dead code in switch ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/33407/1/src/northbridge/intel/pineview/ramin... File src/northbridge/intel/pineview/raminit.c:
https://review.coreboot.org/#/c/33407/1/src/northbridge/intel/pineview/ramin... PS1, Line 490: if (freq == MEM_CLOCK_800MHz) { Unrelated to the change, but shouldn't there be an if (freq == MEM_CLOCK_667MHz) {} here as well?