Über UART Buchstaben einlesen und PORT setzen

[Michael W. (extremrider)]

Hallo Leute,
Ich habe eine Atmega 32 und dort die RS232 Schnittstelle programmiert, 
hardware-mäßig aufgebaut und über ein kleines Testprogramm Daten 
eingelesen und wieder ausgegeben.

Jetz hab ich allerdings ein Problem und zwar möchte ich untersuchen ob 
sich ein a im UDR befindet und dann einen Pin am PORTB setzen.

Überprüfe ob ein a über Uart eingelesen

ja--> LED einschalten

Hier mein Code:

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


// ---------- Konstantendefinitionen ----------------------------------

const char startText[] = "UART Test\r";


// ---------- Globale Variablen ---------------------------------------


// ---------- Funktionsprototypen -------------------------------------

int receiveByte();
int transmitByte(char data);


// ---------- Hauptprogramm -------------------------------------------

int main(void)
{
  uint8_t i;
  char cIn;

  //---------- Initialisierung Port IO ------------------------------

  PORTB = 0x00;                     // LEDs inaktiv setzen
  DDRB = 0xFF;                      // Port B auf Ausgang schalten


  //---------- Initialisierung USART --------------------------------

  //  UCSRA = (0<<U2X);            // disable 2x speed change

  UCSRB = (1<<RXEN)|(1<<TXEN); // enable receiver and transmitter

  UCSRC = (1<<URSEL)|(1<<UCSZ0)|(1<<UCSZ1); // USART Mode Select

  UBRRH = 0;                   //
  UBRRL = 23;                   // set the baudrate

  
  for (i = 0; startText[i] != '\0'; i++) // Initialisierungstext senden
  {
    transmitByte(startText[i]);
  }


  //---------- Hauptschleife ----------------------------------------

  while(1)
  {
    //cIn = receiveByte();
      //      cIn = toupper(cIn);
    //cIn++;
    //transmitByte(cIn);
  
    while (receiveByte==97)
    {
      PORTB=0x01;
    }
    
  
    
  }

  return 0;
}



// ---------- Unterprogramme ------------------------------------------

int receiveByte()
{
  while( !(UCSRA & (1<<RXC)) );     // Wait for incomming data
  return UDR;
}


int transmitByte(char data)
{
  while ( !(UCSRA & (1<<UDRE)) );   // Wait for empty transmit buffer
  UDR = data;
  //    _delay_ms(50);                    // start transmission
  return 0;
}

[Michael W. (extremrider)]

Mein Problem ist, dass es nicht funktioniert...
Kann mir bitte jemand helfen?

[Peter II (Gast)]

wo ist jetzt die Frage?

dein code düfte überhaupt nicht durch den Compiler gehen:

while (receiveByte==97)

ist für eine Funktion kein gültiger Aufruf.

[Michael W. (extremrider)]

Kannst du mir sagen bitte, wie es denn lauten müsste?

Komm irgendwie grad nicht weiter, hatte sich compilieren lassen

[Wolfgang (Gast)]

Michael W. schrieb:
> while (receiveByte==97)
>     {
>       PORTB=0x01;
>     }

Was soll diese while-Schleife? Der Prozessor kreist doch sowieso in 
seiner Hauptschleife.

[Michael W. (extremrider)]

Na ja, ich dachte er soll ständig die Funktion abfragen und dadurch 
beobachten ob sich ein a im UDR befindet und je nachdem darauf reagieren

[Karl H. (kbuchegg)]

Michael W. schrieb:
> Kannst du mir sagen bitte, wie es denn lauten müsste?

Ähm.
Wenn dir nicht geläufig ist, dass ein Funktionsaufruf immer eine 
Argumentliste besitzt, die auch leer sein kann, dann bedeutet das 
hauptsächlich eines: Du brauchst mehr als dringend ein C-Buch

    if( receiveByte() == 97 )

und warum schreibst du eigentlich 97? Soll das möglichst kryptisch sein? 
Nach dem Muster: Wenn es schwer zu schreiben war, soll es auch schwer zu 
lesen sein.

    if( receiveByte() == 'a' )

So braucht niemand erst mal eine ASCII Tabelle konsultieren, um zu 
wissen auf welches Zeichen du deinen Code reagieren lassen willst.


Hast du deine UART schon getestet? Funktioniert die Übertragung vom µC 
zum Terminal auf dem PC? Wenn nicht, dann mach das zuerst. Denn dann 
siehst du erst mal im Terminal ob die UART grundsätzlich funktioniert. 
WEnn du in der Schleife einfach immer nur ein 'x' sendest, dann müssen 
im Terminal auch lauter x auftauchen. Wenn nicht, dann stimmt etwas an 
der UART grundsätzlich nicht. In dem Fall hat es dann nämlich auch 
überhaupt keinen Sinn, vom PC an den AVR zu übertragen. Das ist dann nur 
Stochern im Nebel.


> Komm irgendwie grad nicht weiter, hatte sich compilieren lassen

Du solltest dir angewöhnen, Warnungen vom Compiler ernst zu nehmen! Denn 
in den meisten Fällen hat er recht, wenn er dubiose Konstrukte anwarnt.

[Michael W. (extremrider)]

Okay, danke erst mal.

Ist es nicht so, dass wenn ich die Taste a (Kleinbuchstabe) auf der 
Tastatur drücke der ASCII CODE im UDR steht?

Diesen wollte ich auslesen und hab auf 97 abgefragt.

Kommunikation ist möglich (nur ein Problem was im Bild erkennen kann), 
eigentlich sollte der Controller dies wieder ausgeben, was er empfangen 
hat doch leider bekommen ich andere Zeichen zurück.

Gibts ne Erklärung dafür?

[Wolfgang (Gast)]

Michael W. schrieb:
> Gibts ne Erklärung dafür?

Der Standardtip wäre: Übertragungsgeschwindigkeit von Sender und 
Empfänger stimmen nicht überein. Guck dir auf dem Oszi mal ein vom PC 
gesendetes und ein vom µC gesendetes Zeichen an und vergleiche die 
Symboldauer.