指向易失结构成员的常量指针

时间:2019-10-28 20:09:06

标签: c embedded const volatile

我正在使用微控制器进行一些ADC测量。我尝试使用-O2优化来编译以下代码时遇到问题,当代码中存在PrintVal()函数时,MCU冻结。我进行了一些调试,结果发现,当我添加-fno-inline编译器标志时,即使使用PrintVal()函数,代码也可以正常运行。

这里是一些背景:

AdcIsr.c包含ADC完成其工作时执行的中断。该文件还包含ISRInit()函数,该函数初始化将在转换后保存值的变量。在主循环中将等待中断,然后才访问AdcMeas.value。

AdcIsr.c
static volatile uin16_t* isrVarPtr = NULL;

ISR()
{
    uint8_t tmp = readAdc();
    *isrVarPtr = tmp;
}

void ISRInit(volatile uint16_t *var)
{
    isrVarPtr = var;
}
AdcMeas.c

typedef struct{
    uint8_t id;
    volatile uint16_t value;
}AdcMeas_t;

static AdcMeas_t AdcMeas = {0};

const AdcMeas_t* AdcMeasGetStructPtr()
{
    return &AdcMeas;
}
main.c

void PrintVal(const AdcMeas_t* data)
{
    printf("AdcMeas %d value: %d\r\n", data->id, data->value);
}

void StartMeasurement()
{
    ...
    AdcOn();
    ...
}

int main()
{
    ISRInit(AdcMeasGetStructPtr()->value);
    while(1)
    {
        StartMeasurement();
        WaitForISR();
        PrintVal(AdcMeasGetStructPtr());
        DelayMs(1000);
    }
}

问题:

  1. 使用const AdcMeas_t *数据作为PrintVal()函数的参数是否存在问题?我了解AdcMeas.value可能会在中断内更改,并且PrintVal()可能已过时。

  2. AdcMeas包含一个“通用获取器”。使用这种功能允许对静态结构的只读访问是一种好习惯吗?还是应该实现AdcMeasGetId()和AdcMeasGetValue函数(请注意,此结构只有2个成员,如果有8个成员呢?)?

我知道这段代码有点笨(在while循环中等待中断),这只是一个例子。

1 个答案:

答案 0 :(得分:0)

一些错误:

  • 您没有头文件,没有库包含文件或您自己的文件。这意味着一切都无济于事,直到您修复它。没有头文件,您不能在C中执行多个文件项目。

  • *isrVarPtr = tmp;在这里,您在没有竞争条件的情况下写入变量。如果主程序分几个步骤读取此变量,则可能会有获取错误数据的风险。您需要防范竞争条件或保证原子访问。

  • const AdcMeasGetStructPtr()简直是乱码,里面的return &AdcMeas;不可能用兼容的C编译器进行编译。

    如果您使用的是旧的但合格的C90编译器,则返回类型将被视为int。否则,如果您使用的是现代C编译器,则即使函数定义也不会编译。因此,似乎您的编译器出了点问题,这比此错误要引起更大的关注。

  • 在C文件中声明typedef struct,然后返回指向它的指针没有任何意义。您需要重新设计该模块。如果只有一个实例(单例),则可以使用getter函数将实例返回到私有结构。但是,如上所述,它需要处理比赛条件。

样式问题:

    函数声明中的
  • 空括号()在C中几乎总是错误的。这是过时的样式,表示“接受任何参数”。 C ++在这里有所不同。

  • int main()在微控制器系统中根本没有任何意义。您应该使用一些适合于独立程序的实现定义形式。最常用的形式是void main (void)

  • DelayMs(1000);在任何嵌入式系统中都是高度可疑的代码。永远没有理由要挂掉您的MCU一秒钟没用的最大电流消耗。

总的来说,您似乎将从“连续转换” ADC中受益。支持连续转换的ADC会将其最新读取的数据转储到数据寄存器中,您可以在需要时通过轮询将其读取。捕获所有ADC中断实际上仅适用于硬实时系统,信号处理等。