USART Interrupt feuert nicht

[Judge (Gast)]

Hi ich habe eine Verbindung mit dem AVR und einem Terminal Programm auf 
meinem PC. Bis eben lief es noch wunderbar jetzt feuer das Interrupt 
nicht mehr. Den Code hab ich wieder hergestellt und er funktioniert noch 
immer nicht.

Verbindungen auf dem Steckbrett sind da und Empfangen kann ich noch mit 
dem Terminalprogramm daher ist die Verkabelung auch richtig.

Kerndaten zum Project:
Atmega16a interner RC Oszi

Hier ist der Code:

//definitions
//define Taster 
#define T_DEBOUNCE  100
#define TASTER_OK  (PINA & (1 << PINA0))
#define TASTER_UP  (PINA & (1 << PINA1))
#define TASTER_DOWN  (PINA & (1 << PINA2))

//led
#define LED_STATUS PD3

//define IIC Connection
#define SCK      PC7
#define SCA      PC7

//serial connection
#define BAUD 2400
#define MYUBRR F_CPU/16/BAUD-1
#define RX_PIN PD0
#define TX_PIN PD1

//define general
#define F_CPU    1000000

//libarys
#include <avr/io.h>
#include <util/delay.h>
#include <stdbool.h>
#include <avr/interrupt.h>

//function prototypes
//adc
void adc_init(void);
int adc_read(unsigned char);
void adc_change_channal(unsigned char);
//uart
void USART_Init(unsigned int);
void USART_Transmit(unsigned char);
void USART_String(char* str);
//led
void init_io(void);
void led_flash(unsigned char);

int main(void)
{  
  //init io
  init_io();
  
  sei();
  
  USART_Init(MYUBRR);
  
  while(1)
  {  
    if(TASTER_OK)
    {
      _delay_ms(T_DEBOUNCE);
      if(!TASTER_OK)
      {
        USART_String("OK");
      }
    }
    led_flash(1);
  }
    return 0;
}

//functions
//adc
void adc_init()
{
  int result;
  ADMUX = (1 << REFS1) | (1 << REFS0);
  //einstellen des registers
  ADCSRA  = (1 << ADSC) | (1 << ADPS2);
  //eine wandlung starten
  result = adc_read(0);
}

void adc_change_channal(unsigned char channal)
{
  switch(channal)
  {
    case(1):
      ADMUX &= ~(1 << MUX0);
      break;
    case(2):
      ADMUX |= (1 << MUX0);
      break;
    default:
      ADMUX &= ~(1 << MUX0);
      break;
  }
}

int adc_read(unsigned char channal)
{
  int result;
  
  adc_change_channal(channal);
  
  //start measuring
  ADCSRA |= (1 << ADEN);
  while(ADCSRA & (1 << ADSC));
  
  result = ADCL;
  result += (ADCL<<8);

  return result;
}

//uart
void USART_Transmit( unsigned char data )
{
  /* Wait for empty transmit buffer */
  while ( !( UCSRA & (1<<UDRE)) )
  ;
  /* Put data into buffer, sends the data */
  UDR = data;
}

void USART_String(char* str)
{
  while(*str)
  {
    USART_Transmit(*str);
    str++;
  }
}

void USART_Init( unsigned int ubrr)
{
  //unsigned char flush;
  
  /* Set baud rate */
  UBRRH = (unsigned char)(ubrr>>8);
  UBRRL = (unsigned char)ubrr;
  
  /* Enable receiver and transmitter */
  UCSRB = (1<<RXEN)|(1<<TXEN)|(1<<RXCIE);
  /* Set frame format: 8data, 1stop bit */
  UCSRC = (1<<URSEL)|(3<<UCSZ0);
  
  if(UCSRB & (1 << RXEN))
  {
    USART_String("RX enabled\n");
  }
  
  if(UCSRB & (1 << RXCIE))
  {
    USART_String("RX Interrupt enabled\n");
  }
  
  USART_String("UART Verbindung hergestellt...\n");
}

//gpio
void led_flash(unsigned char n)
{
  unsigned char i;
  for(i = 0; i < n; i++)
  {
    PORTD |= (1 << LED_STATUS);
    _delay_ms(200);
    PORTD &= ~(1 << LED_STATUS);
    _delay_ms(200);
  }
}

void init_io(void)
{
  DDRA = 0x00;
  PORTA = 0xff;
  
  DDRD = 0xff;
  DDRD &= ~(1 << RX_PIN);
  
  PORTD = 0x00;
  PORTD |= (1 << RX_PIN);
}

ISR(USART_RXC_vect)
{
  USART_String("Zeichen empfangen");
}

Bitte um eure Hilfe

[Nico (nico123)]

Der Code sieht gut aus!
Verbinde mal RX und TX von deiner seriellen Verbindung zum PC und schaue 
ob die Daten die Du sendest auch wieder zurück kommen.

[Uwe (Gast)]

> interner RC Oszi
Der ist abhängig von Betriebsspannung und Teperatur und nicht geeignet 
für UART. Man kann jedoch eine autobaud erkennung einbauen. Der PC 
Sendet ein Bestimmtes Zeichen und der µC weiß bescheid was für ein 
Zeichen es Sein soll. Dann mißt der µC ein paar bit längen und berechnet 
daruas die Baudrate. Bzw paßt die Baudtratenregister an oder verstimmt 
den internen RC-Oszillator.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Uwe schrieb:
>> interner RC Oszi
> Der ist abhängig von Betriebsspannung und Teperatur und nicht geeignet
> für UART.
Aber irgendwas wird der UART trotzdem empfangen...

[Uwe (de0508)]

Hallo,

wie es aussieht sind die Taster nach Vcc geschaltet.

Das geht nicht gut, wenn da kein Pull-Down Widerstand existiert.

Die UART Init ist nicht unbedingt portabel. Schaue bitte in's Datenblatt 
und rechne mal nach.

Ich mache das so:

UCSR0C = (1<<UCSZ01) | (1<<UCSZ00)  // 8 Bit
#ifdef URSEL0
     | (1<<URSEL0)    // if UCSR0C shared with UBRR0H
#endif
     ;

Deine ADC Routinen sind falsch, wie willst Du den ADC beschreiben, wenn 
er nicht eingeschaltet wird ?

Also erst in |void adc_init()| aktivieren und dann die Parameter setzen.

ADCSRA |= (1 << ADEN);

Was auch noch fehlt ist das Setzten des ADC Prescalewertes,so dass der 
ADC mit <= 125.00kHz läuft!

Das sind die Bits ADPS2 ADPS1 ADPS0 in ADCSRA, bei 1MHz Takt wäre es ein 
Vorteilerwert von 8, das entspricht, als Vektor geschrieben: 
(ADPS2,ADPS1,ADPS0) = (0,1,1).

Und bei 8MHz ? klar 1:64.
(ADPS2,ADPS1,ADPS0) = (1,1,0).

Und was machst Du in |adc_read()|?

*Keine Messung starten !*

int adc_read(unsigned char channal)
{
  int result;
  
  adc_change_channal(channal);
  
  //start measuring
  ADCSRA |= (1 << ADEN);
  while(ADCSRA & (1 << ADSC));
  
  result = ADCL;
  result += (ADCL<<8);

  return result;
}

Im Datenblatt (von Atmel) steht doch genau, wie das geht !

Und man kommt zu solch einen Ablauf bei einer Messung mit blocking.

int adc_read(unsigned char channal)
{
  adc_change_channal(channal);
  
  //start measuring
  ADCSRA |= (1 << ADSC);
  while(ADCSRA & (1 << ADSC));
  
  return ADCW;
}

[Judge (Gast)]

Danke für die Antworten. Die ADC Routinen kommen erst zum Einsatz, wenn 
ich die UART Schnittstelle hinbekommen habe. Sind ausm alten Projekt mit 
einem andern Controller habe sie abgewandelt und noch nicht getestet. 
Kann gut sein das die falsch sind. Danke schonmal für den Hinweis. Die 
USART INIT routine ist aus dem Datenblatt. Habe nur statt 2 Stopbit 1 
gemacht.

[Judge (Gast)]

Also ich hab jetzt nochmal die Routine aus dem Datenblatt verwendet und 
es funktioniert leider immernoch nicht. Habe das Interrupt enable selbst 
hinzugefügt.

Gibt es eine Methode um zu Prüfen ob die Interrupts global gesetzt sind?

void USART_Init(unsigned int ubrr)
{
  /* Set baud rate */
  UBRRH = (unsigned char)(ubrr>>8);
  UBRRL = (unsigned char)ubrr;
  /* Enable receiver and transmitter */
  UCSRB = (1<<RXEN)|(1<<TXEN)|(1<<RXCIE);
  /* Set frame format: 8data, 2stop bit */
  UCSRC = (1<<URSEL)|(1<<USBS)|(3<<UCSZ0);
}

Im weiteren ist mir vorhin aufgefallen als ich den Controller mit einem 
Model kommunizieren lies, das er auch geantwortet hat. Aber nicht wenn 
ich in mein Terminal Programm etwas eingebe. ich benutze das Programm 
AccessPort. Welches könnt Ihr mir empfehlen?

[spess53 (Gast)]

Hi

>Gibt es eine Methode um zu Prüfen ob die Interrupts global gesetzt sind?

I-Flag in SREG auswerten.

MfG Spess

[Judge (Gast)]

Aus Verzweiflung habe ich jetzt alles nochmal neu gemacht und nur die 
Routinen aus dem Datenblatt des Atmega16a genommen.

#include <avr/io.h>

#define FOSC 1000000// Clock Speed
#define BAUD 2400
#define MYUBRR FOSC/16/BAUD-1

void main( void )
{
  
  USART_Init ( MYUBRR );
  
  
  while(1)
  {
    USART_Transmit('H');
    USART_Transmit('i');
    USART_Transmit('!');  
  }
}

void USART_Init( unsigned int ubrr)
{
  /* Set baud rate */
  UBRRH = (unsigned char)(ubrr>>8);
  UBRRL = (unsigned char)ubrr;
  /* Enable receiver and transmitter */
  UCSRB = (1<<RXEN)|(1<<TXEN);
  /* Set frame format: 8data, 2stop bit */
  UCSRC = (1<<URSEL)|(1<<USBS)|(3<<UCSZ0);
}

void USART_Transmit( unsigned char data )
{
  /* Wait for empty transmit buffer */
  while ( !( UCSRA & (1<<UDRE)) )
  ;
  /* Put data into buffer, sends the data */
  UDR = data;
}

[Karl H. (kbuchegg)]

Judge schrieb:
> Aus Verzweiflung habe ich jetzt alles nochmal neu gemacht und nur die
> Routinen aus dem Datenblatt des Atmega16a genommen.

Schön.
Und?
Siehst du auf deinem Terminal was (egal was, selbst wenn es die falschen 
Zeichen sind)

[Judge (Gast)]

Nein mir wird nichts angezeigt... Habe auch ein anderes ausprobiert. 
Putty aber das startet noch nichtmal.

Ein selbstgemachtes mit C# zeigt nur mist an als noch die baudrate 
zuhoch war.. jetzt wo ich sie angepasst habe kommt nichts mehr.

[Nico (nico123)]

Nico ... schrieb:
> Der Code sieht gut aus!
> Verbinde mal RX und TX von deiner seriellen Verbindung zum PC und schaue
> ob die Daten die Du sendest auch wieder zurück kommen.

Hast Du das mal getestet?

[Karl H. (kbuchegg)]

Judge schrieb:
> Nein mir wird nichts angezeigt... Habe auch ein anderes ausprobiert.
> Putty aber das startet noch nichtmal.
>
> Ein selbstgemachtes mit C# zeigt nur mist an als noch die baudrate
> zuhoch war.. jetzt wo ich sie angepasst habe kommt nichts mehr.


Leg erst mal dein selbstgemachtes PC-Programm zur Seite und sieh zu, 
dass du ein Standard-Terminal Programm auf den PC kriegst. Sonst führst 
du einen 2-Frontenkrieg, weil du dich auf nix Konkretes verlassen 
kannst. Bei einem Standard-Terminal-Programm stehen die Chancen schon 
mal sehr gut, dass dieses zumindest soweit erst mal als korrekt 
angesehen werden kann.


Und dann nimmst du erst mal den µC aus seinem Sockel und brückst im 
Sockel den Tx Pin mit dem Rx Pin. Was auch immer du am Terminal an der 
Tastatur tippst, muss auch wieder im Terminal aufscheinen. Gegentest: 
Brücke im Sockel entfernen, dann muss auch das Echo aufhören.

Funktioniert dieser einfache Test nicht, dann hast du ein Hardware 
Problem. Kabel falsch gekreuzt, irgendwo was falsch angeschlossen (zb am 
MAX), Kabelader abgerissen, usw. usw.