Syntaxfrage Funktionsdefinition mit Optionen

[Veit D. (devil-elec)]

Hallo,

C++

Man kann doch eine Funktionen entweder erst deklarieren und später 
definieren oder gleich definieren und damit auch deklarieren. Ich hätte 
gern gewusst, wie der Syntax für eine Funktionsdefinition lauten muss 
mit diesen Optionen um sie gleich zu definieren? Ohne vorher deklarieren 
zu müssen.

void foo () __asm__("__vector_5") __attribute__((__signal__, __used__, __externally_visible__));

normalerweise irgendwie sowas, aber kompiliert nicht

void foo () __asm__("__vector_5") __attribute__((__signal__, __used__, __externally_visible__))
{
  // ... Code
}

man erhält
" an asm-specification is not allowed on a function-definition"

Der Hintergrund ist, ich möchte gern etwas Header only programmieren, 
ohne .cpp.

[Peter D. (peda)]

Die Syntax von Interrupthandlern ist je nach Target und Compiler 
verschieden.
Daher solltest Du mal beides nennen.

[Veit D. (devil-elec)]

Hallo,

Target wäre AVR ATmega 8Bit und der Compiler avr-gcc 9.3.0. Aktuell 
Atmega2560 und 328P. Mit getrennter Deklaration und Definition in .h. 
und .cpp funktioniert es. Ich hätte es gern, falls möglich, Header only.

Der Ursprung ist von hier: 
https://www.mikrocontroller.net/articles/AVR_Interrupt_Routinen_mit_C%2B%2B

[Peter D. (peda)]

Veit D. schrieb:
> Der Ursprung ist von hier:

Da wird nur erklärt, wie das Macro ISR intern arbeitet. Im Usercode 
haben die Internas nichts verloren. Es sein denn, man will den 
Unleserlichkeits-Kontest gewinnen.
ISRs werden auch nicht von extern aufgerufen, sondern von der 
Interruptlogik. Daher gehören sie nicht in ein h-File.

[Einer K. (Gast)]

Peter D. schrieb:
> Daher gehören sie nicht in ein h-File.

Mein reden seit 1840 ...

Eine ISR darf nur einmal definiert werden, darum hat sie in einer *.h 
nichts zu suchen.

Denn:
Eine *.h welche nur einmal eingebunden werden darf, ist Sinnverzerrt.

[Veit D. (devil-elec)]

Hallo,

jetzt dämmerts, habs kapiert. Danke euch.
Der Hintergrund ist folgender warum ich meine Lib auf Header only 
umbauen wollte.
Ich möchte eine vorhandene Template Klasse/Lib einbinden.
Das funktioniert aber nur wenn meine Lib Header only ist.
Weil ein Template in einer .cpp nicht gebaut werden kann.
Damit sitze ich in der Falle.
Entweder verzichte ich auf die vorhandene "Template Lib" oder ich muss 
mir Gedanken machen wie ich das mit der ISR anderweitig gelöst bekomme.

[Rolf M. (rmagnus)]

Peter D. schrieb:
> Veit D. schrieb:
>> Der Ursprung ist von hier:
>
> Da wird nur erklärt, wie das Macro ISR intern arbeitet. Im Usercode
> haben die Internas nichts verloren. Es sein denn, man will den
> Unleserlichkeits-Kontest gewinnen.

Der Code wird nicht weniger leserlich dadurch, dass er im Header steht.
Und wenn ich als Anwender wissen will, wie ich die Funktionen benutze, 
schaue ich in die Doku, nicht in den Header.

> ISRs werden auch nicht von extern aufgerufen, sondern von der
> Interruptlogik. Daher gehören sie nicht in ein h-File.

Das wiederum sehe ich auch so.

Veit D. schrieb:
> Entweder verzichte ich auf die vorhandene "Template Lib" oder ich muss
> mir Gedanken machen wie ich das mit der ISR anderweitig gelöst bekomme.

Mir ist das Problem nicht klar. Deine ISR soll ein Template sein? Das 
ergibt irgendwie keinen Sinn. Und wenn sie kein Template ist, warum muss 
sie dann im Header stehen?

[Veit D. (devil-elec)]

Hallo,

Missverständnis. Die ISR ist kein Template. Geht ja nicht.
Meine SPI Lib ist auch keine Template Klasse.
Alles funktioniert so wie gewollt.

Jetzt möchte ich zum Pin schalten die PinLib von Arduino Fanboy 
einbinden. Seine Lib ist eine Template Klasse.
Kann ich so nicht einbinden, eben weil Templates.
Darum muss meine SPI Lib auch eine Template Klasse sein - Header only.
Ich habe dazu auch schon eine neue Idee die ich im Moment umsetze.
Es geht allgemein darum wie man eine einzige ISR für mehrere Objekte 
einer Klasse verwendet. Zeige ich sobald das fertig. Wenn das nicht 
klappt, dann hat sich mein Problem hier sowieso erledigt. :-)  Dann 
bleibt alles wie es derzeit ist.

Sorry für das Durcheinander. Konnte anfangs nicht erahnen wie verzwickt 
das wird.  :-)

[Veit D. (devil-elec)]

Hallo,

habe es am laufen.  :-)
Ich habe jedoch noch einen Schönheitsfehler drin.
Und zwar muss die ISR den CS Pin wieder auf High setzen wenn alle Daten 
aus dem Buffer sind.
Derzeit setze ich von allen Instanzen den Pin auf High.
Ich würde auch hier gern über den Zeiger auf den Pin vom aktuellen 
Objekt zugreifen.

Mache ich das in einer Trockenübung klappt das. Es blinkt.

#include <CombiePin.h>
using namespace Combie::Pin;
OutputPin<13> pin;

void setInit (void) 
{            
  pin.init();
}

void setPinHigh (void) 
{            
  pin.setHigh();
} 

void setPinLow (void) 
{            
  pin.setLow();
}
   
void (*ptrFunction)() = nullptr;

void setup(void)
{
  ptrFunction = setInit; 
  (*ptrFunction)();
}

void loop(void)
{
  ptrFunction = setPinHigh;  
  (*ptrFunction)();
  delay(500);
  
  ptrFunction = setPinLow;  
  (*ptrFunction)();
  delay(500);
}

Mach ich das im Headerfile, kann ich dem Zeiger keinen Funktionnamen 
zuweisen. Auch ein verpacken in eine extra Funktion hilft nicht.

void (*ptrFunction)() = nullptr;      // function pointer
ptrFunction = slaveSelectHigh;

Was ist dabei anders? Ist das möglich? Wenn ja wie?

Fehlermeldung ist:

TwoDevices.ino:13:
DevilSpi.h: In instantiation of 'void SpiSlave<csPin, divider>::init() [with unsigned char csPin = 11; unsigned char divider = 32]':
TwoDevices.ino:38:16:   required from here
DevilSpi.h:94:19: error: cannot convert 'SpiSlave<11, 32>::slaveSelectHigh' from type 'void (SpiSlave<11, 32>::)()' to type 'void (*)()'
   94 |       ptrFunction = slaveSelectHigh;
      |       ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
DevilSpi.h: In instantiation of 'void SpiSlave<csPin, divider>::init() [with unsigned char csPin = 12; unsigned char divider = 8]':
TwoDevices.ino:39:16:   required from here
DevilSpi.h:94:19: error: cannot convert 'SpiSlave<12, 8>::slaveSelectHigh' from type 'void (SpiSlave<12, 8>::)()' to type 'void (*)()'

PS: Die messPin Objekte sind nur für den Datalogger/Oszi drin.