emFile with NOR Flash Crash

This site uses cookies. By continuing to browse this site, you are agreeing to our Cookie Policy.

  • emFile with NOR Flash Crash

    I'm using emFILE v3.28 library for evaluating for EA1788 evaluation board with Keil IDE.
    I tried to run the physical layer FS_NOR_PHY_CFI_1x16 and could not.
    The flash is connected as follows:



    So I had to implement my own:

    C Source Code

    1. #include <stdio.h>
    2. #include "norflash_sst39vf3201.h"
    3. #include "FS.h"
    4. #include "RTOS.h"
    6. static int WriteOff(U8 Unit, U32 Off, const void * pSrc, U32 Len);
    7. static int ReadOff(U8 Unit, void * pDest, U32 Off, U32 Len);
    8. static int EraseSector(U8 Unit, unsigned int SectorIndex);
    9. static void GetSectorInfo(U8 Unit, unsigned int SectorIndex, U32 * pOff, U32 * pLen);
    10. static int GetNumSectors(U8 Unit);
    11. static void Configure(U8 Unit, U32 BaseAddr, U32 StartAddr, U32 NumBytes);
    12. static void OnSelectPhy(U8 Unit);
    13. static void DeInit(U8 Unit);
    15. const FS_NOR_PHY_TYPE FS_NOR_PHY_CFI_39FV3201 __attribute__ ((section("EXT_FLASH"))) = {
    16. WriteOff,
    17. ReadOff,
    18. EraseSector,
    19. GetSectorInfo,
    20. GetNumSectors,
    21. Configure,
    22. OnSelectPhy,
    23. DeInit
    24. };
    26. typedef struct
    27. {
    28. U32 BaseAddr;
    29. U32 StartAddr;
    30. U32 NumBytes;
    31. } DRIVER_INSTANCE;
    33. DRIVER_INSTANCE iDriver[10];
    35. static int WriteOff(U8 Unit, U32 Off, const void * pSrc, U32 Len)
    36. {
    37. U32 flash_addr;
    38. U16 *pSource16;
    39. U8 *pSource8;
    41. // printf("WriteOff - Off: %lX, pSrc: %lX, Len: %lu\n", Off, (U32)pSrc, Len);
    42. // fflush(stdout);
    44. pSource16 = (U16 *)pSrc;
    45. flash_addr = iDriver[Unit].StartAddr + ((Off >> 1) << 1);
    47. // Se offset ímpar
    48. if (Off % 2)
    49. {
    50. pSource8 = (U8 *)pSrc;
    51. OS_EnterRegion();
    52. if (NORFLASHWriteWord(flash_addr, ((*pSource8++) << 8) | *((U8 *)flash_addr)) != 0)
    53. {
    54. OS_LeaveRegion();
    55. return 1;
    56. }
    57. OS_LeaveRegion();
    59. pSource16 = (U16 *)pSource8;
    60. Len--;
    61. flash_addr += 2;
    62. }
    64. while (Len >= 2)
    65. {
    66. OS_EnterRegion();
    67. if (NORFLASHWriteWord(flash_addr, *pSource16++) != 0)
    68. {
    69. OS_LeaveRegion();
    70. return 1;
    71. }
    72. OS_LeaveRegion();
    74. Len -= 2;
    75. flash_addr += 2;
    76. }
    78. if (Len == 1)
    79. {
    80. pSource8 = (U8 *)pSource16;
    81. OS_EnterRegion();
    82. if (NORFLASHWriteWord(flash_addr, ((*((U8 *)(flash_addr + 1))) << 8) | *pSource8) != 0)
    83. {
    84. OS_LeaveRegion();
    85. return 1;
    86. }
    87. OS_LeaveRegion();
    88. }
    90. return 0;
    91. }
    93. static int ReadOff(U8 Unit, void * pDest, U32 Off, U32 Len)
    94. {
    95. U32 *pSource32, *pDestin32;
    96. U16 *pSource16, *pDestin16;
    97. U8 *pSource8, *pDestin8;
    99. // printf("ReadOff - pDest: %lX, Off: %lX, Len: %lu\n", (U32)pDest, Off, Len);
    100. // fflush(stdout);
    102. pSource32 = (U32 *) (iDriver[Unit].StartAddr + Off);
    103. pDestin32 = (U32 *) pDest;
    104. if (Len >= 4)
    105. {
    106. while (Len >= 4)
    107. {
    108. *pDestin32++ = *pSource32++;
    109. Len -= 4;
    110. }
    112. if (Len == 0) return 0;
    113. }
    115. if (Len >= 2)
    116. {
    117. pSource16 = (U16 *) pSource32;
    118. pDestin16 = (U16 *) pDestin32;
    120. while (Len >= 2)
    121. {
    122. *pDestin16++ = *pSource16++;
    123. Len -= 2;
    124. }
    126. if (Len == 0) return 0;
    128. pSource32 = (U32 *) pSource16;
    129. pDestin32 = (U32 *) pDestin16;
    130. }
    132. if (Len >= 1)
    133. {
    134. pSource8 = (U8 *) pSource32;
    135. pDestin8 = (U8 *) pDestin32;
    137. while (Len--) *pDestin8++ = *pSource8++;
    138. }
    140. return 0;
    141. }
    143. static int EraseSector(U8 Unit, unsigned int SectorIndex)
    144. {
    145. // printf("EraseSector - SectorIndex: %u\n", SectorIndex);
    146. // fflush(stdout);
    148. OS_EnterRegion();
    149. NORFLASHSectorErase(SectorIndex);
    150. OS_LeaveRegion();
    152. return 0;
    153. }
    155. static void GetSectorInfo(U8 Unit, unsigned int SectorIndex, U32 * pOff, U32 * pLen)
    156. {
    157. // volatile int x, y, z ,w;
    158. //
    159. // if ((SectorIndex >= 512) || (pOff == NULL) || (pLen == NULL))
    160. // {
    161. // x = SectorIndex;
    162. // y = (int)pOff;
    163. // z = (int)pLen;
    164. // w = SectorIndex;
    165. // return;
    166. // }
    168. // printf("GetSectorInfo - SectorIndex: %u, pOff: %lX, pLen: %lX\n", SectorIndex, (U32)pOff, (U32)pLen);
    169. // fflush(stdout);
    171. *pOff = SectorIndex * 4096;
    172. *pLen = 4096;
    173. }
    175. static int GetNumSectors(U8 Unit)
    176. {
    177. // printf("GetNumSectors\n");
    178. // fflush(stdout);
    180. return (iDriver[Unit].NumBytes / 4096);
    181. }
    183. static void Configure(U8 Unit, U32 BaseAddr, U32 StartAddr, U32 NumBytes)
    184. {
    185. // printf("Configure\n");
    186. // fflush(stdout);
    188. iDriver[Unit].BaseAddr = BaseAddr;
    189. iDriver[Unit].StartAddr = StartAddr;
    190. iDriver[Unit].NumBytes = NumBytes;
    191. }
    193. static void OnSelectPhy(U8 Unit)
    194. {
    195. // printf("OnSelectPhy\n");
    196. // fflush(stdout);
    197. }
    199. static void DeInit(U8 Unit)
    200. {
    201. // printf("DeInit\n");
    202. // fflush(stdout);
    203. }
    Display All


    When I write a file with the size of the drive it is saved normally.
    But if I delete the file and write again, there is a Hardware Fault.
    This hardware fault is caused by pLen parameter of GetSectorInfo routine that is 0.



    Could anyone help me?

    The post was edited 2 times, last by leonarff ().

  • Hello leonarff,

    In some cases, the NOR driver needs to know only the address of a physical sector
    and it calls GetSectorInfo() with the pLen parameter set to NULL.
    You can avoid the Hardware Fault error by validating the pLen parameter before dereferencing it.

    C Source Code

    1. if (pLen) {
    2. *pLen = 4096;
    3. }


    Best regards,
    Marius
    Please read the forum rules before posting.

    Keep in mind, this is *not* a support forum.
    Our engineers will try to answer your questions between their projects if possible but this can be delayed by longer periods of time.
    Should you be entitled to support you can contact us via our support system: segger.com/ticket/
    Or you can contact us via e-mail.