Miklós Márton has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/25683 )
Change subject: Add support for National Instruments USB-845x devices ......................................................................
Patch Set 29:
(3 comments)
https://review.coreboot.org/c/flashrom/+/25683/28/flashrom.8.tmpl File flashrom.8.tmpl:
https://review.coreboot.org/c/flashrom/+/25683/28/flashrom.8.tmpl@1141 PS28, Line 1141: In the case if the none of the programmer's supported IO voltage is within the supported voltage range of the detected flash chip the flashrom will abort the operation (to prevent damaging the flash chip). You can override this behaviour by passing "yes" to the
also, a spurious space at the end of the line, and the line should be broken […]
Done
https://review.coreboot.org/c/flashrom/+/25683/28/flashrom.8.tmpl@1141 PS28, Line 1141: the
second "the" in the first sentence seems spurious
Done
https://review.coreboot.org/c/flashrom/+/25683/26/ni845x_spi.c File ni845x_spi.c:
https://review.coreboot.org/c/flashrom/+/25683/26/ni845x_spi.c@440 PS26, Line 440: strlen(ignore_io_voltage_limits_str) == 3 : && strstr("yes", ignore_io_voltage_limits_str) == 0
I meant strcmp(), not strstr().
Done
-
Miklós Márton (Code Review)