Frage zur UART Mega328

[Daniel B. (yzdani)]

Servus,

ich übertrage ein Char von einem Mega zum anderen
Wenn das UDR0 Register ein A enthält soll ein Pin auf HIGH geschalten 
werden.
Einschalten geht aber abschalten nicht

Hier der CODE:

Sender

void uasrt_init(unsigned int ubbr){
  UBRR0H=(unsigned char)(ubbr>>8);
  UBRR0L=(unsigned char)ubbr;
  UCSR0B=(1<<RXEN0)|(1<<TXEN0);
  UCSR0C=(0<<USBS0)|(3<<UCSZ00);
}

int main(void)
{
   portkonfig();
   uasrt_init(MYUBBR);
   
    while (1) 
    {
    if (PINC&(1<<DDC0))
    {
      
    }else{
      UDR0='A';
    }
    }
}

Empfänger

void uasrt_init(unsigned int ubbr){
  UBRR0H=(unsigned char)(ubbr>>8);
  UBRR0L=(unsigned char)ubbr;
  UCSR0B=(1<<RXEN0)|(1<<TXEN0);
  UCSR0C=(0<<USBS0)|(3<<UCSZ00);
}


char chars;
int main(void)
{
    uasrt_init(MYUBBR);
  portkonfig();
    while (1) 
    {
    
    //while(!(UCSR0A&(1<<RXC0)));
    chars=UDR0;
    
    if (chars=='A')
    {
      PORTC=(1<<DDC1);
  
    }
    
    PORTC=0x00;
  
    }
}

FOSC usw. ist bei beiden gleich eingestellt auf das A reagiert der 
Empfänger auch aber Schaltet wenn kein A mehr kommt nicht mehr ab.


Warum?

Es wird auch kein A mehr gesendet wenn ich den Taster los lasse.

Danke im Voraus!

Gruß
Danie

[Georg G. (df2au)]

Nimm dir bitte eine funktionierende Lib für den UART, zB von Fleury und 
arbeite damit.

Nur ein paar Stichpunkte:
Du solltest nur senden, wenn das Datenregister frei ist
Du solltest das Datenregister nur lesen, wenn ein Zeichen richtig 
empfangen wurde.
Wenn du ein Portbit einschaltest, ist es wenig zielführend, es im 
nächsten Befehl gleich wieder aus zu schalten. Das sieht man nur mit 
Anstrengung.

[Wolfgang (Gast)]

Georg G. schrieb:
> Das sieht man nur mit Anstrengung.
... oder mit vernünftigem Werkzeug. Da reich schon ein kleiner LA für 7€

[Daniel B. (yzdani)]

Danke Georg für die Stichpunkte.

Lese jetzt in einer ISR und es funzt.

Ich denke es macht auch wenig Sinn in der Lernphase eine Libary zu 
verwenden.
Trotzdem danke für den Tipp!

Gruß
Dani

[Sascha (Gast)]

"
    if (chars=='A')
    {
      PORTC=(1<<DDC1);

    }

    PORTC=0x00;

    }
"

Das wird in einer sehr schnellen Endlosschleife wiederholt ausgeführt.

Wenn du ein A schickst, ist PC1 für ein bis zwei CPU Takte (paar 
Nanosekunden oder so?) auf High Pegel, direkt danach wieder low. Sieht 
man nicht.
Selbst mit PORTC=0; im else-Teil wärs nur bis zum nächsten 
Schleifendurchlauf, das sind dann vielleicht 8 CPU Takte.

Weiterhin ist (1<<DDC1) nur möglich, weil das vom Präprozessor auf 
(1<<1) umgesetzt wird.
Richtig wäre PORTC=(1<<PC1). Noch richtiger wäre

PORTC |=(1<<PC1); zum Einschalten und

PORTC &= ~(1<<PC1); zum Ausschalten.

Unterschied: Mit diesem beiden Befehlen änderst du nur das eine Bit, was 
du ändern willst, nicht alle 8.

Siehe das AVR-GCC Tutorial oben links.

[Georg G. (df2au)]

Wolfgang schrieb:
> mit vernünftigem Werkzeug

Was vermutest du an Ausstattung und Erfahrung beim TO, wenn du das 
eröffnende Post liest?