USART Problem Atmega32

[neo98]

Hallo zusammen,

ich versuche schon seit einer Weile die Usart-Verbindung von meinem Atmega32 über das MySmartUsb MK2 auf die Reihe zu bekommen. Zum empfangen/senden benutze ich das HyperTerminal. Das was der Atmega sendet empfange ich auch, allerdings kann ich nichts an den Atmega senden da er offenbar aus der Lesefunktion rausspringt. Hier mal meine Einstellungen zwecks Baudrate und Lesefunktion(diese hab ich zugegeben aus der Asuro lib "geklaut").

#include <avr/io.h>

#ifndef MCU
#define MCU atmega32
#endif

#ifndef F_CPU
#define F_CPU 16000000
#endif

#include <util/delay.h>

#define USART_BAUD_RATE 9600
#define USART_BAUD_SELECT (F_CPU/(USART_BAUD_RATE*16L)-1) 

void SerRead (
  unsigned char *data,
  unsigned char length,
  unsigned int timeout)
{
  unsigned char i = 0;
  unsigned int  time = 0;
  UCSRB = (1<<RXEN);                         // Empfaenger einschalten
  if (timeout != 0)
  {
    /*
      Die Funktion wird auf alle Faelle, auch ohne empfangene Daten, wieder
      verlassen. --> nonblocking mode
    */
    while (i < length && time++ < timeout)
    {
      if ((UCSRA & (1<<RXC)))
      {
        data [i++] = UDR;
        time = 0;
      }
    }
    if (time > timeout)
      data [0] = 'T';
  }
  else
  {
    /*
      Die Funktion wird auf alle Faelle erst nach dem Empfang von der
      vorgegebenen Anzahl Zeichen verlassen. blocking mode
    */
    while (i < length)
    {
      while(!(UCSRA & (1<<RXC)));
        data [i++] = UDR;
    }
  }
}

int main()
{
 unsigned char data[4];

UBRRH = (unsigned char) (USART_BAUD_SELECT>>8);
UBRRL = (unsigned char) USART_BAUD_SELECT; 
UCSRB = (1 << RXEN) | (1 << TXEN); 
UCSRC = (1 << URSEL) | (1 << UCSZ1) | (1 << UCSZ0); 

SerRead(data, 2, 0);
//...weiterer Code
}

Schreibe ich in die letzte while-Schleife der SerRead-Funktion rein, dass bei jedem Durchlauf/bei jeder Abfrage eine "hallo" gesendet wird, werd ich "zugespammt". Wenn ich das nicht tue, wird einfach der weitere Code ausgeführt und nichts gelesen.

[markusj]

[...]

  {
    /*
      Die Funktion wird auf alle Faelle erst nach dem Empfang von der
      vorgegebenen Anzahl Zeichen verlassen. blocking mode
    */
    while (i < length)
    {
      while(!(UCSRA & (1<<RXC)));
        data [i++] = UDR;
    }

Da ist der Fehler: In der While-Schleife prüfst du, ob das RXC-Bit NICHT (!!!) gesetzt ist, du möchtest eigentlich das Gegenteil machen (Abbruch wenn das Bit gelöscht ist, sonst weitermachen).

mfG
Markus

[neo98]

Ich würde sagen dass das so richtig ist. Hast du das ';' hinter der Schleife gesehen? (Der Code ist falsch eingerückt) Die Schleife wird solang ausgeführt bis RXC gesetzt ist. Steht auch so in der Doku...

[markusj]

Mea culpa, du hast recht. Das kleine Semikolon habe ich doch glatt übersehen. Später gehe ich den Code nochmal durch, jetzt ist erstmal Klausur schreiben angesagt.

mfG
Markus

[markusj]

Ich kann in dem Code keinen Fehler erkennen, ABER: SerRead schreibt den Wert von UCSRB neu (und macht dabei den Transmitter aus). Evtl. gibt es in anderen Codeteilen ähnliche Seiteneffekte.

mfG
Markus

[seite5]

Hallo,
in main() ist beim serread-aufruf das timeout auf 0 gesetzt ?!
mein Compiler würde auch in dem Ausdruck while (i < length && time++ < timeout) ein paar Klammern anfordern,
aber das ist wohl von Compiler zu Compiler unterschiedlich.
mfg
Achim

[neo98]

@markusj: Ich habs mal rausgenommen, und siehe da, mein Atmega kann auf einmal warten. Aber warum wird dabei der Transmitter(du meinst Empfänger oder?) ausgeschaltet? RXEN wird doch nur nochmal auf 1 gesetzt.
@seite5: Ich hab jetzt mal die Klammern gesetzt, aber auch ohne bekomm ich keine Warnung/Fehler.

Immerhin wartet der Atmega jetzt, allerdings liest er nicht, sondern bleibt immer an der stelle stehn egal was ich ins HyperTerminal eintippe. Mein mySmartUsb zeigt an das Daten gesendet werden, die ignoriert mein Atmega aber anscheinend. Beim anschließen hab ich am Anfang nicht gewusst dass man RxD und Txd kreuzen muss, kann es sein das es mir dabei einen Pinn verheizt hat?

[markusj]

Nein, RXEN wird nicht einfach gesetzt, das Register wird neu beschrieben und NUR RXEN dabei gesetzt. (da steht = und nicht |= ...)
Die ASURO-Routinen sind nicht auf Duplex-Betrieb ausgelegt, entweder sie senden oder sie Empfangen und nehmen daher keine Rücksicht aufeinander. gleichzeitiges Senden und Empfangen ist nicht vorgesehen!

Wenn das Empfangen nicht klappt, stimmt entweder die Verdrahtung nicht, die eingestellte Baudrate, der Takt des AVRs ist zu ungenau oder du hast tatsächlich den Pin gegrillt.
Äh moment, WAIT: Du setzt RXEN jetzt gar nicht? Dann funktioniert das natürlich auch nicht ... du musst UCSRB |= (1 << RXEN); machen!

mfG
Markus

[neo98]

1. Heißt das jetzt dass TXEN oder RXEN nicht gleichzeitig gesetzt sein dürfen? Komm grad nicht ganz mit was du meinst... (Entweder sende ich oder ich empfange)
2. Verdrahtung: Ich hab mal meinen 2. Atmega rausgekramt und den angeschlossen, genau das selbe. Bin mir ziemilch sicher das ich alles richtig angeschlossen habe, da ich schließlich am PC empfangen kann. Und empfange tu ich genau das was der Atmega sendet, also müsste die Baudrate ja auch übereinstimmten (sonst Kauderwelsch oder nicht) und dann denke ich müsste der Takt auch ungefähr stimmten. Pin kann ja jetzt auch fast nicht mehr hinüber sein da ja neuer Atmega.

Vielleicht sollte ich mal den ganzen Code posten??

Ich bekomm hier gleich die Krise

[markusj]

Es können TXEN und RXEN sehr wohl gleichzeitig gesetzt sein, (das ging nur beim ASURO nicht sinnvoll), das Problem besteht darin, dass du eben NUR das Bit RXEN setzen (bzw. evtl auch wieder löschen) möchtest, und nicht alle anderen Bits im Register gleich mit.

Gleichzeitiges Senden/Empfangen funktioniert aber nur, wenn eine der beiden Aufgaben in einer Interruptroutine behandelt wird, oder du deinen Code etwas komplexer gestaltest ...

mfG
Markus