Attention is currently required from: Hsuan Ting Chen, Hsuan-ting Chen, Thomas Heijligen.
5 comments:
File doc/contrib_howtos/how_to_add_new_chip.rst:
Delete "This", start with "Indicates that"
same
Patch Set #1, Line 125: priort
typo, prior
Write-Status-Register (WRSR) Handling
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
``FEATURE_WRSR_EWSR``, ``FEATURE_WRSR_WREN``, and ``FEATURE_WRSR_EITHER``. These bits used for **SPI only**.
The Write Status Register (WRSR) is used to configure various settings within the flash chip, including write protection and
other features. The way WRSR is accessed varies between SPI flash chips, leading to the need for these feature bits.
* ``FEATURE_WRSR_EWSR``:
This indicates that we need an **Enable-Write-Status-Register** (EWSR) instruction which opens the status register for the
immediately-followed next WRSR instruction. Usually, the opcode is **0x50**.
* ``FEATURE_WRSR_WREN``:
This indicates that we need an **Write-Enable** (WREN) instruction to set the Write Enable Latch (WEL) bit. The WEL bit
must be set priort to every WRSR command. Usually, the opcode is **0x06**.
* ``FEATURE_WRSR_EITHER``:
This indicates that either EWSR or WREN is supported in this chip.
Maybe move this section to "Write-protection" , below? You can skip the intro, and rename the section as "Feature bits for Write-Status-Register (WRSR) Handling".
You mentioned you have further plans to update the doc, maybe you can tell me the plan? Then we can come up with a doc structure which aligns with you next plans!
same
To view, visit change 82908. To unsubscribe, or for help writing mail filters, visit settings.