Interrupt-Abfrage >>> Routine vereinfachen

[Ceos]

Sorry das ist etwas zu kurzsilbig, ich versteh der Satz gerade nicht mal

Der ISR und auch das PIN Register funktionieren unabhängig vom DDR Register!

Observe that, if enabled, theinterrupts will trigger even if the INT0 or PCINT[5:0] pins are configured as outputs

du musst den Interrupt also schon aus/blindschalten und dann den Pin über das PORT Register entsprechend setze.

Der Taster an einem auf output geschalteten Pin bedeutet natürlich auch ein Kurzschluss, ist also generell nicht empfehlenswert!

[frabe]

@ceos
Bei meinem Timer-Interrupt werden innerhalb des ISR die Eingänge PA4, PA5 abgefragt.
Im main() werden für gewöhnlich die DDR einmalig als Ein- oder Ausgänge konfiguriert.
Nun möchte ich aber innerhalb der while(1) jeden einzelnen Eingang nach Bedarf "Stumm" schalten, so dass im ISR, if((PINA & (1<<PINA5)) != 0) //1=tu was, ins "leere" läuft.
Daher meine Idee, den entsprechenden Pin, "mal eben" als Ausgang zu konfigurieren.

@021aet04
5V==High==1
Kurzschluss auf der Ausgangsleitung? Wenn der uC/Out==low==0, uC-intern direkt mit dem Gnd verbunden ist, ist das tatsächlich möglich.

[Ceos]

wie aus der Doku zitiert, das geht leider nciht, du kannst also nur den
PCMSK0 umkonfigurieren um deine ISR zu blockieren. Zusätzlich solltest du alelrdings in deinen Abfragen auch den Pin ausmaskieren, sonst löst er trotzdem aus, wenn du beide Taster drückst.

Du musst also alles maskieren

[frabe]

PCMSK0 umkonfigurieren um deine ISR zu blockieren.

PCMSK0 = (1<<PCINT4)|(1<<PCINT5);
konfiguriere ich nur beim PortChange-Interrupt, nicht aber beim Timer-Interrupt.

[021aet04]

Das 5V ein High ist, ist normal. Die Frage ist, wie hast du den Taster angeschlossen (gegen 5V oder Masse, Verwendest du einen Widerstand)?

Normalerweise gibt es kein Problem, solltest du aber versehentlich den Port schalten und du die Taste dürcken, hast du ein Problem (z.B. Ausgang auf high und Taste auf Masse). Es reicht schon wenn du dich verschreibst.

MfG Hannes

[frabe]

Du hast natürlich Recht - in diesem Fall kann nichts passieren - man/ich sollte aber "Narrensicher" entwickeln, könnte ja mal in andere Hände gelangen.
Mittlerweile halte ich die Lösung mit der Ein/Ausgangs-Umschaltung für praxisfern.

Vielleicht ist eine andere Variante mit dem "Stummschalten" durch volatile-Variablen sinnvoll.
Bsp:
Im main(), sequneziell PA4 und/oder PA5; Variable "PA4freischalt=1;", "PA5freischalt=1;"
Innerhalb des ISR (Timer) oder dessen Funktionsaufruf; if (PA4freischalt == 1) {} // Eingangsimpuls wird weiter verarbeitet

[021aet04]

Das ist die Variante die ich gemeint habe. Ich würde aber eine Bytevariable nehmen und einzelne Bits einem Eingang zuordnen. Bei dir ist es "Resourcenverschwendung" weil du für jeden Eingang eine Bytevariable verwendest.

Also z.B. eine Hilfsvariable "char hilfsvariable" => Bit 0 = Freigabe PA4, Bit 1 = Freigabe PA5
Der Code sieht dann so aus

if ((PINA & (1<<PA4)) && (hilfsvariable & (1<<0))){...}

Das gleiche gilt auch für PA5. Die Schleife wird nur ausgeführt wenn PA4 == 1 und hilfsvariable == 1.

MfG Hannes