UART merkwürdiges Verhalten

[Marcel H. (repac3r)]

Guten tag liebe com,

ich versuche gerade via UART 3 Led's zu steuern, was aber nur teilweise 
klappt, denn diese verhalten sich der komisch.

Hier mein Code:

#define F_CPU 3686400UL
#define BAUD 9600UL

#define UBRR_VAL ((F_CPU+BAUD*8)/(BAUD*16)-1)
#define BAUD_REAL (F_CPU/(16*(UBRR_VAL+1)))
#define BAUD_ERROR ((BAUD_REAL*1000)/BAUD)

#if ((BAUD_ERROR<990) || (BAUD_ERROR>1010))
#error Baud_error
#endif

#include <avr/io.h>

void uart_init()
{
  UBRRH = UBRR_VAL >> 8;
  UBRRL = UBRR_VAL & 0xFF;
  
  UCSRB |= (1<<TXEN) | (1<<RXEN);
  UCSRC = (1<<URSEL) | (1<<UCSZ1) | (1<<UCSZ0);
}

int main(void)
{
  DDRB |= (1<<PB0) | (1<<PB1) | (1<<PB2);
  
  uart_init();
  uint8_t i;
  
    while(1)
    {
    i = uart_getdata();
    uart_data(i);
    if(i == '1')
    {
      PORTB |= (1<<PB0);
      PORTB &= ~(1<<PB1) | (1<<PB2);
      
    }
    else if(i == '2')
    {
      PORTB |= (1<<PB1);
      PORTB &= ~(1<<PB0) | (1<<PB2);
    }
    else if(i == '3')
    {
      PORTB |= (1<<PB2);
      PORTB &= ~(1<<PB1) | (1<<PB0);
    }
    else if(i == '0')
    {
      PORTB |= (1<<PB1) | (1<<PB0) | (1<<PB2);
    }
    else{
      PORTB &= ~(1<<PB1) | (1<<PB0) | (1<<PB2);
    }
  
    }
}

uint8_t uart_getdata()
{
  while(!(UCSRA &(1<<RXC)))
  {
    
  }
  return UDR;
}

Wenn ich über ein Terminal die Zahl 1 sende, leuchtet die erste Led.
Danach schreibe ich die Zahl 2, aber diese leuchtet nur sehr kurz auf.
Bei der Zahl 3 leuchtet wieder die dritte Led. Doch wenn ich dann wieder 
die Zahl 1 sende, leuchtet die erste und die dritte ? Normal sollte 
diese ja durch das löschen des Bits ausgehen.

Mache ich irgendwas Falsch ? Denn ich weiß absolut nicht wo der Fehler 
ist, bzw. bin mir sicher das dort kein Fehler ist. Denn wenn ich den 
empfangenen Wert zurückschicke kommt exakt die Zahl, die ich eingegeben 
habe.


lg, marcel

[g457 (Gast)]

Mir dünkt Dir gehen ein paar Klammern ab zu den bitweisen Nichten.

[Marcel H. (repac3r)]

g457 schrieb:
> Mir dünkt Dir gehen ein paar Klammern ab zu den bitweisen Nichten.

Bin ich nur zu doof zum lesen, oder
versteht man wirklich nicht was du mir damit sagen möchtest ?
Wäre nett wenn du das nochmal schreiben würdest ;)

Lg, marcel

[g457 (Gast)]

> PORTB &= ~(1<<PB1) | (1<<PB2);
            ^                  ^
            Gute Stellen für zusätzliche Klammern
           ^
           Bitweises Nicht

..und bei allenen anderen releventen Stellen ebenso.

[Martin (Gast)]

Probiermal mit zusätzlichen klammern :

PORTB &= ~((1<<PB1)|(1<<PB2));

für alles was hinter der Tilde ist....


Ich denke mit "uart_data(i);" sendet du das empfangene Zeichen wieder 
zurück oder? setze das mal hinter die if-Bedingung und probiere es 
nochmal!


Sonst würde ich auch nur auf die Baudrate tippen aber wie das klingt 
funktioniert bei dir die Kommunikation .... aber ich berechne die 
anders:

#define F_CPU 3686400UL
#define BAUD 9600UL

uint16_t baudvalue=(((F_CPU/16)/BAUD)-1);
UBRR0H=(baudvalue>>8);
UBRR0L=baudvalue;

[Mike (Gast)]

Marcel Hoffmann schrieb:
> Bin ich nur zu doof zum lesen, oder
> versteht man wirklich nicht was du mir damit sagen möchtest ?
Das blöde an Computern ist, dass sie machen, was man ihnen sagt und 
nicht unbedingt das, was man von ihnen möchte.

[Marcel H. (repac3r)]

g457 schrieb:
>> PORTB &= ~(1<<PB1) | (1<<PB2);
>             ^                  ^
>             Gute Stellen für zusätzliche Klammern
>            ^
>            Bitweises Nicht
>
> ..und bei allenen anderen releventen Stellen ebenso.

Nein dort fehlen eigentlich kleine Klammern, habe sie trotzdem 
hinzugefügt, aber leiger gleiches Resultat

[Marcel H. (repac3r)]

Okay ein Fehler beim Uart habe gerade ausgeschlossen, denn wenn ich das 
ganze so aufbaue:

1 senden: LED1 an
2 senden: LED1 aus

3 senden: LED2 an
usw...

Funktioniert alles genauso wie es sollte, bloß wenn ich jetzt z.B.: die 
Zahl 2 sende, um alle LED's auszumachen, geht wieder nur eine aus. Das 
heißt irgendwas klappt da mit dem löschen des Bits nicht, oder ?

[Karl H. (kbuchegg)]

Marcel Hoffmann schrieb:
> g457 schrieb:
>>> PORTB &= ~(1<<PB1) | (1<<PB2);
>>             ^                  ^
>>             Gute Stellen für zusätzliche Klammern
>>            ^
>>            Bitweises Nicht
>>
>> ..und bei allenen anderen releventen Stellen ebenso.
>
> Nein dort fehlen eigentlich kleine Klammern,

Doch, die fehlen dort. Darüber brauchen wir nicht diskutieren. Rechne es 
halt mal binär nach!
Und bei allen anderen Verundungen fehlen sie ebenso. Hast du sie ÜBERALL 
nachgetragen oder nur an der einen Stelle?

[Michael .. (bigneal)]

Marcel Hoffmann schrieb:
> g457 schrieb:
>>> PORTB &= ~(1<<PB1) | (1<<PB2);
>
> Nein dort fehlen eigentlich kleine Klammern

http://de.wikibooks.org/wiki/C-Programmierung:_Liste_der_Operatoren_nach_Priorit%C3%A4t