Attention is currently required from: Edward O'Callaghan, Angel Pons, Nikolai Artemiev. Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/62768 )
Change subject: flashrom.8.tmpl: Add raiden_debug_spi doc entry ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
File flashrom.8.tmpl:
https://review.coreboot.org/c/flashrom/+/62768/comment/33939337_03d98c9b PS1, Line 319: Currently supported are This is of course not for this patch, but is there any reason programmers are not in alphabetical order? :)
https://review.coreboot.org/c/flashrom/+/62768/comment/ac829d7f_811756fc PS1, Line 1154: custom_rst=(true,false) It might be a little confusing, because description talks about delay in ms, but parameter is boolean. I understand the parameter is what it is, but maybe a bit more clarity, what does it mean when it is set "true" vs "false"? custom_rst=true -> does it mean 10ms? custom_rst=false -> does it mean 1-3ms? no parameter -> does it mean 1-3ms?