AT90CAN128 uart RX Interrupt

[Franz S. (franz_0815)]

Hallo
Beschäftige mich gerade mit dem Prozessor AT90CAN128

Beim Empfangen von Daten über RXD0 resetet mein System.

Das ende klappt.
Aber auch nur beim Empfangen von einen char tillt alles.

??
Gruss

//C Code Example (1)
#include "pins.h"
#include "uart.h"
#include <avr/interrupt.h>

unsigned int buffercounter = 0;
char usart_rx_buffer[BUFFER_SIZE],newdata;
char *rx_buffer_pointer_in  = &usart_rx_buffer[0];
char *rx_buffer_pointer_out  = &usart_rx_buffer[0];
unsigned char receive_char;  


void USART0_Init (unsigned long baudrate)
//void USART0_Init (unsigned long baudrate)
{

  unsigned long baud = (F_CPU/(16*baudrate))-1;
  SETBIT (DDRE,PE1);
  /* Set baud rate */
  UBRR0H = (unsigned char) (baud>>8);
  UBRR0L = (unsigned char) baud;
  /* Set frame format: 8data, no parity & 2 stop bits */
  UCSR0C = (0<<UMSEL0) | (0<<UPM0) | (1<<USBS0) | (3<<UCSZ0);
  /* Enable receiver and transmitter */
//  UCSR0B = (1<<RXEN0) | (1<<TXEN0);
    UCSR0A =0;
  UCSR0B =(1 << TXEN0 | 1 << RXEN0 | 1<< RXCIE0);
  //return baud;
}
//C Code Example (1)
void USART0_Transmit (unsigned char data)
{
  /* Wait for empty transmit buffer */
  while ( ! ( UCSR0A & (1<<UDRE0)));
  /* Put data into buffer, sends the data */
  UDR0 = data;
}

void SendRs232(char* string)//Nullterminiert
{
  unsigned char ende = 0;
  int i=0;
  while(!ende)
  {
    unsigned char data = string[i++];
    if(data == 0)
      ende = 1;
    else
      USART0_Transmit(data);
  }
}

ISR (USART_RX0)
{


      while (!(UCSR0A & (1<<RXC0)));
       receive_char=UDR0;
/*    receive_char=UDR0 &127;  // &127 damit auch 7 Datenbit funktioniert.
    usart_rx_buffer[buffercounter]=receive_char;
  buffercounter++;

    if((buffercounter > BUFFER_SIZE - 1)||(receive_char==13)) 
   {
     if((buffercounter < BUFFER_SIZE )&&(receive_char==13))
         usart_rx_buffer[buffercounter+1]=0; // f?r stringende !
     buffercounter=0;
   }
 */


}

[Tom (Gast)]

Im Anhang das ganze in lesbar. Jede IDE und jeder bessere Editor hat 
automatische Formatierung. Wie kann man in so einem Wust programmieren 
und hoffen, den Überblick zu behalten?

[Karl H. (kbuchegg)]

Franz Suahnein schrieb:

> ISR (USART_RX0)

Bist du sicher, dass der Name des Interrupt-Vektors, USART_RX0, korrekt 
ist?
Da zb das entsprechende Bit im UCSR0A den Namen RXC0 trägt, hätte ich 
als Name jetzt mal eher USART_RXC0 angenommen. Aber ich gebe zu, dass 
Atmel da mit den Bezeichnungen nicht ganz einheitlich ist. Allerdings 
teilt dir der Compiler ja als Warnung mit, dass er mit dem Namen des 
INterrupts nichts anfangen kann.
Allerdings ist es auch so, dass das Symptom "Reset beim Auftreten eines 
Interrupt Ereignisses" ein deutliches Indiz für einen Schreibfehler im 
Interrupt Namen ist

>       while (!(UCSR0A & (1<<RXC0)));

das ist ziemlich sinnlos.
Die Interrupt Funktion ist aufgerufen worden, WEIL ein Zeichen an der 
UART vorliegt, das abgeholt werden will. Das brauchst du hier nicht mehr 
testen. An dieser Stelle steht einwandfrei und zweifellos fest, dass es 
ein Zeichen in UDR0 gibt.

[Franz S. (franz_0815)]

Hallo
So mal geändert.
Aber der Prozessor resetet nicht sonder kommt garnicht mehr.

Bei dem AT90CAN128 ist das ISP System auch auf TX0 RX0,also 
zusammengeschaltet mit MISO und MOSI.

Auch die Routine ohne Interrupt bringt das programm zu stehen.
Mit dem Ozsi sieht alles gut auch.
Auch nach dem programmieren das den AVR-ISP entfernen bringt nichts.

?
Was kann es noch sein.

ISR(USART0_vec)
/
/ISR (USART0)
{

main_counter++;
//      while (!(UCSR0A & (1<<RXC0)));
     receive_char=UDR0;
/*    receive_char=UDR0 &127;  // &127 damit auch 7 Datenbit funktioniert.
    usart_rx_buffer[buffercounter]=receive_char;
  buffercounter++;

    if((buffercounter > BUFFER_SIZE - 1)||(receive_char==13)) 
   {
     if((buffercounter < BUFFER_SIZE )&&(receive_char==13))
         usart_rx_buffer[buffercounter+1]=0; // f?r stringende !
     buffercounter=0;
   }
 */


}

uint8_t USART0_Receive(void)  
{
  while ( !(UCSR0A & (1<<RXC0)) );
  main_counter++;
  return UDR0;
}

[Karl H. (kbuchegg)]

Franz Suahnein schrieb:

> Auch die Routine ohne Interrupt bringt das programm zu stehen.

Hast du auch die Freigabe des Interrupts hier

  UCSR0B =(1 << TXEN0 | 1 << RXEN0 | 1<< RXCIE0);

rausgenommen?

Man kann es nicht oft genug betonen.
Wenn es eine ISR nicht gibt, du den entsprechenden Interrupt aber 
freigibt, dann kommt es beim Auftreten des Interrupts zu einem Reset.
Ob es die ISR nicht gibt, weil du sie nicht geschrieben hast, oder weil 
sie nicht als solche erkannt wurde, weil du dich im Namen vertan hast, 
ist dabei unerheblich. Nicht benutzbar ist nicht benutzbar und der dann 
benutzte Default-Handler macht nun mal einen Reset.

[Franz S. (franz_0815)]

void USART0_Init (unsigned long baudrate)
//void USART0_Init (unsigned long baudrate)
{

  unsigned long baud = (F_CPU/(16*baudrate))-1;
  /* Set baud rate */
  UBRR0H = (unsigned char) (baud>>8);
  UBRR0L = (unsigned char) baud;
  /* Set frame format: 8data, no parity & 2 stop bits */
  UCSR0C = (0<<UMSEL0) | (0<<UPM0) | (1<<USBS0) | (3<<UCSZ0);
  /* Enable receiver and transmitter */
//  UCSR0B = (1<<RXEN0) | (1<<TXEN0);
  UCSR0B =(1 << TXEN0 | 1 << RXEN0 | 0<< RXCIE0);
}


Will hoffen das mit dem "0<<RXCIE0" richtig ist.

ISR(USART0_vec)

sollte doch auch richtig sein.

?

[Franz S. (franz_0815)]

hallo
Menschen der Elektronik, Gemeinschaft der Elektroniker !!

ISR(USART0_RX_vect)

http://www.atmel.com/webdoc/avrlibcreferencemanual/group__avr__interrupts.html

""Man kann es nicht oft genug betonen.""

Danke Karl Heinz.

[Peter D. (peda)]

Es sollte eigentlich ne Warnung geben:

main.c:8: warning: 'BLA_BLUB_Vect' appears to be a misspelled signal handler