Probleme mit Ausgängen

Daniela Gast 15.01.2009 11:47 #1111341

Lesenswert?

•

Hallo zusammen,

ich hab folgendes Problem:

ich habe ein Programm welches über LEDs die Uhrzeit ausgibt (siehe 
vorherige Foreneinträge).

Am Anfang hat alles wunderbar funktioniert, außer dass die LEDs sehr 
schwach geleuchtet haben.

In der ersten "Version" habe ich das DDRx jeweil bei jedem Ausgang 
einzeln gesetzt.
jetzt sieht es folgendermasen aus:

....
// initalisieren der Ports
void init ( void ) {
  DDRA = 0xFF;
  DDRB = 0xFF;
  DDRC = 0xFF;
  DDRD = 0xFF;
  DDRD &=~ (1 << DDD2); //Eingang
  }
....

Hat auch geholfen, also die LEDs leuchten hell (so wie ich es wollte).

ABER, die LEDs, welche am Port D hängen leuchten nicht mehr.
Ich hab schon auf verschiedene Arten Versucht das Problem zu beheben, 
z.B.
die letzten beiden Zeilen in "DDRD = 0xFB" ändern.

Ich freue mich über jeden Verbesserungsvorschlag!

Angehängte Dateien:

main.c (6,43 KB) | Codeansicht

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

Karl H. (kbuchegg) 15.01.2009 11:50 #1111347

Lesenswert?

•

▲
▼

Daniela wrote:

> ....
> // initalisieren der Ports
> void init ( void ) {
>   DDRA = 0xFF;
>   DDRB = 0xFF;
>   DDRC = 0xFF;
>   DDRD = 0xFF;
>   DDRD &=~ (1 << DDD2); //Eingang
>   }
> ....
>

Schön, aber dein main fängt so an

int main(void)
{

init();
// Kontroll LED

  DDRD = (1 << PD5);

und damit ist deine ganze DDRD Einstellung in init() Geschichte 
geworden.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Daniela Gast 15.01.2009 12:15 #1111381

Lesenswert?

•

▲
▼

versteh ich das richtig, dass es gehen müsste wenn die Zeile

  DDRD = (1 << PD5);

gelöscht wird? Die brauch ich im Prinzip auch nicht.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Klaus F. (kfalser) 15.01.2009 12:36 #1111396

Lesenswert?

•

▲
▼

> DDRD = 0xFF;
> DDRD &=~ (1 << DDD2); //Eingang

Ich finde es überhaupt eine schlechte Idee, ein Port (und sei es auch 
nur für einen Takt) als Ausgang zu definieren und dann wieder zurück auf 
Eingang.
Wenn an dem Pin etwas hängt, dann kommt es zu einem Kurzschluss.

In der Init Routine sollen alle Pins entsprechend der Hardware von 
Anfang an richtig gesetzt werden.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Karl H. (kbuchegg) 15.01.2009 12:36 #1111397

Lesenswert?

•

▲
▼

Daniela wrote:
> versteh ich das richtig, dass es gehen müsste wenn die Zeile
>
>   DDRD = (1 << PD5);
>
> gelöscht wird? Die brauch ich im Prinzip auch nicht.

Ich hab mir den Rest des Programmes nicht durchgesehen.
Aber wenn du hier eine Zuweisung zu DDRD machst, dann ist deine ganze 
EInstellung des DDRD Registers in init() sinnlos geworden, weil sie hier 
ja überschrieben wird.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Daniela Gast 15.01.2009 12:44 #1111402

Lesenswert?

•

▲
▼

wäre dann
> DDRD &= (1 << DDD2); //Eingang
anstatt
> DDRD &=~ (1 << DDD2); //Eingang
die bessre Lösung?

Sonst habe ich nichts an den DDR-Registern geändert.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Hmm... Gast 15.01.2009 12:51 #1111412

Lesenswert?

•

▲
▼

In 99.999% aller Fälle ist es nicht notwendig die Richtung eines 
Portpins im laufenden Programm zu ändern.

Das macht man genau einmal in einer Init-Routine gemäss der 
angeschlossenen Hardware und ändert dann nix mehr dran. Demnach würde 
ich Zugriff auf DDRD ausschließlich in "Init()" machen.

Du bist dir der Funktion der beiden Register "DDRD" und "PORTD" aber 
schon bewusst oder hast du den folgenden Block von irgendwo her kopiert?

// Kontroll LED
  DDRD = (1 << PD5);
  PORTD |= (1 << PD5);

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Daniela Gast 15.01.2009 12:59 #1111420

Lesenswert?

•

▲
▼

Am Anfang habe ich jede LED so "angesteuert."

Die Kontroll LED gibt es inzwischen gar nicht mehr, die hatte ich nur 
drin um zu sehen ob überhaupt was geht. Deshalb hab ich auf die beiden 
Zeilen  auch gar nicht mehr geachtet und jetzt auch komplett gelöscht.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Hmm... Gast 15.01.2009 13:05 #1111433

Lesenswert?

•

▲
▼

> Am Anfang habe ich jede LED so "angesteuert."
>
> Die Kontroll LED gibt es inzwischen gar nicht mehr, die hatte ich nur
> drin um zu sehen ob überhaupt was geht. Deshalb hab ich auf die beiden
> Zeilen  auch gar nicht mehr geachtet und jetzt auch komplett gelöscht.

Gerade im Fehlerfall ist es hilfreich, den Code im Geiste noch einmal 
nach zu vollziehen und sich über die Bedeutung jeder einzelnen Zeile 
klar zu werden.


http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Zugriff_auf_IO-Ports

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Daniela Gast 15.01.2009 13:11 #1111444

Lesenswert?

•

▲
▼

Wäre auf jeden fall vermeidbar gewesen, aber ich bin auf dem Thema nicht 
so fit (wie man merkt), dass mir so was nur durch lesen klar wird.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Hmm... Gast 15.01.2009 13:16 #1111453

Lesenswert?

•

▲
▼

> Wäre auf jeden fall vermeidbar gewesen, aber ich bin auf dem Thema nicht
> so fit (wie man merkt), dass mir so was nur durch lesen klar wird.

Deshalb der Link auf das Tutorial ;)

Wenn du dir das mal in Ruhe durch gehst und ausprobierst, kommt ganz 
sicher noch der eine oder andere 'AHA'-Effekt.

Viel Erfolg mit deiner Funkuhr ;)

Darf ich fragen, was genau an Port B hängt?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Daniela Gast 15.01.2009 13:21 #1111462

Lesenswert?

•

▲
▼

ja klar darfst du da kommt das Signal vom DCF-Modul rein, du meinst doch 
den Eingangsport? Sonst sind überall LEDs oder nichts.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Hmm... Gast 15.01.2009 13:36 #1111482

Lesenswert?

•

▲
▼

Dem Code nach zu urteilen hängt das DCF-Modul an PortD.2 Alle weiteren 
Pins sind Ausgänge.

Ich frage nur, weil die Ausgabe der Uhrzeit sicherlich etwas eleganter 
als über das 2 dutzend if-else Konstrukte gelöst werden kann ;) Schaut 
ein wenig nach simulierter Analog-Uhr (mit "Zeigern") aus

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Daniela Gast 15.01.2009 13:39 #1111486

Lesenswert?

•

▲
▼

ist es auch, war die Aufgabe.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

(prx) A. K. (prx) 15.01.2009 13:47 #1111499

Lesenswert?

•

▲
▼

Hmm... wrote:

> In 99.999% aller Fälle ist es nicht notwendig die Richtung eines
> Portpins im laufenden Programm zu ändern.

Ein paar Neuner weniger tun es auch. Beispielsweise ist das bei 
bidirektionale Protokollen wie I2C und 1-Wire, bei HD44780-LCDs mit R/W 
und für Powerdown/save-Modi erforderlich.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Karl H. (kbuchegg) 15.01.2009 14:16 #1111552

Lesenswert?

•

▲
▼

>> etwas eleganter als über das 2 dutzend if-else Konstrukte
>> gelöst werden kann
>> Schaut ein wenig nach simulierter Analog-Uhr (mit "Zeigern") aus
> ist es auch, war die Aufgabe.

Habt ihr schon von Arrays (auf deutsch: Felder) gelernt?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

yalu Gast 15.01.2009 14:17 #1111555

Lesenswert?

•

▲
▼

Wenn ich das richtig verstehe, wird also von PD2 das DCF-Signal gelesen,
an PD0, PD1, PD3, PD5, PD6 und PD7 hängen die LEDs, und PD4 ist nicht
belegt.

Mit

  DDRD = (1 << PD5);


konfigurierst du alle PD-Pins außer PD5 als Eingang, weswegen die
angeschlossenen LEDs nicht oder nur schwach leuchten. Lass also die
Zeile weg, da die Datenrichtung in init schon für alle Ports richtig
konfiguriert wurde.

Und nimm den Raten von Klaus Falser zu Herzen und ersetze

  DDRD = 0xFF;
  DDRD &=~ (1 << DDD2); //Eingang


durch

  DDRD = 1<<DDD0 | 1<<DDD1 | 1<<DDD3 | 1<<DDD5 | 1<<DDD6 | 1<<DDD7;


oder

  DDRD = ~(1<<DDD2 | 1<<DDD4);

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Daniela Gast 15.01.2009 14:23 #1111564

Lesenswert?

•

▲
▼

JA es geht!!!!!

tausend mal Dank!

>Habt ihr schon von Arrays (auf deutsch: Felder) gelernt?

Gelernt schon aber können nicht.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Probleme mit Ausgängen

Antwort schreiben