Unable to write to flash memory using direct memory access functions like sd_flash_page_erase & sd_flash_write (nrf52840 nrf SDK 17.1.0)

Question

// Register the event handler using the NRF_SDH_SOC_OBSERVER macro NRF_SDH_SOC_OBSERVER(m_sys_evt_observer, 0, sys_evt_dispatch, NULL); // Event handler function prototype void sys_evt_dispatch(uint32_t sys_evt, void * p_context) { // Your system event handling code here // For example, handle the flashBusy flag or other system events ilumi_debug("System event : %d ",sys_evt); if (sys_evt == NRF_EVT_FLASH_OPERATION_SUCCESS){ ilumi_debug("NV op ok "); setFlashIdle(1); }else if (sys_evt == NRF_EVT_FLASH_OPERATION_ERROR){ // will get flash op error if too much command, will investigate later ilumi_debug("NV op fail "); setFlashIdle(0); } } //--------------------------------------------------------------------------------------------------// ret_code_t disable_soft_device(void){ ret_code_t error_code = sd_softdevice_disable(); if (error_code != NRF_SUCCESS && error_code != NRF_ERROR_INVALID_STATE) { ilumi_log("Failed to disable SoftDevice: 0x%x ", error_code); return 0; } return 1; } ret_code_t enable_soft_device(void){ nrf_clock_lf_cfg_t const clock_lf_cfg = { .source = NRF_SDH_CLOCK_LF_SRC, .rc_ctiv = NRF_SDH_CLOCK_LF_RC_CTIV, .rc_temp_ctiv = NRF_SDH_CLOCK_LF_RC_TEMP_CTIV, .accuracy = NRF_SDH_CLOCK_LF_ACCURACY }; ret_code_t error_code = sd_softdevice_enable(&clock_lf_cfg, app_error_fault_handler); if (error_code != NRF_SUCCESS && error_code != NRF_ERROR_INVALID_STATE) { ilumi_log("Failed to enable SoftDevice: 0x%x ", error_code); return 0; } return 1; } //----------------------------------------------------------------------------------------------// #define FLASH_PAGE_VALID ((uint32_t) 0xAABBCCEE) #define USER_FLASH_START_ADDRESS 0x000FF000 //also tried fd0000, fe000 #define USER_FLASH_SIZE 0x1000 #define FLASH_RETRY_TIMES (8) static volatile uint8_t flash_error = 0; static volatile uint8_t flashBusy = 0; extern ret_code_t enable_soft_device(void); extern ret_code_t disable_soft_device(void); void flashErasePage(uint8_t * addr){ uint32_t error_code; disable_soft_device(); // uint32_t page_number = (uint32_t)addr >> 12; //nRF52 page size is 4096Byte uint32_t page_number = (uint32_t)addr / USER_FLASH_SIZE; ilumi_log("page number: %d ",page_number); int8_t retry = FLASH_RETRY_TIMES; retry_flash_op: while((error_code = sd_flash_page_erase(page_number)) == NRF_ERROR_BUSY); //if busy keeps trying, is this efficient? // error_code = sd_flash_write( (uint32_t*) addr, buffer_ptr, word_len)) if(error_code == NRF_SUCCESS){ // this SUCCNSS just means that the erase command have been accpected by stack, we need to wait for erase finish flashBusy = 1; while(flashBusy) (void) sd_app_evt_wait(); if(flash_error){ flash_error = 0; retry--; if(retry){ goto retry_flash_op; }else{ ilumi_log("nv page erase retry fail"); } } // LOG_INFO("nv page erase OK, addr=0x%x", addr); }else{ ilumi_log("nv page erase cmd reject, err:0x%x addr=0x%x", error_code, addr); } } int8_t flashWrite( uint8_t *addr, uint16_t len, uint8_t *buf ){ uint32_t error_code; uint32_t word_len; //uint32_t * buffer_ptr = NULL; // buf address could be not WORD-aligned need to create a buffer for it int8_t ret = 0; if(len == 0) return -1; if((len & 0x3) == 0){ // len has to be divide by 4 because flash is WORD-aligned word_len = len >> 2; // word size is 4x the len }else{ word_len = (len >> 2) + 1; } uint32_t buffer_ptr[word_len]; //if((buffer_ptr = (uint32_t*)malloc(word_len<<2))) { memcpy(buffer_ptr, buf, word_len<<2); disable_soft_device(); int8_t retry = FLASH_RETRY_TIMES; retry_flash_op : while((error_code = sd_flash_write( (uint32_t*) addr, buffer_ptr, word_len)) == NRF_ERROR_BUSY); //sd_flash_write size 32bit word size, addr must be word aligned if(error_code != NRF_SUCCESS){ ret = -1; ilumi_log("sd nv write fail, err=0x%x addr=0x%x len=0x%x wordlen=0x%x srcaddr=0x%x", error_code, addr, len, word_len, buf); }else{ flashBusy = 1; while (flashBusy) (void) sd_app_evt_wait(); if(flash_error){ flash_error = 0; retry--; if(retry){ goto retry_flash_op; }else{ ret = -1; ilumi_log("sd nv write retry fail"); } } // NRF_LOG_INFO("sd nv write OK, addr=0x%x len=0x%x wordlen=0x%x", addr, len, word_len); } } return ret; } void setFlashIdle(uint8_t is_flash_op_success){ // previously this is a potential bug becasue no matter what value in is_flash_op_success, // this function will set flashBusy to idle, while for NRF51 is_flash_op_success is not being used // following path should fix the problem flashBusy = 0; if(is_flash_op_success == 1) flash_error = 0; else flash_error = 1; enable_soft_device(); } /********************************************************************* * Save user data to NV * @param addr addr pointer of area * @param data data buffer point * @param length length of data buffer * @param flash_page_size */ void update_nv_data(uint16_t length, uint8_t* data, uint8_t * addr, uint16_t flash_page_size) { uint32_t signature; int8_t ret; if (length > flash_page_size) { ilumi_log("data size > NV page size ", flash_page_size, addr); return; } // Get the current signature from the flash address signature = *((uint32_t*) addr); // get 32bit signature // If the current flash sector is valid, erase it first if (FLASH_PAGE_VALID == signature) { flashErasePage(addr); } // Write the updated data to the flash ret = flashWrite(addr, length, data); // If flashWrite failed, invalidate the current flash sector by setting the signature to 0 if (0 != ret) { signature = 0x00; flashErasePage(addr); // Always erase first flashWrite(addr, sizeof(signature), (uint8_t *)&signature); // Write this last } } /********************************************************************* * clear the backup flags to user data to NV * @param length length of data buffer * @param data data buffer pointer * @param offset offset to the data struct * @param addr addr pointer of area * @param flash_page_size */ void clear_reset_flag_data(uint16_t length, uint8_t* data, uint32_t offset, uint8_t * addr, uint16_t flash_page_size) { uint32_t signature; if (length > flash_page_size) { ilumi_log("data size > NV page size ", flash_page_size, addr); return; } // Get the current signature from the flash address signature = *((uint32_t*) addr); // get 32bit signature // Write the updated flag to the flash if the current page is valid if (FLASH_PAGE_VALID == signature) { flashWrite(addr + offset, length, data); } } /* * if not valid data, return 0; */ uint8_t load_nv_data(uint16_t length, uint8_t* data, uint8_t * addr, uint16_t flash_page_size) { uint32_t signature; // Log the NV loading attempt ilumi_log("load NV addr: 0x%x, page size: %d ", addr, flash_page_size); if (length > flash_page_size) { ilumi_log("len: %d exceed pg size ", length); return 0; } // Get the current signature from the flash address signature = *((uint32_t*) addr); // get 32bit signature ilumi_log("signature : 0x%x ",signature); // Load from the valid segment if (FLASH_PAGE_VALID == signature) { ilumi_log("user data load ok "); memcpy(data, addr, length); } else { ilumi_log("addr bad "); return 0; } return 1; } void clear_nv_data(uint8_t * addr) { flashErasePage(addr); } Fullscreen 8726.bootloade_flash_placement.xml Download Fullscreen 1882.application_flash_placement.xml Download Unable to write to flash memory using direct memory access functions like sd_flash_page_erase & sd_flash_write (nrf52840 nrf SDK 17.1.0) Also attached application and bootloader flash placement file Application : Memory segment: FLASH1 RX 0x0 0x100000;RAM1 RWX 0x20000000 0x40000 Section placement macros: FLASH_PH_START=0x0 FLASH_PH_SIZE=0x100000 RAM_PH_START=0x20000000 RAM_PH_SIZE=0x40000 FLASH_START=0x27000 FLASH_SIZE=0xd9000 RAM_START=0x20003F08 RAM_SIZE=0x3C0F8 Bootloader : Memory segment: FLASH1 RX 0x0 0x100000;RAM1 RWX 0x20000000 0x40000;mbr_params_page RX 0x000FE000 0x1000;bootloader_settings_page RX 0x000FF000 0x1000;uicr_bootloader_start_address RX 0x10001014 0x4;uicr_mbr_params_page RX 0x10001018 0x4 Section placement macros: FLASH_PH_START=0x0 FLASH_PH_SIZE=0x100000 RAM_PH_START=0x20000000 RAM_PH_SIZE=0x40000 FLASH_START=0xf4000 FLASH_SIZE=0x6000 RAM_START=0x20005978 RAM_SIZE=0x3a688