Atmel Studio Absolut Beginner Coding programmierfragen

[Chris Telmeth (Gast)]

Hallo Guten Tag :)


Ich habe mir einen Atmega8 besorgt, einen kleinen USB Programmierer und 
das auch alles zum Laufen bekommen, mit dem Atmel Studio 7. Den ersten 
Code habe ich auch schon hochgeladen (Momentan eine LED die SOS blinkt)


Ich weiß leider nicht, ob ich hier richtig bin mit meinem Anliegen. Ich 
würde hier gerne meine kleinen Fortschritte vorstellen und Fragen 
stellen, warum dies und jenes gerade nicht funktioniert...vielleicht 
macht sich ja jemand die Mühe, mir dabei behilflich zu sein und mir über 
die Schulter zu schauen.


Ich fange einfach mal an, ein Bild von meinem kleinen Projekt zu posten 
und den dazugehörigen Code:


Die LED ist am pin C5, der als ausgang definiert ist, dranne. Diese 
funktioniert auch und blinkt im SOS-Modus

Der Pin C4 direkt da drunter soll als Eingang definiert sein.


Wenn ich jetzt durch den Button ein High signal an pin C4 schicke, soll 
die LED im SOS Modus blinken...diese blinkt aber unabhängig von dem 
Button dauerhaft im SOS Modus.

Die Blaue LED leuchtet, wenn ich den Button drücke, dieser funktioniert 
also..jemand eine Ahnung wo der Fehler ist :)?


code:


// MyAtmelBlink.c

// put LED on PC5

#ifndef F_CPU                    // if F_CPU was not defined in Project 
-> Properties
#define F_CPU 1000000UL            // define it now as 1 MHz unsigned 
long
#endif

#include <avr/io.h>                // this is always included in AVR 
programs
#include <util/delay.h>            // add this to use the delay function

//////////////////////////////////////////////////////////////////////// 
///////////////////////////
int main(void) {

DDRC |= (1 << PC5);            // set Port C pin PC5 for output
DDRC &= ~(1 << PC4);        // set Port C pin PC4 for intput

while (1)
{                    // begin infinite loop
if (PORTC ^= (1 << PC4))
{
PORTC &= ~(1 << PC5);
}
else
{

PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(1000);
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(1000);
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(1000);
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(5000);


}
}
return(0);                    // should never get here, this is to 
prevent a compiler warning
}

[Mario K. (christelmeth)]

der Code als Code:

// put LED on PC5

#ifndef F_CPU                    // if F_CPU was not defined in Project 
-> Properties
#define F_CPU 1000000UL            // define it now as 1 MHz unsigned 
long
#endif

#include <avr/io.h>                // this is always included in AVR 
programs
#include <util/delay.h>            // add this to use the delay function

//////////////////////////////////////////////////////////////////////// 
///////////////////////////
int main(void) {

DDRC |= (1 << PC5);            // set Port C pin PC5 for output
DDRC &= ~(1 << PC4);        // set Port C pin PC4 for intput

while (1)
{                    // begin infinite loop
if (PORTC ^= (1 << PC4))
{
PORTC &= ~(1 << PC5);
}
else
{

PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(1000);
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(1000);
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(1000);
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(250);
PORTC ^= (1 << PC5);    // flip state of LED on PC5
_delay_ms(250);            // delay 1/2 second
PORTC &= ~(1 << PC5);
_delay_ms(5000);


}
}
return(0);                    // should never get here, this is to 
prevent a compiler warning
}

[Stefan S. (chiefeinherjar)]

Mario K. schrieb:
> if (PORTC ^= (1 << PC4))

Das ist keine Frage, das ist eine Zuweisung. Damit toggelst du Pin C4 am 
Port C.

Die Abfrage müsst sein

if(PINC & (1<<PINC4){
PORTC &= ~(1 << PC5);
}
else {
...
}

[Harry L. (mysth)]

Und an den ATMega gehört ein Abblock-Kondensator!

Auch auf einem Steckbrett!

[Stefan F. (Gast)]

Harry L. schrieb:
> Und an den ATMega gehört ein Abblock-Kondensator!

Der übliche Fehler.

Ich habe noch zwei gesehen: Die LED hat keinen wirksamen Vorwiderstand, 
weil der durch das Steckbrett überbrückt wird. Die fünf Kontakte in der 
Reihe sind (wie in allen Reihen) direkt miteinander verbunden!

Die gelbe Leitung vom Taster wird keine ordentlichen Signalpegel haben. 
Wenn der Taster gedrückt ist hast du dort die Betriebsspannung der LED 
(ca 3V) und wenn der Taster losgelassen wird, hast du eine zufällige 
Spannung zwischen 0 und 3V.

Das musst du ganz anders beschalten:

         Taster
          ____           10k
+5V o------  -------+---[===]---------| GND
                    |
                    +---|>|---[===]---|
                    |   LED
                    o
                  zum AVR

[Hugo H. (hugohurtig1)]

Na ja, da musst Du schon mal ca. 11,5 Sekunden drücken oder den 
richtigen Moment zum Drücken abpassen. Dein SOS wird "durchgezogen" - 
Taste gedrückt oder nicht. Der Prozessor arbeitet alle Anweisungen im 
ELSE-Zweig ab - incl. der delays, bevor er die Taste abfragt (mal 
abgesehen von den anderen Problemen).

[Mario K. (christelmeth)]

Stefan S. schrieb:
> Mario K. schrieb:
>> if (PORTC ^= (1 << PC4))
>
> Das ist keine Frage, das ist eine Zuweisung. Damit toggelst du Pin C4 am
> Port C.
>
> Die Abfrage müsst sein
>

> if(PINC & (1<<PINC4){
> PORTC &= ~(1 << PC5);
> }
> else {
> ...
> }
>

danke :)

Stefanus F. schrieb:
> Harry L. schrieb:
>> Und an den ATMega gehört ein Abblock-Kondensator!
>
> Der übliche Fehler.
>
> Ich habe noch zwei gesehen: Die LED hat keinen wirksamen Vorwiderstand,
> weil der durch das Steckbrett überbrückt wird. Die fünf Kontakte in der
> Reihe sind (wie in allen Reihen) direkt miteinander verbunden!
>
> Die gelbe Leitung vom Taster wird keine ordentlichen Signalpegel haben.
> Wenn der Taster gedrückt ist hast du dort die Betriebsspannung der LED
> (ca 3V) und wenn der Taster losgelassen wird, hast du eine zufällige
> Spannung zwischen 0 und 3V.
>
> Das musst du ganz anders beschalten:
>
>

>          Taster
>           ____           10k
> +5V o------  -------+---[===]---------| GND
>                     |
>                     +---|>|---[===]---|
>                     |   LED
>                     o
>                   zum AVR
>

Ich bin etwas verwirrt...ich habe doch einen Kondensator verbaut? zw + 
und dem Power Supply pin vom Atmega

Ansonsten danke für den Schaltplan :)

[Stefan F. (Gast)]

Du hast einen Elko verbaut. Elkos sind gewickelt, haben daher eine hohe 
Induktivität und sind für hochfrequente Störungen unwirksam. Der Elko 
macht Sinn, aber er muss durch einen kleinen Keramik-Kondensator 
unterstützt werden - typischerweise 100nF.

Und: Der gehört direkt an den Mikrocontroller, weil die Leitungen und 
Kontakte vom Steckbrett störende Übergangswiderstände und Induktivitäten 
haben. Da dein Controller zwei Pin-Paare zur Stromversorgung hat, 
brauchst du sogar zwei Keramik-Kondensatoren.

[Franz M. (elmo64)]

Mario K. schrieb:
> Ich bin etwas verwirrt...ich habe doch einen Kondensator verbaut? zw +
> und dem Power Supply pin vom Atmega

Aber keinen Kerko mit 100nF so nahe wie möglich an den Pins.
Sieht das nur so aus, oder befinden sich beide Beine des Elkos auf der 
GNDschiene?

Außerdem wäre es sehr Ratsam den Reset-Pin mit einem Pull up Widerstand 
und optional einem 100nF Kerko nach GND zu versehen.

siehe auch Appnote AVR042
http://ww1.microchip.com/downloads/en/appnotes/atmel-2521-avr-hardware-design-considerations_applicationnote_avr042.pdf