[riot-notifications] [RIOT-OS/RIOT] drivers/mtd_spi_nor: enable 32 bit addressing, define WP and HOLD pins (#15360)

benpicco notifications at github.com
Mon Nov 2 17:06:53 CET 2020


### Contribution description

If an address width of 4 byte is configured, we also must enable it on the SPI flash device in order for it to accept 32 bit addresses.

If the WP and HOLD pin is connected to the MCU, it must be driven HIGH as otherwise a LOW level will enable both HOLD and WP.

This is not the case for any existing boards, but it is wired up that way on the `same54-xpro`, so add this in preparation for that.


### Testing procedure

No changes in existing boards.


### Issues/PRs references

<!--
Examples: Fixes #1234. See also #5678. Depends on PR #9876.

Please use keywords (e.g., fixes, resolve) with the links to the issues you
resolved, this way they will be automatically closed when your pull request
is merged. See https://help.github.com/articles/closing-issues-using-keywords/.
-->

You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/15360

-- Commit Summary --

  * drivers/mtd_spi_nor: enable 32 bit addressing
  * drivers/mtd_spi_nor: define WP and HOLD pin
  * boards: define mtd_spi_nor HOLD and WP pins

-- File Changes --

    M boards/ikea-tradfri/board.c (2)
    M boards/mulle/board.c (4)
    M boards/nrf52840dk/mtd.c (2)
    M boards/pinetime/board.c (2)
    M boards/serpente/board.c (2)
    M boards/weact-f411ce/board.c (2)
    M drivers/include/mtd_spi_nor.h (2)
    M drivers/mtd_spi_nor/mtd_spi_nor.c (32)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/15360.patch
https://github.com/RIOT-OS/RIOT/pull/15360.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15360
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201102/2ffb24a0/attachment.htm>


More information about the notifications mailing list