Atmega328p: Uart 7 data bits 1 stop bit

[M. K. (sylaina)]

Ich habe ein Problem mit dem UART.
Auf die Gegenseite habe ich keinen Einfluss, ich muss also benutzen was 
da ist. Das Frameformat ist 7 bit, 1 stopbit und parity even. Das 
"blöde" ist, dass den Uart-Receiver die Einstellung beim Stopbit nicht 
interessiert (Datasheet Page 197: Bit 3 – USBSn: Stop Bit Select
This bit selects the number of stop bits to be inserted by the 
Transmitter. The Receiver ignores this setting.). Ich empfange immer nur 
richtig Daten wenn ich am PC auf 2 Stopbits umstelle, bei einem Stopbit 
werden die Zeichen schlicht falsch empfangen. Dazu die Bilder im Anhang:
1. Bild (von 13:30:59 Uhr): "Test"+CR+LF gesendet (bei den gezeigten 
Einstellungen)
2. Bild (von 13:31:33 Uhr): "Test"+CR+LF gesendet (bei den gezeigten 
Einstellungen)
3. Bild (von 13:31:54 Uhr): CR+LF gesendet (bei den gezeigten 
Einstellungen)
Ich muss eigentlich nur hinbekommen, dass der Receiver bei 1 Stopbit die 
Daten richtig empfängt. Muss ich da noch irgendwo was schubbsen wenn 
mein Frameformat nur 7 bit + 1 stop ist? Ich sehs grade nicht.

Mein Testcode:
main

#define ubbrFuer4800BD      185     //Bei 14,318 MHz Quarz fuer 4800 Bd
#define uart_str_length     19


#include <avr/io.h>
#include <avr/interrupt.h>

#include "uart.h"
#include "main.h"

volatile char uartString[uart_str_length] = "050.00\r\n";
volatile uint8_t uart_str_count = 0;
volatile uint8_t uartGetMessage = 0;

int main(void)
{
    /* insert your hardware initialization here */
    setup();
    sei();
    for(;;){
        /* insert your main loop code here */
        while (uartGetMessage == 0);
        uartGetMessage = 0;
        sendString(uartString);
    }
    return 0;   /* never reached */
}

void setup(void) {
    //setup uart/rs232
    uartSetup(ubbrFuer4800BD);
}
ISR(USART_RX_vect){
    unsigned char nextChar;
    // Daten aus dem Puffer lesen
    nextChar = UDR0;
    if ( uartGetMessage == 0) {
        // Daten werden erst in uart_string geschrieben, wenn nicht String-Ende bzw. maximale Zeichenlaenge erreicht ist
        if( (nextChar != '\n') &&
           (nextChar != '\r') &&
           (uart_str_count < uart_str_length) ) {
            // Zeichen abspeichern
            uartString[uart_str_count] = nextChar;
            // und String-Count um 1 erhoehen
            uart_str_count++;
        }
        else {
            // Stringende definieren
            uartString[uart_str_count] = '\0';
            // Zeichenempfangszaehler zuruecksetzen
            uart_str_count = 0;
            // und Flag setzen für Datenempfang Abgeschlossen
            uartGetMessage = 1;
        }
        
    }
}

uart.c

#include "uart.h"


void uartSetup(uint16_t baudrate){
    //set Baudrate
    UBRR0H = baudrate >> 8;
    UBRR0L = baudrate & 0xFF;
    //set other UART Settings
    UCSR0B |= (1<<TXEN0)|(1 << RXEN0)|(1<<RXCIE0);  // UART TX/RX einschalten, RXIE einschalten
    UCSR0C |= (1 << UPM01)|(1<<UCSZ01); // Asynchron 7N1 und Parity even
}
void sendChar(unsigned char zeichen){
    {
        /* Wait for empty transmit buffer */
        while ( !( UCSR0A & (1<<UDRE0)) );
        /* Put data into buffer, sends the data */
        UDR0 = zeichen;
    }
}
void sendString(const char* charOfString){
    //send string from ram
    /* while *charOfString != '\0 */
    while (*charOfString) {
        sendChar(*charOfString);
        charOfString++;
    }
}
void sendString_p(const char* charOfString){
    //send string from flash
    register char c;
    /* while *charOfString != '\0 */
    while ( (c = pgm_read_byte(charOfString++)) ) {
        sendChar(c);
    }
}

[S. Landolt (Gast)]

> UCSR0B |= (1<<TXEN0)|(1 << RXEN0)|(1<<RXCIE0);
> UCSR0C |= (1 << UPM01)|(1<<UCSZ01);

Das 'oder' ist schlecht, UCSZ00 ist ja bereits gesetzt als initial 
value, das führt also zum 8-bit-mode.

[Max D. (max_d)]

Den receiver interessieren die stop-bits deswegen nicht, weil er ein 
zweites stop-bit nicht von einer kleinen Pause unterscheiden kann. Also 
tut er immer so als würde er mit einem einzelnen SB rechnen und 
interpretiert "bei Bedarf" das zweite als Ruhepegel.

[spess53 (Gast)]

Hi

>Hat jemand eine Verbesserungsidee?

Was für einen Oszillator benutzt du beim  Atmega328p?

MfG Spess

[Thomas E. (thomase)]

S. Landolt schrieb:
> das führt also zum 8-bit-mode

So ist es.

Der Empfänger erwartet somit 11 Bit: Start-8-P-Stop
Der PC sendet aber nur 10 Bit: Start-7-P-Stop

Das 11. Bit, welches vom Empfänger erwartet wird, ist dann das 1. Bit 
vom nächsten Frame. Das ist aber das Startbit und immer 0. Erwartet wird 
aber Stop=1. Das führt dann zu einem Frameerror. Mit einem 2.Stopbit 
sind die Daten zwar immer noch falsch, zumindest passt aber der Frame.

mfg.

[M. K. (sylaina)]

S. Landolt schrieb:
> Das 'oder' ist schlecht, UCSZ00 ist ja bereits gesetzt als initial
> value, das führt also zum 8-bit-mode.

Ohm man, natürlich. Du hast recht, der Receiver ist ja auf 8 bit 
default-mäßig eingestellt. Ich war soo blind...

[spess53 (Gast)]

Hi

>Ich war soo blind...

Deswegen macht man in der Initialisierung auch '=' und nicht '|='.

MfG Spess

[Max D. (max_d)]

spess53 schrieb:
> Deswegen macht man in der Initialisierung auch '=' und nicht '|='.

Da aber bitte vorsichtig wenn sich mehere Elemente ein Register teilen.

[M. K. (sylaina)]

spess53 schrieb:
> Hi
>
>>Ich war soo blind...
>
> Deswegen macht man in der Initialisierung auch '=' und nicht '|='.
>
> MfG Spess

Es gibt viele Wege. 99,9% aller Register (in der Regel sinds nur die 
Port-Register, mache nicht soo viel mit µCs), die ich für gewöhnlich 
beim AVR benutze sind komplett mit 0 initialisiert. Mir ist einfach 
entfallen, dass der UART initialmäßig beim Atmeg328 auf 8N1 eingestellt 
ist.