[PIC] Problem mit Interrupt

[Andreas R. (blackpuma)]

Guten Morgen!

Ich habe hier ein Problem auf einem PIC mit dem Interrupt on Change am 
PortB. Ein Drehimpulsgeber ist wie am Bild angeschlossen.

Wenn ich am Drehimpulsgeber drehe funktioniert mein Programm. Wenn ich 
aber auf den Drehimpulsgeber drücke dann Schaltet er zwischen 10 und 100 
Hz um. Nur das er zusätzlich noch die frequenz (frequ) erhöht. Ich finde 
aber nicht heraus warum.

#pragma interruptlow high_isr
void high_isr( void )
{
  char hdt[] = "100";
  char zen[] = " 10";

  // Am Drehimpulsgeber wurde gedreht.
  if( INTCONbits.RBIF && PORTBbits.RB5 )    // Interrupt on Change ist aufgetreten
  {
    if( PORTBbits.RB4 == PORTBbits.RB3 )
    {
      frequ += add_frequ;
    }
    else
    {
      frequ -= add_frequ;
    }
    set_new_frequ();
  }
  // Der Drehimpulsgeber wurde gedrückt
  else if( INTCONbits.RBIF && !PORTBbits.RB5 )    // Interrupt on Change ist aufgetreten
  {
    if( stat == 0x00 )
      stat = 0x01;
    else
      stat = 0x00;
    
    if( add_frequ == 10 )
    {
      add_frequ = 100;
      SetDDRamAddr( 0x4C );
      while( BusyXLCD() );
      putsXLCD( hdt );
      while( BusyXLCD() );    
    }
    else if( add_frequ == 100 )
    {
      add_frequ = 10;  
      SetDDRamAddr( 0x4C );
      while( BusyXLCD() );
      putsXLCD( zen );
      while( BusyXLCD() );    
    }

    while( !PORTBbits.RB5 ); // Warten bis der Impulsgeber wieder los gelassen wird.
  }

  INTCONbits.RBIF = 0;  // Interruptflag löschen
  return;
}

Ich hoffe ihr seht da was. Wenn ich mehr vom Code braucht sagt es mir.

BG
Andreas

[Peter D. (pdiener)]

while( !PORTBbits.RB5 ); // Warten bis der Impulsgeber wieder los gelassen wird.

Das hat zwar mit dem eigentlichen Problem nichts zu tun, aber es ist 
generell keine so gute Idee, im Interrupt auf Benutzereingaben zu 
warten...

 // Am Drehimpulsgeber wurde gedreht.
  if( INTCONbits.RBIF && PORTBbits.RB5 )    // Interrupt on Change ist aufgetreten

Das verstehe ich auch nicht ganz. RB5 ist doch der Taster. Was hat das 
mit dem Drehen zu tun?


Grüße,

Peter

[morph1 (Gast)]

er lässt es nur zu, dass die frequenz verändert wird wenn der taster 
nicht gedrückt ist :)

aber für die leserlichkeit:

#define TASTER PORTBbits.RB5

dann musst du nur mehr TASTER schreiben, geht klarerweise auch für 
anderes :)

[Peter D. (pdiener)]

Nein, ich denke es soll so sein:

 // Am Drehimpulsgeber wurde gedreht.
  if( INTCONbits.RBIF && PORTBbits.RB4 )    // Interrupt on Change ist aufgetreten

[Peter D. (pdiener)]

Der Fehler passiert beim Loslassen vom Taster. Das erzeugt auch einen 
Pinchange-Interrupt.
if( INTCONbits.RBIF && PORTBbits.RB5 )
ist dann einmal erfüllt und
if( PORTBbits.RB4 == PORTBbits.RB3 )
ist auch erfüllt, die Frequenz wird also um einen Schritt erhöht beim 
Loslassen des Tasters.

[Andreas R. (blackpuma)]

Peter Diener schrieb:
> Der Fehler passiert beim Loslassen vom Taster. Das erzeugt auch einen
> Pinchange-Interrupt.
> if( INTCONbits.RBIF && PORTBbits.RB5 )
> ist dann einmal erfüllt und
> if( PORTBbits.RB4 == PORTBbits.RB3 )
> ist auch erfüllt, die Frequenz wird also um einen Schritt erhöht beim
> Loslassen des Tasters.

Das Interruptflag wird aber erst am Ende der Interruptroutine wieder 
gelöscht. Kann dann überhaupt noch ein Interrupt ausgelöst werden?

Ich habe jetzt mal versucht beim Einstieg in die Routine den Interrupt 
on Change auszuschalten und erst am ende der Routine wieder 
einzuschalten. Problem bleibt aber bestehen. :-(

[Peter D. (pdiener)]

>Kann dann überhaupt noch ein Interrupt ausgelöst werden?

Ja,  normalerweise wird ein PinchangeInterrupt bei beiden Flanken 
ausgelöst.

Es ist schon klar, dass er nicht nochmal kommt durch Drücken während der 
Interrupt noch läuft. Aber durch Loslassen triggert er nochmal.

[Andreas R. (blackpuma)]

Peter Diener schrieb:
>>Kann dann überhaupt noch ein Interrupt ausgelöst werden?
>
> Ja,  normalerweise wird ein PinchangeInterrupt bei beiden Flanken
> ausgelöst.
>
> Es ist schon klar, dass er nicht nochmal kommt durch Drücken während der
> Interrupt noch läuft. Aber durch Loslassen triggert er nochmal.

Aber wenn ich beim Einstieg in die Routine den Interrupt on Change 
ausschalte und ich dann die Taste los lasse kann er nicht nochmal einen 
Auslösen oder? Ist ja deaktiviert! Und bei Sprut habe ich noch gelesen 
das GIE beim einstieg in die Interruptroutine auf 0 gesetzt wird. Also 
sind die Interrupts während eines Interrupts überhaupt aus.

[Peter D. (pdiener)]

Ich würde es mal so versuchen:

// Am Drehimpulsgeber wurde gedreht.
if( INTCONbits.RBIF && ((!PORTBbits.RB4) || (!PORTBbits.RB3)) )    // Interrupt on Change ist aufgetreten

[Andreas R. (blackpuma)]

Kein Erfolg.

[Stefan Kunz (Gast)]

Der Interrupt wird nochmal ausgelößt, da die Interrupt um einiges 
schneller ausgeführt wird als der Mensch ihn wieder loslassen kann.

[Stefan Kunz (Gast)]

wenn du wirklich darauf warten willst, dass die Taste losgelassen wird 
in der Interruptroutine dann solltest du

 while( !PORTBbits.RB5 ); // Warten bis der Impulsgeber wieder los 
gelassen wird.

in

 while( PORTBbits.RB5 ); // Warten bis der Impulsgeber wieder los 
gelassen wird.

ändern, da dein Taster den Pin sicher gegen GND zieht und wenn er 
losgelassen wurde gegen VCC.

Davon ab ist dies aber eine DEFENETIV schlechte lösung. Verfolge lieber 
die Anzahl der Interrupts und ihre Gründe, so das du den Interrupt 
unterscheiden kannst.

[Peter D. (pdiener)]

Nein,
while( !PORTBbits.RB5 );
ist schon richtig um auf das Loslassen zu warten.

Der Punkt ist ein ganz anderer. Der Taster prellt.
Und das eventuell so schnell, dass er einen Interrupt auslöst und in 
diesem dann an verschiedenen Stellen verschiedene Werte liefert.

Das Konzept ist wie gesagt eh nicht gut mit dem Warten im Interrupt.

Ich würde einen Timerinterrupt aufsetzen, dort regelmäßig alle Kontakte 
prüfen und entprellen und dann damit weiterarbeiten.

[Peter D. (pdiener)]

Hier gibt es ein paar gute Anregungen:
http://www.mikrocontroller.net/articles/Drehgeber