[coreboot-gerrit] Change in ...coreboot[master]: nb/intel/pineview: Remove dead code in switch

[Jacob Garber (Code Review)]

Jacob Garber 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?

Hmmm, if to avoid the die(), then probably yes