ajuckler commented on code in PR #17264:
URL: https://github.com/apache/nuttx/pull/17264#discussion_r2485229571


##########
Documentation/components/drivers/block/eeprom.rst:
##########
@@ -2,17 +2,19 @@
 EEPROM
 ======
 
-EEPROMs are a form of Memory
-  Technology Device (MTD).  EEPROMs are non-volatile memory like FLASH, but
-  differ in underlying memory technology and differ in usage in many respects:
-  They may not be organized into blocks (at least from the standpoint of the
-  user) and it is not necessary to erase the EEPROM memory before re-writing
-  it.  In addition, EEPROMs tend to be much smaller than FLASH parts, usually
-  only a few kilobytes vs megabytes for FLASH.  EEPROM tends to be used to
-  retain a small amount of device configuration information; FLASH tends
-  to be used for program or massive data storage. For these reasons, it may
-  not be convenient to use the more complex MTD interface but instead use
-  the simple character interface provided by the EEPROM drivers.
+EEPROMs are a form of Memory Technology Device (MTD).

Review Comment:
   For background, this is a port from an older version of NuttX which didn't 
have the MTD driver for AT25EE yet. I believe it is good to keep both the 
EEPROM and the MTD drivers, no matter whether there is any speed difference 
between both, in case some users don't want to compile the MTD part to save 
memory.
   
   I'll do some speed comparison between both drivers this week and get back to 
you. As for the documentation, I'll add information about the actual MTD driver 
(without the need for losetup).



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to