Schalter und LED

[Peter (Gast)]

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)
{

init();

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

[Martin B. (martin_b97)]

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

[Sebastian R. (sepp1985)]

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.

[Martin B. (martin_b97)]

@ 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

[Sebastian R. (sepp1985)]

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

[Mike (Gast)]

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?

[Guest (Gast)]

LED_PORT ist nicht initialisiert