Atmega88 reagiert nicht auf seriellen Eingang

[Micha (Gast)]

Bin grade am verzweifeln - eine Schaltung mit einem Atmega 88 der aus 
einem seriellen Input einen Output zum parallelen Drucker machen soll. 
Letzteres hab ich bereits ausgiebig getestet, das funktioniert.

Heute hab ich den seriellen Input drangebastelt, aber der Atmega 
reagiert nicht auf Input. Sieht eventuell jemand einen offensichtlichen 
Fehler in dem folgenden Code? Vielen Dank schon mal.

/*
 * DruckerSteuerung.c
 *
 * Created: 14.10.2012 10:42:54
 *  Author: Michael Berger
 */ 

#include "main.h"
#include <util/delay.h>
#include <avr/io.h>
#include <avr/interrupt.h>

volatile uint8_t buf[256];  // print buffer
volatile uint8_t ix = 0;  // index for incoming chars
volatile uint8_t ox = 0;  // index for outgoing chars

ISR(USART_RX_vect)  // usart receive char ISR
{
  buf[ix++] = UDR0;
}

int main(void)
{
  initPorts();
  initUSART();
    while(1)
    {
    if (ix != ox) {
      printChar(buf[ox++]);
      if (((ix-ox) % 256) > 100) {  // modulo operator to determine abs. difference
        // send XOFF
        while (ox != ix) printChar(buf[ox++]);
        // send XON
      }
    }    
  }  
}

void initPorts()
{
  DDRB = 0b11111111;  // PB0 is Strobe, PB1 is Init
  DDRC = 0b11111111;  // PC0..PC5 is D0..D5
  DDRD = 0b11000011;  // Port D pins used as follows:
  /*
    PD7,PD6 is D7,D6
    PD5  Ack
    PD4  Busy
    PD3  Paper Out
    PD2  Sel
  */  
  PORTC = CR;        // sometimes this gets printed after a reset...
  PORTD &=0b00111111;    // 2 high bits of CR, both 0

  PORTB &= ~(1<<INIT_PIN);  // init printer
  _delay_ms(10);
  PORTB |= 1<<INIT_PIN;
  _delay_ms(100);    // give the printer some time to settle
}

void printChar(char c)
{
  while (PIND & 1<<BUSY_PIN) {}    // wait for Busy signal
  PORTC = c & 0b00111111;        // data bits 0..5
  PORTD &= 0b00111111;        // delete pins 6,7
  PORTD |= c;              // set pins 6,7 to char pattern
  PORTB &= ~(1<<STROBE_PIN);
  _delay_ms(5);
  PORTB |= (1<<STROBE_PIN);
  while (!(PIND & 1<<ACK_PIN)) {}   // wait for /Ack signal
}

void initUSART()
{
  UCSR0C = 0b00001110;  // 8N1 format
  UCSR0B = 0b00011000;  // enable TX and RX
  UBRR0H = 0;
  UBRR0L = 119; // 9600 Baud for F_CPU = 18.432 MHz
  UCSR0A = 0b00000000;
  UCSR0B |= (1 << RXCIE0);  // enable interrupt for RxD
  sei();    // allow interrupts
}

[Karl H. (kbuchegg)]

Das erste, was du bei einer seriellen Schnittstelle IMMER machen 
solltest, ist die Richtung umzudrehen.
Anstatt den µC empfangen zu lassen, lässt du ihn erst mal senden!
Einfach zb in einer Endlosschleife ein 'X' senden lassen. Ev. mit einer 
kleinen Pause zwischen den Zeichen.

Am anderen Ende des Kabels sitzt dann ein PC mit einem Terminalprogramm, 
welches dir anzeigt was es vom µC erhalten hat, was immer das auch ist.

Warum ist das das erste?
Weil du am µC normalerweise keine vernünftige Anzeigemöglichkeit hast, 
an der du verfolgen kannst, ob überhaupt etwas ankommt, bzw. wenn etwas 
ankommt, ob das das richtige ist. So gesehen ist die Serielle 
Schnittstelle im PC sowie das Terminalprogramm und dessen Anzeige für 
dich eine 'getestete Komponente', von der erst mal angenommen werden 
kann, dass sie funktioniert. Wenn daher die Übertragung nicht 
funktioniert, dann muss das Problem entweder zb am Kabel oder im µC 
liegen.
Beim Kabel gibt es die Möglichkeit, das es falsch gekreuzt ist. Wenn gar 
nichts auf dem PC rauskommt, dann ist das eine nicht unwahrscheinliche 
Möglichkeit.
Wenn was rauskommt, allerdings die Zeichen falsch sind, dann stimmt die 
Baudrate im µC nicht. Etwas, das wiederrum in mehr als 98% der Fälle auf 
eine fehlerhafte Taktrate im µC zurückzuführen ist.  Allerdings ist 
damit nachgewiesen, das das Kabel schon mal richtig rum gekreuzt ist

Klappt dann die Übertragung µC -> PC, dann klappt auch die Übertragung 
PC -> µC (die sich leider in den meisten Fällen nicht so einfach 
debuggen lässt) auf Anhieb. Wenn dann immer noch Probleme im µC sind, 
dann liegen die im Programm, welches die eingetrudelten Zeichen 
weiterverarbeiten bzw. weiterverteilen. Daher ist es wichtig, dass du 
erst mal feststellst, ob die USART grundsätzlich funktioniert. Und das 
geht am einfachsten, wenn der µC sendet und der PC empfängt. Immer so 
wenig Unbekannte wie möglich und ungetestete Systeme die Fehler 
enthalten können im System haben!

[Karl H. (kbuchegg)]

>   UCSR0C = 0b00001110;  // 8N1 format
>  UCSR0B = 0b00011000;  // enable TX and RX

Sorry. Aber das tu ich mir nicht an, da die Einzelbits 
auseinanderzudröseln.
Benutze die Bitnamen. Da bleiben dann noch genug andere Stolpersteine 
übrig - man muss sich nicht mutwillig Prügel zwischen die Beine werfen.

[Magnus M. (magnetus)]

An welchem Pin fütterst du den Mega88 denn mit den seriellen Daten? (ich 
will jetzt kein "RXD" hören!)

Was sitzt alles zwischen dem Controller und dem COM-Port des PCs?

Was hast du bisher unternommen um sicherzustellen dass die seriellen 
Daten auch am Controller ankommen?

[Micha (Gast)]

ich hab inzwischen mal nach einem brauchbaren Muster für eine USART_init 
spezifisch für den Atmega88  gesucht und bin hier fündig geworden:
Beitrag "Atmega88 USART"

Danach funktioniert jetzt erst mal was, d.h. der Drucker druckt, bis er 
sich verstolpert, was aber normal ist da ich das XOFF/XON Protokoll 
bisher weder auf Sender- noch auf Empfängerseite implementiert habe.

Daher erst mal vielen Dank für die Antworten! Problem gelöst :)

[Magnus M. (magnetus)]

Micha schrieb:
> ich hab inzwischen mal nach einem brauchbaren Muster für eine USART_init
> spezifisch für den Atmega88  gesucht und bin hier fündig geworden:
> Beitrag "Atmega88 USART"

Dann zeig doch mal bitte, wie du diese Initialisierung übernommen hast.

[Micha (Gast)]

Hallo Magnus,

hier ist der mittlerweile für den Atmega88, mit 18,432MHz getaktet, 
funktionierende Code, bzw. der interessante Teil davon in der initUSART 
Funktion. Ursprünglich hatte ich in das UCSR0A Register 0 geschrieben, 
damit ging es nicht. In dieser Version wird das UCSR0A Register in Ruhe 
gelassen so wie es ist, damit geht es. Muss wohl mal genauer nachlesen.
Das Datenblatt das ich mir von Reichelt geladen hatte geht nur bis Seite 
35, muss wohl mal ein komplettes Datenblatt suchen...

/*
 * DruckerSteuerung.c
 *
 * Created: 14.10.2012 10:42:54
 *  Author: Michael Berger
 */ 

#include "main.h"
#include <util/delay.h>
#include <avr/io.h>
#include <avr/interrupt.h>

volatile uint8_t buf[256];  // print buffer
volatile uint8_t ix = 0;  // index for incoming chars
volatile uint8_t ox = 0;  // index for outgoing chars

ISR(USART_RX_vect)  // usart receive char ISR
{
  buf[ix++] = UDR0;
}

int main(void)
{
  initPorts();
  initUSART();
    while(1)
    {
    if (ix != ox) {
      if (buf[ox]>10) printChar(buf[ox]);  // supress LF because printer misinterprets it
      ox++;
      if (((ix-ox) % 256) > 100) {  // modulo operator to determine abs. difference
        // implement XOFF here
        while (ox != ix) {
          if (buf[ox]>10) printChar(buf[ox]);
          ox++;
        }        
        // implement XON here
      }
    }    
  }  
}

void initPorts()
{
  DDRB = 0b11111111;  // PB0 is Strobe, PB1 is Init
  DDRC = 0b11111111;  // PC0..PC5 is D0..D5
  DDRD = 0b11000011;  // Port D pins used as follows:
  /*
    PD7,PD6 is D7,D6
    PD5  Ack
    PD4  Busy
    PD3  Paper Out
    PD2  Sel
  */  
  PORTC = CR;        // sometimes this gets printed after a reset...
  PORTD &=0b00111111;    // 2 high bits of CR, both 0

  PORTB &= ~(1<<INIT_PIN);  // init printer
  _delay_ms(10);
  PORTB |= 1<<INIT_PIN;
  _delay_ms(100);    // give the printer some time to settle
}

void printChar(char c)
{
  while (PIND & 1<<BUSY_PIN) {}    // wait for Busy signal
  PORTC = c & 0b00111111;        // data bits 0..5
  PORTD &= 0b00111111;        // delete pins 6,7
  PORTD |= c;              // set pins 6,7 to char pattern
  PORTB &= ~(1<<STROBE_PIN);
  _delay_ms(5);
  PORTB |= (1<<STROBE_PIN);
  while (!(PIND & 1<<ACK_PIN)) {}   // wait for /Ack signal
}

void initUSART()
{
  UBRR0H = 0;
  UBRR0L = 119; // 9600 Baud for F_CPU = 18.432 MHz
  UCSR0B = (1<<RXEN0) | (1<<TXEN0) | (1<<RXCIE0);  // enable interrupt for RxD
  UCSR0C = (3<<UCSZ00);  // 8N1 format
  sei();    // allow interrupts
}

[Micha (Gast)]

Die Anwendung dahinter nimmt Gestalt an ;)