[GitHub] [incubator-nuttx] jlaitine commented on pull request #1042: stm32h7 progmem fixes

[GitBox <gi...@apache.org>]

jlaitine commented on pull request #1042:
URL: https://github.com/apache/incubator-nuttx/pull/1042#issuecomment-628399659


   The erase block size is sector size (128K). The page size is the nand page size, i.e. the smallest writable unit which also has the flash internal ecc code. This is a nand device, so it is not possible to write smaller units than page size, otherwise you'll end up with nand ecc errors or wrong data (caused by ecc).
   
   The original version only worked when you were writing pretty much the full flash sequentially,  and even then it might have failed in the end doing some partial page write.
   
   It was also not possible to use the original version with e.g. filesystems (map it to mtd like layers) because of the page/sector mismatches and partial page writes.
   
   I'd guess that the px4 and the bootloader are originally written for NOR type flash memories, and this is where the problems originate from.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org