Zwei Interrupts/Auswertungen gleichzeitig

[Lokus P. (derschatten)]

Ich möchte gerne Tasten abfragen und gleichzeitig eine LED (Punkt einer 
7-Segment) blinken lassen.
Wie macht man das am geschicktesten?

Ich habe mir dazu dieses Beispiel mal angesehen und ausprobiert:
https://www.mikrocontroller.net/articles/Multitasking
Hier wird das ganze ohne Interrupts gezeigt.

Allerdings funktioniert das bei mir irgendwie nicht:
"Funktion (void blinken(uint8_t taste)"

/************************************************************************/
/*                                    */
/*            Commodore 1541-II Bubble            */
/*                Toolbox                  */
/*                                    */
/************************************************************************/
#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/eeprom.h>
#include <util/delay.h>

// CPU: ATtiny2313-20PU 8MHz
// =======================================================================

#define LEDS_PORT    PORTB                  // 7-Segment PORT
#define TASTER_PORT    PORTA                  // Taster PORT
#define SCHALTER_PORT  PORTD                  // Schalter PORT

#define LEDS_DDR    DDRB
#define TASTER_DDR    DDRA
#define SCHALTER_DDR  DDRD

#define TASTER_PIN    PINA

#define MENU_TASTER    (1 << PA0)
#define LICHT_EIN    (PINA & 1 << PA1)


// Schaltmuster
// -----------------------------------------------------------------------
#define EINSCHALT    0x70                  // 0111 0000
#define ALL_LOW      0x00                  // 0000 0000
#define ALL_HI      0xFF                  // 1111 1111
#define RESET      0x40                  // 0100 0000
#define ID_HI      0x30                  // 0011 0000
#define ID_8      0x30                  // 0011 0000
#define ID_9      0x20                  // 0010 0000
#define ID_10      0x10                  // 0001 0000
#define ID_11      0x00                  // 0000 0000
#define WRITE      0x08                  // 0000 1000

// Tastendruck
// -----------------------------------------------------------------------
#define REPEAT_START  50                    // nach 500ms
#define REPEAT_NEXT    20                    // alle 200ms

#define DELAY      50

// Segment Anzeige
// -----------------------------------------------------------------------
#define _8        0x7F                  // 0111 1111
#define _9        0x67                  // 0110 0111
#define _10        0x3F                  // 0011 1111
#define _11        0x36                  // 0000 0110
#define _I        0x30                  // 0011 0000
#define _D        0x5E                  // 0101 1110
#define _S        0x6D                  // 0110 1101
#define _R        0x50                  // 0101 0000
#define _A        0x77                  // 0111 0111
#define _C        0x39                  // 0011 1001
#define _J        0x1E                  // 0000 1110
#define _E        0x79                  // 0111 1001
#define _H        0x80                  // 1000 0000

volatile int8_t nSelect = 0;
volatile int8_t ende;
volatile uint8_t marker;

static int8_t lichtval;
static int16_t timeCount = 0;
static int16_t lichtCount = 0;

uint8_t key_state;                        // Entprellt und invertierter Tastenstatus:
                                // bit = 1: Taste gedrückt
uint8_t key_press;                        // Tastendruck erkannt
uint8_t key_rpt;                        // Langer Tastendruck mit Wiederholung
uint8_t eeprom_floppyid EEMEM = 1;                // Speicherstelle 1 im EEPROM für Floppy-ID
uint8_t licht = 1;
uint8_t laufwerkid;
uint8_t laufwerkidled;

ISR(TIMER0_OVF_vect)
{
  static uint8_t ct0, ct1, rpt;
  uint8_t i;

  TCNT0 = (uint8_t)(int16_t)-(F_CPU / 1024 * 10e-3 + 0.5);

  i = key_state ^ ~TASTER_PIN;                // Taste gewechselt ?
  ct0 = ~(ct0 & i);                      // Reset oder zähle ct0
  ct1 = ct0 ^ (ct1 & i);                    // Reset oder zähle ct1
  i &= ct0 & ct1;                        // count until roll over ?
  key_state ^= i;                        // then toggle debounced state
  key_press |= key_state & i;                  // 0 -> 1: Tastendruck registrieren

  if((key_state & MENU_TASTER) == 0)              // Überprüfe Wiederholfunktion
    rpt = REPEAT_START;                    // Verzögerung starten
  if(--rpt == 0)
  {
    rpt = REPEAT_NEXT;                    // Wiederhole Verzögerung
    key_rpt |= key_state & MENU_TASTER;
  }

  if(timeCount > 0)                      // Timer für Menüwechsel
  {
    timeCount--;
    if(timeCount == 0)
    {
      nSelect = 0;
      ende = 1;
    }
  }

  if(lichtCount > 0)                      // Timer für Diskwechselimpuls
  {
    SCHALTER_PORT |= (WRITE);
    lichtCount--;
    if(lichtCount == 0)
      lichtval = LICHT_EIN;
  }

  if(licht == 0)                        // Schreibschutzerkennung = AUS
  {
    if(LICHT_EIN)                      // wenn schreibgeschützt (HIGH)
    {
      SCHALTER_PORT &= ~(WRITE);              // LICHTAUSGANG = LOW / Schreiben erlaubt
      marker &= ~(_H);                  // 7-Segment Punkt löschen
    }
    if(!(LICHT_EIN))                    // wenn nicht schreibgeschützt (LOW)
      licht = 1;
  }

  if(licht == 1)                        // Schreibschutzerkennung = EIN
  {
    if(LICHT_EIN)                      // wenn schreibgeschützt (HIGH)
    {
      SCHALTER_PORT |= (WRITE);              // LICHTAUSGANG = HIGH / Schreiben nicht erlaubt
      marker |= (_H);                    // 7-Segment Punkt anzeigen
    }
    if(!(LICHT_EIN))                    // wenn nicht schreibgeschützt (LOW)
    {
      SCHALTER_PORT &= ~(WRITE);              // LICHTAUSGANG = LOW / Schreiben erlaubt
      marker &= ~(_H);                  // 7-Segment Punkt löschen
    }
  }

  if(licht == 2)                        // Schreibschutzerkennung = IMMER AUS
  {
    if((LICHT_EIN != lichtval) && (lichtCount == 0))    // wenn schreibgeschützt (HIGH) und Diskwechsel erfolgt
      lichtCount = 25;                  // LICHTAUSGANG für 150ms auf HIGH

    if(LICHT_EIN == lichtval)
      SCHALTER_PORT &= ~(WRITE);              // LICHTAUSGANG = LOW / Schreiben nicht erlaubt

  }
}

uint8_t get_key_press(uint8_t key_mask)
{
  cli();                            // Lese und lösche Atomic
  key_mask &= key_press;                    // Tastenzustand lesen
  key_press ^= key_mask;                    // Tastenzustand löschen
  sei();
  return key_mask;
}

uint8_t get_key_rpt(uint8_t key_mask)
{
  cli();                            // Lese und lösche Atomic
  key_mask &= key_rpt;                    // Tastenzustand lesen
  key_rpt ^= key_mask;                    // Tastenzustand löschen
  sei();
  return key_mask;
}

uint8_t get_key_short(uint8_t key_mask)
{
  cli();                            // Tastenzustand lesen und Tastendruck Atomic
  return get_key_press(~key_state & key_mask);
}

uint8_t get_key_long(uint8_t key_mask)
{
  return get_key_press(get_key_rpt(key_mask));
}

void blinken(uint8_t taste)
{
    static uint16_t zaehler=0;

    if (taste)
  {
        if (zaehler>=999)
    {
      marker ^= (_H);
            zaehler=0;
        }
    }
    else
  {
        if (zaehler>=99)
    {
            marker ^= (_H);
            zaehler=0;
        }
    }
  _delay_ms(1);
    zaehler++;
}

void floppy_id(int8_t floppyid)
{
  SCHALTER_PORT |= (ID_HI);                  // Alle ID-PIN = HIGH  
  SCHALTER_PORT &= ~(floppyid);                // ID-PIN = LOW

  switch(floppyid)
  {
    case ID_8:
      laufwerkidled = _8;
      break;
    case ID_9:
      laufwerkidled = _9;
      break;
    case ID_10:
      laufwerkidled = _10;
      break;
    case ID_11:
      laufwerkidled = _11;
      break;
  }
}

void reset()
{
  SCHALTER_DDR |= (RESET);                  // ResetPIN = Ausgang
  SCHALTER_PORT &= ~(RESET);                  // ResetPIN = LOW
  _delay_ms(DELAY);                      // Warten
  SCHALTER_DDR &= ~(RESET);                  // ResetPIN = Eingang
  SCHALTER_PORT |= (RESET);                  // ResetPIN = PullUp
}

void eeprom_var(void)
{
  if(eeprom_read_byte(&eeprom_floppyid) != ALL_HI)
  {
    laufwerkid = eeprom_read_byte(&eeprom_floppyid);    // Floppy-ID aus EEPROM auslesen
    floppy_id(laufwerkid);
  }
}

void ioinit()
{
  LEDS_DDR = ALL_HI;                      // 7-SEGMENT-PORT = Ausgang

  SCHALTER_DDR = ALL_HI;                    // SCHALT-PORT = Ausgang
  SCHALTER_PORT = EINSCHALT;                  // EINSCHALT-ZUSTAND

  TASTER_DDR = ALL_LOW;                    // LICHTSCHRANKE = Eingang
  TASTER_PORT = ALL_HI;                    // LICHTSCHRANKE = PullUp

  TCCR0B = (1 << CS02) | (1 << CS00);              // Interrupt durch 1024 teilen
  TIMSK = 1 << TOIE0;                      // Timer Interrupt aktivieren
}

int main(void)
{
  int32_t val = 0;
  ioinit();
  reset();
  eeprom_var();

  sei();                            // Interrupts aktivieren

  for(;;)
  {
    LEDS_PORT = laufwerkidled | marker;

    if(get_key_short(MENU_TASTER))
    {
      timeCount = 300;                  // Zeitzähler auf 5sek setzen
      val = 0;                      // Tasterwert zurücksetzen
    }

    while(timeCount > 0)
    {
      if(get_key_short(MENU_TASTER))
      {
        timeCount = 300;
        val++;                      // Tasterwert um 1 erhöhen
      }

      if(nSelect == 0)                  // Haupt-Menü
      {
        if (val > 2) val = 0;
      }

      if(nSelect == 1)                  // Floppy-ID Menü
      {
        if (val > 6) val = 3;
      }

      if(nSelect == 2)                  // Schreibschutz-Menü
      {
        if (val > 9) val = 7;
      }

      switch(val)
      {
        // Hauptmenü
        case 0:
          LEDS_PORT = _I;                // I -> ID
          if(get_key_long(MENU_TASTER))
          {
            val = 3;
            nSelect = 1;
          }
          break;
        case 1:
          LEDS_PORT = _S;                // S -> Schreibschutz
          if(get_key_long(MENU_TASTER))
          {
            val = 7;
            nSelect = 2;
          }
          break;
        case 2:
          LEDS_PORT = _R;                // R -> Reset
          if(get_key_long(MENU_TASTER))
          {
            reset();
            val = 0;
            nSelect = 0;
            timeCount = 1;
          }
          break;

        // Floppy-ID
        case 3:
          LEDS_PORT = _8;                // 8 -> ID 8

          if(get_key_long(MENU_TASTER))
          {
            floppy_id(ID_8);
            eeprom_write_byte(&eeprom_floppyid, ID_8);
            reset();
            timeCount = 1;
          }
          break;
        case 4:
          LEDS_PORT = _9;                // 9 -> ID 9

          if(get_key_long(MENU_TASTER))
          {
            floppy_id(ID_9);
            eeprom_write_byte(&eeprom_floppyid, ID_9);
            reset();
            timeCount = 1;
          }
          break;
        case 5:
          LEDS_PORT = _10;              // 10 -> ID 10

          if(get_key_long(MENU_TASTER))
          {
            floppy_id(ID_10);
            eeprom_write_byte(&eeprom_floppyid, ID_10);
            reset();
            timeCount = 1;
          }
          break;
        case 6:
          LEDS_PORT = _11;              // 11 -> ID 11

          if(get_key_long(MENU_TASTER))
          {
            floppy_id(ID_11);
            eeprom_write_byte(&eeprom_floppyid, ID_11);
            reset();
            timeCount = 1;
          }
          break;

        // Schreibschutzerkennung
        case 7:
          LEDS_PORT = _A;                // A -> Aus
          if(get_key_long(MENU_TASTER))
          {
            licht = 0;
            timeCount = 1;
          }
          break;
        case 8:
          LEDS_PORT = _E;                // E -> Ein
          if(get_key_long(MENU_TASTER))
          {
            licht = 1;
            timeCount = 1;
          }
          break;
        case 9:
          LEDS_PORT = _I;                // I -> Immer
          if(get_key_long(MENU_TASTER))
          {
            lichtval = LICHT_EIN;
            licht = 2;
            blinken(LICHT_EIN);
            timeCount = 1;
          }
          break;
      }
    }
  }
}

Die LED leuchtet ständig.

Allerdings würde ich das ganze gerne mittels Interrupts machen.
Nimmt man sich dazu einen zweiten Timer oder wie geht man da vor?
Oder wie würdet ihr das machen?

[Karl M. (Gast)]

Hallo,

da fehlt die Init von Pedas Enprellroutine.

ct0 = ct1 = 0xff;

[Regelcop (Gast)]

Lokus P. schrieb:
> Oder wie würdet ihr das machen?

Ich würde mich erst mal an die Regeln halten.
Du bist lang genug hier angemeldet um das zu wissen.

>>Wichtige Regeln - erst lesen, dann posten!
>>........
>>Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang

[Karl M. (Gast)]

Hallo,

so mit if - then - else Konstrukten hast Du keine Ahnung ?

Oder warum wird mind. 2 mal der Zustand einer Variable abgefragt ?

Dann könnte man der Übersicht halber alles ist Select - Case - Endselect 
verpacken.

[Lokus P. (derschatten)]

Karl M. schrieb:

> da fehlt die Init von Pedas Enprellroutine.
>
> ct0 = ct1 = 0xff;
Danke für den Hinweis.

> Oder warum wird mind. 2 mal der Zustand einer Variable abgefragt ?
>
Wo meinst du?

[Karl M. (Gast)]

Guten Morgen Lokus Pokus,

Der Code wird durch diese Schreibweise nicht leserlicher und ist auch 
nicht optimal, in Bezug auf Speichernutzung.
Z.B.

if(LICHT_EIN) // wenn schreibgeschützt (HIGH)
  { /* .. */ }
if(!(LICHT_EIN))  // wenn nicht schreibgeschützt (LOW)
  { /* .. */ }

[Lokus P. (derschatten)]

Karl M. schrieb:
> Guten Morgen Lokus Pokus,
>
> Der Code wird durch diese Schreibweise nicht leserlicher und ist auch
> nicht optimal, in Bezug auf Speichernutzung.
> Z.B.
>
> if(LICHT_EIN) // wenn schreibgeschützt (HIGH)
>   { /* .. */ }
> if(!(LICHT_EIN))  // wenn nicht schreibgeschützt (LOW)
>   { /* .. */ }
Hallo Karl,

da hast du recht, das kann man mit einem ELSE besser lösen.
Danke.

Karl M. schrieb:
> Dann könnte man der Übersicht halber alles ist Select - Case - Endselect
> verpacken.

Was meinst du mit "alles"?

[Einer K. (Gast)]

Karl M. schrieb:
> Guten Morgen Lokus Pokus,
>
> Der Code wird durch diese Schreibweise nicht leserlicher und ist auch
> nicht optimal, in Bezug auf Speichernutzung.
> Z.B.if(LICHT_EIN) // wenn schreibgeschützt (HIGH)
>   { /* .. */ }
> if(!(LICHT_EIN))  // wenn nicht schreibgeschützt (LOW)
>   { /* .. */ }

? ? ? ?

Hier haben wir einen Programmierer, welcher das if mag.
Und gerne einsetzt.
Auch diese gerne auch 4 bis 5 Ebene tief schachtelt.
Dann ein Switch und da drin noch ein paar if.

Ganz sicher ist sowas nicht mein Stil.
Aber die Schreibweise:
if(dies) tuDas();
if(!dies) TuWasAnderes();
Mindert die Schachtelungstiefe, vermeidet zumindest das else.
Und ist von daher auch gut!

Ansonsten stelle ich den Code auf meinem Monitor recht groß ein, klarer: 
So dass ich den Text flüssig und leicht (konzentrationsschonend) lesen 
kann.

Hier geht das nicht.
Hier ist die Regel, dass ein ganzer Block, also {} und das dazwischen, 
auf eine Bildschirmseite passen muss nicht eingehalten.

Bei dieser Unübersichtlichkeit wäre mir jedes Mittel recht, welches den 
Code "flach" hält.
Mehr als recht.

[DraconiX (Gast)]

Arduino F. schrieb:
> Aber die Schreibweise:
> if(dies) tuDas();
> if(!dies) TuWasAnderes();
> Mindert die Schachtelungstiefe, vermeidet zumindest das else.
> Und ist von daher auch gut!

Nee, ist nicht gut - kostet Rechenzeit, ist unleserlich und völlig 
Sinnbefreit. Und wenn ich diese Verschachtelungen und 
Mehrfachabfragungen in der Timer ISR sehe - dann wird mir Himmelangst 
und Bange. Und das alles auf nem Tiny23 wo es eh knapp mit den Resourcen 
aussieht. Das das Ding macht was es soll, ist arg verwunderlich.

Wenn man schon ein Art Threading einsetzt, sollte dies auch strukturiert 
sein.

[Lokus P. (derschatten)]

Tipps wie man es richtig macht?
Bin für alles offen...

Und zu meinem ursprünglichen Problem wäre ich auch für Tipps dankbar.

[Peter II (Gast)]

Lokus P. schrieb:
> Tipps wie man es richtig macht?
> Bin für alles offen...

dann musst du in Vorleistung gehen und den Code zeigen.

[DraconiX (Gast)]

Lokus P. schrieb:
> Tipps wie man es richtig macht?

Willst du da meine ehrliche Meinung hören? Wenn ja: Fang neu an! Nimm 
dir ein Papier und ein Stift zur Hand, lese dir folgendes durch: 
https://de.wikipedia.org/wiki/Zustandsdiagramm_(UML) und arbeite deine 
State Machine durch.

Dann wirst du sehen das du vieles, sehr vieles in deinem Code einfach 
verkomplizierst. Das Ergebnis, solltest du es richtig machen, wird ein 
sehr effizienter Code, dein µC wird das machen was du willst und die 
Resourcen reichen für mehr.

[Einer K. (Gast)]

Lokus P. schrieb:
> Tipps wie man es richtig macht?
> Bin für alles offen...

Keine Funktion länger, als 25 Zeilen
Schachtelungstiefen über 3 werden unleserlich
Schachtelungstiefen über 7 sind böse, Übermenschlichkeit ist gefragt

Variablen heißen nicht val, auch nicht value, denn wir wissen, dass da 
ein Value drin steckt. Ein Variable heißt z.B. keyPressedCount, wenn sie 
die Anzahl Tastendrücke halten soll.



Funktionen kann man auch inline machen.
Damit erhält man die Übersichtlichkeit, vermeidet aber den 
Call-Return-Stack overhead

[Lokus P. (derschatten)]

ok, wie verhindert man Schachtelungstiefen über 3 ?

Sehr vieles kann in dem Code nicht verkompliziert sein. Der Großteil 
stammt ja von hier aus den Forumsbeispielen.

[Einer K. (Gast)]

Lokus P. schrieb:
> ok, wie verhindert man Schachtelungstiefen über 3 ?

Hmm...

Vielleicht hast du mich nicht richtig verstanden...

Mein Priorität lag eher auf:
Arduino F. schrieb:
> Keine Funktion länger, als 25 Zeilen
Damit erledigt sich das andere Problem meist gleich mit.


Also den Code in sinnvolle Teilbereiche aufteilen.
Diese Teilbereiche lassen sich dann auch gut einzeln testen.

Man muss sich ja nicht sklavisch an die 25 Zeilen halten...
Aber als Richtwert... geht das schon...

[Lokus P. (derschatten)]

Nur ich sehe bei meinem Code nicht wirklich den Sinn etwas auszulagern.
Die CASE? Die IF?

Maximal einige Teile im Interrupt.

Wozu gibt es denn die Hauptschleife, wenn man da nicht die meisten Code 
hin setzt?

Bisher leider nur sehr oberflächliche Tipps.

Und mein Hauptproblem kann ich damit auch nicht lösen.

[Einer K. (Gast)]

Lokus P. schrieb:
> Und mein Hauptproblem kann ich damit auch nicht lösen.

Dein Hauptproblem ist offensichtlich, dass dein Programm nicht tut was 
es soll. Und wenn es übersichtlich wäre, dann würdest du den Fehler 
vermutlich schnell finden. Es ist aber nicht so übersichtlich, dass du 
den Fehler schnell findest.
Und ich finde ihn auch nicht.
Mir stellen sich die Nackenhaare auf, wenn dich die Wurst sehe.

Ich wende mich ab.

[Lokus P. (derschatten)]

Das Grundprogramm funktioniert problemlos.
Lediglich das "blinken" Beispiel aus meinem ersten Beitrag funktioniert 
nicht.

Ich mach das nur als Hobby nicht beruflich, somit werde ich hier niemals 
perfekt optimierten CODE herbeizaubern können.

[Einer K. (Gast)]

Lokus P. schrieb:
> Lediglich das "blinken" Beispiel aus meinem ersten Beitrag funktioniert
> nicht.

Dann dividiere das Blinken aus der Wurst raus.
Betrachte das Blinken als einen eigenständigen endlichen Automaten.
Eine einzige globale Variable reicht. Eigentlich so gar nur ein Bit.
Ich nenne das Bit z.B. "Blinkanforderung"
Programmiere es so, dass dein Blinken "nebenläufig" arbeitet.

Teste es einzeln.

[Route_66 H. (route_66)]

Lokus P. schrieb:
> Ich mach das nur als Hobby nicht beruflich, somit werde ich hier niemals
> perfekt optimierten CODE herbeizaubern können.

Es geht doch nicht um perfekt optimierten Code, sondern um 
übersichtliche Strukturen des Programmtextes.

Zum Blinken: ich nutze gern ein Verfahren das mir erlaubt, mitten im 
Programmablauf einfach einer Bytevariable (z.B. blink) einen Wert 
zuzuweisen und die LED blinkt ganz automatisch in einem bestimmten 
Muster:
blink = 0 -> LED aus
blink= 255 -> LED dauernd an
blink= 0b11110000 -> LED blinkt langsam
blink = 0b10101010 -> LED blinkt schnell
blink = 0b10000000 -> LED blitzt regelmäßig kurz auf
uws.

Dazu wird im Timer-Interrupt die Variable blink in regelmäßigen 
Abständen durch das Carry rotiert und dieses Flag zum LED-Pin gesandt.
Ganz einfach.

[Dieter F. (Gast)]

Beitrag "Externe Signale mit AVR Schalten"

[Lokus P. (derschatten)]

Dieter F. schrieb:
> Beitrag "Externe Signale mit AVR Schalten"

Ja, was ist damit?
Hast du etwas dazu beizutragen?

[Dieter F. (Gast)]

Lokus P. schrieb:
> Ja, was ist damit?

Hilft ein wenig zu verstehen, wo Du unterwegs bist - mehr nicht. Ggf. 
könntest Du ja auch einen Schaltplan einstellen - evtl. ist ja nicht 
unbedingt (nur) die Logik ursächlich.

(Habe übrigens selbst noch einen Brotkasten - aber kein Dolphin-DOS)

Lokus P. schrieb:
> Hast du etwas dazu beizutragen?

Ja, ich würde die Variable "licht" volatile deklarieren.

Lokus P. schrieb:

 static int8_t lichtval;
 static int16_t timeCount = 0;
 static int16_t lichtCount = 0;

Und da passt auch was nicht ... die verwendest Du doch auch in der ISR - 
oder?

Lokus P. schrieb:

 TCNT0 = (uint8_t)(int16_t)-(F_CPU / 1024 * 10e-3 + 0.5);

Dieses Konstrukt muss man erst mal verstehen ..

[dummschwaetzer (Gast)]

Hallo,
mach mal bei deinen ganzen #define die Kommentare weg, z.B:
org:
#define EINSCHALT    0x70                  // 0111 0000
geändert:
#define EINSCHALT    0x70

[Lokus P. (derschatten)]

Dieter F. schrieb:
> static int8_t lichtval;
>  static int16_t timeCount = 0;
>  static int16_t lichtCount = 0;
>
> Und da passt auch was nicht ... die verwendest Du doch auch in der ISR -
> oder?

Nicht nur, auch im main.

Dieter F. schrieb:
> TCNT0 = (uint8_t)(int16_t)-(F_CPU / 1024 * 10e-3 + 0.5);
>
> Dieses Konstrukt muss man erst mal verstehen ..

Muß man das, wenns funktioniert.
Aber ich denke das kann am besten der Peter beantworten. Das stammt von 
ihm und ist Teil der Tastenentprellung: 
https://www.mikrocontroller.net/articles/Entprellung

dummschwaetzer schrieb:
> Hallo,
> mach mal bei deinen ganzen #define die Kommentare weg, z.B:
> org:
> #define EINSCHALT    0x70                  // 0111 0000
> geändert:
> #define EINSCHALT    0x70

Ähm, warum?

[dummschwaetzer (Gast)]

spiel mal präprozessor:
#define Name Ausdruck //Kommentar
im Programm:
if(Name)
wird eventuell ersetzt durch
if(Ausdruck //Kommentar)

[Lokus P. (derschatten)]

Wird das in die IF-Schleife mit einbezogen?
Welche Auswirkung hat das denn?

Ich kann hier leider nicht folgen aufgrund fehlenden Wissens.

Wie sollte man sonst mit Kommentaren umgehen?

[S. R. (svenska)]

Lokus P. schrieb:
> Wird das in die IF-Schleife mit einbezogen?

Erstens: ES GIBT KEINE IF-SCHLEIFEN.
Zweitens: Ja, der C-Präprozessor versteht kein C.

Lokus P. schrieb:
> Welche Auswirkung hat das denn?

Ein Doppelslash leitet einen Kommentar bis zum Ende der Zeile ein. Der 
frisst im Zweifelsfall keine Klammer weg und führt zu Syntaxfehlern.

Nutze einfach normale C-Kommentare,
also mit /* anfangen und mit */ aufhören.

[Dieter F. (Gast)]

Lokus P. schrieb:
> Welche Auswirkung hat das denn?

Verzeih bitte, wenn ich das krass ausdrücke:

"Ich kann nicht irgendetwas in die Gegend rotzen und hoffen, dass andere 
es schon richten."

Du wirst Dich also selbst kümmern müssen oder einen Kümmerer finden 
(gegen Entgelt), der Dir das richtet.

[A. S. (Gast)]

dummschwaetzer schrieb:
> if(Name)
> wird eventuell ersetzt durch
> if(Ausdruck //Kommentar)

Der Code ist nicht optimal. Und // ist nicht ANSI C, wird aber von den 
allermeisten Compilern unterstützt. Aber solche Kommentare erschweren es 
dem UP dann doch, nützliche Hilfestellungen von dummgeschwaetztem zu 
unterscheiden.

[Lokus P. (derschatten)]

Route 6. schrieb:

> Muster:
> blink = 0 -> LED aus
> blink= 255 -> LED dauernd an
> blink= 0b11110000 -> LED blinkt langsam
> blink = 0b10101010 -> LED blinkt schnell
> blink = 0b10000000 -> LED blitzt regelmäßig kurz auf
> uws.
>
> Dazu wird im Timer-Interrupt die Variable blink in regelmäßigen
> Abständen durch das Carry rotiert und dieses Flag zum LED-Pin gesandt.
> Ganz einfach.

Kannst du mir das genauer erläutern wie das im Interrupt aussehen muß?
Wo genau definiere ich denn dann die Blinkzeit?

Dieter F. schrieb:
> Du wirst Dich also selbst kümmern müssen oder einen Kümmerer finden
> (gegen Entgelt), der Dir das richtet.

Warum soll ich das andere machen lassen? Das macht ja Spass dran 
herumzubasteln, sonst wäre ich ja nicht hier. Ich suche mir nur Tipps 
zusammen.

[Dieter F. (Gast)]

Lokus P. schrieb:
> Ich suche mir nur Tipps
> zusammen.

Nö,

Dieter F. schrieb:
> "Ich kann nicht irgendetwas in die Gegend rotzen und hoffen, dass andere
> es schon richten."

So sieht es aus und nicht anders. Mehr sage (schreibe) ich nicht dazu.

[S. R. (svenska)]

Lokus P. schrieb:
> Kannst du mir das genauer erläutern wie das im Interrupt aussehen muß?

Nö. Dafür gibt es Tutorials.

[Lokus P. (derschatten)]

Ja? Wie nennt sich denn das "Verfahren" ?

[Zitadelle (Gast)]

Dieter F. schrieb:
> So sieht es aus und nicht anders. Mehr sage (schreibe) ich nicht dazu.

Halte Dich dann aber auch dran. Das wäre sehr hilfreich für den weiteren 
Fortgang.

[Dieter F. (Gast)]

Zitadelle schrieb:
> Halte Dich dann aber auch dran. Das wäre sehr hilfreich für den weiteren
> Fortgang.

Gäste-Kommentare blende ich dazu mal aus :-)

[Lokus P. (derschatten)]

Ich habe nun was gefunden das wunderbar funktioniert:

int16_t timerFreq = 1000; 
int16_t ledCount = 1000;
int8_t ledFreq = 10;

...
// Im Interrupt: 

  ledCount -= ledFreq;

  if (ledCount <= 0)
  {
     marker ^= (_H);
    ledCount += timerFreq;
  }

Was mir jetzt allerdings sehr seltsam vorkommt ist, dass es genau so 
funktioniert wie ich gerne hätte.
Nur warum?

Der Interrupt sollte doch ständig laufen, und die LED auch immer 
blinken.
Tatsächlich blinkt sie jedoch nur dann wenn die Variable licht=2 ist.

Warum?

Kompletter code im Anhang...

[Walter S. (avatar)]

dummschwaetzer schrieb:
> spiel mal präprozessor:
> #define Name Ausdruck //Kommentar
> im Programm:
> if(Name)
> wird eventuell ersetzt durch
> if(Ausdruck //Kommentar)

was heißt eventuell?
Bei gcc jedenfalls schon Mal nicht,
was für einen Compiler verwendest du?