关于宏的副作用_应该多使用inline函数，少使用宏

今天写 rtc 驱动， rtc 寄存器使用了 bcd 编码， 于是写了一个宏进行转换：
#define FROMBCD8(val)   (((val >> 4)  & 0xf) * 10 + (val & 0xf))
 
然后为了获取某个寄存器，比如小时, 是这么调用的：
tm->tm_hour     = FROMBCD8(micco_read(RTC_HOUR_REG));
 
但是， micco_read 的打印语句表明该函数被调用了两次！
 
仔细研究原来这是由于宏的副作用导致的！
宏扩展后变为：
tm->tm_hour     =  (((micco_read(RTC_HOUR_REG)>> 4)  & 0xf) * 10 + (micco_read(RTC_HOUR_REG)& 0xf))
 
这导致了低效（有时是错误）的代码！micco_read 可是使用低速 i2c 在访问寄存器！
 
修改为：
u8 val;
val = micco_read(RTC_HOUR_REG);
tm->tm_hour     = FROMBCD8(val);
 
即可！
 
该实例，表明宏的缺陷！正确的方法是应该使用 inline 函数， 少使用宏。
上面的宏改为：
static u8 inline frombcd8(u8 val)
{
        return ((val >> 4)  & 0xf) * 10 + (val & 0xf);
}
 
然后， 调用改为：
tm->tm_hour     = frombcd8 (micco_read(RTC_HOUR_REG));
也没有问题， 而且代码更加简洁。