Hi,
File: fsl_flexspi_nor_flash.c
In the function Nor_Flash_Page_Program, are the following lines:
uint32_t pageSize = handle->bytesInPageSize;
.
.
.
flashXfer.dataSize = pageSize;
This means that calling this function does not allow you to program less than a page size. The flash parts I am using allow you to program down to a single byte using the page program command. The fix I put in was to add a length argument to the function:
status_t Nor_Flash_Page_Program(nor_handle_t *handle, uint32_t address, uint8_t *buffer, uint32_t length)
And then, use it in the flexspi call:
flashXfer.dataSize = length;
And not use pageSize at all.
I also added an assert near the top of the function to prevent length going over a page boundary, otherwise the data will be page wrapped by the device (may not be the most efficient way to do it):
#ifdef DEBUG
uint32_t nextPageStartBoundary = handle->bytesInPageSize * ((address + handle->bytesInPageSize) / handle->bytesInPageSize);
assert((address + length) <= nextPageStartBoundary); // address + length <= next page boundary.
#endif
Also the function Nor_Flash_Program would need to be modified to allow lengths that are not page aligned.
Is there a reason Nor_Flash_Page_Program limits the data size to a page? Or does my fix look reasonable? The current API will need to be maintained, so a new function with a length argument may be required.
FYI:
MCUXpresso IDE v11.4.0 [Build 6224] [2021-07-15]
SDK 2.10.0
The demo I originally looked at is evkmimxrt1020_flash_component_nor, which contains a folder called nor_flash. That is where the code I refer to is located.
Kind regards,
Ronnie
