UART-2Microkontroller-gleichzeitig senden und empfangen

[hans w. (elektropepi)]

Hallo,
folgende Aufgabe:ATMEGA328P
2uC: S1..TASTER1, S2..TASTER2,L1..Led1, L2..LED2
S1 an uC1 soll LED2 an uC2 schalten(Wenn Tasrer betätigt->LED EIN)
S2 an uC2 soll LED1 an uC1 schalten(Wenn Tasrer betätigt->LED EIN)
das ganze soll mit UART realisiert werden.



Mein Programm:

#include <avr/io.h>
#include <stdio.h>

void UART_init(); //Definition der Funktion UART_init
unsigned char UART_receive();//UART Empfangs-funktion
void UART_transmit(unsigned char data);//UART Sende-funktion

volatile unsigned char wert = 0;


int main ()
{
  UART_init();//UART initialisieren
  
  DDRC = 0x00;//Alles Eingänge

  PORTC = 0x00;

  DDRB |= (1<<PORTB5);//AUSGANG

  PORTB |= (1<<PORTB5);//PB5 ist Ausgang (LED low aktiv)



  while(1)
  {
    wert = UART_receive();//Wert empfangen

    switch(wert)
    {
      case 'E':  PORTB &= ~(1<<PORTB5);//LED leuchtet
      case 'A':  PORTB |= (1<<PORTB5);//LED leuchtet nicht
    }

    if ((PINC & (1<<PINC5)))
    {
      UART_transmit('E');//LED Ein-Befehl schicken
    }
    else 
    {
      UART_transmit('A');
    }

    
      
  }

  
}





void UART_init()//Funktion UART_init
{
      DDRD |= (1<<1);

   DDRD &= ~(1<<0);

   PORTD |= 0x02;

   UCSR0B = (1<<RXEN0) | (1<<TXEN0);//RX und TX (Empfangen und Senden) enabled

   UCSR0C = (1<<UCSZ00) | (1<<UCSZ01);//8-Bit Betrieb und 1 Stopbit

   UBRR0H = 0;
   UBRR0L = 103;//Laut Tabelle (Datenblatt) bei 16 MHz für Baud = 9600 --> 103;  UBRR0H nicht benötigt, da 2^8 = 256


}

unsigned char UART_receive()
{
  while ( ! (UCSR0A & (1<<RXC0)));//Warten auf Empfang

  return UDR0;//Empfangene Daten zurückgeben
}

void UART_transmit(unsigned char data)
{
  while( ! (UCSR0A & (1<<UDRE0)));//Warten auf leeres Datenregister

  UDR0 = data;//Daten senden
}

Funktioniert nicht;(
Vielen dank wenn mir evtl. jemand weiterhelfen kann

mfg

[Peter D. (peda)]

hans werner schrieb:
> while ( ! (UCSR0A & (1<<RXC0)));//Warten auf Empfang

Deadlock.

[Thomas K. (thomas_k39)]

Oder etwas ausführlicher: beide uC warten darauf, dass der andere als 
erster ein Zeichen sendet, aber keiner will den ersten Schritt machen.

[hans w. (elektropepi)]

vielen Dank erstmals:)

Wie könnte das gelöst werden?

mfg

[Markus O. (pipimaxi)]

Ein break vermisse ich in der switch-case ebenso

[Karl H. (kbuchegg)]

Indem man hier

 while(1)
  {
    wert = UART_receive();//Wert empfangen

bzw. in der Funktion selber

unsigned char UART_receive()
{
  while ( ! (UCSR0A & (1<<RXC0)));//Warten auf Empfang

  return UDR0;//Empfangene Daten zurückgeben
}

nicht so lange wartet, bis irgendwas eingetroffen ist, sondern ganz 
einfach den UART befragt: Hast du was für mich?
Und nur dann ein Zeichen holt, wenn eines da ist. Im anderen Falle wird 
dann eben nicht solange gewartet, bis ein Zeichen eingetroffen ist 
sondern mit dem Rest des programms weiter gemacht.

Oder bleibst du neben dem Postkasten stehen, solange bis der Briefträger 
einen Brief einwirft? Dann wirds aber nix mit einer Brieffreundschaft, 
wenn du immer nur neben dem Postkasten rumlungerst und deswegen nie zum 
selber schreiben kommst.

Das könnte dann zb so aussehen

...

  while(1)
  {
    if( UART_available() )    // gibts was an der UART?
    {
      wert = UART_receive();//Wert empfangen

      switch(wert)
      {
        case 'E':  PORTB &= ~(1<<PORTB5); break; //LED leuchtet
        case 'A':  PORTB |= (1<<PORTB5); break;  //LED leuchtet nicht
      }
    }

    if ((PINC & (1<<PINC5)))
    {
      ...
  }

bzw.

uint8_t UART_available()
{
  if( UCSR0A & (1<<RXC0) )    // gibts was an der UART zu holen?
    return TRUE;

  return FALSE;
}

[hans w. (elektropepi)]

#include <avr/io.h>
#include <stdio.h>

void UART_init(); //Definition der Funktion UART_init
unsigned char UART_receive();//UART Empfangs-funktion
void UART_transmit(unsigned char data);//UART Sende-funktion

volatile unsigned char wert = 0;


int main ()
{
  UART_init();//UART initialisieren
  
  DDRC = 0x00;//Alles Eingänge

  PORTC = 0x00;

  DDRB |= (1<<PORTB5);//AUSGANG

  PORTB |= (1<<PORTB5);//PB5 ist Ausgang (LED low aktiv)



  while(1)
  {
     if( UART_available() )    // gibts was an der UART?
    {
      wert = UART_receive();//Wert empfangen

      switch(wert)
      {
        case 'E':  PORTB &= ~(1<<PORTB5); break; //LED leuchtet
        case 'A':  PORTB |= (1<<PORTB5); break;  //LED leuchtet nicht
      }
    }

    if ((PINC & (1<<PINC5)))
    {
      UART_transmit('E');//LED Ein-Befehl schicken
    }
    else 
    {
      UART_transmit('A');
    }

    
      
  }

  
}





void UART_init()//Funktion UART_init
{
      DDRD |= (1<<1);

   DDRD &= ~(1<<0);

   PORTD |= 0x02;

   UCSR0B = (1<<RXEN0) | (1<<TXEN0);//RX und TX (Empfangen und Senden) enabled

   UCSR0C = (1<<UCSZ00) | (1<<UCSZ01);//8-Bit Betrieb und 1 Stopbit

   UBRR0H = 0;
   UBRR0L = 103;//Laut Tabelle (Datenblatt) bei 16 MHz für Baud = 9600 --> 103;  UBRR0H nicht benötigt, da 2^8 = 256


}

uint8_t UART_available()
{
  if( UCSR0A & (1<<RXC0) )    // gibts was an der UART zu holen?
    return TRUE;

  return FALSE;
}


unsigned char UART_receive()
{
  while ( ! (UCSR0A & (1<<RXC0)));//Warten auf Empfang

  return UDR0;//Empfangene Daten zurückgeben
}

void UART_transmit(unsigned char data)
{
  while( ! (UCSR0A & (1<<UDRE0)));//Warten auf leeres Datenregister

  UDR0 = data;//Daten senden
}

So?
Muss ich noch etwas inkludieren bzw. TRUE und FALSE deklarieren?

Vielen Dank nochmals und entschuldigt die dummen Fragen. Bin Noob im uC 
programmieren:)

[Karl H. (kbuchegg)]

hans werner schrieb:

> Vielen Dank nochmals und entschuldigt die dummen Fragen. Bin Noob im uC
> programmieren:)

Dann wirds Zeit für ein C Buch, wenn du noch nicht mal weißt, wie die 
Defintionen für die boolschen Werte TRUE bzw. FALSE lauten bzw. wie C 
das Thema Wahrheitswerte behandelt.

"Bin Noob" lass ich seit geraumer Zeit nicht mehr als Ausrede für 
fehlende Grundlagen gelten.
Du willst programmieren? Gut. Dann lern deine Sprache. Du musstest ja 
auch erst mal in der Fahrschule Autofahren lernen, ehe du auf den 
Strassenverkehr losgelassen wurdest.

[hans w. (elektropepi)]

Dann erkläre es mir bitte:)

Für euch Profis ist das sicherlich ein klacks;)