stm32l1xx库明显bug

只看该作者 · 2012-2-9 14:06

本帖最后由 knight_21ic 于 2012-2-9 14:10 编辑

/*******************************************************
  * @file stm32l1xx_rtc.h
  * @author  MCD Application Team
  * @version V1.0.0
  * @date 31-December-2010
  * @brief This file contains all the functions prototypes for the RTC firmware
  *       library.
  *****************************************************

......

#define RTC_Month_January             ((uint32_t)0x00000001)
#define RTC_Month_February          ((uint32_t)0x00000002)
#define RTC_Month_March             ((uint32_t)0x00000003)
#define RTC_Month_April             ((uint32_t)0x00000004)
#define RTC_Month_May                ((uint32_t)0x00000005)
#define RTC_Month_June                ((uint32_t)0x00000006)
#define RTC_Month_July                ((uint32_t)0x00000007)
#define RTC_Month_August             ((uint32_t)0x00000008)
#define RTC_Month_September          ((uint32_t)0x00000009)
#define RTC_Month_October             ((uint32_t)0x000000010)/*<------此处应该是 0x00000000A*/

#define RTC_Month_November          ((uint32_t)0x000000011)/*<------此处应该是 0x00000000B*/

#define RTC_Month_December          ((uint32_t)0x000000012) /*<------此处应该是 0x00000000C*/
#define IS_RTC_MONTH(MONTH)          (((MONTH) >= 1) && ((MONTH) <= 12))/*<------要么就是此处验证MONTH范围有错误 */

#define IS_RTC_DATE(DATE)             (((DATE) >= 1) && ((DATE) <= 31))

1万 · 2012-2-9 14:21

没看出来怎么“明显”呀？宏定义要看怎么用它，只看一个定义怎么看出来有问题呢？

请LZ指教一二。

只看该作者 · 2012-2-9 14:40

#define RTC_Month_October             ((uint32_t)0x000000010)/*<------此处应该是 0x00000000A*/

#define RTC_Month_November          ((uint32_t)0x000000011)/*<------此处应该是 0x00000000B*/

#define RTC_Month_December          ((uint32_t)0x000000012) /*<------此处应该是 0x00000000C*/
#define IS_RTC_MONTH(MONTH)          (((MONTH) >= 1) && ((MONTH) <= 12))/*<------要么就是此处验证MONTH范围有错误 */

这个还不明显？

试验一下就可以了分别设置
RTC_DateStructure->RTC_Month = 0x000000012；
和
RTC_DateStructure->RTC_Month = 0x00000000C；
看看结果就知道了

再说了 (((MONTH) >= 1) && ((MONTH) <= 12)) 这个范围明显不包括 0x10 0x11 0x12 还不明显？

我是写程序的时候设置RTC时间遇到奇怪的问题追踪到此才看到的，

香水城你试试就知道了

1万 · 2012-2-9 15:15

你能确定这个宏定义应该这么用吗？你怎么知道你的用法是正确的？

我没有看过这个库里是怎么用的，所以我不知道它是否正确。请你最好也先看看库里是怎么用的，然后再下结论。

只看该作者 · 2012-2-9 15:27

先不说 RTC_Month_December =  0x12 还是 12
这句话“ (((MONTH) >= 1) && ((MONTH) <= 12)) 这个范围明显不包括 0x10 0x11 0x12 还不明显？”
不知道香水城你能不能看懂

再说了我试验过当调用 RTC_SetDate 库函数设置 RTC_Month = 0x12 时没有反应就是因为 IS_RTC_MONTH(MONTH) 通不过

当设置  RTC_Month = 12 时就OK 了

我没有贬低STM32，我一直在用STM32的
香水城你不必紧张

我只是遇到bug 通报一下，想找出到底是库编写者笔误，还是定有玄妙！望得到确认

1万 · 2012-2-9 15:34

我才不会因为你通报一个Bug而紧张，我只是认为你的结论不严谨，认为你把别人都当成傻瓜的这种态度要不得，:L

我请你好好看看库里面是怎么用的这些宏定义，不知道你看了没有。

如果没有看过，你可以看看下面这个截图中，我标出来的部分，如果想看完整，就把原文件打开看看：

1万 · 2012-2-9 15:41

这是BCD表示法 #define RTC_Month_October ((uint32_t)0x000000010)

这是BIN表示法 #define IS_RTC_MONTH(MONTH) (((MONTH) >= 1) && ((MONTH) <= 12))

你怎么能够不做变换，就生往里套呢？

只看该作者 · 2012-2-9 15:56

香版主果然厉害，什么问题都难不倒。

只看该作者 · 2012-2-9 16:08

本帖最后由 knight_21ic 于 2012-2-9 16:09 编辑

7# 香水城

唉不是我硬套只能说定义的太那个了

谁使用库前还要仔细看函数里面的，这句程序呢
  if ((RTC_Format == RTC_Format_BIN) && ((RTC_DateStruct->RTC_Month & 0x10) == 0x10))
  {
RTC_DateStruct->RTC_Month = (RTC_DateStruct->RTC_Month & (uint32_t)~(0x10)) + 0x0A;
  }

一般都是根据定义的来的是谁看到 IS_RTC_MONTH(MONTH)  就会知道是检查month的合法性

要不就不要偷懒加一个 BIN 和 BCD 关键字比如：
IS_RTC_BIN_MONTH(MONTH) 或 IS_RTC_BCD_MONTH(MONTH)

#define RTC_Month_BIN_December  0x0C
#define RTC_Month_BCD_December  0x12