Posted on April 20, 2013 at 18:17
Couple of observations:
sBuff3[bytesRecieved3] = (uint8_t) (USART3->DR & 0x000000FF); //grab the latest packet
bytesRecieved3++;
cnt--;
if (bytesRecieved3 > sBuffSize)
Your test here is flawed, C uses zero based indices, at 128 your already violating the buffer. It's Received BTW (I before E, except after C).
Your interrupt handler seems to be designed to dwell there waiting for multiple bytes. Don't do that! Don't disable RXNEIE, just handle one byte at a time, accumulate them, and when you have enough process them. I haven't dug through all your logic there, but it looks to be unnecessarily complex, and could be done more simply/efficiently/correctly.
Your string output function back tests the USART SR register for TC, you should front test for TXE before jamming data in DR, and not linger waiting for TC.
