关于V4例程片内flash读写的一个小BUG

夜歌

一个项目中需要保存几个参数在内部，板子是客户的，坑爹的客户板子没有eeprom，好在不是很频繁所以就存在了片内flash里面，使用的安富莱的V4的片内flash例程，其中在写入内容的时候会调用一个函数bsp_CmpCpuFlash对比写入内容和片内存的内容，如果不相等时候判断片内是不是ff如果是表示可以直接写不用擦除，代码如下

1. ucIsEqu = 1;                        /* 先假设所有字节和待写入的数据相等，如果遇到任何一个不相等，则设置为 0 */
   
2.         for (i = 0; i < _ulSize; i++)
   
3.         {
   
4.                 ucByte = *(uint8_t *)_ulFlashAddr;
   
5. 
   
6.                 if (ucByte != *_ucpBuf)
   
7.                 {
   
8.                         if (ucByte != 0xFF)
   
9.                         {
   
10.                                 return FLASH_REQ_ERASE;                /* 需要擦除后再写 */
    
11.                         }
    
12.                         else
    
13.                         {
    
14.                                 ucIsEqu = 0;        
    
15.                         }
    
16.                 }
    
17. 
    
18.                 _ulFlashAddr++;
    
19.                 _ucpBuf++;
    
20.         }

复制代码

1. 
   

复制代码

问题就在黑体部分，对应ucIsEqu = 0;        的分支，假如我写入的数据是 aa bb fd 00 内部存的是aa bb ff 00，这时候就会判断写入的内容可以直接写入到芯片内容，实际这个是错误的，因为写入时候同时会把aa bb 在此写入，最后跟踪问题找到了对应的描述，所以在判断数据的时候不能通过一个字节是否为ff来判断是否需要擦除，不然会有问题

eric2013

谢谢楼主分享，如果方便的话，把你修改后的贴上，方便后面也有人咨询了，直接看这个帖子。

主要是F1系列不再维护了，让他们直接看这个帖子方便些。

夜歌

eric2013 发表于 2020-7-16 18:06
谢谢楼主分享，如果方便的话，把你修改后的贴上，方便后面也有人咨询了，直接看这个帖子。

主要是F1系列 ...

我是将8-15行代码只保留第10行，就是只要数据不相等就进行一次擦除，如果要判断是否是ff还需要在扫描是不是全部是ff，我觉得除了第一次写入一片没有写过数据的内存时可能全部是FF，感觉判断有点没太大意义

eric2013

夜歌 发表于 2020-7-16 18:21
我是将8-15行代码只保留第10行，就是只要数据不相等就进行一次擦除，如果要判断是否是ff还需要在扫描是不 ...

好的，强制擦除。

1847123212

我觉得原来的代码是有意义的，只是要改成 如果和原来相同，就不再重复写，这样才最好。

eric2013

1847123212 发表于 2020-7-17 07:17
我觉得原来的代码是有意义的，只是要改成 如果和原来相同，就不再重复写，这样才最好。

是这样的，前面讨论的就是这个意思。

我贴下正确的代码，取消掉了一个特例情况，即要填写的Flash地址有部分oxFF的，可以直接改写的情况。

1. /*
   
2. *********************************************************************************************************
   
3. *        函 数 名: bsp_CmpCpuFlash
   
4. *        功能说明: 比较Flash指定地址的数据.
   
5. *        形    参: _ulFlashAddr : Flash地址
   
6. *                         _ucpBuf : 数据缓冲区
   
7. *                         _ulSize : 数据大小（单位是字节）
   
8. *        返 回 值:
   
9. *                        FLASH_IS_EQU                0   Flash内容和待写入的数据相等，不需要擦除和写操作
   
10. *                        FLASH_REQ_WRITE                1        Flash不需要擦除，直接写
    
11. *                        FLASH_REQ_ERASE                2        Flash需要先擦除,再写
    
12. *                        FLASH_PARAM_ERR                3        函数参数错误
    
13. *********************************************************************************************************
    
14. */
    
15. uint8_t bsp_CmpCpuFlash(uint32_t _ulFlashAddr, uint8_t *_ucpBuf, uint32_t _ulSize)
    
16. {
    
17.         uint32_t i;
    
18.         uint8_t ucIsEqu;        /* 相等标志 */
    
19.         uint8_t ucByte;
    
20. 
    
21.         /* 如果偏移地址超过芯片容量，则不改写输出缓冲区 */
    
22.         if (_ulFlashAddr + _ulSize > FLASH_BASE_ADDR + FLASH_SIZE)
    
23.         {
    
24.                 return FLASH_PARAM_ERR;                /*　函数参数错误　*/
    
25.         }
    
26. 
    
27.         /* 长度为0时返回正确 */
    
28.         if (_ulSize == 0)
    
29.         {
    
30.                 return FLASH_IS_EQU;                /* Flash内容和待写入的数据相等 */
    
31.         }
    
32. 
    
33.         ucIsEqu = 1;                        /* 先假设所有字节和待写入的数据相等，如果遇到任何一个不相等，则设置为 0 */
    
34.         for (i = 0; i < _ulSize; i++)
    
35.         {
    
36.                 ucByte = *(uint8_t *)_ulFlashAddr;
    
37. 
    
38.                 if (ucByte != *_ucpBuf)
    
39.                 {
    
40.                         return FLASH_REQ_ERASE;                /* 需要擦除后再写 */
    
41.                 }
    
42. 
    
43.                 _ulFlashAddr++;
    
44.                 _ucpBuf++;
    
45.         }
    
46. 
    
47.         if (ucIsEqu == 1)
    
48.         {
    
49.                 return FLASH_IS_EQU;        /* Flash内容和待写入的数据相等，不需要擦除和写操作 */
    
50.         }
    
51.         else
    
52.         {
    
53.                 return FLASH_REQ_WRITE;        /* Flash不需要擦除，直接写 */
    
54.         }
    
55. }

复制代码

eric2013

确诊了下，这个是F1的特色啊，F4系列就没有这个要求，也没有这个错误标志
F1：





F407:


F429:

sunwindsz

增加0xffff的判断...
        if (_ulFlashAddr + _ulSize > FLASH_BASE_ADDR + FLASH_SIZE)
        {
                return FLASH_PARAM_ERR;                /*　函数参数错误　*/
        }

        /* 长度为0时返回正确 */
        if (_ulSize == 0)
        {
                return FLASH_IS_EQU;                /* Flash内容和待写入的数据相等 */
        }

        ucIsEqu = 1;                        /* 先假设所有字节和待写入的数据相等，如果遇到任何一个不相等，则设置为 0 */
        ucReqErase = 0;              //suppose not require to erase
        for (i = 0; i < _ulSize; i++)
        {
                ucByte = *(uint8_t *)_ulFlashAddr;
             if (ucByte != *_ucpBuf)
                {
                        ucIsEqu = 0;        // 不相等，需要写 //
                }
                if (ucByte != 0xFF)  
                {   
                        ucReqErase = 1;
                }
               
                _ulFlashAddr++;
                _ucpBuf++;
        }

        if (1 == ucIsEqu )
        {
                return FLASH_IS_EQU;        /* Flash内容和待写入的数据相等，不需要擦除和写操作 */
        }
        else if (1 == ucReqErase) //"ucIsEqu == 0" and "ucReqErase == 1"
        {
                return FLASH_REQ_ERASE;
        }
        else   //"ucIsEqu == 0" and "ucReqErase == 0"
        {
                return FLASH_REQ_WRITE;        /* Flash不需要擦除，直接写 */
        }

eric2013

sunwindsz 发表于 2020-7-27 11:02
增加0xffff的判断...
        if (_ulFlashAddr + _ulSize > FLASH_BASE_ADDR + FLASH_SIZE)
        {

这么改貌似有问题。

如果是待写入的和Flash里面已经有的内容相同，你程序里面ucByte != 0xFF会造成需要擦除的情况。

sunwindsz

        if (1 == ucIsEqu )
        {
                return FLASH_IS_EQU;        /* Flash内容和待写入的数据相等，不需要擦除和写操作 */
        }

后面先判断相等就return了， 当然把一开始判断0xff写在判断内容相不相等内部更好理解和严谨

eric2013

sunwindsz 发表于 2020-7-29 09:17
if (1 == ucIsEqu )
        {
                return FLASH_IS_EQU;        /* Flash内容和待 ...

这个判断不太好，6楼的修改更靠谱些。

if (ucByte != 0xFF)  
{   
        ucReqErase = 1;
}

sunwindsz

flash操作，如1楼，第一次写入一片没有写过数据的内存时可能全部是FF， 其他时候基本上是要擦除后再写的。所以判断0xff意义不是很大。  1楼和6楼的方式是实用的。
  
纯属探讨，
6楼的，这段不会执行到。
        else
        {
                return FLASH_REQ_WRITE;        /* Flash不需要擦除，直接写 */
        }