Atmega32 seltsames Verhalten bei einfachem Code (1 Taster, 2 LEDs)

[slashbrain (Gast)]

Hallo,

hab mir neulich einen Atmel Atmega32 gekauft, um mal einen Einblick in 
die Welt der µCs zu bekommen. Als ISP nutze ich Usbprog. Allerdings 
hänge ich jetzt an einem seltsamen Problem, dass ich bei besten Willen 
und viel nachdenken nich erklären kann.

Die Beschaltung:
LEDG an D6
LEDR an D7
Taster an A7

Funktion sollte klar sein: immer eine LED an, je nach dem ob der Taster 
gedrückt ist.

#include <avr/io.h>
#define LEDG_OUTPUT()  DDRD|=~(1<<6)
#define LEDR_OUTPUT()  DDRD|=~(1<<7)
#define LEDG_ON()  PORTD&=~(1<<6)
#define LEDG_OFF()  PORTD|=~(1<<6)
#define LEDR_ON()  PORTD&=~(1<<7)
#define LEDR_OFF()  PORTD|=~(1<<7)

int main(void) {
  LEDG_OUTPUT();
  LEDR_OUTPUT();
  while (1) {
    if ((PINA & (1 << PINA7))) {
      LEDG_ON();
      LEDR_OFF();
    } else {
      LEDG_OFF();
      LEDR_ON();
    }
  }
}

Das Ergebnis ist allerdings, dass die rote LED alles richtig macht, die 
grüne jedoch immer an ist, und in dem Fall wo sie eiglt aus sein sollte, 
noch Heller wird (nicht viel).

Steh ich da jetzt einfach programmiertechnisch total auf dem Schlauch 
oder was geht da ab?

Gruß slashbrain

[Stefan E. (sternst)]

Hier http://www.mikrocontroller.net/articles/Bitmanipulation
nochmal nachlesen, warum deine "|=~"-Zeilen nicht das machen, was du 
denkst.

[Falk B. (falk)]

@  slashbrain (Gast)

Eher so.

#include <avr/io.h>
#define LEDG_OUTPUT()  DDRD |= (1<<6)
#define LEDR_OUTPUT()  DDRD |= (1<<7)
#define LEDG_ON()   PORTD &= ~(1<<6)
#define LEDG_OFF()  PORTD |= (1<<6)
#define LEDR_ON()   PORTD &= ~(1<<7)
#define LEDR_OFF()  PORTD |= (1<<7)

Siehe Bitmanipulation

MFG
Falk

[slashbrain (Gast)]

vielen Dank für die Antworten. Leider muss ich sagen, dass sich die 
Fehler in meiner Bitmanipulation sich erst im Laufe meiner 
Fehlerbehebung eingeschlichen haben. Der Code ist nun korrekt, 
allerdings tritt der besagte Fehler immer noch auf.

Um alle Fehler in der Bitmaske auszulöschen hab ich den Code so 
geschrieben:

#include <avr/io.h>

int main(void) {
  DDRD = 192;
  while (1) {
    if ((PINA & (1 << PINA7))) {
      PORTD = 64;
    } else {
      PORTD = 128;
    }
  }
}

Der Fehler tritt jedoch immernoch auf. Wo ist mein Fehler?

[Thomas E. (thomase)]

slashbrain schrieb:
> if ((PINA & (1 << PINA7)))

Das ist immer 0, da der Pullup nicht gesetzt ist: PORTA |= (1 << 7);

mfg.

[slashbrain (Gast)]

Okay, so funktionierts. Vielen Dank für die Antwort.
Aber zum Verständnis: warum spinnen die LEDs wenn ich vergessen habe den 
Pullup des Tasters zu setzen. Und an sich hat der Taster ja 
funktioniert. Also es wurde immer der richtige Teil des if/else 
ausgeführt.

[Karl H. (kbuchegg)]

slashbrain schrieb:
> Okay, so funktionierts. Vielen Dank für die Antwort.
> Aber zum Verständnis: warum spinnen die LEDs wenn ich vergessen habe den
> Pullup des Tasters zu setzen.

Es sind nicht die LEDs die spinnen. Du hast, wenn der Taster nicht 
gedrückt ist, einen offenen Eingang, der sich jedes dahergelaufene 
elektromagnetische Feld aus der näheren Umgebung einfängt und wahlweise 
als 0 oder 1 interpretiert.

[slashbrain (Gast)]

so langsam versteh ich das ganze mit den Pullups usw...

aber jetzt hängts an der Übertragung, ich bekomme kein neues Prog auf 
den Atmel.

avrdude: stk500v2_command(): command failed
avrdude: stk500v2_recv_mk2: error in USB receive
avrdude: stk500v2_program_enable(): bad STK600 connection status: Unknown (0x64)
avrdude: initialization failed, rc=-1
avrdude: AVR device initialized and ready to accept instructions
avrdude: Device signature = 0x000000
avrdude: Yikes!  Invalid device signature.
avrdude: Expected signature for ATMEGA32 is 1E 95 02
avrdude: NOTE: FLASH memory has been specified, an erase cycle will be performed
         To disable this feature, specify the -D option.

-F hat nicht geholfen, und die Verbindungen hab ich 3 mal gecheckt...