[C] Sehe den Fehler nicht - multiple definition of "flag"

[Stephan W. (stipo)]

Hallo zusammen,
ich sehe da den Fehler nicht. Füge einfach mal den betreffenden Code 
ein.

Timer.h

#ifndef TIMER_H_
#define TIMER_H_

volatile uint8_t flag = 0;

void InitialTimer(void);
ISR (TIMER0_OVF_vect);



#endif /* TIMER_H_ */

Timer.c

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

uint8_t count = 0;


void InitialTimer(void)
{
  // Timer 0 konfigurieren
  TCCR0B = ((1<<CS00)|(1<<CS02)); // Prescaler 8 <-- Hier zeigt der Fehler hin.
  
  // Overflow Interrupt erlauben
  TIMSK0 |= (1<<TOIE0);
}


ISR (TIMER0_OVF_vect)
{
  if (count == 30)
  {
    //PORTD ^= (1<<PD0);
    flag = 1;
    count = 0;
  }
  count++;
}

Main.c

// µC Frequenz
#define F_CPU 8000000UL

... 

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


int main(void)
{
  ...

         InitialTimer();
  sei();
  
  ...
  
  // Beginn der Hauptschleife
  while(1)
  {    
    if (flag == 1)
    {
      DATAPORT ^= (1<<PD0);
      flag = 0;
    }
                  ...    
  }
}

Ich muss doch, wenn ich eine Variable nicht nur innerhalb einer 
Quelldatei nutzen möchte, die in der Header Datei angeben. Das 
funktioniert soweit auch, das mir Intellisens die Variable anbietet, 
wenn ich in der main.c die Header Datei einbinde.

Nur warum meckert der Compiler, das ich die Variable in folgender Zeile 
doppelt definiere.

TCCR0B = ((1<<CS00)|(1<<CS02)); // Prescaler 8

Wo muss ich meine Gedanken noch ordnen, das es funktioniert?

Grüße
Stephan

[B e r n d W. (smiley46)]

Timer.h
extern volatile uint8_t flag;

Timer.c
volatile uint8_t flag = 0;

[Stephan W. (stipo)]

Hallo Bernd,

danke erstmal. Das hat geklappt.
Jetzt muss ich mal nachlesen, was das extern bewirkt und warum da mein 
Gedanke falsch war.

[Peter D. (peda)]

Stephan W. schrieb:
> Nur warum meckert der Compiler

Man muß die Meldung auch mal lesen.
Da steht ganz genau drin, welches File,Zeile er anmeckert und wo es 
zuerst steht.


Peter

[Stephan W. (stipo)]

Peter Dannegger schrieb:
> Man muß die Meldung auch mal lesen.
> Da steht ganz genau drin, welches File,Zeile er anmeckert und wo es
> zuerst steht.
Peter, klar hab ich den Fehler gelesen und auch verstanden was mir der 
Compiler da sagen will "Doppelte Definition der Variable", aber ich hab 
mal so garnicht verstanden "Warum".

Nur deshalb hab ich da nachgefragt. Ganz auf den Kopf gefallen bin ich 
nicht und ich kann mit Compiler Fehler umgehen (C# Programmierung).

[B e r n d W. (smiley46)]

> Nur warum meckert der Compiler

Falls Du eine Variable im Header definierst, wird diese in jedem C-File, 
welcher den Header included, im Speicher angelegt. Der Linker bemerkt 
dann, dass eine globale Variable mit selbem Namen mehrfach vorhanden 
ist.

[Stephan W. (stipo)]

B e r n d W. schrieb:
>> Nur warum meckert der Compiler
>
> Falls Du eine Variable im Header definierst, wird diese in jedem C-File,
> welcher den Header included, im Speicher angelegt. Der Linker bemerkt
> dann, dass eine globale Variable mit selbem Namen mehrfach vorhanden
> ist.

Danke, das ist eine Erklärung die verständlich ist und einleuchtet.
Ich hab schon mein C-Buch aufgeschlagen und werde mal noch da nachlesen.