UART in C - Problem

[Freddy (Gast)]

Hallo. :)

einfache Frage, folgender CODE:

/*
 * UART_Test.c
 *
 * Created: 08.04.2014 14:17:40
 *  Author: Freddy
 */
#define F_CPU 16000000

#include <avr/io.h>
#include <stdlib.h>
#include <util/delay.h>



int main(void)
{
  init_uart();
    while(1)
    {
    send_uart();
    _delay_ms(1000);
    }
}

void init_uart(){
  
  //setze BAUD-Wert
  UBRRL =  103;  //9600
  
  //Frame Format konfigurieren: 8n1
  //Tx und Rx aktivieren
  UCSRB = (1<<RXEN) | (1<<TXEN);
  UCSRC = (1<<UCSZ1) | (1<<UCSZ0) | (0<<UMSEL);
}

void send_uart(){
  
  while (!(UCSRA & (1<<UDRE)))
  {
  }
  UDR = 'x';
}

Wenn ich mein PC dran hänge, dann kommt im 1s - Takt zweimal eine Null.. 
Warum? Das sollte doch alles passen oder?

Hab die RegisterKonfiguration aus dem Datenblatt.

Danke! Viele Grüße!

[Karl H. (kbuchegg)]

Welcher µC?


Die hier
> #define F_CPU 16000000
stimmen und sind auch geprüft? Macht dein µC tatsächlich 16Mhz?

[Hubert G. (hubertg)]

Kontroller?
Fehler beim beschreiben des UCSRC? UMSEL fehlt.

[Thomas E. (thomase)]

Hubert G. schrieb:
> Kontroller?
> Fehler beim beschreiben des UCSRC? UMSEL fehlt.
Nein. URSEL!
Aber raten wir doch erstmal, was für ein Controller das ist.

( ) Atmega8
( ) Atmega16
( ) Atmega32

Die URSEL-befreiten haben ja UMSEL0 und UMSEL1.

mfg.

[Freddy (Gast)]

Ja tut mir leid, das habe ich vergessen.

Ist ein Atmega8 :)

Das URSEL ist standart mäßig 1 und das UMSEL 0... beide werte stimmen 
mit dem was ich konfigurieren möchte überein.

Ich möchte eine 8n1 bei 9600 b/s einrichten :)


Wegen der Frequenz, auf das Herz genau vlt nicht, aber mit dem Oszi 
gemessen lieg ich 50Hz daneben..

Grüße!

[Karl H. (kbuchegg)]

Freddy schrieb:
> Ja tut mir leid, das habe ich vergessen.
>
> Ist ein Atmega8 :)
>
> Das URSEL ist standart

standard

> mäßig 1

wo findet sich das in deinem Code wieder?

Wenn du UCSRC einen neuen Wert zuweist, hast du URSEL definitiv nicht 
angegeben. Das musst du aber, denn sonst gilt diese Zuweisung als 
Zuweisung an UBRRH.

[g457 (Gast)]

> Das URSEL ist standart mäßig 1

Fehlt trotzdem. Im Zweifelsfalls einfach das Datenplatt nochmals lesen.

HTH

[Freddy (Gast)]

UCSRC = (1<<UCSZ1) | (1<<UCSZ0) | (0<<UMSEL) | (0<<UMSEL) | (1<<URSEL);

Wäre dass dann so quasi korrekt? :)

Danke für die Hilfe!

[Thomas E. (thomase)]

Freddy schrieb:
>

> UCSRC = (1<<UCSZ1) | (1<<UCSZ0) | (0<<UMSEL) | (0<<UMSEL) | (1<<URSEL);
>

>
> Wäre dass dann so quasi korrekt? :)
Ja. Aber da die UCSZ-Werte ohnehin 1 sind, kannst du dir das Ganze auch 
sparen.

mfg.

[Paul B. (paul_baumann)]

Die Ursel und das Umsel,
die hatten einen Streit:
Wer wohl am Besten sendet,
wer wohl am Besten sendet,
zur schönen Maien-zei-heit-heit
zur schönen Maien-Zeit....

;-)
MfG Paul

(ich bin ein großer Fan von schönen Variablen -und Registernamen)

[Freddy (Gast)]

Thomas Eckmann schrieb:
> Ja. Aber da die UCSZ-Werte ohnehin 1 sind, kannst du dir das Ganze auch
> sparen.

Okay, aber dann wär ja auch die Angabe der UMSEL und URSEL Werte 
überflüssig.. Die sind standardmäßig ja auch entsprechend gesetzt.

MfG

[Thomas E. (thomase)]

Freddy schrieb:
> Thomas Eckmann schrieb:
>> Ja. Aber da die UCSZ-Werte ohnehin 1 sind, kannst du dir das Ganze auch
>> sparen.
>
> Okay, aber dann wär ja auch die Angabe der UMSEL und URSEL Werte
> überflüssig.. Die sind standardmäßig ja auch entsprechend gesetzt.

Darum geht es nicht. Vergiss das UMSEL. Das schaltet eine Funktion im 
UART und steht per Default auf Asynchron. Das ist das, was du brauchst.

Das URSEL-Bit schaltet aber keine Funktion im UART, sondern steuert den 
Zugriff auf UBRRH und UCSRC. Diese Register haben dieselbe(!) Adresse.
Wenn URSEL 1 ist, geht der Wert ins UCSRC, wenn es 0 ist, ins UBRRH. Per 
Default ist dieses Bit eins. Wenn man zuerst das UCSRC setzt, braucht 
man sich nicht darum kümmern. Man darf nur nicht das ganze Byte 
zuweisen.
> UCSRC = (1<<UCSZ1) | (1<<UCSZ0) | (0<<UMSEL) = 0x06;
Damit ist URSEL nämlich 0!

Es dürfen nur die Bits geändert werden:

UCSRC |= (1<<UCSZ1) | (1<<UCSZ0;

oder

UCSRC = (1 << URSEL) | (1<<UCSZ1) | (1<<UCSZ0;

Sonst landet das Byte im UBRRH und setzt es in diesem Fall auf 6. Welche 
Baudrate du dann hast, kannst du selber ausrechnen. Die, die du haben 
haben willst auf jeden Fall nicht.

Nach einem Zugriff auf UBRRH, für das der Maximalwert 15 ist und kein 
normaler Mensch einzelne Bits setzt, wird das URSEL automatisch auf 0 
gesetzt.

UBRRH = 7;

Damit ist das Bit7, also URSEL, 0 und der Wert geht ins UBRRH.

Deswegen ist es grundsätzlich keine gute Idee, das URSEL beim Schreiben 
nicht explizit zu setzen.

mfg.

[Freddy (Gast)]

Ahh! Ich verstehe! Sehr eingängige und klare Erklärung! :)
Danke!

Das URSEL bit ist quasi eine Art Steuerbit das sozusagen eine Art 
Switch-Funktion hat zwischen den Registern!
Und da ich das "|=" nich hatte hat es logischerweise das URSEL immer 0 
gesetzt und dadurch den Wert 6 ins BAUD-Bestimmende Register UBRR 
geschrieben...

Macht echt total sinn!

Vielen Dank! :)
Ich probier das mal!

[Freddy (Gast)]

EDIT:!

Grandios! Läuft. Danke! :)