Problem mit dem senden von Zeichen per UART

Hi

Ich habe das myAVR 2 USB Board.
Nun möchte ich ein Zeichen vom PC an den Mikrocontroller senden und je nach Zeichen soll reagiert werden.
Nur kommen die Zeichen leider ganz anders an als sie sollen.
Die mitgelieferten Beispiel Programme funktionieren auch nicht.

Mein Testprogramm:

Code:

---

#define F_CPU 3686400
#include <avr\io.h>
#include <util\delay.h>
#include        <avr/io.h>
void uartPutString(char *buffer);
//----------------------------------------------------------------------
void uartInit()
{
        UBRRL = 23;                      //9600Baud siehe Baudratentabelle
        UCSRB = 8 + 16;        //Sender enable, Empfänger enable
}
//----------------------------------------------------------------------
char uartGetChar()
{
        char data=0;
        //warte bis RX-complete
        while (!(UCSRA&128));
        //empfangen
        data=UDR;
        return data;
}
//----------------------------------------------------------------------
void uartPutChar(char data)
{
        //warte bis UDR leer ist UCSRA / USR bei z.B.: 2313
        while (!(UCSRA&32));
        //sende
        UDR=data;
}
//----------------------------------------------------------------------
main ()
{
        DDRB = 0xFF;
        PORTB = 0xFF;
        _delay_ms(500);       
        PORTB = 0x00;

        char zeichen;

        uartInit();
        while (true) // Mainloop
        {
                zeichen=uartGetChar();

                PORTB = (1<<PINB1);
                _delay_ms(500);       
                PORTB = (0<<PINB1);
                _delay_ms(500);       

                if(zeichen=='0')
                        PORTB = (0<<PINB0);
                if(zeichen=='1')
                        PORTB = (1<<PINB0);

                _delay_ms(500);       
        }
}

---

Durch die LED an PINB1 kann ich sehen das etwas angekommen ist, aber es ist nicht das was es soll.

Ich habe schon die Baudraten 2400, 9600 und 115200 ausprobiert.
Zum senden bzw. empfangen benutze ich HTerm.

Und wie gesagt, mit den mitgelieferten Beispielprogramm welches ein String an den PC senden soll klappt es auch nicht! Es kommt zwar ein String auf dem PC an aber das ist nur Zeichensalat...

Was kann ich noch probieren?

gruß

Am besten, du siehst dir das genau an,
https://www.roboternetz.de/wissen/in...RT_mit_avr-gcc

Werd ich tun.

Es muss doch aber auch mit den Beispielprogrammen funktionieren!
Wo kann da der Fehler noch liegen?

Das ist c-mässig ein Stiefel, auch wenn es scheinbar funktioniert;
if(zeichen=='0')
PORTB = (0<<PINB0);

Ein Bit löschen geht so
PORTB &= ~(1<<PINB0);


Du solltest, so wie in der Wikil, für die Register-Bits die korrekten Symbole verwenden

sowas ist pfui gack:
UBRRL = 23; //9600Baud siehe Baudratentabelle
UCSRB = 8 + 16; //Sender enable, Empfänger enable

Besser:

Code:

---

    uint16_t ubrr = (uint16_t) ((uint32_t) F_CPU/(16*BAUDRATE) - 1);
 
    UBRRH = (uint8_t) (ubrr>>8);
    UBRRL = (uint8_t) (ubrr);
 
    // UART Receiver und Transmitter anschalten
    // Data mode 8N1, asynchron
    UCSRB = (1 << RXEN) | (1 << TXEN);
    UCSRC = (1 << URSEL) | (1 << UCSZ1) | (1 << UCSZ0);

---

Noch was: Bevor "Hello, world" nicht geht, ist an empfange nicht zu denken

https://www.roboternetz.de/wissen/in...lo.2C_world.29

Nun kommt was anderes an, aber immernoch nicht das richtige...

Mein Programm:

Code:

---

#define F_CPU 3686400
#include <avr\io.h>
//----------------------------------------------------------------------
void uartInit()
{
        uint16_t ubrr = (uint16_t) ((uint32_t) F_CPU/(16*9600) - 1);
 
    UBRRH = (uint8_t) (ubrr>>8);
    UBRRL = (uint8_t) (ubrr);
 
    // UART Receiver und Transmitter anschalten
    // Data mode 8N1, asynchron
    UCSRB = (1 << RXEN) | (1 << TXEN);
    UCSRC = (1 << URSEL) | (1 << UCSZ1) | (1 << UCSZ0);
}
//----------------------------------------------------------------------
void uartPutChar(char data)
{
        //warte bis UDR leer ist UCSRA / USR bei z.B.: 2313
        while (!(UCSRA&32));
        //sende
        UDR=(uint8_t)data;
}
//----------------------------------------------------------------------
void print(char buffer[])
{
        for (int i=0;buffer[i]!=0;i++)
                uartPutChar(buffer[i]);
}
//======================================================================
main ()
{
        uartInit();
        while (true) // Mainloop
        {
                print("x");
        }
}

---

Edit:
Ich habe das Beispiel aus der Wiki übernommen (die ohne Interupts):

Code:

---

#define        F_CPU 3686400        // Taktferquenz des myAVR-Boards
#include        <avr\io.h>                // AVR Register und Konstantendefinitionen
#include        <util\delay.h>

void uart_init()
{
        uint16_t ubrr = (uint16_t) ((uint32_t) F_CPU/(16*9600) - 1);

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

        // UART Receiver und Transmitter anschalten
        // Data mode 8N1, asynchron
        UCSRB = (1 << RXEN) | (1 << TXEN);
        UCSRC = (1 << URSEL) | (1 << UCSZ1) | (1 << UCSZ0);

        // Flush Receive-Buffer (entfernen evtl. vorhandener ungültiger Werte)
        do
        {
                UDR;
        }
        while (UCSRA & (1 << RXC));
}

int        uart_putc (const uint8_t c)
{
        // Warten, bis UDR bereit ist für einen neuen Wert
        while (!(UCSRA & (1 << UDRE)))
                ;

        // UDR Schreiben startet die Übertragung
        UDR = c;

        return 1;
}

uint8_t        uart_getc_wait()
{
        // Warten, bis etwas empfangen wird
        while (!(UCSRA & (1 << RXC)))
                ;

        // Das empfangene Zeichen zurückliefern
        return UDR;
}

int        uart_getc_nowait()
{
        // Liefer das empfangene Zeichen, falls etwas empfangen wurde; -1 sonst
        return (UCSRA & (1 << RXC)) ? (int) UDR : -1;
}

//----------------------------------------------------------------------
main ()                                                // Hauptprogramm, startet bei Power ON und Reset
{
        uart_init();

        //...                                        // hier Init-Code eintragen
        do
        {
                // Schleifenanfang Mainloop
                uart_putc('x');
                _delay_ms(50);
        }
        while (true);                        // Schleifenende Mainloop
}

---

Aber immernoch das selbe Ergebnis.

Hallo,

wie sind die Fusebits gesetzt?

Gruß

Jens

So sind sie gesetzt:

Hallo,

wenn ich das richtig sehe, hast du den internen Oszillator mit 4Mhz aktiviert. Aber rechne ruhig noch einmal mit deinen binären Werten nach.

FuseCalculator

Gruß

Jens

Hab schon andere ausprobiert
Bin jetzt auf "Ext Crsytal.... 1k ck+64ms.."

Die Ausgabe ist nun anders aber immernoch nicht korrekt...
Auf dem Board sitzt ein 3,6864MHz Oszilator.

Edit:
Wenn ich ne Null an den PC sende bekomm ich auch ne 00, wenn ich was anderes sende bekomm ich immer C0

Hallo,

es hilft nichts, andere Fusebits zu probieren. Es helfen nur die richtigen. O:)

Ändere

Code:

---

  uint16_t ubrr = (uint16_t) ((uint32_t) F_CPU/(16*9600) - 1);

---

in

Code:

---

  uint16_t ubrr = (uint16_t) ((uint32_t) F_CPU/(16*9600UL) - 1);

---

Gruß

Jens

Also das UL bringt nichts...

Nun geht es wenn ich die Zeile so umgeändert:

Code:

---

uint16_t ubrr = (uint16_t) ((uint32_t) F_CPU/(16*38400));

---

Also -1 weg, und auch nur mit der Baudrate 38400...
War ein Zufallstreffer, ist also noch lange nicht die Lösung des Problems. Aber wenigstens kann ich nun erstmal weiter testen

Hallo,

interessant, kannst du mal schreiben, mit welchem Compiler du arbeitest? Ich hatte mich schon bei deinem true im Beispielcode gewundert. Ich habe es mal im AVR-Studio simuliert. Ohne das UL wird der Ausdruck in der Klammer mit nur 16 Bit gerechnet.

Gruß

Jens

Also ich benutze das Workpad Plus mit WinAvr.
Ich werde mir die Tage aber mal das AVR-Studio installieren.

Das Workpad PLUS bietet einfach nichts was für richtiges Programmi9eren erforderlich ist, alleine schon ohne einen Debugger, bzw. eine ordentliche Simulation ist die Fehlersuche nicht leicht...

EDIT:
Mit dem http://www.gjlay.de/helferlein/avr-uart-rechner.html
Kann ich die passenden Werte für ubrr ausrechnen, so funktioniert es schonmal

Hallo,

Avr-Studio nutzt auch WinAvr. Was mich halt wundert ist, dass 'true' da eigentlich nicht definiert ist. Der Simulator von Avr-Studio ist schon nicht schlecht. Man darf ihn aber auch nicht überschätzen.
Wenn du Werte von der Seite einsetzt, dann geht es jetzt?
Die 23 bei 9600 Baud ist genau der Wert, der hierbei

Code:

---

  uint16_t ubrr = (uint16_t) ((uint32_t) F_CPU/(16*9600UL) - 1);

---

ausgerechnet wurde. Wenn ich 16*9600 in 16 Bit rechne bekomme ich ein ubrr von 162. Wenn du damit an den PC sendest ist dein Startbit schon so lang, dass Teile als Datenbits erkannt werden. Vielleicht möchtest du ja doch noch einmal in der Richtung ermitteln.

Gruß

Jens

Also ich nutze den internen 4MHz Oszilator.
das wäre dann bei 9600 Baud ein Wert von 25.

Bei 4MHz und mit "UL" funktioniert es auch!

Mit 4 MHz hatte ich es schon vorher probiert, nur leider ohne das "UL".
Also, es geht jetzt! Danke für eure Hilfe!!


Zu dem "true":
Das steht da nur drin weil ich den teil aus einem Beispiel habe, ich selber setz da immer ne 1 ein.
Warum es mit true geht, keine Ahnung, so tief hab ich mich mit dem WinAVR noch nicht beschäftigt, programmiere sonst mit anderen Compilern.

EDIT:
Vom PC zum Testboard funktioniert jetzt auch!
Nochmal Danke.