ATmega328 Timer1- Konfigurationsproblem

[Pimi (Gast)]

Hallo,
Leider funktioniert mein Timer immer noch nicht. Pure Verzweiflung!
Alle Komponenten meines Programms außer Timer 1 funktionieren.
Darum habe ich nur ein kleines Timer-Programm zugeschickt. Den Timer 
hier habe ich mit Code Vizard (SiSy-AVR) generiert. Das Programm sollte 
eine Markise steuern:
Wenn der Wind schwach ist, das heißt wenige als 10 Impulse am Interrupt 
D3 innerhalb von 5 Sekunden -> öffnet die Markise.
Bei starken Wind Markise schließt und die Windanzeige geht an.

Bitte um Hilfe!
Mit freundlichen Grüßen

[Karl H. (kbuchegg)]

Pimi schrieb:
> Hallo,
> Leider funktioniert mein Timer immer noch nicht. Pure Verzweiflung!
> Alle Komponenten meines Programms außer Timer 1 funktionieren.
> Darum habe ich nur ein kleines Timer-Programm zugeschickt. Den Timer
> hier habe ich mit Code Vizard (SiSy-AVR) generiert.

Vergiss doch einfach mal dieses ganze SiSy Zeugs und diverse 
Code-Wizards. Verstehen wie die Dinge tatsächlich funktionieren kann 
durch keinen Wizard ersetzt werden!

Wenn ich sowas ....

   TCCR1B=0x05;   // Teiler 1/1024
   TCCR1B|=0x02;   // Modus: Zählen bis Vergleichswert

schon sehe, krieg ich die Krise und meine Bereitschaft da irgendwelchen 
Code zu analysieren tendiert rapide gegen 0.


Was ist denn nun?

#define   F_CPU   3686400
#include <avr\io.h>
#include <util\delay.h>

int main()
{
  Port auf Ausgang schalten

  while( 1 )
  {
    LED einschalten
    _delay_ms( 1000 );
    LED ausschalten
    _delay_ms( 1000 );
  }
}

blinkt deine LED mit 1 Sekunde an/aus oder blinkt sie nicht in diesem 
Zeitraster? Ehe das nicht stimmt hat es doch keinen Sinn irgendwelche 
weiteren Berechnungen oder Versuche mit Timerzeiten zu machen, weil dann 
etwas ganz Grundlegendes nicht stimmt! Dein µC taktet dann eben nicht so 
schnell, wie der Wert, mit dem alles andere berechnet wurde, aussagt. 
(Entweder das, oder der Optimizer ist nicht eingeschaltet)

Du kannst stundenlang irgendwelche Berechnungen darüber anstellen, wie 
weit du mit 250km/h fahren kannst bzw. wo du da am Sonntag Nachmittag 
überall hin einen Ausflug machen kannst. Wenn aber dein VW-Käfer (das 
Original) nun mal nicht mehr als 100km/h schafft, dann ist die ganze 
Rechnerei sinnlos und deine Ergebnisse werden gnadenlos falsch sein.

[Peter D. (peda)]

Liefert der Sensor überhaupt prellfreie Impulse oder ist das ein 
Kontakt?

Laß Dir doch erstmal den Meßwert über UART oder ein LCD ausgeben.


Peter

[Pimi (Gast)]

Hallo,

Karl Heinz Buchegger schrieb:
> TCCR1B=0x05;   // Teiler 1/1024
>    TCCR1B|=0x02;   // Modus: Zählen bis Vergleichswert
> schon sehe, krieg ich die Krise und meine Bereitschaft da irgendwelchen
> Code zu analysieren tendiert rapide gegen 0.

Danke für Ihre Hilfsbereitschaft!
Ich weiß dass bei mir fehlen die Grundkenntnisse. Bis jetzt war für mich 
das „Sisy Zeug“ sehr hilfreich. Ich weiß nicht mal wie ich das ganze 
Zeug lernen kann, bis jetzt fand ich keine entsprechende Literatur 
(C-Sprache), wo alles auf einfache art und weiße erklärt wird. Wenn du 
mir mit einem Tipp dabei helfen willst wäre ich sehr dankbar!

Den Fehler habe ich mit Hilfe behoben:
1.  Falsche Fuses
2.  Falsche Timerkonfiguration:
Ist:  TCCR1B=0x05;    // Teiler 1/1024
      TCCR1B|=0x02;   // Modus: Zählen bis Vergleichswert
korrekt heißt es
      TCCR1B=0x05;    // Teiler 1/1024
      TCCR1B|=0x08;   // Modus: Zählen bis Vergleichswert

An Alle die mir Helfen wollten.
Besten Dank!

[Jahat I. (jaib)]

Hallo Pimi,

hast es schneller hin bekommen als ich dazu kam dir nen Lösungsvorschlag 
leider nur für den ATmega8 zusammenzuschustern :-( ich poste den 
trotzdem mal, vielleicht kommst ja dazu das mal auszuprobieren brauchst 
aber dafür schon das aktuellste Sisy ... interessiert mich unheimlich ob 
das geht oder ob die Klasse Button zu träge für deinen Windsensor ist.

//----------------------------------------------------------------------
// Titel     : Automatische Markisensteuerung, kleines Programm AVR C++
//----------------------------------------------------------------------
//----------------------------------------------------------------------
// Schaltung : Windsensor liefert einen Impuls je Umdrehung -> an PortD3 -> Zählen mit Interrupt
//             Schalter an PortC5 öffnet/schießt die Markise (es wird der soll-Zustand [High=offen; Low=geschlossen] ausgegeben)
// Funktion  : Markise ist offen:
//             Wenn innerhalb eines Intervalls 1 der Schwellwert überschrittenwird,
//             soll die Markise geschlossen werden.
//             Markise ist geschlossen:
//             Wenn innerhalb von 120xIntervall 1 der Schwellwert nicht überschritten wurde
//             soll die Markise geöffnet werden.
// Prozessor : ATmega328P
// Takt : 3686400 Hz
// Sprache : C++
//-------------------------------------------------------------------------
#define MAX_IMPULSE   10
#define MAX_INTERVALL  120

class Markise 
{
  public: void init()   {ddrC.bit0=1;}
  public: void close()  {portC.bit0=0;}
  public: void open()   {portC.bit0=1;}
  public: bool isOpen() {return portC.bit0;}
};

class Anzeige 
{
  public: void init() {ddrB.bit0=1;}
  public: void on()   {portB.bit0=0;}
  public: void off()  {portB.bit0=1;}
};


class Application : public Controller
{
    protected: Button     windsensor;
    protected: uint8_t    impulsCounter;
    protected: uint8_t    intervallCounter;
    protected: Markise    markise;
    protected: Anzeige    windAnzeige;
    
    public: void onStart()
    {
    windsensor.config(portD,bit3);
    impulsCounter=0;
    markise.init();    
    windAnzeige.init();
    }
    
    public: void onWork()
    {
        // leer
    }
    
    public: void onTimer1s()
    {
    if(markise.isOpen())
    {
      if(impulsCounter >= MAX_IMPULSE)
      {
        markise.close();  
        windAnzeige.on();  
      }
      intervallCounter=0;  // wenn offen werden die Intervalle nicht gezählt.
    }
    // wenn geschlossen
    else
    {
      intervallCounter++;
      // wenn zu windig
      if(impulsCounter >= MAX_IMPULSE)
      {
        intervallCounter=0;
      }
  
      if(intervallCounter >= MAX_INTERVALL)
      {
        markise.close();   
        windAnzeige.off();  
      }
    }
    impulsCounter = 0;  // Impulszähler für nächses Intervall zurücksetzen          
    }
    
    public: void onEvent(const Object& sender, uint8 data)
    {
    // Windsensor sollte wie ein Button arbeiten und jeder Impuls als Click ankommen,
    // Button ist bereits entprellt aber vielleicht zu langsam
    if ( sender == windsensor && data == Button::Click )
      impulsCounter++;        
    }
    
} app;

[Karl H. (kbuchegg)]

Pimi schrieb:

> korrekt heißt es
>       TCCR1B=0x05;    // Teiler 1/1024
>       TCCR1B|=0x08;   // Modus: Zählen bis Vergleichswert

Nein.
Korrekt heisst es:

   TCCR1B = ( 1 << CS12 ) | ( 1 << CS10 );
   TCCR1B |= ( 1 << WGM12 );


Ob das WGM12 Bit nun das Bit 2 oder das Bit 4 ist, ist etwas was mich 
nicht interessieren braucht. Das Bit hat einen Namen! Und über den setze 
ich es.

Dann passieren dir auch nicht solche Bit-Unfälle.

Du hast jetzt am eigenen Leib erfahren, wie mühsam es ist, 
Hex-Konstanten wie 0x02 oder 0x05 mit dem Datenblatt in Übereinstimmung 
zu bringen und rauszukriegen, was da wirklich passiert. Benutze die 
Bitnamen und du schaltest zumindest eine Fehlerquelle erst mal aus.