UART mit Atmega8535 Problem

[izaseba]

Hallo Kater,
Da ich eher ein praktischer Mensch bin, habe ich das eben auf meinem Dragon ausprobiert, die Wette hättest Du gewonnen, egal welche Werte man in die Register reinschreibt, nach einem Watchdog Reset sind sie wieder auf 0

Gruß Sebastian

[kater]

Dankeschön .

[booty]

Hallo!
Ich brauche mal nochmal Hilfe. Ich habe versucht über RS232 vom PC auf den Controller eine 1 zu senden und dann soll eine LED ausgehen. Das habe ich per Interrupt versucht, doch es funbktioniert leider nicht.
Kann einer mal den Quelltext anschauen und sagen was ich falsch gemacht habe?
Danke!

#include <avr/io.h>
#include <avr/interrupt.h>
#define USART_RXC_vect _VECTOR(14)


#define F_CPU 1000000UL
#define BAUD 9600UL
#define UBRR_BAUD ((F_CPU/(16L*BAUD))-1)

uint8_t buffer;

void uart_init(void)
{

UBRRH = (uint8_t) (UBRR_BAUD>>;
UBRRL = (uint8_t) (UBRR_BAUD & 0xFF);
UCSRB = (1<<TXEN) | (1<<RXEN);
UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0);
}


ISR(USART_RXC_vect)
{
UCSRB = (1<<RXEN)|(1<<TXEN)|(1<<RXCIE);
while (!(UCSRA & (1<<RXC)))
buffer = UDR;
}


void led_init(void)
{
if(buffer==0x31)
PORTA=0x01;

else
PORTA=0x00;
}


int main(void)
{
sei();
DDRA = 0xff;
led_init();
uart_init();

}

[izaseba]

Hallo booty,
eigentlich steht alles in dem Link, den ich Dir gepostet habe drin, aber egal, ich erkläre es nochmal

1. Problem:

#define USART_RXC_vect _VECTOR(14)

Das ist unnötig, kann eventuell schaden, lass das weg, das ist Aufgabe von Compiler (Präprozessor) den Vektor richtig zuzuordnen.
2. Problem:

uint8_t buffer;

buffer muß als volatile deklariert werden, warum, sehe link oben

volatile uint8_t buffer;

3. Problem

UCSRB = (1<<TXEN) | (1<<RXEN);

Ja, hier mußt Du noch den Interrupt einschalten, sonst wird er nie ausgeführt, also

UCSRB = (1<<TXEN) | (1<<RXEN)|(1<<RXCIE);

4. Problem, der Interrupt sollte etwa so aussehen:

ISR(USART_RXC_vect) 
{ 
buffer = UDR; 
}

Du brauchst hier keine flags mehr setzen, oder auf irgendwas warten.
5. Problem, es fehlt wieder die Endlosschleife, mach das etwa so:

int main(void) 
{ 
DDRA = 0xff; 
uart_init(); 
sei(); 
for(;;){
led_init(); 
}
return 0;
}

So dann schaumal...
Anmerkung noch Du kannst auch anstatt von 0x31 '1' schreiben, ist vielleicht einfacher zu lesen(wenn man die ASCII Werte nicht im Kopf hat.

Viel Erfolg

@kater,
Kein Problem, der Dragon hat sich auch gefreut mal ein paar Bytes schaufeln zu dürfen, außerdem wollte ich es auch wissen...
Gruß Sebastian

[booty]

Danke für deine schnelle Hilfe!!!!!!!!

[booty]

Hallo!

Ich habe alles geändert, so wie es beschrieben ist, doch es funktioniert immer noch nicht. Hat einer noch ne Idee?
Danke!

Hier noch mal der geänderte Quellcode:

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


#define F_CPU 1000000UL
#define BAUD 9600UL
#define UBRR_BAUD ((F_CPU/(16L*BAUD))-1)

volatile uint8_t buffer;

void uart_init(void)
{

UBRRH = (uint8_t) (UBRR_BAUD>>;
UBRRL = (uint8_t) (UBRR_BAUD & 0xFF);
UCSRB = (1<<TXEN) | (1<<RXEN)|(1<<RXCIE);
UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0);
}


ISR(USART_RXC_vect)
{
buffer = UDR;
}


void led_init(void)
{
if(buffer==0x31)
PORTA=0x01;

else
PORTA=0x00;
}


int main(void)
{
DDRA = 0xff;
uart_init();
sei();
while(1){
led_init();
}
return 0;
}

[mogel]

Moin,

Ich habe alles geändert, so wie es beschrieben ist, doch es funktioniert immer noch nicht. Hat einer noch ne Idee?

überprüf mal ob Du auch wirklich 0x31 sendest und nicht 0x01

ISR(USART_RXC_vect) 
{  
   buffer = UDR; 
}


void led_init(void)
{
if(buffer==0x31) 								
PORTA=0x01;
												
else 											
PORTA=0x00; 
}

bevor in led_init() buffer ausgelesen werden kann, kann ein andere Interrupt buffer wieder geändert haben (ja ich weis - wird bei diesem kleinen Programm nicht passieren)

also - entweder

ISR(USART_RXC_vect) 
{  
    led_init(UDR)
}
void led_init(char value)
{
    PORTA = value
}

oder die kurze Variante ... da es nur ein Testprogramm ist

ISR(USART_RXC_vect) 
{  
    PORTA = (UDR == 0x31) ? 0x01 : 0x00;
    // PORTA = UDR;
}

um als Alternative zu sehen ob überhaupt was am UART/Interrupt ankommt

ISR(USART_RXC_vect) 
{  
    static char dummy = 0;
    PORTA = dummy++;
    if (dummy == 255) dummy = 0;
}

Detailänderung

int main(void) 
{ 
    DDRA = 0xff; 
    uart_init(); 
    sei(); 
    while(1) { 
        // CPU schlafen legen ... spart Akkus
    } 
    return 0; 
}

hand, mogel