Проблемы с передачей USART на PIC

Немного сбивает с толку то, что код в вопросе отличается от кода в pastebin, но если предположить, что код pastebin правильный, я бы сказал, что проблема в том, что вы отправляете неинициализированные данные из массива вместо символов, которые вы положить в него.

TXREG = TXBuffer[TXEndPos];           // send next byte

должно быть

TXREG = TXBuffer[TXCurrentPos];           // send next byte

Важно копировать/вставлять код напрямую (использование pastebin — это здорово!), потому что вы фактически исправили это самостоятельно в вопросе, из-за чего людям трудно обнаружить проблему :)

Возможно, это было бы легче заметить самостоятельно, если бы вы инициализировали буфер известным значением при запуске (например, 0x5A).

Вы также упомянули, что в какой-то момент вы сбросили переменные TxCurrentPos& TxEndPosдо нуля, но я не вижу этого в вашем коде. Лучше всего делать это всякий раз, когда вы увеличиваете эти значения. Например

TxCurrentPos++;
TxCurrentPos %= TX_BUFFER_SIZE;

В сторону: мне нравится, что вы удалили избыточную переменную TxBufferSize в вопросе - это следует принципу DRY и устраняет источник ошибки. Дублирование данных в двух местах вызывает проблемы.

Несколько вещей, которые можно попробовать:
1) Убедитесь, что ваш буфер инициализирован, это скажет вам, исходит ли 0x98 из буфера или откуда-то еще.
2) Отключить прерывания при добавлении элементов в буфер

Я не вижу ничего, что делало бы ваш буфер круглым. Я думаю, что вы полагаетесь на переполнение, чтобы получить от 127 до 0... но беззнаковый символ может представлять значения от 0 до 255. Таким образом, ваш TXCurrentPos будет идти прямо с конца вашего массива.

Я также видел, как компиляторы PIC C делали некоторые странные вещи без уважительной причины. Большинство компиляторов также могут выводить собранную версию программы; возможно, стоит некоторое время просмотреть листинг ассемблера и посмотреть, как компилируется C. В зависимости от типа данных, объявленного как TXREG, компилятор может попытаться выполнить приведение типов.

Кроме того, в вашем коде приема в прерывании вы ДОЛЖНЫ проверить приемник на наличие состояния переполнения и очистить его. Каждый. Одинокий. Время. Приемник RS232 отказывается принимать дополнительные данные, пока установлен бит ошибки переполнения.

void ClearRXOverRun()
{
    // If there's a frame overrun, RS232 will refuse to receive
    if (OERR == 1)
    {
        // Clear overrun condition by turning RX off and back on
        CREN = 0;
        CREN = 1;   
    }   
}   

Мне жаль, что это может показаться очень простым, но пробовали ли вы записывать свои данные в EEPROM, а затем читать их позже, чтобы убедиться, что вы действительно назначаете правильные данные массиву? Если данные, которые назначаются буферу, не являются допустимыми данными, то это может быть вашей проблемой?

Для этого типа проблемы я делю проблему еще дальше, чем обычно смотрю на сборку. Также я видел проблемы, когда RX мог повредить или переполнить буфер, поэтому я отключил RX, пока у вас не заработает TX.

Первое, что я бы сделал, это двигаться

if(waitingForACK == 0){
    PIE1bits.TXIE = 1; //Enable the TX interrupt
} 

из addToBuffer в собственную функцию. Затем вызывайте его, когда буфер заполнен, а не по ходу дела. Это может быть какая-то проблема с синхронизацией, когда вы не можете добавить, пока прерывание отправляет данные. Когда я передаю пакет, я обычно отключаю прерывание TX, заполняю буфер TX, а затем включаю прерывание TX.

Затем проверьте кольцевой буфер вне прерывания. В вашей основной функции заполните буфер, а затем прочитайте данные.

Затем измените код прерывания, чтобы пройти через промежуточную летучую память. Таким образом, вы можете поставить точку останова в своем прерывании и посмотреть, что происходит. Волатильность заключается в том, что компилятор не удаляет временную переменную.

volatile unsigned char temp;
if((TXEndPos - TXCurrentPos) > 0)            // if there is data in the buffer
{                                              
    temp  = TXBuffer[TXCurrentPos]; 
    Nop();                                   // for adding breakpoints
    Nop();
    Nop();
    // add error checking to make sure TXREG is empty.
    TXREG = temp;                            // send next byte
    TXCurrentPos = TXCurrentPos + 0x01;      // update to the new position

Еще одна маленькая вещь заключается в том, что я бы также рассмотрел возможность изменения

else if (PIR1bits.TXIF == 1){

к

if (PIR1bits.TXIF == 1){

чтобы убедиться, что другие прерывания не истощают прерывание TX.

Затем, если у вас все еще есть проблема, посмотрите на код сборки, я видел, где компилятор делает что-то странное, и это обычно очевидно, если вы посмотрите на сборку.