Interrupt Vector Atmega4809

[Sandro (sandro91)]

Hallo Leute

Ich habe grade ein Projekt mit dem Atmega4809.
Ich habe folgenden Code in einer Bibliotheksdatei.

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


void Ultra_Sonic_Sensor_init( void )
{
  // Aktiviere Externen Interrupt auf Echo Signal
  Echo_CTRL  =  PORT_ISC_FALLING_gc;
  
  // Schalte Trig als Ausgang 
  Trig_Dir  |=  Trig;
  
  //config Timer
  
  // select prescaler
  // mit prescaler 2 ein increment alle 0.4uS bei einem 20MHz Takt
  TCA0.SINGLE.CTRLA |= TCA_SINGLE_CLKSEL_DIV8_gc;
  
  // Aktiviere Interrupt "Compare Match 0"
  TCA0.SINGLE.INTCTRL = TCA_SINGLE_CMP0EN_bm;
  
  // Zähle clock ticks
  TCA0.SINGLE.EVCTRL &= ~(TCA_SINGLE_CNTEI_bm);
  
  // Compare Match Interrupt nach 25mS
  TCA0_SINGLE_CMP0 = 62500;
    
  // starte Timer
  TCA0.SINGLE.CTRLA |=  TCA_SINGLE_ENABLE_bm;

}

/*
ISR(TCA0_CMP0_vect)
{
  //lösche Interrupt flag
  TCA0.SINGLE.INTFLAGS = TCA_SINGLE_CMP0EN_bm;
  
  //Reset Counter Register
  TCA0.SINGLE.CNT  =  0; 
    
  // Start signal für 10uS
  Trig_Out |= Trig;
  _delay_us(10);
  Trig_Out &= ~Trig;
}

*/

uint32_t RohWert;

ISR(PORTB_PORT_vect)
{
  
  // lösche Interrupt flag
  PORTB.INTFLAGS = PORT_INT2_bm;
  
  RohWert = TCA0.SINGLE.CNT;
}

uint32_t Ultra_Sonic_Sensor_read (void)
{
  uint32_t Distanz = 0;
  
  // Berechne Distanz in mm
  Distanz = RohWert;
  Distanz *= 25;
  Distanz *= 23.3;
  Distanz /= 2;
  
  return Distanz;
}

Leider kriege ich beim compilieren immer die Fehlermeldung.

multiple definition of `__vector_34'.
Diese bezieht sich auf den Interrupt Vector ISR(PORTB_PORT_vect).
Wenn ich aber diese Routine in mein main File reinkopiere kommt diese 
Fehlermeldung nicht.

Was mach ich falsch?
Vielen Dank für die Hilfe.

[Johann L. (gjlayde)]

Sandro schrieb:

> multiple definition of `__vector_34'.

Beim ATmega4809 ist Vector 34 PORTB_PORT_vect.

Es gibt also mehr als eine Definitionen dieser ISR im Project.

Grund kann sein, dass unterschiedliche Compilation Units die gleiche ISR 
definieren, dass eine Compilation Unit mehrfach im Makefile angegeben 
wurde, oder dass eine ISR in einem Header definiert wurde der in mehr 
als einer Compilation Unit inkludiert wird.

Davon ab fehlt "RohWert" ein volatile, und der Zugriff darauf in 
Ultra_Sonic_Sensor_read() ist nicht atomar.

[Harald K. (kirnbichler)]

> Ich habe folgenden Code in einer Bibliotheksdatei.

Was soll eine "Bibliotheksdatei" sein?

[Wilhelm M. (wimalopaan)]

Harald K. schrieb:
>> Ich habe folgenden Code in einer Bibliotheksdatei.
>
> Was soll eine "Bibliotheksdatei" sein?

Na, was wohl?

[Harald K. (kirnbichler)]

Wilhelm M. schrieb:
> Na, was wohl?

*.lib bzw. *.a, das wären Bibliotheksdateien.

*.h ist keine Library. In C++ kann man zwar libraryartige Dinge damit 
anstellen, der da oben gezeigte Code aber fällt definitiv nicht 
darunter. Der hat in einer *.h-Datei absolut nichts zu suchen. Das ist 
ein Fehler.

[Wilhelm M. (wimalopaan)]

Wo steht, dass das ein Header Datei sei?

[Harald K. (kirnbichler)]

Wilhelm M. schrieb:
> Wo steht, dass das ein Header Datei sei?

Das steht nirgendswo, es ist aber zu befürchten, weil im Arduino-Umfeld 
Headerdateien als "Library" bezeichnet werden.

Obendrein lässt das geschilderte Symptom

Sandro schrieb:
> multiple definition of `__vector_34'.

darauf schließen, daß das eine von mehreren Sourcefiles eingebundene 
Headerdatei ist.

[Wilhelm M. (wimalopaan)]

Harald K. schrieb:
> Wilhelm M. schrieb:
>> Wo steht, dass das ein Header Datei sei?
>
> Das steht nirgendswo, es ist aber zu befürchten, weil im Arduino-Umfeld
> Headerdateien als "Library" bezeichnet werden.

Wo steht Arduino?

>
> Obendrein lässt das geschilderte Symptom
>
> Sandro schrieb:
>> multiple definition of `__vector_34'.
>
> darauf schließen, daß das eine von mehreren Sourcefiles eingebundene
> Headerdatei ist.

Oder einfach die o.g. Datei als TU mehrfach gebunden wird.

[Harald K. (kirnbichler)]

Wilhelm M. schrieb:
> Oder einfach die o.g. Datei als TU mehrfach gebunden wird.

Um so einen Fehler zu begehen, braucht es deutlich mehr krimineller 
Energie.

Außerdem würde dann der Begriff

Sandro schrieb:
> Bibliotheksdatei

auch noch weniger passen.