在原子SPI通信程序中发现了一个小错误

书生的剑

在使用原子哥的SPI通信中的 W25QXX_Write（）函数时发现了有一处错误，第一次可以进行正确的读写，可当我第二次改变写的数据，并往同一地址写的时候就出现了乱码，经过查找发现了一个问题，下面是截取的一段代码
    W25QXX_Read(W25QXX_BUF,secpos*4096,4096);
        for(i=0;i<secremain;i++)
        {
            if(W25QXX_BUF[secoff+i]!=0XFF)break;     
        }
        if(i<secremain)//&#208;è&#210;&#170;&#178;&#193;&#179;&#253;
        {
            W25QXX_Erase_Sector(secpos);       //此处错误
            for(i=0;i<secremain;i++)   
            {
                W25QXX_BUF[i+secoff]=pBuffer;
            }
            W25QXX_Write_NoCheck(W25QXX_BUF,secpos*4096,4096);

       代码的意思是当检测到写入的扇区内有数据的时候，执行扇区删除操作，W25QXX_Erase_Sector（）函数传递进去的参数应该是一个24位的地址，可是在上面的程序中传进去的不是一个地址，而是secpos=WriteAddr/4096;，即第几个扇区，所以错误的地方应该改为W25QXX_Erase_Sector(secpos*4096);  

liuchang

呵呵，上代码......这是原子源码......

书生的剑

我错了我自己写的擦除函数少写了那个4096，导致有了这个错误，第一次发帖，真是尴尬

xuande

如果属实，值得点赞。

龙之谷

应该如楼主所言，点个赞

jackielau

贴上相关代码
//写SPI FLASH  
//在指定地址开始写入指定长度的数据
//该函数带擦除操作!
//pBuffer:数据存储区
//WriteAddr:开始写入的地址(最大32bit)                                               
//NumByteToWrite:要写入的字节数(最大65535)   
u8 W25QXX_BUFFER[4096];                 
void W25QXX_Write(u8* pBuffer,u32 WriteAddr,u16 NumByteToWrite)   
{
        u32 secpos;
        u16 secoff;
        u16 secremain;          
        u16 i;   
        u8 * W25QXX_BUF;          
           W25QXX_BUF=W25QXX_BUFFER;             
        secpos=WriteAddr/4096;//扇区地址  
        secoff=WriteAddr%4096;//在扇区内的偏移
        secremain=4096-secoff;//扇区剩余空间大小   
        //printf("ad:%X,nb:%X\r\n",WriteAddr,NumByteToWrite);//测试用
        if(NumByteToWrite<=secremain)secremain=NumByteToWrite;//不大于4096个字节
        while(1)
        {       
                W25QXX_Read(W25QXX_BUF,secpos*4096,4096);//读出整个扇区的内容
                for(i=0;i<secremain;i++)//校验数据
                {
                        if(W25QXX_BUF[secoff+i]!=0XFF)break;//需要擦除            
                }
                if(i<secremain)//需要擦除
                {
                        W25QXX_Erase_Sector(secpos);//擦除这个扇区
                        for(i=0;i<secremain;i++)           //复制
                        {
                                W25QXX_BUF[i+secoff]=pBuffer[i];          
                        }
                        W25QXX_Write_NoCheck(W25QXX_BUF,secpos*4096,4096);//写入整个扇区  

                }else W25QXX_Write_NoCheck(pBuffer,WriteAddr,secremain);//写已经擦除了的,直接写入扇区剩余区间.                                   
                if(NumByteToWrite==secremain)break;//写入结束了
                else//写入未结束
                {
                        secpos++;//扇区地址增1
                        secoff=0;//偏移位置为0          

                           pBuffer+=secremain;  //指针偏移
                        WriteAddr+=secremain;//写地址偏移          
                           NumByteToWrite-=secremain;                                //字节数递减
                        if(NumByteToWrite>4096)secremain=4096;        //下一个扇区还是写不完
                        else secremain=NumByteToWrite;                        //下一个扇区可以写完了
                }         
        };         
}

//擦除一个扇区
//Dst_Addr:扇区地址 根据实际容量设置
//擦除一个扇区的最少时间:150ms
void W25QXX_Erase_Sector(u32 Dst_Addr)   
{  
        //监视falsh擦除情况,测试用   
        //printf("fe:%x\r\n",Dst_Addr);          
        Dst_Addr*=4096;
    W25QXX_Write_Enable();                  //SET WEL          
    W25QXX_Wait_Busy();   
          W25QXX_CS=0;                            //使能器件   
    SPI5_ReadWriteByte(W25X_SectorErase);   //发送扇区擦除指令
    if(W25QXX_TYPE==W25Q256)                //如果是W25Q256的话地址为4字节的，要发送最高8位
    {
        SPI5_ReadWriteByte((u8)((Dst_Addr)>>24));
    }
    SPI5_ReadWriteByte((u8)((Dst_Addr)>>16));  //发送24bit地址   
    SPI5_ReadWriteByte((u8)((Dst_Addr)>>8));   
    SPI5_ReadWriteByte((u8)Dst_Addr);  
        W25QXX_CS=1;                            //取消片选                  
    W25QXX_Wait_Busy();                                       //等待擦除完成
}  


应该没有问题！

龙之谷

ok,看到帖子还专门去查了一下程序，看的不仔细，没注意到函数中又*4096了
My mistake.....

yuzeyuan1

龙之谷 发表于 2017-11-8 14:02
ok,看到帖子还专门去查了一下程序，看的不仔细，没注意到函数中又*4096了
My mistake.....

我很好奇，这东西为什么我们一直用着正常？咋突然就说某个地方错了呢如果以前用着不正常，应该很多人反馈的呀。难道它错了不影响用？

龙之谷

yuzeyuan1 发表于 2017-11-8 16:21
我很好奇，这东西为什么我们一直用着正常？咋突然就说某个地方错了呢如果以前用着不正常，应该很多 ...

起初也有这样的感觉，专门对照了代码，也是粗心，没注意到代码处理语句
错了影不影响用不太确定，但如果错了这么多网友应该会早些时候就发现问题了，毕竟代码推出已经有一段时间了

yuzeyuan1

龙之谷 发表于 2017-11-8 16:55
起初也有这样的感觉，专门对照了代码，也是粗心，没注意到代码处理语句
错了影不影响用不太确定，但如果 ...

例程代码没有问题，是楼主自己写的擦除函数写错了，看你之前说的都弄糊涂了。
W25QXX_Erase_Sector(secpos);的参数就应该是第几个扇区，而不是地址。W25QXX_Erase_Sector内部自己带有地址计算。而且个人感觉这个W25QXX_Write函数封的很难受，判断有数据就要擦除，我们就不能很好地控制哪些数据该擦，哪些不该擦，还是自己控着安心。