c - 将 C 语言与 TI EZ430-RF2500 开发芯片结合使用

Question

所以我想做的是接收一个数据包并将有效负载打印到串行端口。在端口上监听的是一个 python 脚本，它重新组装有效负载并用它做一些事情。

这是代码:

#include "mrfi.h"
int main(void)
{
  BSP_Init();
  MRFI_Init();
  //Initialize the serial port
  P3SEL    |= 0x30;
  UCA0CTL1  = UCSSEL_2;
  UCA0BR0   = 0x41;
  UCA0BR1   = 0x3;
  UCA0MCTL  = UCBRS_2;                     
  UCA0CTL1 &= ~UCSWRST;
  MRFI_WakeUp();
  MRFI_RxOn();
  __bis_SR_register(GIE+LPM4_bits);
}
//This is run when a packet is received
void MRFI_RxCompleteISR()
{
  uint8_t i;
  P1OUT ^= 0x02;
  mrfiPacket_t packet;
  MRFI_Receive(&packet);
  char output[] = {"                   "};
  for (i=9;i<29;i++) {
    output[i-9]='a';
    if (packet.frame[i]=='\r') {
      output[i-9]='\n';
      output[i-8]='\r';
    }
  }
  TXString(output, (sizeof output));
}

我发送了一个包含测试数据的数据包，但什么也没有。有人有任何见解吗？另外，虽然只是为了让你知道我在这样做的同时正在学习 C，所以任何关于设计的指导也会很棒。

谢谢。

Answer 1

我不知道为什么你的代码不起作用，但这里有一些按要求的设计提示。

• 由于这似乎是一个无主机系统，因此 main() 很可能返回 void。我假设您没有发布所有代码，因为在无主机中， main() 中也应该有一个永恒循环。

• 从代码中删除所有“魔数(Magic Number)”，并将其替换为#定义的位掩码或常量。

• 将中断内的所有代码减少到最少。最佳中断只设置一些标志。

• 不要对数组/字符串使用未指定的宽度 (output[])。嵌入式系统设计的目的是使事物具有确定性和固定性。

• 下面是编写该循环的另一种方法的示例。由于我不知道这个程序应该做什么，因此将常量名称替换为有意义的名称。

uint8_t output[OUTPUT_N];

memset(output, ' ', SPACES_N);
output[OUTPUT_N - 1] = '\0';

for(i=0; i < SOMETHING; i++)
{
  output[i + A_OFFSET] = 'a';

  if(packet.frame[i + FRAME_OFFSET] == '\r')
  {
    output[i + CR_OFFSET] = '\r';
    output[i + LF_OFFSET] = '\n';
  }
}