USART überträgt falsche Zeichen

[TM F. (p_richner)]

Hallo Zusammen

Ich versuche die Kommunikation zwischen uC Atmega 1284p mit 16MHz und PC 
über USART. Dabei benutze ich folgendes Programm, wobei dies 
grösstenteils dem Beispiel aus dem Datenblatt entspricht.
Das Problem liegt darin, dass komplett falsche Zeichen ausgegeben 
werden. Ich sehe jedoch nicht weiter, wo das Problem liegen könnte.

Im Anhang ist die Ausgabe vom Putty Terminal. Dieses habe ich auf eine 
Baudrate von 9600 eingestellt, Parity: none, Flow Control: XON/XOFF, 8 
Datenbit, 1 Stopbit

void USART_init( void )
{
  // Initialize the whole USART
  // Set the baud rate value
  uint16_t baud = 9600;
  // Set baud rate 
  UBRR0H = (unsigned char)(baud>>8);
  UBRR0L = (unsigned char)baud;
  // Set the XCK0 pin to output
  DDRB   = 0x01;
  // Enable receiver and transmitter 
  UCSR0B |= (1<<RXEN0)|(1<<TXEN0);
  // Set frame format: 8data, 1stop bit and synchronous mode 
  UCSR0C |= (1<<UCSZ01) | (1<<UCSZ00) | (1<<UMSEL00);
}

void USART_transmit( unsigned char data )
{
  // Send the data to the PC
  // Wait for empty transmit buffer  
  while ( !( UCSR0A & (1<<UDRE0)) )
  ;
  // Put data into buffer, sends the data  
  UDR0 = data;
}

int main(void)
{
  USART_init();
    while (1) 
    {  
    if(TASTER_EINGANG)
      USART_transmit('A');

    }
}

[Peter II (Gast)]

welche Taktquelle hast du am Atmega?

[Cyblord -. (cyblord)]

Baudrate wird völlig falsch gesetzt. Schaue bitte ins Datenblatt was in 
UBRR rein muss und wie das aus der Baudrate berechnet wird.

[Peter II (Gast)]

TM F. schrieb:
> uint16_t baud = 9600;
>   // Set baud rate
>   UBRR0H = (unsigned char)(baud>>8);
>   UBRR0L = (unsigned char)baud;

das kann doch nicht funktionieren, ließ mal das Datenblatt was in UBRR0 
für ein Wert rein muss.

[TM F. (p_richner)]

Peter II schrieb:
> das kann doch nicht funktionieren, ließ mal das Datenblatt was in UBRR0
> für ein Wert rein muss.

Ok danke.

Ich habe mal 103 in dieses Register geschrieben und den Modus in den 
asynchronen gewechselt und jetzt funktioniert es:)

Habe den Baud-wert in der Initialisierung falsch interpretiert.

Danke für die schnelle Hilfe

[Frank M. (ukw) (Moderator)]

TM F. schrieb:
> Ich habe mal 103 in dieses Register geschrieben

Das geht auch noch etwas portabler:

#define UART_BAUD_RATE 9600
...
   uint16_t baud = F_CPU / (UART_BAUD_RATE * 16L) -1;

   UBRRH = (uint8_t) (baud >> 8);
   UBRRL = (uint8_t) baud;

Da kommt dann für baud ebenso die 103 raus.

Vorteil: Solltest Du mal Deine UART-Routinen für einen AVR mit anderer 
Taktrate verwenden wollen, brauchst Du am Code nichts zu ändern. 
Dasselbe gilt für UART_BAUD_RATE. Bei einer Änderung auf 19200 ist 
lediglich das #define dafür zu ändern.

Es gibt noch weitere Optimierungsmöglichkeiten, wie zum Beispiel die 
Berücksichtigunge des U2X-Flags - falls der verwendete AVR dies hat.

[TM F. (p_richner)]

Frank M. schrieb:
> Da kommt dann für baud ebenso die 103 raus.

Ok danke, dann mache ichs universell.

Frank M. schrieb:
> falls der verwendete AVR dies hat.

Ja das hat er.