IRMP reagiert langsam

[DerSchatten (Gast)]

Ich habe auf einer Logitech Harmony Touch die Funktionen eines SONY 
K770ES Kassettendecks programmiert.

Das Deck nutzt das Sony SIRCS Protokoll.

Damit wollte ich nun ein Nakamichi Dragon Kassettendeck steuern das nur 
eine Kabelgebundene Fernbedienung zulässt.

Dazu habe ich mir folgendes Programm geschrieben:

/* IR Receiver:  TSOP34836            
CPU: ATtiny44A-PU @ 8 MHz internal RC        
Fuses: l:0xE2 h:0xDB e:0xFF
*/

#include "irmp.h"
#include <util/delay.h>
#include "avr/eeprom.h"

#define LED_PORT     PORTB
#define LED_DDR      DDRB

#define OUT_PORT     PORTA
#define OUT_DDR      DDRA
#define OUT_PIN1      0
#define OUT_PIN2     1
#define OUT_PIN3      2
#define OUT_PIN4     3

// Schaltmuster
// -----------------------------------------------------------------------
#define ALL_HI      0xFF
#define ALL_LOW      0x00
#define LED      0x02

// Ausgabe
// -----------------------------------------------------------------------
#define REW      0x01
#define REC      0x02
#define FREW      0x04
#define FFORW      0x08
#define FORW      0x10
#define PAUSE      0x20
#define STOP      0x40

// EEPROM Variablen definieren
// -----------------------------------------------------------------------
uint16_t eeprom_speicherstelle EEMEM = 1;
uint16_t ir_command;

void timer_init(void)
{
  OCR1A = (F_CPU / F_INTERRUPTS) - 1;              // Vergleichswert: 1/15000 der CPU-Frequenz
  TCCR1B = (1 << WGM12) | (1 << CS10);            // schalte CTC-Modus ein, setze Vorteiler auf 1
  TIMSK1 = 1 << OCIE1A;                    // OCIE1A: Interrupt mittels Timer-Vergleich
}

ISR(TIM1_COMPA_vect)                      // Timer1 aufruf alle 1/15000 sek
{
  (void) irmp_ISR();                      // irmp ISR aufrufen
}

void led_callback(uint8_t on)
{
  if(on)
  {
    LED_PORT |= (LED);
  }
  else
  {
    LED_PORT = ALL_LOW;
  }
}

void ioinit()
{
  OUT_DDR = ALL_HI;                      // SCHALTER = Ausgang
  OUT_PORT = ALL_LOW;                      // EINSCHALT-ZUSTAND

  #if IRMP_USE_CALLBACK == 1
    LED_DDR |= (LED);                    // SignalLED = Ausgang
    LED_PORT = ALL_LOW;                    // EINSCHALT-ZUSTAND
    irmp_set_callback_ptr (led_callback);          // Aktivität anzeigen
  #endif
}

int main(void)
{
  IRMP_DATA irmp_data;

  ioinit();                          // Ein/Ausgänge initialisieren
  irmp_init();                        // IRMP initialisieren
  timer_init();                        // Timer initialisieren

  sei ();                            // Interrupts aktivieren

  for (;;)
  {
    if (irmp_get_data (&irmp_data) && (irmp_data.protocol == IRMP_SIRCS_PROTOCOL) && (irmp_data.address == 0x0000))
    {
      // wenn ein IR-Signal erkannt wird, dann...
      // irmp_data.protocol ist das protokoll, siehe irmp.h
      // irmp_data.address ist die Adresse / Hersteller Code des IR Sender
      // irmp_data.command ist der Befehlscode
      // irmp_protocol_names[irmp_data.protocol] ist der Protokollname (wenn aktiviert, siehe irmpconfig.h)

      #if IR_TO_EEPROM == 1                // IR-Code in EEPROM speichern
        ir_command = irmp_data.command;
        eeprom_write_word(&eeprom_speicherstelle, irmp_data.command);
      #endif
      
      if (irmp_data.flags & IRMP_FLAG_REPETITION)
        {
        switch (irmp_data.command)
        {
          case 0x0832: OUT_PORT = FORW;  break;    // Vorlauf
          case 0x0837: OUT_PORT = REW;  break;    // Rücklauf
          case 0x0834: OUT_PORT = FFORW;  break;    // Vorspulen
          case 0x0833: OUT_PORT = FREW;  break;    // Zurückspulen
          case 0x0838: OUT_PORT = STOP;  break;    // Stop
          case 0x0839: OUT_PORT = PAUSE;  break;    // Pause
          case 0x071F: OUT_PORT = REC;  break;    // Aufnahme + Pause
        }
        _delay_ms(500);
        OUT_PORT = ALL_LOW;
      }
    }
  }
}

Das ganze reagiert nur etwas träge, sag ich jetzt mal.
Wenn ich nur kurz eine Taste antippe, sehe ich zwar das gesendet wird, 
die Schaltung reagiert jedoch noch nicht.
Erst wenn ich etwas länger auf der Taste bleibe reagiert sie.

Woran könnte das liegen?

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

DerSchatten schrieb:
> if (irmp_data.flags & IRMP_FLAG_REPETITION)

Lass da mal das Flag weg. Damit wartest du nämlich darauf, das die FB 
bereits den Wiederholungcode sendet und damit wird das erste Kammando 
verschluckt. Ich mach das so:

    if (irmp_get_data (&irmp_data))
    {
    if ((irmp_data.protocol == IRMP_SIRCS_PROTOCOL) && (irmp_data.address == 0x0000)) 
      {
      switch (irmp_data.command) {
     
       case    0x0832  :OUT_PORT = FORW; break;
// und so weiter.

[Oliver R. (superberti)]

Hi,

durch die Zeile

if (irmp_data.flags & IRMP_FLAG_REPETITION)
...

beachtest Du ja auch nur länger gedrückte Kommandos. Das erste erkannte 
IR-Kommando hat immer das Repetition-Flag NICHT gesetzt, erst die 
nachfolgenden bei dauerhaft gedrückter Taste.

Gruß,
Oliver

[DerSchatten (Gast)]

Spitze! Danke! So geht's schon viel besser.

[Frank M. (ukw) (Moderator)]

DerSchatten schrieb:
> if (irmp_data.flags & IRMP_FLAG_REPETITION)

Das ist so nicht richtig.

Du meinst hier wohl eher

if (! (irmp_data.flags & IRMP_FLAG_REPETITION))

Damit beachtest Du den ersten Frame und ignorierst die 
Wiederholungsframes bei lang gedrückter Taste. So ist das im 
IRMP-Artikel auch dokumentiert ("Entprellen" von Tasten).

Wenn Du die Wiederholungsframes gar nicht ignorieren willst, dann lass 
das if weg und führe das darunter stehende immer aus. Ist hier aber wohl 
nicht nötig, da Du keinen VOLUME-Befehl nutzt, um die Lautstärke zu 
regeln.

>         _delay_ms(500);

Damit bremst Du den µC eine halbe Sekunde. Du setzt den Pin am Port erst 
eine halbe Sekunde nach dem Empfang des IR-Befehls. Und Du wunderst Dich 
dann, dass das Programm "träge" reagiert? Mach das weg. Gerade dann, 
wenn Wiederholungsframes ignoriert werden sollen, ist das unsinnig.

IRMP kannst Du mit dem Delay-Befehl sowieso nicht daran hindern, 
mindestens einen weiteren Frame im Hintergrund einzusammeln. Korrigiere 
das oben stehende if wie oben angegeben und lösche den Delay-Befehl.

[DerSchatten (Gast)]

ok, das

if (! (irmp_data.flags & IRMP_FLAG_REPETITION))

habe ich jetzt wieder rein gepackt.

Das delay soll den Ausgang nach kurzer Zeit wieder auf HI setzen.
Der Ausgang soll ja nur ein mal kurz geschalten werden. Eben wie bei 
einer Fernbedienung.

Gibt's denn da einen anderen Weg?

[Frank M. (ukw) (Moderator)]

DerSchatten schrieb:
> Das delay soll den Ausgang nach kurzer Zeit wieder auf HI setzen.

Achso, dann habe ich es jetzt kapiert.

> Gibt's denn da einen anderen Weg?

Wenn Dein µC innerhalb dieser halben Sekunde sowieso auf nichts anderes 
reagieren soll, dann ist es okay.

Sonst könntest Du eine Countdown-Variable auf 500 setzen, in der ISR 
herunterzählen und wenn diese auf 0 gelaufen ist, in der Hauptschleife 
den Port wieder resetten. Dann hätte die Hauptschleife während dieser 
500 msec auch noch Zeit für andere Dinge. Ist aber wohl für Deine 
Aufgabenstellung oversized.

Trotzdem solltest Du das im Hinterkopf behalten, wenn Du mal ein 
anspruchsvolleres Projekt angehen möchtest.