PIC UART fehlerhafte Übertragung

[Lasse01 (Gast)]

Hallo zusammen,

ich habe hier einen PIC24FV32KA301 mit dem ich Text über UART übertragen 
will.

Den nötigen Code habe ich mir zum größten Teil vom Code Configurator in 
MPLAB X generieren lasse. Ich habe den generierten Code natürchlich auch 
schon selber durch geschaut, aber habe keinen Fehler des Generators 
gefunden.

In der main habe ich dann nur noch diesen einfachen Code.

    char buffer[50] = "Hallo-Welt123\n";
    
    SYSTEM_Initialize();
    
    __delay_ms(10);
    
    while(1)
    {
        UART1_WriteBuffer((const uint8_t*)buffer, strlen(buffer));
        __delay_ms(1000);
    }
    
    return 0;

auf dem PC emfange ich aber nur abwechselnd

HalloWl13<\n>
l1lHal-et2<\n>




der Initialisierungs Code des UART sieht so aus

#define UART1_CONFIG_TX_BYTEQ_LENGTH 100
#define UART1_CONFIG_RX_BYTEQ_LENGTH 100

static uint8_t uart1_txByteQ[UART1_CONFIG_TX_BYTEQ_LENGTH] ;
static uint8_t uart1_rxByteQ[UART1_CONFIG_RX_BYTEQ_LENGTH] ;

void UART1_Initialize (void)
{
   // STSEL 1; IREN disabled; PDSEL 8N; UARTEN enabled; RTSMD disabled; USIDL disabled; WAKE disabled; ABAUD disabled; LPBACK disabled; BRGH enabled; RXINV disabled; UEN TX_RX; 
   U1MODE = (0x8008 & ~(1<<15));  // disabling UARTEN bit   
   // UTXISEL0 TX_ONE_CHAR; UTXINV disabled; OERR NO_ERROR_cleared; URXISEL RX_ONE_CHAR; UTXBRK COMPLETED; UTXEN disabled; ADDEN disabled; 
   U1STA = 0x0000;
   // BaudRate = 9600; Frequency = 4000000 Hz; BRG 103; 
   U1BRG = 0x0067;

   IEC0bits.U1RXIE = 1;

    //Make sure to set LAT bit corresponding to TxPin as high before UART initialization
   U1MODEbits.UARTEN = 1;  // enabling UART ON bit
   U1STAbits.UTXEN = 1;
   
   

   uart1_obj.txHead = uart1_txByteQ;
   uart1_obj.txTail = uart1_txByteQ;
   uart1_obj.rxHead = uart1_rxByteQ;
   uart1_obj.rxTail = uart1_rxByteQ;
   uart1_obj.rxStatus.s.empty = true;
   uart1_obj.txStatus.s.empty = true;
   uart1_obj.txStatus.s.full = false;
   uart1_obj.rxStatus.s.full = false;
}

vielleicht kann mir ja jemand weiter helfen.

[Volker S. (vloki)]

Ein Oszilloskop hast du wohl nicht gerade zur Hand?
Dann könntest du einfach mal nachschauen, was auf der Leitung...

[Lasse01 (Gast)]

Nein leider nicht.

[Dieter W. (dds5)]

Sieht so aus, als ob UART1_WriteBuffer den Hardware-Sendepuffer 
überfährt.

[Volker S. (vloki)]

Warum fehlt dann jedes zweite Zeichen?
1000ms sollten auch mehr als genug sein.

@lasse01
Kommen die (falschen)  Strings ungefähr im Sekundenrythmus?

[Lasse01 (Gast)]

Ja, das timing stimmt.

[Volker S. (vloki)]

Mit was empfängst du die Daten auf dem PC?
Irgendein Terminal (Br@y, TeraTerm, HTerm....) oder was selber 
geschriebenes?

Kann man bei dem PIC24 irgendwie 2 Stopbits einstellen?
Du könntest auch mal testen, was passiert, wenn der Wert für die BAUD 
Rate leicht verändert wird...
(Ohne Oszi ist alles doof ;-)

[waflija (Gast)]

Volker S. schrieb:
> Mit was empfängst du die Daten auf dem PC?
> Irgendein Terminal (Br@y, TeraTerm, HTerm....) oder was selber
> geschriebenes?
>
> Kann man bei dem PIC24 irgendwie 2 Stopbits einstellen?
> Du könntest auch mal testen, was passiert, wenn der Wert für die BAUD
> Rate leicht verändert wird...
> (Ohne Oszi ist alles doof ;-)

Das wäre auch mein erster Anstatz. Prüfe in jedem Fall mal die 
Verbindungseinstellungen auf beiden Seiten.

->Baudrate korrekt?
->Anzahl Startbits korrekt?
->Anzahl Stoppbits korrekt?
->Parity richtig?
->Wartet eine Seite eventuell auf bestimmte Signale wie Ready-to-recive 
or ähnliches?

Auf Seiten des PIC muss man etwas im Datenblatt wühlen bevor man alles 
zusammen hat. Teilweise stellen die Bibiliotheken von Microchip das dann 
auch als "default" auf andere Werte. Am besten du lässt dir die im 
Debugger / Simulation mal ausgeben, kurz bevor du die Verbindung 
öffnest.

[Lasse01 (Gast)]

Ich benutze HTerm auf dem PC.

Ja er kann 2 StopBits, bringt leider auch nichts.
Andere Baud-Raten machen auch keinen Unterschied.

[Lasse01 (Gast)]

Ich habe jetzt mal absichtlich unpassende einstellungen gewählt, einfach 
um zu sehen was passiert.

HTerm:
- 9600 Baud
- 7 Datenbits
- 1 Stopbit
- Odd Parity

PIC:
- 9600 Baud
- 8 Datenbits
- 2 StopBits
- keine Parität

komischer weise ändert das nichts an dem Ergebniss. Ist das normal ?

[waflija (Gast)]

Lasse01 schrieb:
> Ich habe jetzt mal absichtlich unpassende einstellungen gewählt,
> einfach
> um zu sehen was passiert.
>
> HTerm:
> - 9600 Baud
> - 7 Datenbits
> - 1 Stopbit
> - Odd Parity
>
> PIC:
> - 9600 Baud
> - 8 Datenbits
> - 2 StopBits
> - keine Parität
>
> komischer weise ändert das nichts an dem Ergebniss. Ist das normal ?

Das sollte einiges Ändern. Falsche Anzahl Datenbit und ein weiteres bit 
wird als 1 gliefert und dann als Parität interpretiert. Das müsste jede 
Menge Paritätsfehler produzieren und könnte durchaus deinen Fehler 
erklären.

Versuch im Zweifel mal ein anderes Terminal z.B. Putty.

[Lasse01 (Gast)]

Ich hab den Fehler gefunden.

Ich konnte mich noch erinnern das es früher immer ohne probleme 
funktioniert hat, also habe ich mal eine ältere Version vom MPLAB X 
CodeConfigurator ausprobiert und siehe da, es geht wieder.

MCC v3.26.2

/**
    Maintains the driver's transmitter state machine and implements its ISR
*/
void __attribute__ ( ( interrupt, no_auto_psv ) ) _U1TXInterrupt ( void )
{ 
    if(uart1_obj.txStatus.s.empty)
    {
        IEC0bits.U1TXIE = false;
        return;
    }

    IFS0bits.U1TXIF = false;


    while(!(U1STAbits.UTXBF == 1)) // <---------
    {


        U1TXREG = *uart1_obj.txHead;

        uart1_obj.txHead++;

        if(uart1_obj.txHead == (uart1_txByteQ + UART1_CONFIG_TX_BYTEQ_LENGTH))
        {
            uart1_obj.txHead = uart1_txByteQ;
        }

        uart1_obj.txStatus.s.full = false;

        if(uart1_obj.txHead == uart1_obj.txTail)
        {
            uart1_obj.txStatus.s.empty = true;
            break;
        }
    }
}

MCC v3.15

/**
    Maintains the driver's transmitter state machine and implements its ISR
*/
void __attribute__ ( ( interrupt, no_auto_psv ) ) _U1TXInterrupt ( void )
{ 
    if(uart1_obj.txStatus.s.empty)
    {
        IEC0bits.U1TXIE = false;
        return;
    }

    IFS0bits.U1TXIF = false;

    int count = 0;
    while((count < UART1_TX_FIFO_LENGTH)&& !(U1STAbits.UTXBF == 1)) // <---------
    {
        count++;

        U1TXREG = *uart1_obj.txHead;

        uart1_obj.txHead++;

        if(uart1_obj.txHead == (uart1_txByteQ + UART1_CONFIG_TX_BYTEQ_LENGTH))
        {
            uart1_obj.txHead = uart1_txByteQ;
        }

        uart1_obj.txStatus.s.full = false;

        if(uart1_obj.txHead == uart1_obj.txTail)
        {
            uart1_obj.txStatus.s.empty = true;
            break;
        }
    }
}

Da hat wohl jemand zu viel "optimiert" ;)

trotzdem Danke für die Hilfe.