[C] ATmega32 tut nicht was er soll. Tasterabfrage

[Valentin S. (lochrasterer)]

Hallo Leute

Ich hab hier nen ATmega32, der, wenn die Variable "Wuerfel" auf 1 
gesetzt ist, einen Motor einschalten soll und, wenn PINB,7 gedrückt (auf 
null gesetzt) ist, den Motor wieder ausschalten soll.
Code:

  case 4:    // Zucker ausgeben Teil 1
        if (wuerfel==0)
        {menu = 6;}
        if (wuerfel==1)
        {menu = 5;}
          break;
      case 5:    // Teil 2
        PORTA = 0x01;    
        loop_until_bit_is_clear(PINA,7);  
        PORTA = 0x00;    
        menu++; 
          break;

Wenn wuerfel also null ist wird der Teil (case 5) übersprungen. Das 
macht er auch. Wenn wuerfel aber 1 ist, springt er wie gewollt auch in 
case 5 und schaltet den motor an. Nach schätzungsweise 100ms schaltet er 
ihn aber wieder ab. Egal ob der Taster gedrückt ist oder nicht. Hab mit 
Multimeter und Oszi schon nachgemessen welche Spannung vom Taster kommt. 
Er ist aber nicht dauerhaft gedrückt.

Liegt das Problem am Code?

Mit freundlichen Grüßen
Valentin S.

[Valentin S. (lochrasterer)]

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.

#include <avr/io.h>
#define F_CPU 1000000
#include <util/delay.h>
#include "lcd-routines.c"

int main(void)
{
  lcd_init();
  DDRA = 0b00000001;  //Motor an PORTA 0
  PORTA = 0b01000000;  //Pullup für Taster
    while(1)
    {
  lcd_setcursor(0,1);
  PORTA = 0x01;    //Motor anschalten
  loop_until_bit_is_clear(PINA,7);  //Schleife bis PINA 7 gedrückt ist
  PORTA = 0x00;    //Motor ausschalten    
  _delay_ms(1000);
  lcd_string("1");
  _delay_ms(1000);
  lcd_clear();
  
  } //while(1)
}  //Main

[g457 (Gast)]

> PORTA = 0b01000000;  //Pullup für Taster
[..]
> PORTA = 0x01;    //Motor anschalten
[..]
> PORTA = 0x00;    //Motor ausschalten

∗hüstel∗

[Valentin S. (lochrasterer)]

Genau das hab ich doch gemacht.

[BattMan (Gast)]

Valentin S. schrieb:
> #include "lcd-routines.c"

Wer macht denn so was?

Valentin S. schrieb:
> loop_until_bit_is_clear

Zeig mal diesen Teil der Source.

[Valentin S. (lochrasterer)]

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?

[Informer (Gast)]

Hallo,

unabhängig vom schon genannten Problem: wenn der Taster an PinA.7 hängt 
(zweites Programm) und mit der Anweisung

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

der Pullup für PinA.6 eingeschaltet wird, passt das auch nicht zusammen 
...

[Valentin S. (lochrasterer)]

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

[g457 (Gast)]

> Genau das hab ich doch gemacht.

Nein, hast Du nicht. Um genau zu sein fehlt bei den Kommentaren "Motor 
anschalten" und "Motor ausschalten" die Hälfte.

HTH

[BattMan (Gast)]

Löte einen externen Pullup Widerstand an den PA7 (10k oder so),
dann funktioniert dein Programm.
Mit der Anweisung

PORTA = 0x01;    //Motor anschalten

schaltest Du den internen Pullup aus. Weil Du das Bit PA7 mit 0 
beschreibst.
Dass es einmal funktionieren soll wundert mich deshalb sehr.

[Valentin S. (lochrasterer)]

Sieht inzwischen so aus:

while ( (PINB & (1<<PINB2)) )
        {PORTA = 0x01;}
        PORTA = 0x00;    //Motor ausschalten
        menu++;

funktioniert aber immer noch nicht.

[g457 (Gast)]

> funktioniert aber immer noch nicht.

Wird immer schlimmer. Geh mal zurück auf Anfang, vergleich die 
Codezeilen, die ich Dir oben rauszitiert hab, nochmals ∗genau∗ mit den 
mitzitierten Kommentaren und stelle fest, dass die beiden letzteren 
Kommentare FALSCH sind (die kommentieren nicht das, was Du tatsächlich 
machst).

Dann überleg nochmal, was Du machen musst, damit die Kommentare RICHTIG 
werden.

[Nil (nilsnilss)]

Stichwort Bitmanipulation

[Valentin S. (lochrasterer)]

Hab jetzt nen externen Pullup dran. funktioniert aber nicht.

[Valentin S. (lochrasterer)]

Nils Friess schrieb:
> Stichwort Bitmanipulation

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

[Valentin S. (lochrasterer)]

push

[Felix P. (fixxl)]

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

[Valentin S. (lochrasterer)]

A7

[Felix P. (fixxl)]

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.

[Valentin S. (lochrasterer)]

Hab wie gesagt desshalb ja den ext. Pullup

[Felix P. (fixxl)]

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

#include <avr/io.h>
#define F_CPU 1000000
#include <util/delay.h>
#include "lcd-routines.c"

int main(void)
{
  lcd_init();
  DDRA  |= 1<<PA0;  //Motor an PORTA 0
  PORTA |= 1<<PA7;  //Pullup für Taster
    while(1)
    {
  lcd_setcursor(0,1);
  PORTA |= (1<<PA0);    //Motor anschalten
  loop_until_bit_is_clear(PINA,7);  //Schleife bis PINA 7 gedrückt ist
  PORTA &= ~(1<<PA0);    //Motor ausschalten    
  _delay_ms(1000);
  lcd_string("1");
  _delay_ms(1000);
  lcd_clear();
  
  } //while(1)
}  //Main

Wie ist der Motor angeschlossen? Direkt am Mikrocontroller?

[Valentin S. (lochrasterer)]

@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.

[Valentin S. (lochrasterer)]

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

[Felix P. (fixxl)]

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

while(PINA & (1<<PA7))

ersetzt?

Freilaufdiode ist vorhanden?

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

[Valentin S. (lochrasterer)]

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.

[Valentin S. (lochrasterer)]

Push

[Magnus M. (magnetus)]

Valentin S. schrieb:
> push

Valentin S. schrieb:
> Push

Lass das bitte.

[Felix P. (fixxl)]

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.

[BattMan (Gast)]

g457 schrieb:
> letzteren
> Kommentare FALSCH sind

Ach ja? Dann teile dein Wissen mit uns und sprich nicht wie ein Orakel.

[g457 (Gast)]

> Ach ja?

Ja! Ich zitiere zweckmäßigerweise nochmals eine der bemängelten Zeilen:

> PORTA = 0x00;    //Motor ausschalten

Links (Code) steht im wesentlichen sowas wie 'setzte den gesamten Port 
auf 0 und schalte damit alle Pullups aus (wo auf Eingang konfiguriert) 
bzw. den Ausgang auf low (wo auf Ausgang konfiguriert)'.
Rechts (Kommentar) steht im wesentlichen sowas wie 'mach was an einem 
einzelnen bestimmten Pin uns sonst gar nie nicht nirgens woanders'.

Jetzt klarer? Dann ganz konkret: Der Code schaltet die 
Pull-Up-Widerstände (und den Motor) aus, der Kommentar behauptet er 
würde nur den Motor ausschalten. Das mit den Widerständen wurde 
mittlerweile mehrfach diskutiert, ändert aber nix daran dass im Original 
Code und Kommentare nicht zusammenpassen.

HTH und ich konnte damit alle Unklarheiten beseitigen.