USART anpassen Atmega8 -> Atmega168

[Thomas G. (Gast)]

Hallo,

möchte mein Programm vom Atmega8 auf den Atmega168 zum laufen bringen. 
Ich lese Daten von einem GPS-Empfänger mit der USART Schnittstelle aus. 
Auf dem Atmega168 sind die empfangenen Daten jetzt jedoch teilweise 
falsch (teilweise stimmts).

Hier mein Code vom Atmega8:

//---------------------
//  USART.h
//---------------------
#ifndef _USART_H_
#define _USART_H_

#define F_CPU 3686400
#define USART_BAUD_TEILER(baudRate,cpuClk) ((cpuClk)/((baudRate)*16l)-1)

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

void usart_init(unsigned int baudrate);
char usart_get_char(void);

#endif

//---------------------
// USART.c
//---------------------
#include "usart.h"

void usart_init(unsigned int baudrate)
{
  UBRRH = (unsigned char)(baudrate>>8);
    UBRRL = (unsigned char) baudrate;

    // Enable USART: Empfänger
    UCSRB = (1<<RXEN); //UCSRB = (1<<RXCIE)|(1<<RXEN); 
    
    //Frame Format: asynchron, 8bit, no parity, 1stop bit
    UCSRC = (1<<URSEL)|(3<<UCSZ0);
}


char usart_get_char(void)
{    
  char zeichen;

  while(!(0x80 & UCSRA))
  {}
  
  zeichen = UDR;
  return zeichen;
}

USART.c habe ich für den Atmega168 folgendermaßen angepasst:

//---------------------
// USART.c
//---------------------

#include "usart.h"

void usart_init(unsigned int baudrate)
{
  UBRR0H = (unsigned char)(baudrate>>8);  
    UBRR0L = (unsigned char) baudrate;    

    // Enable USART: Empfänger
    UCSR0B = (1<<RXEN0);
    
    //Frame Format: asynchron, 8bit, no parity, 1stop bit
  UCSR0C = (1<<UCSZ01)|(1<<UCSZ00);
}

char usart_get_char(void)
{    
  char zeichen;

  while(!(0x80 & UCSR0A))
  {}
  
  zeichen = UDR0;
  return zeichen;
}

Die init-Funktion rufe ich dann so auf:

   usart_init(USART_BAUD_TEILER(4800,F_CPU));

Bin schon seit ewig am Fehlersuchen... wäre super wenn mal jemand drauf 
schauen könnte... oder hat jemand eine Idee, was ich beim Anpassen noch 
beachten muss?

Gruß Thomas

[... (Gast)]

die Fuse /8 gesetzt?

[... (Gast)]

was ist 0x80 in
while(!(0x80 & UCSR0A)) ?

sollte das nicht so aussehen:

/* Wait for data to be received */
while ( !(UCSR0A & (1<<RXC0)) )

[Thomas G. (Gast)]

Vielen Dank, das war ein guter Hinweis! Mit "while (!(UCSR0A & 
(1<<RXC0)))" funktioniert es :-) super!

[Stefan E. (sternst)]

Thomas G. schrieb:
> Vielen Dank, das war ein guter Hinweis! Mit "while (!(UCSR0A &
> (1<<RXC0)))" funktioniert es :-) super!

Sorry, aber das ist Unsinn. Das ist bloß eine andere Schreibweise deiner 
Variante (allerdings die bessere). Wenn diese Zeile in deinem Code 
falsch gewesen wäre, dann hätte es gar nicht funktioniert, aber nicht:
> teilweise falsch (teilweise stimmts).

Lass mich raten, du verwendest beim Mega168 den internen RC-Oszillator, 
oder? Der ist zu ungenau und dann kann genau so was passieren:
> teilweise falsch (teilweise stimmts).
Dass es jetzt geht liegt dann eher daran, dass er heute Abend etwas 
weniger daneben liegt, als heute Mittag (z.B. wegen einer anderen 
Temperatur).