UART Libary. Register sind undeclared

[Sören T. (stimmy)]

Hallo Forum,

ich habe heute morgen versucht mein Programm zu ordnen.
Ich habe mir einen Artikel über Header durchgelesen und dann die beiden 
Dateien fabriziert. Leider kommen immer die Fehlermeldungen:

Error  1  'UBRR0H' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  7  3  test uart
Error  2  each undeclared identifier is reported only once for each function it appears in  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  7  3  test uart
Error  3  'F_CPU' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  7  13  test uart
Error  4  'BAUD' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  7  13  test uart
Error  5  'UBRR0L' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  8  3  test uart
Error  6  'UCSR0B' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  9  3  test uart
Error  7  'TXEN0' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  9  17  test uart
Error  8  'RXEN0' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  9  30  test uart
Error  9  'UCSR0C' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  11  3  test uart
Error  10  'UCSZ01' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  11  16  test uart
Error  11  'UCSZ00' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  11  28  test uart
Error  12  'UCSR0A' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  17  14  test uart
Error  13  'UDRE0' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  17  27  test uart
Error  14  'UDR0' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  21  5  test uart
Error  15  'UCSR0A' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  38  14  test uart
Error  16  'RXC0' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  38  27  test uart
Error  17  'UDR0' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  40  12  test uart
Warning  18  control reaches end of non-void function  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  41  1  test uart

Leider finde ich den Fehler nicht. Falls einer gerade Zeit hat, könnte 
er sich das ja mal ansehen.

Vielen Dank im voraus!!

Datein:

/*
 * uart_timmy.h
 *
 * Created: 10.12.2012 12:03:14
 *  Author: Timm
 */ 


#ifndef UART_TIMMY_H_

#define UBRR_VAL ((F_CPU+BAUD*8)/(BAUD*16)-1)
#define BAUD_REAL (F_CPU/(16*(UBRR_VAL+1)))
#define BAUD_ERROR ((BAUD_REAL*1000)/BAUD)

#define UART_TIMMY_H_

extern void uart_init(void);
extern int uart_putc(unsigned char c);
extern void uart_puts (char *s);
extern char uart_getc(void);
extern void uart_gets( char* Buffer, uint8_t MaxLen );

#endif /* UART_TIMMY_H_ */

#include <avr/io.h>
#include "uart_timmy.h"


void uart_init(void)
{
  UBRR0H = (UBRR_VAL>>8);
  UBRR0L =  UBRR_VAL;
  UCSR0B |= (1<<TXEN0) | (1<<RXEN0);// | (1<<RXCIE0);
  // Frame Format: Asynchron 8N1
  UCSR0C = (1<<UCSZ01)|(1<<UCSZ00);
}  


int uart_putc(unsigned char c)
{
    while (!(UCSR0A & (1<<UDRE0)))  /* warten bis Senden moeglich */
    {
    }                             
 
    UDR0 = c;                      /* sende Zeichen */
    return 0;
}
 
 
/* puts ist unabhaengig vom Controllertyp */
void uart_puts (char *s)
{
    while (*s)
    {   /* so lange *s != '\0' also ungleich dem "String-Endezeichen" */
        uart_putc(*s);
        s++;
    }
}

char uart_getc(void)
{
    while (!(UCSR0A & (1<<RXC0)))   // warten bis Zeichen verfuegbar
        ;
    return UDR0;                   // Zeichen aus UDR an Aufrufer zurueckgeben
}

void uart_gets( char* Buffer, uint8_t MaxLen )
{
  uint8_t NextChar;
  uint8_t StringLen = 0;
 
  NextChar = uart_getc();         // Warte auf und empfange das nächste Zeichen
 
                                  // Sammle solange Zeichen, bis:
                                  // * entweder das String Ende Zeichen kam
                                  // * oder das aufnehmende Array voll ist
  while( NextChar != '\n' && StringLen < MaxLen - 1 ) {
    *Buffer++ = NextChar;
    StringLen++;
    NextChar = uart_getc();
  }
 
                                  // Noch ein '\0' anhängen um einen Standard
                                  // C-String daraus zu machen
  *Buffer = '\0';
}
  
  // Passende Größe des Arrays wählen!
  // char Line[40];      // String mit maximal 39 zeichen 
  // uart_gets( Line, sizeof( Line ) );

[Ingo (Gast)]

Dein Header fehlt...


Ingo

[Walter S. (avatar)]

wie schaut der makefile aus?
Mir scheint du hast den Prozessor nicht definiert

[Sören T. (stimmy)]

Der Header ist doch die erste datei ;)

Ich arbeite mit AVR Studio 5.0 .
Habe die Dateien in dem selben Projekt drin wie auch die eigentliche C 
Datei.
Wenn ich den selben Code in der C Datei lasse, funktioniert es. Wenn ich 
ihn aber so auslagere wie beschreiben, kommt es zu den Fehlern.

[applefarmer (Gast)]

Um welchen uC gehts?

[Sören T. (stimmy)]

Es geht um einen ATMEGA 8

[Marius W. (mw1987)]

Tja, dann guck mal nach, was der Mega8 so für UART-Register hat und wie 
die heißen...

Gruß
Marius

[Sören T. (stimmy)]

Okay sind fast alle Fehlermeldungen weg. Danke!!!

Die einzigen beiden die noch sind:

Error  1  'F_CPU' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  7  11  test uart
Error  2  each undeclared identifier is reported only once for each function it appears in  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  7  11  test uart
Error  3  'BAUD' undeclared (first use in this function)  C:\Users\Timm\Documents\AVRStudio\test uart\test uart\uart_timmy.c  7  11  test uart

Dabei hab ich doch in der Datei das stehen:

/*
 * test_uart.c
 *
 * Created: 07.12.2012 15:28:36
 *  Author: Timm
 */ 

#define F_CPU 16000000UL
#define BAUD 9600

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

int main(void)
{
    uart_init();
  uart_putc(0xC);
  uart_puts("UART inistalisiert!\r \n");
  DDRD = 0b1111111;
  
  while(1)
    {         
      uart_puts("So ein Scheiss, dass es an einer null lag!\r\n");
    
    PORTD = (1<<PD6) | (1<<PD5); // LED auf Pollin-Board
    _delay_ms(500);
    PORTD = (0<<PD6) | (0<<PD5);
    _delay_ms(500);
    }
}

Korrigiert:

#include <avr/io.h>
#include "uart_timmy.h"


void uart_init(void)
{
  UBRRH = UBRR_VAL >> 8;
  UBRRL = UBRR_VAL & 0xFF;
 
  UCSRB |= (1<<TXEN);  // UART TX einschalten
  UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0);  // Asynchron 8N1 
}  


int uart_putc(unsigned char c)
{
    while (!(UCSRA & (1<<UDRE)))  /* warten bis Senden moeglich */
    {
    }                             
 
    UDR = c;                      /* sende Zeichen */
    return 0;
}
 
 
/* puts ist unabhaengig vom Controllertyp */
void uart_puts (char *s)
{
    while (*s)
    {   /* so lange *s != '\0' also ungleich dem "String-Endezeichen(Terminator)" */
        uart_putc(*s);
        s++;
    }
}

char uart_getc(void)
{
    while (!(UCSRA & (1<<RXC)))   // warten bis Zeichen verfuegbar
        ;
    return UDR;                   // Zeichen aus UDR an Aufrufer zurueckgeben
}

void uart_gets( char* Buffer, uint8_t MaxLen )
{
  uint8_t NextChar;
  uint8_t StringLen = 0;
 
  NextChar = uart_getc();         // Warte auf und empfange das nächste Zeichen
 
                                  // Sammle solange Zeichen, bis:
                                  // * entweder das String Ende Zeichen kam
                                  // * oder das aufnehmende Array voll ist
  while( NextChar != '\n' && StringLen < MaxLen - 1 ) {
    *Buffer++ = NextChar;
    StringLen++;
    NextChar = uart_getc();
  }
 
                                  // Noch ein '\0' anhängen um einen Standard
                                  // C-String daraus zu machen
  *Buffer = '\0';
}
  
  // Passende Größe des Arrays wählen!
  // char Line[40];      // String mit maximal 39 zeichen 
  // uart_gets( Line, sizeof( Line ) );

[Karl H. (kbuchegg)]

Sören Timm schrieb:

> Dabei hab ich doch in der Datei das stehen:

Die, in der main() steht, interessiert nicht.

Deine Fehlermeldungen beziehen sich auf uart_timmy.c


> Korrigiert:
>

> #include <avr/io.h>
> #include "uart_timmy.h"
> 
> 
> void uart_init(void)
> {
>   UBRRH = UBRR_VAL >> 8;
>   UBRRL = UBRR_VAL & 0xFF;
> 
>   UCSRB |= (1<<TXEN);  // UART TX einschalten
>   UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0);  // Asynchron 8N1
> }
> ....
> ....
>

So und jetzt gehst du diese Datei, und zwar nur diese C-Datei von oben 
nach unten durch und stellst dir die Frage.

Wo kommt da F_CPU her?
Wo kommt da BAUD her?

Wie gesagt: Was in main.c steht, interessiert nicht. Das hier ist die 
uart_timmy.c. Und uart_timmy.c wird für sich alleine kompiliert. Wenn 
sich der Compiler uart_timmy.c vornimmt, hat er alles vergessen, was er 
je aus main.c erfahren hat.
In C ist eine C-Datei eine abgeschlossene Einheit. Sie wird für sich 
selbst und ohne Ansehen anderer C-Dateien compiliert. Was du daher in 
anderen C-Dateie stehen hast, mag zwar für die dortige C-Datei super 
sein, ist aber für diese hier uninteressant.

[Rufus Τ. F. (rufus)]

Sören Timm schrieb:
> Dabei hab ich doch in der Datei das stehen:

In welcher Datei steht das und für welche Datei wird der Fehler 
ausgegeben? Genau hinsehen.

Diese Definition gehört aus diesem Grund nicht in eine Quelltextdatei, 
sondern als Compileroption ins Makefile.

[Walter S. (avatar)]

Sören Timm schrieb:
> #define F_CPU 16000000UL
> #define BAUD 9600

hast du das auch in deiner zweiten C Datei?d
Wenn nein dann ist das der Fehler,
wenn ja ist es Käse weil es sinnvollerweise nur an einer Stelle 
definiert wir

[Sören T. (stimmy)]

So es funktioniert!
Senden und empfangen!

Nur wie bekomme ich es jetzt hin, das die Baudrate und die Frequenz nur 
in der test_uart.c angegeben werden muss?

Das es sinnvoll ist ist mir klar.
Bei delay.h geht es ja auch irgendwie...

[Walter S. (avatar)]

Sören Timm schrieb:
> Nur wie bekomme ich es jetzt hin, das die Baudrate und die Frequenz nur
> in der test_uart.c angegeben werden muss?

indem du sie im makefile definierst,
wenn du keinen hast dann in irgendwelchen Options

[Karl H. (kbuchegg)]

Sören Timm schrieb:

> Nur wie bekomme ich es jetzt hin, das die Baudrate und die Frequenz nur
> in der test_uart.c angegeben werden muss?

Und wenn du noch mal scharf darüber nachdenkst, dann kommst du zu dem 
Schluss: gar nicht!

> Das es sinnvoll ist ist mir klar.
> Bei delay.h geht es ja auch irgendwie...

Nö.
Auch bei delay.h geht es nicht. Jede C-Datei, die delay.h inkludiert, 
muss für sich selbst wieder dafür sorgen, dass sie irgendwo eine 
Definition für F_CPU herkriegt.

Bei F_CPU ist es sinnvoll, die in die Projektoptionen mit aufzunehmen. 
Denn diese Einstellungen werden dem Compiler bei jegem COmpiliervorgang 
immer mitgegeben.

Es spricht aber auch nichts dagegen, dass man sich ein Header-File 
macht, in dem alle diese Konfigurationssachen gesammelt werden UND man 
die Forderung erhebt, dass jedes C-File genau dieses Header File 
inkludieren MUSS.