atMega32 zieht PA1 immer auf +

[Bernhard S. (snowy1509)]

hallo! ich habe mir das Atmel Evaluations Board von Pollin zugelegt und 
hab jetzt aus dem Internet folgenden Code kopiert und rübergespielt.

Code:

    #include <avr/io.h>
    #include <util/delay.h>

    char _port1 = 0xFF;
    char _port2 = 0xFF;

     int main ()
    {
      DDRA |= 0xFF;
      PORTA = 0xFF;
      while (1)
      { 
                    //Vorwärts        
        for (int _ledPos=0;_ledPos < 7;_ledPos++)
                    {               
                      _port1 = (64 >> _ledPos);          
                      _port2 = (128 >> _ledPos);                          
                      PORTA = ~(_port1 | _port2);
                      _delay_ms (100);         
                    }
                    //Rückwärts
                    for (int _ledPos=7;_ledPos > 0; _ledPos--)
                    {
                      _port1 = (128 >> _ledPos);       
                      _port2 = (64 >> _ledPos);                             
                      PORTA = ~(_port1 | _port2);
                      _delay_ms (100);
                    }
      }
    }

Das ist so eine Art Knight-Rider-Lauflicht für 8 LED. Funktioniert 
eigentlich auch alles, aber mir ist aufgefallen, das sich die 2. LED nie 
ausschaltet. Habe nachgemessen und festgestellt, dass PA1 immer auf + 
bleibt und nie ausschaltet. Hab die Schaltung im ausgeschaltenen Zustand 
nachgemessen und festgestellt das keine Hardware Verbindung in der 
Schaltung zwischen PA1/LED2 und + vorliegt.

Könnte mir wer sagen wo der Fehler liegen kann?

Danke!
mfg
Snowy

[Helfer (Gast)]

> Das ist so eine Art Knight-Rider-Lauflicht für 8 LED.
                                                 ^
>       for (int _ledPos = 0; _ledPos < 7; _ledPos++)
                                      ^^^
7 LEDs!

> Probleme mit PA1...

LEDs so (active low) angeschlossen?

Pin o----|<--###---o +

Sicher, dass es kein Problem mit PA0 ist?

>    char _port1 = 0xFF;
>    char _port2 = 0xFF;

 uint8_t _port1 = 0xFF;
 uint8_t _port2 = 0xFF;

[Helfer (Gast)]

> _ledPos < 7

Sorry, passt, du schiebst ja 2 Bits.

[Bernhard S. (snowy1509)]

sry, dürfe ein Problem mit meinem Kabel zum Addon-Board haben^^ habs 
grad noch mal durchgemessen und weiß jetzt, dass der Fehler da wo 
liegt...

Danke trotzdem, könnte mir vl noch wer erklären was bei folgendem Befehl 
passiert:

 
_port1 = (64 >> _ledPos);          
_port2 = (128 >> _ledPos);

Verschiebe ich da die Variable _ledPos um den Wert 64 bzw 128?

Danke!

Snowy

[Jörg W. (dl8dtl) (Moderator)]

Bernhard Schneeweiß schrieb:

> _port1 = (64 >> _ledPos);
> _port2 = (128 >> _ledPos);
>
> Verschiebe ich da die Variable _ledPos um den Wert 64 bzw 128?

Nein, du verschiebst die Zahl 64 bzw. 128 um den Wert von _ledPos.

Bitte streich die führenden Unterstriche von Variablennamen.
Bis auf wenige Ausnahmen sind Bezeichner, die mit einem Unterstrich
beginnen, im C-Standard als "reserved" markiert, d. h. sie dürfen
nicht durch die Anwendung selbst belegt werden.

[Bernhard S. (snowy1509)]

ah gut danke, jetzt verstehe ich das Programm ;)

Danke!