Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39663 )
Change subject: soc/intel/braswell: Clean up ......................................................................
Patch Set 2: Code-Review+1
(2 comments)
https://review.coreboot.org/c/coreboot/+/39663/2/src/soc/intel/braswell/gpio... File src/soc/intel/braswell/gpio.c:
https://review.coreboot.org/c/coreboot/+/39663/2/src/soc/intel/braswell/gpio... PS2, Line 111: w why capital above but not here?
https://review.coreboot.org/c/coreboot/+/39663/2/src/soc/intel/braswell/tsc_... File src/soc/intel/braswell/tsc_freq.c:
https://review.coreboot.org/c/coreboot/+/39663/2/src/soc/intel/braswell/tsc_... PS2, Line 59: Speed Step SpeedStep, per Intel