Treppenstufenbeluchtung - Code Umbauen - Hilfe

[Scubydoo (Gast)]

Hallo zusammen,

ich habe in einem anderen Forum folgenden Code für eine 
Treppenstufenbeleuchtung gefunden. Dieser Code funktioniert soweit 
einwandfrei.

Auf der Platine sind Jumper zur Vorwahl der Anzahl an LEDs vorhanden. 
Die LEDs werden durch 2 Lichtschranken als INT0 und INT1 gesteuert.

Nun möchte ich dieses Script gerne verändern: Der Jumper soll durch 
einen Schalter ersetzt werden mit dem ich alle LEDs einschalten kann. So 
möchte ich quasi ein Dauerlicht erzeugen.

Es wäre nett, wenn mir einer bei der Umsetzung helfen könnte.

Ich muß ja quasi den Interrupt sperren und alle Ausgänge auf 1 setzen.

Habe schon ein bisschen probiert, aber irgendwie hat es nicht 
funktioniert.


Vielen Dank für eure Hilfe


Gruß

Scubydoo

Hier der Quelltext

/*
 * main.c
 *
 *  Created on: 14.10.2010
 *  (C) 2010 D.Goersch
 */


#include <avr/io.h>
#include <avr/pgmspace.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include <stdint.h>
#include "soft-pwm.h"

#define DELAY_FADE 11       // wait XX ms between each fade step
#define DELAY_LED 20      // wait XX ms between each LED
#define DURATION_SHORT 15    // autofade off after XX sec if jumper is open (high)
#define DURATION_LONG 25    // autofade off after XX sec if jumper is closed (low)
#define LTR 0          // Fade fom PortA0 forwards
#define RTL 1          // Fade from the last (amount) to PortA0 backwards
#define OFF 0          // State of
#define ON 1          // the LEDs

uint8_t amount=0;          // Amount of LEDs
uint8_t time;          // Fade-Off time
uint8_t direction;        // 0 - LTR 1 - RTL
uint8_t state=OFF;        // 0 - OFF 1 - ON
volatile uint8_t int0=0, int1=0;// flags for interrupts
volatile uint8_t tenms=0;     // +1 each 10ms
volatile uint8_t running=0;    // +1 each second

// dimming steps
uint8_t pwmtable[32] PROGMEM = {0, 1, 2, 2, 2, 3, 3, 4, 5, 6, 7, 8, 10, 11,
                                13, 16, 19, 23, 27, 32, 38, 45, 54, 64, 76,
                                91, 108, 128, 152, 181, 215, 255};


// patchtable for outputs
// uint8_t patchtable[24] PROGMEM = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9,
//                                  10, 11, 12, 13, 14, 15, 16,
//                                  17, 18, 19, 20, 21, 22, 23};

// patchtable for outputs (u.wehner)
uint8_t patchtable[24] PROGMEM = {0, 1, 2, 3, 4, 5, 6, 7, 23, 22,
          21, 20, 19, 18, 17, 16, 8, 9,
                                  10, 11, 12, 13, 14, 15};


void init(void);
void fade_on(uint8_t direction);
void fade_off(uint8_t direction);

int main(void)
{
  init();

  for(;;)
  {

    if(state==OFF)
    {
      if(int0 || int1) amount = ~((PIND & 0x03) | ((PIND >>2) & 0x1C)) & 0x1f;    // read PD0,1,4,5,6 as a dec. number
      if(int0)                  // if LEDs off, turn on LTR
      {
        state = ON;
        direction = LTR;
        fade_on(direction);
        int0 = 0;

        TCNT0 = 100;              // Preload
        TCCR0 |= (1<<CS00) | (1<<CS02);      // Timer0 prescaler 1024
      }
      else if(int1)                // LEDs off, turn on RTL
      {
        state = ON;
        direction = RTL;
        fade_on(direction);
        int1 = 0;

        TCNT0 = 100;              // Preload
        TCCR0 |= (1<<CS00) | (1<<CS02);      // Timer0 prescaler 1024
      }
    }
    else if(state==ON)
    {
      if((!time & (running >= DURATION_SHORT)) | (time & (running >= DURATION_LONG))) {  // turn of automaticly
        TCCR0 = 0;                // disable timer0
        tenms = 0; running = 0;          // reset turnoff-timer
        fade_off(direction);
        state = OFF;

      }
      if(int0 & (direction==LTR)) int0=0;      // LEDs LTR on, left switch ignored
      else if(int1 & (direction==RTL)) int1=0;  // LEDS RTL on, right switch ignored
      else if(int0 & (direction==RTL))      // LEDs RTL on, fade off on left switch
      {
        TCCR0 = 0;                // disable timer0
        tenms = 0; running = 0;          // reset turnoff-timer
        fade_off(direction);
        state = OFF;
        int0 = 0;
      }
      else if(int1 & (direction==LTR))      // LEDs LTR on, fade off on right switch
      {
        TCCR0 = 0;                // disable timer0
        tenms = 0; running = 0;          // reset turnoff-timer
        fade_off(direction);
        state = OFF;
        int1 = 0;
      }
    }
  }
}

// do basic initialisation
void init(void)
{
  DDRA = 0xff;  // PortA
  DDRB = 0xff;  // PortB
  DDRC = 0xff;  // PortC as Output
  DDRD = 0x00;  // PortD as Input
  PORTD = 0xff;  // PullUps on PortD

//  moved to mainloop
//  amount = ~((PIND & 0x03) | ((PIND >>2) & 0x1C)) & 0x1f;    // read PD0,1,4,5,6 as a dec. number
  time   = PIND & 0x20;                    // read PinD5 as fadeout mode

//  activated in mainloop after LEDs on
//  TCCR0 |= (1<<CS00) | (1<<CS02);  // Timer0 prescaler 1024
  TCNT0 = 100;          // Preload
  TIMSK |= (1<<OCIE0);      // enable Timer0

  TCCR1B = 1;               // Timer1 runs full system clock
    TIMSK |= (1<<OCIE1A);     // enable Timer1

    MCUCR |= (1<<ISC01);    // INT0 on falling edge
    MCUCR |= (1<<ISC11);    // INT1 on falling edge
    GICR |= (1<<INT0);      // enable INT0
    GICR |= (1<<INT1);      // enable INT1

    sei();
}

// fade the LEDs on
void fade_on(uint8_t direction)
{
  if (direction == LTR)
  {
    for(uint8_t led=0;led<amount;led++)
    {
      for(uint8_t bri=0;bri<32;bri++)
      {
        pwm_setting[pgm_read_byte(patchtable+led)] = pgm_read_byte(pwmtable+bri);
//        pwm_setting[patchtable[led]] = pwmtable[bri];
        _delay_ms(DELAY_FADE);
      }
      _delay_ms(DELAY_LED);
    }
  }
  else
  {
    for(uint8_t led=amount;led>0;led--)
    {
      for(uint8_t bri=0;bri<32;bri++)
      {
        pwm_setting[pgm_read_byte(patchtable+(led-1))] = pgm_read_byte(pwmtable+bri);
        _delay_ms(DELAY_FADE);
      }
      _delay_ms(DELAY_LED);
    }
  }
}

// fade the LEDs off
void fade_off(uint8_t direction)
{
  if (direction == LTR)
  {
    for(uint8_t led=0;led<amount;led++)
    {
      for(uint8_t bri=32;bri>0;bri--)
      {
        pwm_setting[pgm_read_byte(patchtable+led)] = pgm_read_byte(pwmtable+bri-1);
        _delay_ms(DELAY_FADE);
      }
      _delay_ms(DELAY_LED);
    }
  }
  else
  {
    for(uint8_t led=amount;led>0;led--)
    {
      for(uint8_t bri=32;bri>0;bri--)
      {
        pwm_setting[pgm_read_byte(patchtable+(led-1))] = pgm_read_byte(pwmtable+bri-1);
        _delay_ms(DELAY_FADE);
      }
      _delay_ms(DELAY_LED);
    }
  }
}

// set flags on interrupts
ISR(INT0_vect) { int0 = 1; }
ISR(INT1_vect) { int1 = 1; }

// timer for auto-off
ISR(TIMER0_COMP_vect)
{
  tenms++;
  if (tenms >= 100)
  {
    running++;
    tenms = 0;
  }
}

[Wegstaben V. (wegstabenverbuchsler)]

Scubydoo schrieb:
> Habe schon ein bisschen probiert, aber irgendwie hat es nicht
> funktioniert.

was (wo) ist "ein bichen", und was ist "irgendwie"?

Solche Aussagen zwecks Fehleranalyse und Unterstützungshilfe sind so 
sinnvoll wie mein Tip: "Du must irgendwie irgendwas an dem Programmcode 
ändern, um die gewünschte Funktion zu erhalten."

[Jonas B. (jibi)]

Ist das der verfuschte Code oder das Original?

Gruß Jonas

[Sven W. (Firma: basement industries) (dj8nw)]

#include "soft-pwm.h"

^...geht das?

[Krapao (Gast)]

Wenn du die Jumper für die Anzahl der LEDs anders benutzt und die 
Lichtschranken nicht mehr benutzt, was bleibt dann vom Originalprogramm?
Wäre es da nicht sinnvoller ein eigenes Programm zu schreiben?
Braucht man dafür überhaupt einen µC oder reicht eine AN/AUS Schaltung?

Beim vorhandnen Programm solltest du Dauerlicht bekommen, wenn

* die Lichtschranke fürs Einschalten mit dem Schalter so eingestellt 
wird, dass immer der betreffende Interrupt ausgelöst wird (ODER, 
Schalter parallel)

* und die Lichtschranke für das Ausschalten so eingestellt wird, dass 
nie ein Interrupt ausgelöst wird (UND, Schalter in Serie)

* das time auto-off im Code auskommentiert wird (z.B. running nicht 
hochzählen).

Schaltplan und Beschreibng der Originalschaltung wäre nicht schlecht.

[Daniel B. (scubydoo)]

Hallo,

danke für die erste Hilfestellung:

1) Ja, das Programm ist original und nicht "verpfuscht"

2) Schaltplan und die Include Dateien habe ich mal angehängt

3) Ich habe mir das so vorgestellt, dass bei geöffnetem Schalter die 
LEDs nacheinander an-/bzw. ausgehen. So wie es zur Zeit läuft. Lege ich 
den Schalter um, bleiben alle LEDs an. Und das unabhängig ob jemand 
durch die Lichtschranken läuft (INT0/INT1) oder nicht.

Auskommentieren ist also nicht!

4.) Versucht habe ich folgendes:
In Zeile 63: if(int0 || int1) amount = ~((PIND & 0x03) | ((PIND >>2) & 
0x1C)) & 0x1f;    // read PD0,1,4,5,6 as a dec. number

steht meiner Meinung nach die Abfrage der Jumper. Das habe ich über die 
For Schleife kopiert und nur nach z.B. PIND7 abgefragt. Dann wollte ich 
mit  PORTA = 0xFF; alle Ausgänge setzen.

Aber so ganz hat das nicht funktioniert.

Nun die Frage: Was muß ich wo einfügen um meine Idee umsetzen zu können.


Danke und Gruß

Scubydoo

[Krapao (Gast)]

> 2) Schaltplan und die Include Dateien habe ich mal angehängt

Welche Lichtschranken hängen wie an IN 1 und IN 2?
Wie hast du den/die Schalter derzeit angeschlossen?

> Auskommentieren ist also nicht!

Dann mach diesen time auto-off Teil und und die Reaktion aut int0/int1 
von einem Pinzustand abhängig. Hast du noch einen freien Pin am AVR, 
z.B. einen auf JP1, den du mit dem Schalter füttern kannst?

[guest (Gast)]

Tippfehler suchen macht besoffen doppelt so viel Spaß!

[Daniel B. (scubydoo)]

Die Lichtschranken hängen per Masse an IN1. Sind NPN-Lichtschranken

Schalter habe ich im Moment keinen dran. Der Versuch ist ja gescheitert!

Jumper ist noch frei. Die Anzahl der LEDs braucht nicht variabel zu 
sein. Daher kann ich PD7 für einen Taster nehmen

Wo muß das im Code implementiert werden?

[Krapao (Gast)]

> Daher kann ich PD7 für einen Taster nehmen

Wie jetzt, Taster oder Jumper/Schalter?

Hier die Fassung mit Jumper/Schalter, d.h. der Schaltzustand braucht die 
Software nicht zu archivieren, sondern kann stets das Bauteil abfragen.

uint8_t dauerlicht(void)
{
  DDRD &= ~(1<<PD7); // PD7 Eingang
  PORTD |= (1<<PD7); // interner Pullup an
  asm volatile("nop");
  return (PIND & (1<<PD7)) ^ (1<<PD7); // active-low Jumper/Schalter
}

int main(void)
{
  init();

  // Abfrage der Anzahl LEDs hochgezogen 
  // (Ändert sich im Betrieb ja wohl nicht, oder)?
  // read PD0,1,4,5,6 as a dec. number
  amount = ~((PIND & 0x03)|((PIND >>2) & 0x1C)) & 0x1f;    

  for(;;)
  {
    if (dauerlicht() && state==OFF)
    {
      state = ON;
      direction = LTR; // je nach Gusto
      fade_on(direction);
      TCNT0 = 100;
      TCCR0 |= (1<<CS00) | (1<<CS02); // Timer0 prescaler 1024
    }
    else if (dauerlicht() && state==ON)
    {
      // Auto-off auf z.B. 3 Sekunden setzen für den Fall,
      // dass Dauerlicht ausgeschaltet wird
      running = time ? DURATION_LONG-3 : DURATION_SHORT-3;
    }
    else if(state==OFF)// && !dauerlicht()
    {
      ... wie vorher aber Fehler &, | zu &&, || korrigieren!
    }
    else if(state==ON) // && !dauerlicht()
    {
      ... wie vorher aber Fehler &, | zu &&, || korrigieren!
    }
  }
}

[Daniel B. (scubydoo)]

Hallo Krapao,

vielen vielen Dank für Deine Hilfe.
Es wird ein Schalter. Sorry für die Verwirrung!


Diesen Absatz verstehe ich leider nicht

else if(state==OFF)// && !dauerlicht()
    {
      ... wie vorher aber Fehler &, | zu &&, || korrigieren!
    }
    else if(state==ON) // && !dauerlicht()
    {
      ... wie vorher aber Fehler &, | zu &&, || korrigieren!
    }

Kannst Du mir das vielleicht etwas näher erklären?

DANKE!


Im Originalcode sind ja noch weitere if bzw elseif Abfragen:

if(int0 & (direction==LTR)) int0=0;      // LEDs LTR on, left switch 
ignored
      else if(int1 & (direction==RTL)) int1=0;  // LEDS RTL on, right 
switch ignored
      else if(int0 & (direction==RTL))      // LEDs RTL on, fade off on 
left switch
      {
        TCCR0 = 0;                // disable timer0
        tenms = 0; running = 0;          // reset turnoff-timer
        fade_off(direction);
        state = OFF;
        int0 = 0;
      }
      else if(int1 & (direction==LTR))      // LEDs LTR on, fade off on 
right switch
      {
        TCCR0 = 0;                // disable timer0
        tenms = 0; running = 0;          // reset turnoff-timer
        fade_off(direction);
        state = OFF;
        int1 = 0;
      }


Diese benötige ich aber weiterhin, oder?
Aber eine Schalterabfrage muß da nicht mehr hin.


Danke und Gruß

Scubydoo