Attiny2313 UART Empfang klappt nicht

[Elias F. (strangelet)]

Hallo,

Ich habe das Problem das mein Attiny2313 mit dem Code unten einfach 
nicht per Uart empfangen will, der Interrupt wird nicht ausgelöst was 
ich an der LED erkennen kann.
Wenn ich den Code zum Led blinken im normalen  Code verwende blinkt die 
Led wie sie soll.
Ein externer Quarzoszillator(Ja der mit 4 Beinchen) mit 16Mhz ist dran, 
nachdem's mit dem Internen auch nicht funktioniert hat.
Die Fusebits sind richtig auf Externen Takt und ohne Clockdiv gesetzt.
Am PC Empfang ich die gesendeten Daten 1A und wenn ich mit dem PC sende 
kommen auch die richtigen Daten an, das hab ich per LA am Pin geprüft.

Also sieht irgendjemand einen Fehler in dem Code? Mir fällt einfach 
nichts mehr ein was ich noch debuggen könnte...

/*************************************************************************
Title:    Servo Cam
Author:   Strangelet  http://lloyd3000.wordpress.com/
Hardware: Attiny2313
      
**************************************************************************/

#include <stdlib.h>
#include <avr/io.h>
#include <avr/pgmspace.h>
#include <avr/interrupt.h>
#include <util/delay.h>


#define UART_BAUD  9600

#define UART_BAUD_CALC(UART_BAUD,F_OSC) ((F_CPU)/((UART_BAUD)*16L)-1)

#define F_CPU 16000000UL 

volatile char data;  


void USART_Transmit(void);


int main(void)
{

  DDRB = 0b01111110;

  UBRRH=(unsigned char)(UART_BAUD_CALC(UART_BAUD,F_CPU)>>8);
  UBRRL=(unsigned char)UART_BAUD_CALC(UART_BAUD,F_CPU);
   

  // UART Receiver und Transmitter anschalten
  UCSRB = (1 << RXEN) | (1 << TXEN) | (1<<RXCIE);
  // Data mode 8N1, asynchron
  UCSRC =  (1 << UCSZ1) | (1 << UCSZ0);
  
  sei();  
  
  data = 'x';

 
while(1){

   USART_Transmit();
   _delay_ms(200);
  
}
  
}


ISR(USART_RX_vect){

    PORTB |= (1<<PB2);          //Turn LEDs on
    _delay_ms(200);
    PORTB &= ~(1<<PB2);          //Turn LEDs on


    /* Wait for data to be received */
    while ( !(UCSRA & (1<<RXC)) );

    /* Get and return received data from buffer */
    data = UDR;

}  
  
  
void USART_Transmit(void)
{
    /* warten bis Senden moeglich                   */
    while (!(UCSRA & (1<<UDRE)))  
    {
    }
    /* schreibt das Zeichen x auf die Schnittstelle */
    UDR = data;                    
}

[Thomas E. (thomase)]

Elias F. schrieb:
> ISR(USART_RX_vect){

Der Vektor heisst USART0_RX_vect
                       ^
                       ^
                       ^

Also 1. meckert der Compiler, weil er deinen Vektor nicht kennt und 2. 
stürzt dein Programm ab, weil es für RX keine ISR gibt.

mfg.

[Elias F. (Gast)]

Thomas Eckmann schrieb:
> Also 1. meckert der Compiler, weil er deinen Vektor nicht kennt und 2.
> stürzt dein Programm ab, weil es für RX keine ISR gibt.

Also 
http://www.nongnu.org/avr-libc/user-manual/group__avr__interrupts.html 
meint auch "USART_RX_vect" Und mein Compilerer meckert auch nich, noch 
stürzt das Programm ab...

Testen kann ichs allerdings gerade nich...(hab grad mein Windows platt 
gemacht)

[Karl H. (kbuchegg)]

Elias F. schrieb:

> Also
> http://www.nongnu.org/avr-libc/user-manual/group__avr__interrupts.html
> meint auch "USART_RX_vect" Und mein Compilerer meckert auch nich, noch
> stürzt das Programm ab...

Stimmt.
Das AVR-Include File für den Tiny2313 sagt auch, dass
USART_RX_vect
korrekt ist.

Ich seh allerdings auch keinen Fehler (das delay ordne ich jetzt mal dem 
Debug-Versuch zu). Lediglich einen Schönheitsfehler.

    /* Wait for data to be received */
    while ( !(UCSRA & (1<<RXC)) );

ist unsinnig.
Die Interrupt Funktion wurde aufgerufen, WEIL ein Zeichen vorhanden ist! 
Wenn kein Zeichen da wäre, dann wäre die Funktion nicht aufgerufen 
worden. D.h. das brauchst du hier nicht testen. An dieser Stelle weißt 
du mit 100% Sicherheit, dass ein Zeichen vorhanden ist.

[Thomas E. (thomase)]

Elias F. schrieb:
> http://www.nongnu.org/avr-libc/user-manual/group__avr__interrupts.html
> meint auch "USART_RX_vect" Und mein Compilerer meckert auch nich, noch
> stürzt das Programm ab...
Interessanterweise ist das wohl so. Obwohl der Compiler laut Datenblatt 
den Vektor verweigern und demzufolge das Programm abstürzen müsste.
Somit müsste das Programm aber auch laufen.

Hardware? Lötstelle?

mfg.

[CamToCam (Gast)]

Hallo,
muß F_CPU denn nicht vor der Delay.h definiert werden ?
Die Idee dahinter ist das du evtl. gar nicht sehen kannst das die Led 
leuchtet.

Die utils\setbaud.h ist übrigens sehr viel komfortabler:
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Der_UART

Ansonsten noch viel Erfolg bei deinem Projekt, habe sowas auch mal 
aufgebaut damit kann man nette Spielereien anstellen.
http://www.codeproject.com/Articles/10248/Motion-Detection-Algorithms

[Karl H. (kbuchegg)]

CamToCam schrieb:
> Hallo,
> muß F_CPU denn nicht vor der Delay.h definiert werden ?
> Die Idee dahinter ist das du evtl. gar nicht sehen kannst das die Led
> leuchtet.

Gutes Argument.

Wobei man die ganze Delay Sache auch komplett über Bord wecrfen kann

ISR(USART_RX_vect)
{
    PORTB ^= (1 << PB2);

    data = UDR;
}

jetzt flackert die LED. Wenn alles klappt, dann ist das kaum zu 
übersehen.

[Elias F. (Gast)]

Thomas Eckmann schrieb:
> Hardware? Lötstelle?

OMG, ich hab den fehler gefunden dank dir :D

Ich hab das Ganze aufm Breadboard aufgebaut, allerdings hing genau die 
klammer am RX pin ein Stückchen nach unten raus, so das Zwar LA Kontakt 
zum Ft232 hatte, nur der eben genau der UC nicht :(
Ich hatte deshalb bisher alle Hardware Fehler ausgeschlossen...


CamToCam schrieb:
> Ansonsten noch viel Erfolg bei deinem Projekt, habe sowas auch mal
> aufgebaut damit kann man nette Spielereien anstellen.
> http://www.codeproject.com/Articles/10248/Motion-Detection-Algorithms

Danke Interessant werd ich mir mal anschauen ;)

Und auch nochmal Danke an alle anderen die Versucht haben mir zu helfen! 
:)