C-Programmieren LED & Schalter

[Marc S. (geforce8100)]

Hallo Leute,

ich habe eben hier angemeldet, da ich bitte unbedingt Hilfe von euch 
benötige.

Und zwar machen wir gerade in der Schule Projekte, dort geht es auch ums 
programmieren.

Mein Projekt soll ein Timer werden. - Bestückt wurde die Platine mit 
einem ATtiny2313, 9 LEDs und einem Taster.

Jede 10 Sekunden soll eine LED angehen, sobald 60 Sekungen vergangen 
sind, sollen diese wieder ausgehen und LED 7 angehen. Danach sollen die 
6 LED wieder hinauf zählen und LED 9 soll sich anschalten. Zum Schluss 
soll sich LED 8 einschalten.

Wenn der Taster gedrückt wird, soll der Timer resetet werden.

Pin belegung:

LED1    Port D 0x08
LED2    Port D 0x10
LED3    Port D 0x20
LED4    Port D 0x40
LED5    Port B 0x01
LED6    Port B 0x02
LED7    Port B 0x04
LED8    Port B 0x08
LED9    Port B 0x10
Taaster Port D 0x04


Leider bin ich ein totaler Neuling und verstehe nicht sehr viel von der 
C-Programmierung. - Vllt. hat ja jemand Lust & Zeit von euch den 
Programmiercode zu schreiben :-) !!

LG und Danke

[Düsendieb (Gast)]

lese und lerne
http://www.mikrocontroller.net/articles/Absolute_Beginner-AVR_Steckbrettprojekte

[Janeistklar (Gast)]

Präsentiere uns doch mal deinen Ansatz, dann können wir punktuell 
helfen.
Copy&Paste ist eher üblich bei Promotionen.

[Marc S. (Gast)]

Also, mein Testprogramm sieht zurzeit so aus:

#include <avr/io.h>


int main ()
{
// Init der Ports
DDRB = 0xFF;
DDRD = 0xFB;

PORTB = 0xFF;
PORTD = 0xFF;

// Start Hauptprogramm mit Tastendruck
while (!(PORTD &= 0b00000100));

while (1)
  {
    PORTB = 0x00;
    PORTD &=0x0F;  
  }
}

Wenn ich aber den Taster nun betätige, dann gehen die LEDs nicht aus, 
sondern werden nur dunkler. Hat jemand ne Ídee wo der Fehler liegt?

Danke

[Lutz H. (luhe)]

Siehe  Beitrag
Idiotischer Anfängerfehler

von gestern.

In der while-Schleifewird die LED ganz schnell ein- und ausgeschaltet.
Nach dem Ein- oder Ausschalten   mus eine Pause rein. Zum Beispiel von 
0,5 Sekunden.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Hier wird dir niemand ein Programm schreiben, im Gegenteil, die meisten 
sind hier, um es wieder auseinander zu nehmen :-)
Jetzt mal Ernst beiseite, zerlege die Aufgabenstellung in einzelne 
Schritte:

Marc S. schrieb:
> Jede 10 Sekunden soll eine LED angehen

Du brauchst also einen 10 Sekunden Timer. Lies dir im AVR Tutorial den 
Abschnitt über Timer durch und erschaffe einen Timer, der alle 10 
Sekunden eine LED an bzw. ausschaltet.

> Wenn der Taster gedrückt wird, soll der Timer resetet werden.
Lies dir den Artikel über Input/Output durch und erfinde eine 
Unterroutine, dir dir den Zustand des Taster zuverlässig zurückgibt, am 
einfachsten über True und False.

> LED1    Port D 0x08
> LED2    Port D 0x10
> LED3    Port D 0x20
> LED4    Port D 0x40
> LED5    Port B 0x01
> LED6    Port B 0x02
> LED7    Port B 0x04
> LED8    Port B 0x08
> LED9    Port B 0x10

Nun machst du dir Gedanken über die Anordnung der LED. Da ihr eine 
'Kraut- und Rüben' Pinanordung gewählt habt, ist es evtl. am 
einfachsten, ein Array (-> C-Strukturen) zu erschaffen, in dem alle 
möglichen Zustände der LED gespeichert sind.

Noch ein Tipp. Wenn du so deine Ports ansprichst

> while (!(PORTD &= 0b00000100));
weisst du nach einem Monat nicht mehr, was du da eigentlich machst. 
Abgesehen von dem Fehler, das du den PORT liest und nicht das PIN 
Register , sind klare Definitionen besser:

#define Buttonport PORTD
#define ButtonDir DDRD
#define ButtonPin PIND  
#define Button 2

Damit sieht das dann deutlicher aus:

// Tastenport init
ButtonDir = ~(1 << Button);
// lese Taste
result = !(ButtonPin &= (1 << Button));

[Karl H. (kbuchegg)]

Die wichtigste Frage ist doch:
wie ist dein Kenntnisstand? Welche Dinge solltest du eigentlich schon 
können?

Denn eine handwerklich solide Lösung mit einem Timer nimmt dir dein 
Lehrer nicht ab. Halte die nicht für blöd. Der weiß auch ganz genau, 
dass da was nicht stimmt, wenn die größte Pfeife in der Klasse plötzlich 
mit dem handwerklich besten Programm daher kommt.

Wenn du noch nichts anderes gelernt hast, dann ist _delay_ms dein 
Freund, auch wenn es eine schlechte Lösung ist. Aber irgendwo muss man 
mal anfangen.

[Marc S. (Gast)]

Okay, vielen Dank.

Das Testprogramm funktioniert jetzt :-)

[Matthias L. (Gast)]

Wahrscheinlich mit Tippfehlern. Aber darüber darfst du dir jetzt 
Gedanken machen:

//------------------------------------------------------------------------
struct T_Data
{
    uint8_t   u08Port_b;
    uint8_t   u08Port_d;
    uint32_t  u32Delay;
} 
scLedData[] = { {0x01,0x00, 10000},
                {0x02,0x00, 10000},
                {0x04,0x00, 10000},
                {0x08,0x00, 10000},
                {0x10,0x01, 10000},
                {0x00,0x08, 10000},
                {0x00,0x10, 10000},
                {0x00,0x20, 10000},
                {0x00,0x40, 10000}  };

volatile  uint32_t  u32Tick;
volatile  uint8_t   u8State;

//------------------------------------------------------------------------
void main (void)
{
  //-- outputs --------------------
  DDRB |= 0x1F;
  DDRD |= 0xF8;

  //-- init timer to normal mode, clk/64 => 1ms 
  TIMSK  |= (1<<TOIE0);
  TCCR0B |= (3<<CS00);
  sei();

  //-- run program ----------------
  while ( 1 )
  {
    if ( PIND & 0x04 )
    {
      u8State = 0;
    }

  }
}


//------------------------------------------------------------------------
isr ( TIMER0_OVF_vect)
{
  uint32_t  u32TimeToWait = scLedData[u8State];
  if ( ++u32Tick < u32TimeToWait ) return;

  u32Tick = 0;
  u8Max   = (uint8_t) ( sizeof(scLedData)/sizeof(scLedData[0])  );
  if ( ++u8State >= u8Max ) 
  {
    u8State = 0;
  }

  PORTB = scLedData[u8State].u08Port_b;
  PORTD = scLedData[u8State].u08Port_d;
}

[Studentle (Gast)]

Ooooh Lippy... so lernt er das doch nie!!!

Und da wundert man sich, dass sich so viele mit Plagiatsvorwürfen 
herumschalgen müssen, wenn es einem so einfach gemacht wird.

[Matthias L. (Gast)]

Aber Erklärung gibt es keine dazu.

Wie Karl Heinz schon sagte:
>wenn die größte Pfeife in der Klasse plötzlich
>mit dem handwerklich besten Programm daher kommt.

Aber wenn er es kapiert hat und erklären kann....

Und wenns keine Klausur im geschlossenen Raum ist, ist Hilfe zulässig...

[Marc S. (Gast)]

Vielen vielen Dank!

Selbstverständlich mache ich mir meine Gedanken dazu, Hilfe ist 
natürlich in diesem Projekt von allen Seiten zulässig!

Tests und Schularbeiten gibt es in diesem Fach auch nicht ;-)

Danke!!
LG

[Karl H. (kbuchegg)]

Marc S. schrieb:

> Selbstverständlich mache ich mir meine Gedanken dazu,

Na da bin ich neugierig.

Wenn ich deine Fragen zu Grunde lege, dann dauert das 3 Monate, bis du 
den Code verstanden hast.

Ganz ehrlich Matthias. Ich denke nicht, dass du ihm damit einen Gefallen 
getan hast. Das geht weit darüber hinaus, was er mit seinem Wissen noch 
verstehen kann. Da kannst nicht einen Grundschüler mit 
Differentialgleichungen überfallen, wenn er Probleme mit dem kleinen 
Einmal Eins hat. Selbst wenn Diff-Gleichungen die richtige Lösung für 
eines seiner Probleme wäre.

[Matthias L. (Gast)]

>Ich denke nicht, dass du ihm damit einen Gefallen
>getan hast.

Vielleicht hast du recht. Aber vielleicht kann er das als Richtung 
ansehen...

[Ingo (Gast)]

Zumal es auch für geübte Menschen nicht leicht ist zu sehen was du da 
treibst!

[Marc S. (Gast)]

Haha sehr lustig, ist wohl einer der Experten hier^^

Also ich versuche das irgendwie nachzuvollziehen.

Ob das Programm funktioniert oder nicht kann ich nicht sagen, da ich den 
Fehler bekomme:

Build FAILED.
========== Build: 0 succeeded or up-to-date, 1 failed, 0 skipped 
==========

[Dave C. (dave_chappelle)]

@lippy

Du hättest noch irgendwo ein Zeichen weglassen sollen o.ä., dann hätte 
er sich wenigstens überlegen müssen, was da so abgeht ;)

Gruss

[Karl H. (kbuchegg)]

OK.
Nachdem jetzt einige Zeit vergangen ist, mein Rat:
Leg das Programm beiseite. Man könnte zwar die paar Tippfehler 
korrigieren aber für jemanden, der in seiner Entwicklungsumgebung noch 
nicht einmal die Fehlermeldungen findet ist das wohl aussichtslos.
Nur um das klar zu stellen: Die Vorlage ist absolut in Ordnung. Da gibt 
es nichts daran auszusetzen. Genau so würde man das machen, wenn man 
sein Handwerk versteht.

Schreib das so, wie es deinem Können angemessen ist. Du bist nun mal 
noch kein Meister, sondern fängst gerade erst mit der Ausbildung an. 
Schreib es so, wie es dem entspricht, was du kannst und was du schon 
gelernt hast. Und wenn ich nach der Jahreszeit gehe und daran denke, 
dass die Semester gerade erst angefangen haben, dann kann das noch nicht 
viel sein.
D.h. Portpin setzen, delay, nächsten Portpin setzen, delay, etc. etc.

[Marc S. (Gast)]

Da hast du wohl recht, ich versuche es gerade.
Zurzeit habe ich einen Timer, der jede 10 Sekunden eine LED hinaufzählt 
und sich danach resetet.

[Marc S. (Gast)]

Generell wäre das programm nun fertig.

Jedoch funktioniert bei mir die Sleep Funktion noch nicht ganz.

Der Timer soll nach der Slee Funktion erwachen. - Weiß wer an was es 
liegt?

/*
 * AVRGCC1.c
 *
 * Created: 16.10.2013 13:01:56
 *  Author: Marc Svata
 */ 

#include <avr/io.h>
#include <avr/sleep.h>
#include <avr/interrupt.h>



//#define ten_secs 271000
#define ten_secs 27100

long i;

/*
void sleep()              // Timer befindet sich im Sleep Modus
{
  
  PORTD = 0x07;
  PORTB = 0;
  for (i=0;i = ten_secs;i++);  ____________
}
*/

void clear_leds()
{
  PORTD &= 0x07;
  PORTB &= 0xFC;
}


void cast_my_dice()                  // continue after keypress

{
  PORTD = 0x04;
}

/* %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% */

void eine_minute()                  // Counter fuer eine Minute
{

      //for(i=0; i < ten_secs; i++);  
      PORTD |= 0x08;
      PORTB &= 0xFC;
      for(i=0; i < ten_secs; i++);  
      PORTD |= 0x18;
      for(i=0; i < ten_secs; i++);  
      PORTD |= 0x38;
      for(i=0; i < ten_secs; i++);  
      PORTD |= 0x78;
      for(i=0; i < ten_secs; i++);  
      PORTB |= 0x01;
      for(i=0; i < ten_secs; i++);  
      PORTB |= 0x03;
      for(i=0; i < ten_secs; i++);  
}

/* %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% */



int main(void)
{  
  
  DDRB = 0x1F;                  // Pins PB0, PB1, PB2, PB3, PB4  "high"  Leds als Ausgang definieren
  DDRD = 0x78;                  // Pins PD3, PD4, PD5 und PD6 "high"   Leds als Ausgang definieren  
  PORTD = 0x07;                  // Pull Up Widerstand fuer den Taster setzen
  PORTB = 0x00;

sei();  
    while(1)
    {
    
    eine_minute();          
    clear_leds();                // 60 Sekunden LEDs schalten sich aus
    PORTB |= 0x10;                //  1. Minuten LED schaltet sich ein
    
    
    
    eine_minute();
    clear_leds();                // 60 Sekunden LEDs schalten sich aus
    PORTB |= 0x04;                //  2. Minuten LED schaltet sich ein
      
    
    
    eine_minute();            
    clear_leds();                // 60 Sekunden LEDs schalten sich aus
    PORTB |= 0x08;                //  3. Minuten LED schaltet sich ein
  
  clear_leds();
  
  
  for(i=0; i = ten_secs; i++)            // Setze fuer 10 Sekunden die 3 Minuten LED's auf "high" 
  
  {
    
    PORTB |= 0x10;
    PORTB |= 0x04;
    PORTB |= 0x08;  
  
  }
  
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);      // Setze Timer in Schlafmodus 
  sleep_mode();
  cast_my_dice();                  // continue after keypress
  
  
  
  }
}

[Karl H. (kbuchegg)]

> - Weiß wer an was es liegt?

Das funtkioniert so nicht.
Um den µC wieder aus dem Sleep rauszuholen, benötigt es einen Interrupt. 
Von alleine wacht der µC nicht mehr auf.

Lass den sleep erst mal beiseite.

Dein Programm krankt noch an ganz anderen Stellen. Zb wird dir jeder 
vernünftige Compiler, wenn er das hier

     for(i=0; i < ten_secs; i++);

optimieren darf, die Schleife einfach rauswerfen.

Um derartige Zeitverzögerungen zu machen, gibt es die Funktion

   _delay_ms( .... )

die so gebaut wurde, dass sie gezielt Zeit verbrutzelt.
In Anbetracht dessen, dass du mit Timern noch nicht umgehen kannst, ist 
das das Mittel der (momentanen) Wahl.

Sleep ist etwas, das kommt irgendwann. Aber noch nicht jetzt. Dazu fehlt 
dir noch viel zu viel.

[Marc S. (Gast)]

Der Sleep muss aber (leider) bei meinem Programm gemacht werden
-> Anforderung der Lehrperson.

Problem hierbei ist zurzeit das er nicht in den Slee Modus geht.
Interrupt muss dann noch gemacht werden...

Ansonsten funktioniert der Timer wunderbar...

[Karl H. (kbuchegg)]

Marc S. schrieb:
> Der Sleep muss aber (leider) bei meinem Programm gemacht werden
> -> Anforderung der Lehrperson.

Jetzt muss ich lachen.

Dein Programm hat Probleme ohne Ende.
Da sind gefühlte 8-hundert-2-und-fünfzig Problemkreise die wichtiger und 
dringender in deinem Programm sind.
Und du kümmerst dich jetzt um Problem Nummer 853?

Sorry. Aber da muss ich lachen.

[Marc S. (Gast)]

Verstehe ich nicht.

Das Programm funktioniert ja (Timer zählt 3 Minuten herunter mit LED 
Ansteuerung).

Vllt. ist es 'komisch' geschrieben, weil ich eben ein Anfänger bin ;-)

[Karl H. (kbuchegg)]

> Ansonsten funktioniert der Timer wunderbar...

Das tut ein Klebeband an der durchgerosteten Autokarrosserie auch. 10 
Minuten lang.

PS: Ein 'Timer' ist in diesem Zusammenhang eine Hardwarekomponente im 
µC. Eine eingebaute Funktionalität. Das hat nichts damit zu tun, dass du 
das ganze Gerät einen 'Timer' nennst.
Wenn ich von "du kannst mit Timern noch nicht umgehen" rede, dann meine 
ich mit "Timer" ersteres und nicht letzteres.

[Marc S. (Gast)]

Du bist ja heute wieder zum Scherzen aufgelegt ;-)

[Karl H. (kbuchegg)]

Nicht wirklich.
Aber was soll ich dir hier erklären?
Mehr als "der sleep kann nicht funktionieren, weil es nichts gibt was 
den µC wieder aus dem Sleep herausholen könnte" kann ich dir jetzt nicht 
sagen.
Ich kann hier nicht mit dir 5 Stunden Unterricht im Schnelldurchlauf auf 
3 Bildschirmseiten kompremiert nachholen.