Attention is currently required from: Hsuan-ting Chen, Thomas Heijligen.
Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/81114?usp=email )
Change subject: classic_cli_manpage.rst: Update doc for custom_rst of raiden_debug_spi ......................................................................
Patch Set 1:
(3 comments)
Patchset:
PS1: Thank you so much for improving documentation!
File doc/classic_cli_manpage.rst:
https://review.coreboot.org/c/flashrom/+/81114/comment/e0d44405_233a6469 : PS1, Line 906: I want to visually separate the long text, so maybe do like this (two empty lines):
----- syntax, where: <empty line here> ``custom_rst=false`` is the implicit default timeout of 3ms <empty line here> and ``custom_rst=true`` set ``RAIDEN_DEBUG_SPI_REQ_ENABLE_AP_CUSTOM`` instead of ``RAIDEN_DEBUG_SPI_REQ_ENABLE_AP``. This custom reset modify the timeout from 3ms to 10ms and <... the rest of text>
https://review.coreboot.org/c/flashrom/+/81114/comment/f7ceb566_1f1e4142 : PS1, Line 909: More Let's add empty line before this line (so that More information is visually separated from the previous text)