[riot-notifications] [RIOT-OS/RIOT] drivers/mtd_spi_nor: derive capacity from JEDEC ID (#14084)

benpicco notifications at github.com
Wed May 20 13:31:30 CEST 2020


@benpicco commented on this pull request.



> +    if ((id->manuf == SPI_NOR_JEDEC_ADESTO) &&
+        (id->device[0] & 0xE0) == 0x80) {

Hm thinking about it I wonder if it makes sense to carry IDs that we don't use when the list is so much longer.

Initially I thought this would be a good marker to indicate where that density scheme works, but since that appears to be the standard and only Adesto being an outliner, I think we could just drop it.

-- 
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/14084#discussion_r427937304
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200520/e74f9cb6/attachment.htm>


More information about the notifications mailing list