Blinken (Wieso geht der Code nicht?)

@spess:
du hast die for()-Schleife übersehen. 1000*1ms= 1sek

vorrausgesetzt XTAL im Makefile stimmt

Um die eigentliche Frage zu beantworten: Der Code funktioniert nicht, 
weil du  nur anschaltest, dann eine Verzögerung hast, ausschaltest, und 
dann sofort wieder die Mainloop begonnen wird, dh. eingeschaltet wird. 
Folglich ist nur extrem kurz aus.

@holger: kürzer geht's noch:

1

while(1) {

2

      PORTA = PINA ^ (1 << PA0);

3

4

      for(i=0;i<1000;i++) {

5

        _delay_ms(1);

6

      }

7

}

Und was heisst da, 'scheisse gesetzt'? Das ist vollkommen 
standardkonform.

holger wrote:

>       for(i=0;i<1000;i++) { _delay_ms(1); }

Bei nur einem Befehl können die Brackets auch weggelassen werden.

for (i=0;i<1000;i++) _delay_ms (1);


dem OP bleibt zu raten, sich mal die wikiseite über bitmanipulation sich 
mal zu gemüte zu führen.

Jemand -_- wrote:

> Und was heisst da, 'scheisse gesetzt'? Das ist vollkommen
> standardkonform.

Ist reine Gewöhnungssache. Ich bin es ebenfalls gewohnt, das Brackets in 
einzelenen Zeilen stehen.

Nachtrag:

Jemand -_- wrote:
>

1

> while(1) {

2

>       PORTA = PINA ^ (1 << PA0);

3

[...]

4

> }

5

>

PINx ist nur für Eingänge gedacht. Du meinst sicherlich
  PORTA = PORTA ^ (1<<PA0);

Oder einfach das funktionierende Beispiel aus dem Tutorial nehmen.

http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Warteschleifen_.28delay.h.29

MFG
Falk

>PINx ist nur für Eingänge gedacht. Du meinst sicherlich
>  PORTA = PORTA ^ (1<<PA0);

Nein, denn dann hätte ich PORTA ^= (1<<PA0); geschrieben.

Siehe auch Datenblatt:
"Independent of the setting of Data Direction bit DDxn, the port pin can 
be read through the PINxn Register Bit."

Was natürlich nicht unbedingt heissen muss, dass deine Lösung nicht 
funktioniert, wobei ich mich aber zu erinnern meine, dass das mit dem 
PORTx auslesen, nicht zum gewünschten Ergebnis führt.

spess53 wrote:

> Als Assemblerprogrammierer wurde ich einfach eine XOR-Verknüpfung mit 1
> machen. Dann wird das Bit jedesmal invertiert.

Richtig. Die Lösung hatte "jemand" schon geschrieben.

 PORTA^=1;

^=XOR und ~=Complement

Hat eigentlich noch niemand gemerkt, dass 1000 nicht in einen uint8_t 
passen?

Edit: Doch Thomas P. ;)

Jemand -_- wrote:

> Was natürlich nicht unbedingt heissen muss, dass deine Lösung nicht
> funktioniert, wobei ich mich aber zu erinnern meine, dass das mit dem
> PORTx auslesen, nicht zum gewünschten Ergebnis führt.


Ich glaube du verstehst da etwas miss.
Wenn du PINx liest, erhälst du die Zustände der Portpins, die auf 
Eingang konfiguriert sind. Aus PORTx liesst du die Zustände der 
Portpins, die auf Ausgang geschaltet sind.

PORTx = PORTx ^ 1

ist in diesem Fall richtig.

"Nein, denn dann hätte ich PORTA ^= (1<<PA0); geschrieben."

Welch selbiges du auch haettest tun sollen. Mit Pinx wird das nix (das 
KANN funktionieren), denn du moechtest auf den derzeitigen Zustand der 
Ausgangsstufe zugreifen.