Schalter und LED

Peter Gast 21.02.2014 20:02 #3544676

Lesenswert?

•

Ich stehe gerade absolut auf dem Schlauch! :(

Ziel: BUTTON1-4 beim starte auslesen und die zugehörige Led anmachen.
Aufbau: 4 Schalter (keine Taster!) + 4 Leds mit Vorwiderstand
Problem: Alle Leds leuchten obwohl nicht alle Schalter an sind. d.h. 
b1-4 wird auf 1 gesetzt obwohl es nicht sein dürfte.
Frage: Sieht jemand den Fehler?

#include <avr/io.h>
#include <stdlib.h>
#include <util/delay.h> 
#define LED_DDR DDRB
#define LED_PORT PORTB
#define LED1 PB3
#define LED2 PB4
#define LED3 PB5
#define LED4 PB6
#define BUTTON_DDR DDRA
#define BUTTON_PORT PORTA
#define BUTTON_PIN PINA
#define BUTTON1 PINA1
#define BUTTON2 PINA2
#define BUTTON3 PINA3
#define BUTTON4 PINA4
#define BUTTON1_p PA1
#define BUTTON2_p PA2
#define BUTTON3_p PA3
#define BUTTON4_p PA4
void init(void);
int main (void)
uint8_t b1,b2,b3,b4;
b1 = 0; b2 = 0; b3 = 0; b4 = 0;
if ( BUTTON_PIN & (1<<BUTTON1) ) {  b1 = 1;}
if ( BUTTON_PIN & (1<<BUTTON2) ) {  b2 = 1;}
if ( BUTTON_PIN & (1<<BUTTON3) ) {  b3 = 1;}
if ( BUTTON_PIN & (1<<BUTTON4) ) {  b4 = 1;}
if (b1 == 1) { LED_PORT |= (1<<LED1); }
if (b2 == 1) { LED_PORT |= (1<<LED2); }
if (b3 == 1) { LED_PORT |= (1<<LED3); }
if (b4 == 1) { LED_PORT |= (1<<LED4); }
void init(void)
  LED_DDR |= (1<<LED1) | (1<<LED2) | (1<<LED3) | (1<<LED4);
  BUTTON_DDR &= ~((1<<BUTTON1_p) | (1<<BUTTON2_p) | (1<<BUTTON3_p) | (1<<BUTTON4_p));
  BUTTON_PORT |= (1<<BUTTON1_p) | (1<<BUTTON2_p) | (1<<BUTTON3_p) | (1<<BUTTON4_p);


mfg Peter

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

Martin B. (martin_b97) 21.02.2014 21:13 #3544733

Lesenswert?

•

▲
▼

Hallo,

schon mal dran gedacht ein

while(1)
{
}

um das ganze zu legen?

Sonst wird das nur einmal beim Programmstart ausgeführt.

Grüße,
Martin

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Sebastian (sepp1985) 21.02.2014 21:33 #3544773

Lesenswert?

•

▲
▼

Wie sind die led denn angeschlossen? Senk oder quellbetrieb?

Das mit dem while(1); würde ich trotzdem am Ende setzen.
Zur Antwort davor... Es soll nur einmal ausgeführt werden.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Martin B. (martin_b97) 21.02.2014 22:01 #3544819

Lesenswert?

•

▲
▼

@ Sebastian R.

ich stimme dir ja zu, daß meine Lösung nicht der gewünschten Funktion 
des Programms entspricht. Zum Debuggen wäre es jedoch einfacher direkt 
zu sehen, was passiert.

Mit der While-Schleife könnte der TO halt direkt sehen was passiert wenn 
er einen Schalter umlegt und daraus erkennen was evtl. falsch ist.

Die LEDs könnten natürlich auch noch verpolt sein.

Grüsse,
Martin

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Sebastian (sepp1985) 22.02.2014 07:37 #3545130

Lesenswert?

•

▲
▼

Martin B. schrieb:
> @ Sebastian R.
>
> ich stimme dir ja zu, daß meine Lösung nicht der gewünschten Funktion
> des Programms entspricht. Zum Debuggen wäre es jedoch einfacher direkt
> zu sehen, was passiert.
>
> Mit der While-Schleife könnte der TO halt direkt sehen was passiert wenn
> er einen Schalter umlegt und daraus erkennen was evtl. falsch ist.
>
> Die LEDs könnten natürlich auch noch verpolt sein.
>
> Grüsse,
> Martin

Joah dafür wäre es net schlecht. Sollte ja kein Vorwurf sein.

Ich persönlich mag diese ganzen definie aus den Bibliotheken net.
Mit den PB1 etc. Ich definiere mir da lieber selber was.

Macht man eigentlich beim Init. auch den Port vorstellen auf nen Wert? 
Also das ich da die Ausgänge passend Stelle und nicht drauf gehe, das 
sie Null sind.

Was für ne Programmiersoftware nutzt du eigentlich?

Die Schalter arbeiten ja im Senkbetrieb oder? Also laut Programm wäre 
das so.

MfG
Sebastian

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Mike Gast 22.02.2014 08:31 #3545145

Lesenswert?

•

▲
▼

Sebastian R. schrieb:
> Ich persönlich mag diese ganzen definie aus den Bibliotheken net.
> Mit den PB1 etc. Ich definiere mir da lieber selber was.

Das kannst du gerne machen.
Und wie machst du das ohne die defines aus den Bibliotheken, wenn du mit 
deinem Code auf einen anderen Prozessor ziehst und nicht das Rad neu 
erfinden möchtest?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Guest Gast 22.02.2014 09:32 #3545185

Lesenswert?

•

▲
▼

LED_PORT ist nicht initialisiert

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Schalter und LED

Antwort schreiben