Flankenerkennung von Peter D.

[Login (Gast)]

Guten Abend,
ich seh den Wald vor lauter Bäumen nicht.
Ich möchte die Flankenerkennung von Peter verwenden.
Beitrag "Re: einfache und übersichtliche Flanken erkennung"
Aber ich bin zu doof dazu.

if (get_edge(PINB & (1<<0))==RISE)
  {
    uputs_("PINB0 RISE\n\r");
  }

Wenn ich diesen Pin auf 1 lege wird der Text bei jeden Aufruf 
ausgegeben.
Was mach ich falsch?
Danke für die Hilfe

[batman (Gast)]

LOGISCHES UND?

[Theor (Gast)]

Zeige mal den vollständigen Code. Der Schnipsel alleine ist OK. (Ich 
habe den Code von Peter nicht probiert, habe aber ziemlich grosses 
Vertrauen darin, dass er funktioniert).

Abgesehen davon mag das unerwartete Verhalten an elektrischen Ursachen 
liegen.

[Login (Gast)]

Zeige mal den vollständigen Code. Der Schnipsel alleine ist OK. (Ich
habe den Code von Peter nicht probiert, habe aber ziemlich grosses
Vertrauen darin, dass er funktioniert).

int main(void)
{
    DDRA |=  (1<<OUT_FAN)|(1<<OUT_GENERATOR)|(1<<OUT_PUMP)|(1<<OUT_RELAY)|(1<<OUT_LED_YELOW1)|(1<<OUT_LED_YELOW2)|(1<<OUT_RUNLED);
  DDRB |=  (1<<OUT_LED_RED)|(1<<OUT_LED_YELOW3)|(1<<OUT_LED_YELOW4);
  
  #ifdef APP_DEBUG
    fdevopen(s_sputchar, NULL);
  #endif  
  // MAIN LOOP
    while (1) 
    {
    #ifdef APP_DEBUG
      if (state != old_state)
      {
        old_state=state;
        printf("ACTUAL STATE %d\n\r",state);
      }
    #endif
    if (akt_testmode_time)
    {
      //if (get_edge(1<<0))==RISE)
      if (get_edge(PINB & (1<<IN_DIP_SW4))==LOW)
      {
        #ifdef APP_DEBUG
          printf("DIP4_R\r\n");
        #endif
      }
    }
}

Hier mein Code.
Ich traue Peter auch zu 100% und bin mir sicher das ich schuld bin

[Gert (Gast)]

Prüfe  'mal die { } Paare!

[Englisch Leera (Gast)]

Login schrieb:
> printf("ACTUAL STATE %d\n\r",state);

Für die, die im Englisch-Unterricht nicht aufgepasst haben
oder in dümmlicher Weise einfache Übersetzungen wählen wollen
(es liesse sich im Wörterbuch ja einfach nachschlagen):

"actual" bedeuted im Englischen nicht "aktuell" sondern "tatsächlich".

Wenn ein Englisch-Sprachiger "aktuell" meint sagt er "current".

In diesem programmtechnischen Zusammenhang von tatsächlichen und
nicht-tatsächlichen Status zu sprechen erscheint mir nicht sinnvoll.
Dann also: "aktuell", "current".

[Theor (Gast)]

@ Login

Abgesehen davon, das der Code unvollständig und syntaktisch inkorrekt 
ist, sehe ich darin kein Problem.

Ich empfehle, den Code auf das absolute Minimum zu reduzieren, bei dem 
der Fehler noch auftritt. Auch sollte der Code ohne Fehler und Warnungen 
compilierbar sein.

Vielleicht so:
ACHTUNG: das define muss noch vervollständigt werden.
Dazu fehlt noch die Definition von s_sputchar.

#define IN_DIP_SW4 ???
int main(void)
{
   fdevopen(s_sputchar, NULL);
   while (1) 
   {
      if (get_edge(PINB & (1<<IN_DIP_SW4))==LOW)
      {
         printf("DIP4_R\r\n");
      }
   }
}

Bitte versuche diesen Code einmal und berichte.

[W.S. (Gast)]

Login schrieb:
> Ich möchte die Flankenerkennung von Peter verwenden.
> ...
> Aber ich bin zu doof dazu.

Ich möchte dir da keineswegs widersprechen. Aber was du an Beispiel 
zeigst, ist unverständlich.

Merke: Eine Flankenerkennung soll erkennen, ob sich in einem oder 
mehreren Signalen seit der letzten Flankenerkennung etwas geändert hat.

Also etwa so:

var AlleMeineFlanken,  
    MerkeAlleMeineBits : IrgendeinpassenderTyp;

AlleMeineFlanken = AlleMeineBits XOR MerkeAlleMeineBits;
if AlleMeineFlanken <> 0
then
begin
  for every Bit in AlleMeineFlanken do
  if diesesBit=1 then behandle_es
  MerkeAlleMeineBits = AlleMeineBits;
end

kurzum, zum Erkennen von Flanken muß man den Ist-Zustand mit dem 
vorherigen Zustand vergleichen, der vor der letzten Behandlung der 
geänderten Bits war. Und wenn man die Unterschiede ermittelt hat, muß 
man den jetzt aktuellen Zustand für später speichern.

ist doch easy!

W.S.

[Login (Gast)]

Gert schrieb:
> Prüfe  'mal die { } Paare!

Ja danke beim Kürzen fürs Forum passiert,
Aber trotzdem wird die Flanke nicht erkannt.

[Gerald K. (geku)]

Login schrieb:
> Ich möchte die Flankenerkennung von Peter verwenden.

Ich würde den MACRO in eine INLINE Funktion umwanden.
Ich vermute das Problem liegt an der Parameterrückgabe.


Was gibt

printf("<%x>",get_edge(PINB & (1<<0));

aus

[Rolf M. (rmagnus)]

Gerald K. schrieb:
> Login schrieb:
>> Ich möchte die Flankenerkennung von Peter verwenden.
>
> Ich würde den MACRO in eine INLINE Funktion umwanden.
> Ich vermute das Problem liegt an der Parameterrückgabe.

Eigentlich ist das Makro so auch gar kein gültiges C. Ich nehme an, es 
wird irgendeine gcc-Erweiterung genutzt.

W.S. schrieb:
> kurzum, zum Erkennen von Flanken muß man den Ist-Zustand mit dem
> vorherigen Zustand vergleichen, der vor der letzten Behandlung der
> geänderten Bits war. Und wenn man die Unterschiede ermittelt hat, muß
> man den jetzt aktuellen Zustand für später speichern.

Du bist ganz offensichtlich dem Link nicht gefolgt, sonst hättest du 
gesehen, dass das verwendete Makro das bereits tut.

[Gerald K. (geku)]

Gerald K. schrieb:
> printf("<%x>",get_edge(PINB & (1<<0));

wird RISE  zurück geliefert?

[Theor (Gast)]

Rolf M. schrieb:
> Gerald K. schrieb:
>> Login schrieb:
>>> Ich möchte die Flankenerkennung von Peter verwenden.
>>
>> Ich würde den MACRO in eine INLINE Funktion umwanden.
>> Ich vermute das Problem liegt an der Parameterrückgabe.
>
> Eigentlich ist das Makro so auch gar kein gültiges C. Ich nehme an, es
> wird irgendeine gcc-Erweiterung genutzt.
>
> [...]

Du hast völlig recht, Rolf. 
https://gcc.gnu.org/onlinedocs/gcc/Statement-Exprs.html

[Gerald K. (geku)]

Theor schrieb:
> Du hast völlig recht, Rolf.
> https://gcc.gnu.org/onlinedocs/gcc/Statement-Exprs.html

Außerdem ist der Einsatz von Macro kein guter Prorammierstil.
Daher der Vorschlag mit den Inlinefunktionen.

Die Frage ist, ob ein Pollen von Eingangssignalen überhaupt ein 
zuverlässiger guter Lösungansatz ist. Ich würde eine Interrupt 
gesteuerte Lösung  bevorzugen.