[C] ATmega32 tut nicht was er soll. Tasterabfrage

Wenn ich folgendes Programm ausführe macht er alles 1x richtig und dann 
blinkt nur noch die 1 auf dem Display. den Rest lässt er aus.

1

#include <avr/io.h>

2

#define F_CPU 1000000

3

#include <util/delay.h>

4

#include "lcd-routines.c"

5

6

int main(void)

7

{

8

  lcd_init();

9

  DDRA = 0b00000001;  //Motor an PORTA 0

10

  PORTA = 0b01000000;  //Pullup für Taster

11

    while(1)

12

    {

13

  lcd_setcursor(0,1);

14

  PORTA = 0x01;    //Motor anschalten

15

  loop_until_bit_is_clear(PINA,7);  //Schleife bis PINA 7 gedrückt ist

16

  PORTA = 0x00;    //Motor ausschalten    

17

  _delay_ms(1000);

18

  lcd_string("1");

19

  _delay_ms(1000);

20

  lcd_clear();

21

22

  } //while(1)

23

}  //Main

Genau das hab ich doch gemacht.

Das mit lcd-routines.c ist ein Fehler von mir. da hab ich im Dateinamen 
.c und .h verwechselt.
Meinst du das Problem liegt an der Tasterabfrage?

Pullup ist jetzt am richtigen pin. das ganze funktioniert aber immer 
noch nicht.

Sieht inzwischen so aus:

1

while ( (PINB & (1<<PINB2)) )

2

        {PORTA = 0x01;}

3

        PORTA = 0x00;    //Motor ausschalten

4

        menu++;

funktioniert aber immer noch nicht.

Hab jetzt nen externen Pullup dran. funktioniert aber nicht.

Nils Friess schrieb:
> Stichwort Bitmanipulation

Ich weiß was du meinst aber nurch den exteren Pullup brauche ich das 
nicht zu tun

push

Wo hängt der externe Pullup? An A7 oder an B2?

A7

Valentin S. schrieb:
> A7
Wie soll dann die Abfrage mit B2 funktionieren?

Das Problem sind einfach die Zuweisungen an den Port mittels 
Holzhammermethode. Bitmanipulation wurde ja bereits angesprochen, 
das sollte man sich von Beginn an so angewöhnen, denn gerade wenn man 
unterschiedliche Dinge an den Pins eines Ports hängen hat, wird man mit 
Direktzuweisungen früher oder später wahnsinnig.

Hab wie gesagt desshalb ja den ext. Pullup

Was wäre so schlimm, das so zu schreiben?

1

#include <avr/io.h>

2

#define F_CPU 1000000

3

#include <util/delay.h>

4

#include "lcd-routines.c"

5

6

int main(void)

7

{

8

  lcd_init();

9

  DDRA  |= 1<<PA0;  //Motor an PORTA 0

10

  PORTA |= 1<<PA7;  //Pullup für Taster

11

    while(1)

12

    {

13

  lcd_setcursor(0,1);

14

  PORTA |= (1<<PA0);    //Motor anschalten

15

  loop_until_bit_is_clear(PINA,7);  //Schleife bis PINA 7 gedrückt ist

16

  PORTA &= ~(1<<PA0);    //Motor ausschalten    

17

  _delay_ms(1000);

18

  lcd_string("1");

19

  _delay_ms(1000);

20

  lcd_clear();

21

22

  } //while(1)

23

}  //Main

Wie ist der Motor angeschlossen? Direkt am Mikrocontroller?

@Felix:
Der Motor ist über einen BC548 mit 1kohm Basisvorwiderstand an den 
ATmega angesclossen.

Wenn ich das von dir geschriebene Programm starte, dreht sich der Motor 
unendlich.

Der Taster wird übrigens 70ms lang gedrückt (mit Oszi gemessen).
Reicht das?

Ich bin davon ausgegangen, dass du einen Taster an A7 hast, der beim 
Drücken den Pfad nach GND schließt. Richtig? Ich kenne den Inhalt der 
Funktion loop_until_bit_is_clear(...) nicht, was passiert, wenn du das 
durch

1

while(PINA & (1<<PA7))

ersetzt?

Freilaufdiode ist vorhanden?

Edit: 70ms sind für den Controller eine Ewigkeit. Das muss reichen.

Felix Pflaum schrieb:
> (...]was passiert, wenn du das[...]
> durch
> while(PINA & (1<<PA7))
> ersetzt[...]

Gar nichts. Der Motor fängt gar nicht an sich zu drehen.
loop_until_bit_is_clear(...) heißt dass er wartet bis ein Taster 
gedrückt (gelöscht) ist.
while(PINA & (1<<PA7)) Als ersatz dafür funktioniert nicht.

Push

Valentin S. schrieb:
> push

Valentin S. schrieb:
> Push

Lass das bitte.

while(PINA &  (1<<PA7)) heißt auch nur, dass gewartet werden soll, 
solange an A7 High-Pegel anliegt.

Blinkt die 1 noch? Wenn ja, liegt A7 wahrscheinlich aus irgendeinem 
Grund dauerhaft auf Low.