OpenEdv-开源电子网

 找回密码
 立即注册
正点原子全套STM32/Linux/FPGA开发资料,上千讲STM32视频教程免费下载...
查看: 3360|回复: 9

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

[复制链接]

2

主题

4

帖子

0

精华

新手上路

积分
42
金钱
42
注册时间
2017-10-30
在线时间
8 小时
发表于 2017-11-7 10:07:05 | 显示全部楼层 |阅读模式
在使用原子哥的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);  

2017-11-07_095353.png
正点原子逻辑分析仪DL16劲爆上市
回复

使用道具 举报

119

主题

439

帖子

0

精华

金牌会员

Rank: 6Rank: 6

积分
1185
金钱
1185
注册时间
2015-9-18
在线时间
422 小时
发表于 2017-11-7 14:44:22 | 显示全部楼层
呵呵,上代码......这是原子源码......
搜狗截图20171107144109.png

电子爱好者
回复 支持 反对

使用道具 举报

2

主题

4

帖子

0

精华

新手上路

积分
42
金钱
42
注册时间
2017-10-30
在线时间
8 小时
 楼主| 发表于 2017-11-8 17:56:14 | 显示全部楼层
我错了我自己写的擦除函数少写了那个4096,导致有了这个错误,第一次发帖,真是尴尬
回复 支持 反对

使用道具 举报

58

主题

6294

帖子

1

精华

资深版主

Rank: 8Rank: 8

积分
11544
金钱
11544
注册时间
2014-4-1
在线时间
1315 小时
发表于 2017-11-7 11:46:13 | 显示全部楼层

如果属实,值得点赞。


回复 支持 反对

使用道具 举报

72

主题

2711

帖子

2

精华

论坛大神

Rank: 7Rank: 7Rank: 7

积分
3505
金钱
3505
注册时间
2014-8-4
在线时间
696 小时
发表于 2017-11-7 14:16:23 | 显示全部楼层
应该如楼主所言,点个赞
回复 支持 反对

使用道具 举报

2

主题

123

帖子

0

精华

金牌会员

Rank: 6Rank: 6

积分
1472
金钱
1472
注册时间
2014-4-8
在线时间
172 小时
发表于 2017-11-7 15:15:20 | 显示全部楼层
贴上相关代码
//写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();                                       //等待擦除完成
}  


应该没有问题!
回复 支持 反对

使用道具 举报

72

主题

2711

帖子

2

精华

论坛大神

Rank: 7Rank: 7Rank: 7

积分
3505
金钱
3505
注册时间
2014-8-4
在线时间
696 小时
发表于 2017-11-8 14:02:15 | 显示全部楼层
ok,看到帖子还专门去查了一下程序,看的不仔细,没注意到函数中又*4096了
My mistake.....
以我资质之鲁钝,当尽平心静气、循序渐进、稳扎稳打之力。
回复 支持 反对

使用道具 举报

84

主题

766

帖子

0

精华

金牌会员

Rank: 6Rank: 6

积分
2775
金钱
2775
注册时间
2015-6-1
在线时间
394 小时
发表于 2017-11-8 16:21:42 | 显示全部楼层
龙之谷 发表于 2017-11-8 14:02
ok,看到帖子还专门去查了一下程序,看的不仔细,没注意到函数中又*4096了
My mistake.....

我很好奇,这东西为什么我们一直用着正常?咋突然就说某个地方错了呢如果以前用着不正常,应该很多人反馈的呀。难道它错了不影响用?
自在随心
回复 支持 反对

使用道具 举报

72

主题

2711

帖子

2

精华

论坛大神

Rank: 7Rank: 7Rank: 7

积分
3505
金钱
3505
注册时间
2014-8-4
在线时间
696 小时
发表于 2017-11-8 16:55:32 | 显示全部楼层
yuzeyuan1 发表于 2017-11-8 16:21
我很好奇,这东西为什么我们一直用着正常?咋突然就说某个地方错了呢如果以前用着不正常,应该很多 ...

起初也有这样的感觉,专门对照了代码,也是粗心,没注意到代码处理语句
错了影不影响用不太确定,但如果错了这么多网友应该会早些时候就发现问题了,毕竟代码推出已经有一段时间了
以我资质之鲁钝,当尽平心静气、循序渐进、稳扎稳打之力。
回复 支持 反对

使用道具 举报

84

主题

766

帖子

0

精华

金牌会员

Rank: 6Rank: 6

积分
2775
金钱
2775
注册时间
2015-6-1
在线时间
394 小时
发表于 2017-11-9 08:17:59 | 显示全部楼层
本帖最后由 yuzeyuan1 于 2017-11-9 08:20 编辑
龙之谷 发表于 2017-11-8 16:55
起初也有这样的感觉,专门对照了代码,也是粗心,没注意到代码处理语句
错了影不影响用不太确定,但如果 ...

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

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则



关闭

原子哥极力推荐上一条 /2 下一条

正点原子公众号

QQ|手机版|OpenEdv-开源电子网 ( 粤ICP备12000418号-1 )

GMT+8, 2025-6-6 19:40

Powered by OpenEdv-开源电子网

© 2001-2030 OpenEdv-开源电子网

快速回复 返回顶部 返回列表