Atmega8 UART ext. Quarz

[Dag0bert (Gast)]

Guten Abend zusammen,

ich habe ein Problem mit der seriellen Datenübertragung bei einem 
Atmega8.
Befeuert wird der µC mit einem externen 8 MHz Quarz und den passenden 
Keramikkondensatoren. Die Fuse-Bits habe ich mit dem "AVR8 Burn-O-Mat" 
von Torsten Brischalle eingestellt. Also: external crystal, 3-8 MHz 
frequency range, large output swing und slowly rising power.
In der Software habe ich dann den UBBR-Wert für eine Baudrate von 9600 
Baud bei f_clk=8 MHz berechnet und in das entsprechende Register 
geschrieben. Einstellungen bzw. entsprechende Bits gesetzt für: 8-Bit 
Datenrahmen, 1 Stopbit und keine Parität.

Nun zu meinem Problem:
Wenn ich mit dem Atmega8 reden möchte (hterm), dann funktioniert das nur 
bei einer Baudrate von 300 Bit/s. Bei der eingestellten bzw. gewünschten 
Baudrate von 9600 Bit/s funktioniert es nicht. Es ist so, als wäre es 
ihm einfach zu schnell.

Woran könnte das liegen?

Gruß
Dag0bert

[Einer K. (Gast)]

Da hilft nur "vor Ort Diagnose"

LA oder Oszi zeigen dir das Verhalten.

[Herman Kokoschka (Gast)]

In den Fuses den Vorteiler DIV8 abgeschaltet?
UBBR-Wert korrekt berechnet?

Kommen denn bei den 300Baud die RICHTIGEN Daten, oder nur irgendein 
Müll?

[H.Joachim S. (crazyhorse)]

Dag0bert schrieb:
> In der Software habe ich dann den UBBR-Wert für eine Baudrate von 9600
> Baud bei f_clk=8 MHz berechnet und in das entsprechende Register
> geschrieben.

Schön, dass du das berechnet hast...
Vielleicht zeigst du mal die UART-Initialisierung.

Dann gibts noch die div8-fuse.

[spess53 (Gast)]

Hi

> Also: external crystal, 3-8 MHz
>frequency range, large output swing und slowly rising power.
>In der Software habe ich dann den UBBR-Wert für eine Baudrate von 9600
>Baud bei f_clk=8 MHz berechnet und in das entsprechende Register
>geschrieben. Einstellungen bzw. entsprechende Bits gesetzt für: 8-Bit
>Datenrahmen, 1 Stopbit und keine Parität.

Und wie sehen die genauen Werte aus?

MfG Spess

[Wolfgang (Gast)]

Dag0bert schrieb:
> Woran könnte das liegen?

Das du dich verrechnet hast oder eine Fuse falsch eingestellt hast.

Wie sieht deine Initialisierung für den Baud Rate Generator aus?
Wie hast du deine Fuses gesetzt?
Wie schnell blinkt eine LED mit einem 1Hz LED-Blink Programm?
Hast du ein Oszi oder einen LA?

[Wolfgang (Gast)]

H.Joachim S. schrieb:
> Dann gibts noch die div8-fuse.

Bei einem ATmega8?

Ein Faktor 8 würde außerdem einen Faktor 32 in der Symbolrate alleine 
nicht erklären.

[Dag0bert (Gast)]

Arduino Fanboy D. schrieb:
> Da hilft nur "vor Ort Diagnose"
>
> LA oder Oszi zeigen dir das Verhalten.

Ich habe mir das vom PC gesendete Signal schon mit dem Oszi angeschaut, 
bei verschiedenen Baudraten, passt alles soweit. Taktfrequez des µC habe 
ich auch überprüft (Timer -> toggle Pin).


Herman Kokoschka schrieb:
> In den Fuses den Vorteiler DIV8 abgeschaltet?
> UBBR-Wert korrekt berechnet?
>
> Kommen denn bei den 300Baud die RICHTIGEN Daten, oder nur irgendein
> Müll?

Von einem Vorteiler in den Fuses höre ich das erste mal, muss ich noch 
einmal nachschauen. Allerdings hätte dann die gemessene Taktfrequenz am 
Pin nicht gestimmt.
Den UBBR-Werte sollte ich richtig berechnet haben (nach Datenblatt).
Bei 300 Baud kommen die richtigen Daten an.


H.Joachim S. schrieb:
> Dag0bert schrieb:
>> In der Software habe ich dann den UBBR-Wert für eine Baudrate von 9600
>> Baud bei f_clk=8 MHz berechnet und in das entsprechende Register
>> geschrieben.
>
> Schön, dass du das berechnet hast...
> Vielleicht zeigst du mal die UART-Initialisierung.
>
> Dann gibts noch die div8-fuse.

Habe den code gerade nicht zur Hand, reiche ich nach.

[Einer K. (Gast)]

Dag0bert schrieb:
> Ich habe mir das vom PC gesendete Signal schon mit dem Oszi angeschaut,

Und, die andere Richtung?

Tipp:
PC->µC kann man sich anschauen, zum Vergleich
µC->PC sollte man sich dringenst anschauen, wenn man Probleme hat.

Dag0bert schrieb:
> Den UBBR-Werte sollte ich richtig berechnet haben (nach Datenblatt).
Natürlich musst du deine Berechnung(ERgebnis) und die UART 
Initialisierung geheim halten.
Sonst wird das Problem ja langweilig, wenn jeder sofort den Fehler sehen 
kann.

[Dag0bert (Gast)]

Arduino Fanboy D. schrieb:
> Dag0bert schrieb:
>> Ich habe mir das vom PC gesendete Signal schon mit dem Oszi angeschaut,
>
> Und, die andere Richtung?
>
> Tipp:
> PC->µC kann man sich anschauen, zum Vergleich
> µC->PC sollte man sich dringenst anschauen, wenn man Probleme hat.

Bei 300 Baud passt auch das Signal µC->PC.
Bei größerer Baudratekommt nur Müll an. Das liegt aber vermutlich daran, 
dass die gesendeten Daten abhängig davon sind, welche Daten empfangen 
wurden. Soll heißen, wenn er die zu empfangenden Daten nicht richtig 
versteht, kommt bei den darrauf gesendeten auch nur Müll raus.

[Wolfgang (Gast)]

Dag0bert schrieb:
> Von einem Vorteiler in den Fuses höre ich das erste mal, muss ich noch
> einmal nachschauen.

Ich auch, zumindest beim ATmega8 ;-)

[spess53 (Gast)]

Hi

Dir ist auch bekannt das die Register UBRRH und UCSRC die gleiche 
Adresse besitzen und mit dem Bit URSEL ausgewählt wird, welches der 
beiden Register angesprochen wird?

MfG Spess

[Herman Kokoschka (Gast)]

Dag0bert schrieb:
> Bei 300 Baud passt auch das Signal µC->PC.

Ok, sieht nach einem simpel-zu-lösenden Problem aus:
Deine UART-Init ist nicht korrekt.

DA JA NUN ABER Dein Init-Code offizielle Geheimsache bleibt,
ist kaum zu helfen.

UND mit Verlaub:
Es ist schon etwas sehr-albern, dies geheimzuhalten.
Dann frag aber auch halt nicht!

[Frank B. (frank501)]

Mal wieder typisch µC.net
Es heißt zwar, wer lesen kann, ist klar im Vorteil, aber wer es nicht 
TUT, ist noch mehr im Nachteil als jener, der es nicht kann:


Dag0bert schrieb:
> Habe den code gerade nicht zur Hand, reiche ich nach.

[Herman Kokoschka (Gast)]

@Frank B.:
Dag0bert in der Autowerkstatt:

"Ich habe an meinem Auto gebastelt, aber das funktioniert nicht.
Ich habe aber ALLES RICHTIG GEMACHT. Woran liegt das jetzt?"

Werkstatt:
"Können Sie etwas mehr dazu sagen?"

Dag0bert:
"Nö, ist unnötig weil ICH HABE ALLES RICHTIG GEMACHT!"

Werkstatt:
"Sind Sie da sicher, hier mal 3-4 theoretische Fehlerursachen"

Dag0bert:
"Nö, ICH HABE ALLES RICHTIG GEMACHT!
Warum funktioniert das nicht?"

Werkstatt:
"Können wir denn das Auto wenigstens mal sehen?"

Dag0bert:
"Nö, Habe ich gerade nicht zur Hand."

[Dag0bert (Gast)]

spess53 schrieb:
> Hi
>
> Dir ist auch bekannt das die Register UBRRH und UCSRC die gleiche
> Adresse besitzen und mit dem Bit URSEL ausgewählt wird, welches der
> beiden Register angesprochen wird?
>
> MfG Spess

Danke Spess für den Hinweis, daran liegt es vermutlich. Konnte es aber 
noch nicht probieren.


An den Rest: Bitte die Kirche im Dorf lassen. Wie gesagt, reiche ich den 
Code nach (siehe unten).

Mit der seriellen Schnittstelle sollen Werte an für einen Drehzahlregler 
übertragen werden. Zu Testzwecken habe ich es aktuelle so realisiert, 
dass bei empfangen eines der Zeichen (n,p,i), jweils der entsprechende 
Werte eingegeben werden muss (Drehzahlsollwert, proportional-/ 
integral-Verstärkungsfaktor)

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

#define F_CPU 8000000UL  // Prozessorfrequenz
#define BAUD 9600UL  //Baudrate
#define UBBR_Value  F_CPU/16/BAUD-1

volatile uint8_t Auswahl=0;  //globale variable
char *Drezahlanfrage = "\r\nDrehzahlwert eingeben: \r\n"; // string kommen in " " Zeichenkette als char-array: *Satz, \r\n für Zeilenumbruch
char *I_Anfrage = "\r\nI-Verstaerkungswert eingeben: \r\n";
char *P_Anfrage = "\r\nP-Verstaerkungswert eingeben: \r\n";
char *Eingabe = "\r\n\r\nn fuer Drehzahl,\r\np fuer P-Verstaerkungsfaktor,\r\ni fuer I-Verstaerkungsfaktor\r\neingeben: \r\n";
char data,data_n,data_p,data_i; // char kommen in ' '


//-------------------Funktionen-------------------


void USART_init (uint16_t ubrr)
{
  UBRRH = (uint8_t)(ubrr >> 8);  // Berechneter Wert aus Baudrate und Prozessortakt
  UBRRL = (uint8_t)ubrr;
  UCSRC = (1<<UCSZ1)|(1<<UCSZ0);  // Datenrahmen einstellen 8bit
  UCSRB = (1<<RXEN)|(1<<TXEN)|(1<<RXCIE);  // RXEN/TXEN Sende-,Empfangskanal aktivieren, Empfangsinterrupt aktivieren
  
}

void USART_Flush(void)
{
  uint8_t dummy;        //entfernen evtl. vorhandener ungültiger Werte
  while (UCSRA & (1<<RXC));
  dummy=UDR;
}

char USART_Receive (void)
{
  while( !(UCSRA & (1<<RXC)) );  //Wenn Bit RXC gesetzt, dann Daten im Empfangspuffer
  return UDR;  //Funktion gibt Byte aus Datenregister UDR zurück
}

void USART_Transmit (char data) // war uint8_t
{
  while ( !( UCSRA & (1<<UDRE)) );
  UDR = data;
}

void USART_Transmit_STRING (char *s)
{
  while(*s)
  {
    USART_Transmit (*s);
    s++;
  }
}


//-------------------Hauptprogramm-------------------


int main(void)
{

DDRC=0x0F;  // alle Pins an PortC als Ausgänge
DDRD=0x04;  // alle Pins an PortD als Ausgänge
DDRB=0xFF;  // alle Pins an PortD als Ausgänge
USART_init(UBBR_Value);
USART_Flush();
sei();

while(1)
  {
    //PORTC = 0x00;
    //data=USART_Receive();
    //USART_Transmit(data);
    //PORTC = data;
    PORTD ^= ( 1 << PD2 );
  }

}


//-------------------ISR-------------------


ISR(USART_RXC_vect)
{
///*
  if (Auswahl == 0)
  {
    data=USART_Receive();
  }


  if (Auswahl == 1)
  {
    data_n=USART_Receive();
      PORTB ^= ( 1 << PB1 );
    if (data_n == 8)
    {
      Auswahl=0;
      data_n=0;
      data=0;
    }
       
  }
  else if (Auswahl == 2)
  {
    data_p=USART_Receive();
    if (data_p == 8)
    {
      Auswahl=0;
      data_n=0;
      data=0;
    }
  }
  else if (Auswahl == 3)
  {
    data_i=USART_Receive();
    if (data_i == 8)
    {
      Auswahl=0;
      data_n=0;
      data=0;
    }
  }


  if (data == 'n')
  {
    USART_Transmit_STRING(Drezahlanfrage);
    Auswahl=1;
  }
  else if (data == 'p')
  {
    USART_Transmit_STRING(P_Anfrage);
    Auswahl=2;
  }
  else if (data == 'i')
  {
    USART_Transmit_STRING(I_Anfrage);
    Auswahl=3;
  }
  else
  {
    USART_Transmit_STRING(Eingabe);
    Auswahl=0;
  }
//*/
}

[S. Landolt (Gast)]

> Danke Spess für den Hinweis, daran liegt es vermutlich.

So ist es. Die Übertragungsrate wird nicht auf 9600 Bd eingestellt, 
sondern auf 315. Dieses

UCSRC = (1<<UCSZ1)|(1<<UCSZ0);

geht in Wirklichkeit nach UBRRH, damit wird UBRR = 6*256 + 51.

[Stefan F. (Gast)]

Ich empfehle, die Berechnung der Baudrate nicht selber vorzunehmen, 
sondern die dafür vorgesehene Header Datei der AVR C Bibliothek zu 
verwenden. Dann kann man an dieser Stelle auch nichts falsch machen.
https://www.nongnu.org/avr-libc/user-manual/group__util__setbaud.html

[Dag0bert (Gast)]

Danke für die konstruktiven Beiträge und die Hlfe bei der 
Lösungsfindung!

[Dyson (Gast)]

Stefan ⛄ F. schrieb:
> Ich empfehle

Ich empfehle, diesen unsinnigen Beitrag zu ignorieren, da das in diesem 
Fall zu gar nichts führt.

[Stefan F. (Gast)]

Dyson schrieb:
> Ich empfehle, diesen unsinnigen Beitrag zu ignorieren, da das in diesem
> Fall zu gar nichts führt.

Du bist ja echt eine heiße Luftpumpe.

Hat er die Baudrate richtig berechnet? Nein!

Habe ich ihm den allgemein gültigen korrekten Algorithmus für alle AVR 
samt Doku empfohlen? Ja.

Hat Dag0bert sich dafür bedankt? Ja.

Was hast du hier überhaupt verloren? Du hast hier gar nichts 
intelligentes beigetragen. Denke mal darüber nach!

[Dyson (Gast)]

Stefan ⛄ F. schrieb:
> Dyson schrieb:
>> Ich empfehle, diesen unsinnigen Beitrag zu ignorieren, da das in diesem
>> Fall zu gar nichts führt.
>
> Du bist ja echt eine heiße Luftpumpe.

Da gehen die Meinungen auseinander.

> Hat er die Baudrate richtig berechnet? Nein!

Doch, hat er. Er hat das URSEL-Bit nicht gesetzt und damit die korrekte 
Baudrate unwissentlich überschrieben.

> Habe ich ihm den allgemein gültigen korrekten Algorithmus für alle AVR
> samt Doku empfohlen? Ja.

Deine Epmpfehlungen kannst du dir... Lassen wir das.
> Hat Dag0bert sich dafür bedankt? Ja.

Bei den Leuten mit den konstruktiven Beiträgen, also nicht bei dir.

> Was hast du hier überhaupt verloren?

Was geht dich das an?

> Du hast hier gar nichts intelligentes beigetragen. Denke mal darüber
> nach!

Den Ball spiele ich einfach mal zurück.

[Einer K. (Gast)]

Klasse!
Zwei Hähne auf dem gleichen Misthaufen und plustern sich mit 
Selbstbeweihräucherungen auf.

Sieht von weitem etwas borniert/blamabel aus.

Aber lasst euch nicht stören, die Show ist toll.
Weiter machen!