Daniel Kurtz has posted comments on this change. ( https://review.coreboot.org/28906 )
Change subject: mb/google/kahlee: Set 100MHz fast spi read for supported boards ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/28906/1/src/mainboard/google/kahlee/bootbloc... File src/mainboard/google/kahlee/bootblock/bootblock.c:
https://review.coreboot.org/#/c/28906/1/src/mainboard/google/kahlee/bootbloc... PS1, Line 36: on on one too many 'on'
https://review.coreboot.org/#/c/28906/1/src/mainboard/google/kahlee/bootbloc... PS1, Line 37: set_100mhz_fast_read perhaps "supports_100mhz_fast_spi_read()" ? To me set_ implies a function that will change something.