Printf-Ausgabe mit dem ATTINY2313 fehlerhaft

[Frederik R. (f8bit)]

Hallo Community,

um eine serielle Schnittstelle für mein Projekt zu realisieren, versuche 
ich mithilfe von UART vom µC aus einen String an einen "USB to TTL" 
Treiber zu schicken (CP2102), welchen dieser dann an den PC schicken 
soll.

Der "USB to TTL" Treiber funktioniert(Habe RX direkt an TX angeschlossen 
und über gtkterm getestet.)

Nur wenn ich das nachfolgende Programm auf den Tiny lade und mit dem 
Treiber verbinde, erhalte ich unter gtkterm immer nur: "Ha!" statt 
"Hallo Welt!"

Ich habe schon die Möglichkeit mit ein und zwei Stopbits ausprobiert, da 
dies ja bei langsameren Komponenten teilweise empfohlen wird. Hat sich 
aber nichts geändert.

Ihr wisst sicherlich mehr! Vielen Dank für eure Hilfe!

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


#define F_CPU 1000000UL


void init();
void UART_transmit(uint8_t);
void UART_transmit_String(char *);

int main() {
  init();
  
  while(1) {
    UART_transmit_String("Hallo Welt!\0");
    _delay_ms(1000);
    
  }
  
}

void init() {
  UBRRH = 0x00;    //Baudrate Initialisierung
  UBRRL = 0x33;    //0x33 fuer 1200, 0x0C fuer 4800
  
    
  UCSRB =  (1 << RXEN) | (1 << TXEN);        //Enable von Senden und Empfangen
  
  
  UCSRC = (0 << UCSZ2) | (1 << UCSZ1) | (1 << UCSZ0) | (1 << USBS);      //8 Daten Bits und 2 Stop Bits
}

void UART_transmit(uint8_t data) {
    while(UDRE == 0){}
    UDR = data;
    _delay_ms(1);
}

void UART_transmit_String(char * text) {
  while( *text != '\0') {
    UART_transmit(*text++);
  }
}

[da1l6 (Gast)]

1. Das ist falsch rum:

#include <util/delay.h>


#define F_CPU 1000000UL

F_CPU muss VOR delay.h definiert sein, sonst kommt der Wert dort nicht 
an. Normalerweise definiert man das über Compiler Flags im Makefile.
Der Fehler sollte auch eine Compilerwarnung auslösen! Warnungen sind zum 
ernst nehmen da!

 while(UDRE == 0){}
    UDR = data;
    _delay_ms(1);

Zwei mal falsch.
2. UDRE ist eine Konstante:

#define    UDRE    5

Die ist immer != 0, die Schleife tut also nichts.
Du meinst das hier:

while (USR & (1 << UDRE)) {}

3. Das _delay_ms hat da nichts zu suchen und funktioniert aufgrund von 
Fehler 1 auch gar nicht richtig.

Grüße
da1l6

[Dirk B. (dirkb2)]

Frederik R. schrieb:
> "Hallo Welt!\0"

Für die Null sorgt der Compiler. Die musst du nicht explizit angeben.

[Dirk B. (Gast)]

da1l6 schrieb:
> 3. Das _delay_ms hat da nichts zu suchen und funktioniert aufgrund von
> Fehler 1 auch gar nicht richtig.
in dem Fall: 8 Zeichen("llo Welt")=8ms 9 bitzeiten 1/1200 7,5ms ("!" bei 
8ms) zufällig richtig.
Ändert natürlich nichts daran, dass es unpassend/falsch ist. Vermutlich 
wäre 2) sogar ohne einfacher zu finden.