0

所以我想做的是接收一个数据包并将有效负载打印到串行端口。在端口监听是一个 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 时正在学习 C，但任何有关设计的指针也都很棒。

谢谢。

4

1 回答 1

1

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

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

  • 从代码中删除所有“幻数”并将其替换为#defined 位掩码或常量。

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

  • 不要对数组/字符串使用未指定的宽度 (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';
  }
}
于 2011-05-05T08:16:56.403 回答