SpiFlash操作函数bug+FatFs操作SpiFlash的bug讨论，再问硬汉一个问题

小狐狸

如题，我移植F407开发板例程中【V5-102_串行Flash读写例程（V1.2）.rar】的SpiFlash操作函数时发现sf_AutoWritePage函数的一个bug，请硬汉哥确认下。

bug：第87行应该加一句：ucNeedErase = 1;原因见图片中的解释。
问题1：写完数据后为什么要比较两次呢？sf_CmpData条用了2次，谢谢

问题2：f_mkfs("0:",0,0);第三个参数为什么写0，这个值到底写多少合适？

1. static uint8_t sf_AutoWritePage(uint8_t *_ucpSrc, uint32_t _uiWrAddr, uint16_t _usWrLen)
2. {
3.   uint16_t i;
4.   uint16_t j;         /* 用于延时 */
5.   uint32_t uiFirstAddr;   /* 扇区首址 */
6.   uint8_t ucNeedErase;    /* 1表示需要擦除 */
7.   uint8_t cRet;
9.   /* 长度为0时不继续操作,直接认为成功 */
10.   if (_usWrLen == 0)
11.   {
12.     return 1;
13.   }
15.   /* 如果偏移地址超过芯片容量则退出 */
16.   if (_uiWrAddr >= g_tSF.TotalSize)
17.   {
18.     return 0;
19.   }
21.   /* 如果数据长度大于扇区容量，则退出 */
22.   if (_usWrLen > g_tSF.PageSize)
23.   {
24.     return 0;
25.   }
27.   /* 如果FLASH中的数据没有变化,则不写FLASH */
28.   sf_ReadBuffer(s_spiBuf, _uiWrAddr, _usWrLen);
29.   if (memcmp(s_spiBuf, _ucpSrc, _usWrLen) == 0)
30.   {
31.     return 1;
32.   }
34.   /* 判断是否需要先擦除扇区 */
35.   /* 如果旧数据修改为新数据，所有位均是 1->0 或者 0->0, 则无需擦除,提高Flash寿命 */
36.   ucNeedErase = 0;
37.   if (sf_NeedErase(s_spiBuf, _ucpSrc, _usWrLen))
38.   {
39.     ucNeedErase = 1;
40.   }
42.   uiFirstAddr = _uiWrAddr & (~(g_tSF.PageSize - 1));
44.   if (_usWrLen == g_tSF.PageSize)   /* 整个扇区都改写 */
45.   {
46.     for (i = 0; i < g_tSF.PageSize; i++)
47.     {
48.       s_spiBuf[i] = _ucpSrc[i];
49.     }
50.   }
51.   else            /* 改写部分数据 */
52.   {
53.     /* 先将整个扇区的数据读出 */
54.     sf_ReadBuffer(s_spiBuf, uiFirstAddr, g_tSF.PageSize);
56.     /* 再用新数据覆盖 */
57.     i = _uiWrAddr & (g_tSF.PageSize - 1);
58.     memcpy(&s_spiBuf[i], _ucpSrc, _usWrLen);
59.   }
61.   /* 写完之后进行校验，如果不正确则重写，最多3次 */
62.   cRet = 0;
63.   for (i = 0; i < 3; i++)
64.   {
66.     /* 如果旧数据修改为新数据，所有位均是 1->0 或者 0->0, 则无需擦除,提高Flash寿命 */
67.     if (ucNeedErase == 1)
68.     {
69.       sf_EraseSector(uiFirstAddr);    /* 擦除1个扇区 */
70.     }
72.     /* 编程一个PAGE */
73.     sf_PageWrite(s_spiBuf, uiFirstAddr, g_tSF.PageSize);
75.     if (sf_CmpData(_uiWrAddr, _ucpSrc, _usWrLen) == 0)
76.     {
77.       cRet = 1;
78.       break;
79.     }
80.     else
81.     {
82.       if (sf_CmpData(_uiWrAddr, _ucpSrc, _usWrLen) == 0)
83.       {
84.         cRet = 1;
85.         break;
86.       }
88.       /* 失败后延迟一段时间再重试 */
89.       for (j = 0; j < 10000; j++);
90.     }
91.   }
93.   return cRet;
94. }

复制代码

编辑原因：又发现一个FatFs操作SpiFlash的bug。例程为：【V5-107e_FatFS文件系统例程（SPI串行Flash）（V1.2）.rar】

dickio.c文件中写flash有问题：count次调用了sf_WriteBuffer，但是参数却没有变化，这里有问题哦。

1. #if _USE_WRITE
2. DRESULT disk_write (
3.     BYTE pdrv,            /* Physical drive nmuber (0..) */
4.     const BYTE *buff,    /* Data to be written */
5.     DWORD sector,        /* Sector address (LBA) */
6.     BYTE count            /* Number of sectors to write (1..128) */
7. )
8. {
9.     switch (pdrv) {
10.         case FS_SPI_FLASH :
11.             {
12.                 uint8_t i;
13.                 for(i = 0; i < count; i++)
14.                 {
15.                     sf_WriteBuffer((uint8_t *)buff, sector << 12, 4096);
16.                 }
17.                 return RES_OK;
18.             }
19.     }
20.     return RES_PARERR;
21. }
22. #endif

复制代码

改法一：

1. for(i = 0; i < count; i++)
2. {
3.       sf_WriteBuffer((uint8_t *)buff, sector << 12, 4096);   
4. }
6. 上面的for直接改为：
7. sf_WriteBuffer((uint8_t *)buff, sector << 12, count<<12);
9. 因为sf_WriteBuffer函数支持写多个扇区的功能，所以可以这样写

复制代码

改法二：

1. for循环改为如下的方式：
2. for(i = 0; i < count; i++)
3. {
4.       sf_WriteBuffer((uint8_t *)buff, sector << 12, 4096);   
5.       sector++;
6.       buff +=4096;
7. }

复制代码

eric2013

你的这种做法可以的。感觉是个bug，这个驱动是站长做的，我跟站长说一下。

armfly

如果比较OK 就返回， 如果出错，就再比较一次确认有错再重试。大多数情况，写一次就会OK，如果由于干扰导致比较出错，那么就再比较一次，如果还有错，才能认为确实错了，此时需要重试。

小狐狸

armfly:如果比较OK 就返回， 如果出错，就再比较一次确认有错再重试。大多数情况，写一次就会OK，如果由于干扰导致比较出错，那么就再比较一次，如果还有错，才能认为确实错了，此时需要重试。 (2016-04-13 01:09) 

哦，明白了，为了保险起见比较两次

小狐狸

eric2013:你的这种做法可以的。感觉是个bug，这个驱动是站长做的，我跟站长说一下。
 (2016-04-13 00:52) 

好的，谢谢了啦

小狐狸

eric2013:你的这种做法可以的。感觉是个bug，这个驱动是站长做的，我跟站长说一下。
 (2016-04-13 00:52) 

呼叫硬汉哥 ，，@eric2013

eric2013

小狐狸:呼叫硬汉哥 ，，@eric2013    

 (2016-04-15 10:24) 

不要用这个例子，这个有bug，我已经修正了，参考这个帖子我更新，：
http://www.armbbs.cn/forum.php?mod=viewthread&tid=14894

小狐狸

eric2013:不要用这个例子，这个有bug，我已经修正了，参考这个帖子我更新，：
http://www.armbbs.cn/forum.php?mod=viewthread&tid=14894 (2016-04-15 10:29)

谢谢回复 @eric2013  

后面的问题确实在新的例程中更新了，，但是我说的第一个bug还没改，是bug吗？

即在函数：
sf_AutoWritePage中第一次写没写正确，第二次写前需要把：ucNeedErase = 1;如果前面ucNeedErase = 0;时第二次就无法写了。。。

eric2013

小狐狸:谢谢回复 @eric2013   

后面的问题确实在新的例程中更新了，，但是我说的第一个bug还没改，是bug吗？

....... (2016-04-15 23:22)

for循环里面ucNeedErase 是一直等于1 的。

小狐狸

eric2013:

for循环里面ucNeedErase 是一直等于1 的。

@eric2013   

嗯，只要ucNeedErase=1后，for里面是没人清它的。。。关键是ucNeedErase进入for前不一定是1哦，所以for里面应该把ucNeedErase置位一次。。

请看截图： ucNeedErase置位是有条件的。

eric2013

小狐狸:@eric2013      

嗯，只要ucNeedErase=1后，for里面是没人清它的。。。关键是ucNeedErase进入for前不一定是1哦，所以for里面应该把ucNeedErase置位一次。。

....... (2016-04-16 10:27) 

对的，这样更加保险。[s:142]

小狐狸

eric2013:对的，这样更加保险。[s:142] (2016-04-16 10:46) 

嗯，，，好的

再次感谢，spiflash的驱动写的很完善。