ATtiny44 AD Intialisierung

[Michael (Gast)]

Hallo, ich habe eine Frage und hoffe, dass ihr mir helfen könnt!

Erstmal zu den Fakten!
Ich habe eine Platine, auf welcher sich unter anderem der tiny, 
fototransistor und etliche LED's befinden.

Ich habe eine Eingangsspannung von max. 24V welche aber auf 5V runter 
geschraubt wird!

Was ich machen möchte ist je nachdem wie dunkel es ist mehr LED's 
leuchten lassen! Wenn ich mein Programm flashe, passiert jedoch nichts..

Könnt ihr mir bitte helfen evtl. Fehler zu finden?

Lg Michael

#include <avr/io.h>
#include <util/delay.h>



int main(void)
{
  unsigned int x;
  
  DDRA = 0xFE;
  DDRB = 0xFF;
  
  ADCSRA |= ((1<<ADPS2)|(1<<ADPS1)|(1<<ADPS0));
  ADMUX |= (1<<REFS0);
  ADMUX &= (1<<REFS1);
  ADCSRB &= ~((1<<ADTS2)|(1<<ADTS1)|(1<<ADTS0));   
  ADCSRA |= (1<<ADATE);               
  ADCSRA |= (1<<ADEN);               
  ADCSRA |= (1<<ADSC);    
  
  while (ADCSRA & (1<<ADSC));
    x = ADC;
    
  while (1)
  {
    ADCSRA |= (1<<ADSC);
    while (ADC>=1 && ADC<=128);
    PORTB |= (1<<PB2);
    
    while (ADC>=1 && ADC<=256);
    PORTB |= (1<<PB2);
    PORTA = 0x80;
    
    while (ADC>=1 && ADC<=384);
    PORTB |= (1<<PB2);
    PORTA = 0xC0;
    
    while (ADC>=1 && ADC<=512);
    PORTB |= (1<<PB2);
    PORTA = 0xE0;
    
    while (ADC>=1 && ADC<=640);
    PORTB |= (1<<PB2);
    PORTA = 0xF0;
    
    while (ADC>=1 && ADC<=768);
    PORTB |= (1<<PB2);
    PORTA = 0xF8;
    
    while (ADC>=1 && ADC<=896);
    PORTB |= (1<<PB2);
    PORTA = 0xFC;
    
    while (ADC>=1 && ADC<=1023);
    PORTB |= (1<<PB2);
    PORTA = 0xFE;
  }        
}

[Sudo (Gast)]

Weil dein Programm Murks ist !
>while (ADC>=1 && ADC<=128);
meist du vielleicht if(...)

[Uwe (Gast)]

Weil du warten mußt bis der ADC fertig gewandelt hat, der sagt dir das 
indem er ein Bit im Statusregister setzt.

[Uwe (Gast)]

Wenn man es so schreibt geht natürlich etwas, aber er würde sich immer 
die höchste Helligkeit merken und dei lampen erst wieder auschalten 
nachdem das licht ausgeschaltet worden ist.
Ich denke es ist nicht das was du willst.

  while (1)
  {
    while (ADCSRA & (1<<ADSC));
    x = ADC;


    ADCSRA |= (1<<ADSC);
    while (x>=1 && x<=128);
    PORTB |= (1<<PB2);

    while (x>=1 && x<=256);
    PORTB |= (1<<PB2);
    PORTA = 0x80;

    while (x>=1 && x<=384);
    PORTB |= (1<<PB2);
    PORTA = 0xC0;

    while (x>=1 && x<=512);
    PORTB |= (1<<PB2);
    PORTA = 0xE0;

    while (x>=1 && x<=640);
    PORTB |= (1<<PB2);
    PORTA = 0xF0;

    while (x>=1 && x<=768);
    PORTB |= (1<<PB2);
    PORTA = 0xF8;

    while (x>=1 && x<=896);
    PORTB |= (1<<PB2);
    PORTA = 0xFC;

    while (x>=1 && x<=1023);
    PORTB |= (1<<PB2);
    PORTA = 0xFE;
  }

[Karl H. (kbuchegg)]

Uwe schrieb:

> Ich denke es ist nicht das was du willst.

Aber deines auch nicht.

Die while Schleifen sind Quatsch.
Du sollst nur einmal den ADC auslesen und dann entscheidest du anhand 
des Messwertes, wieviele LED einzuschalten sind.

"entscheidest" - da steckt schon der Hinweis drinn. Eine Entscheidung 
wird mit einem if gemacht und nicht mit einer SChleife.

  while( 1 )
  {
    while (ADCSRA & (1<<ADSC))
      ;
    x = ADC;

    if( x < 128 )
      1 LED einschalten

    else if( x < 256 )
      2 LED einschalten

    else if( ....
  }

@TO
Die Initialisierung ist auch zu überdenken. Es gibt in diesem Programm 
keinen wirklichen Grund ADATE zu verwenden. Benutze den ADC ganz 
konventionell, d.h. du startest ihn manuell und du wartest darauf, dass 
er fertig wird.

  while( 1 )
  {
    ADCSRA |= ( 1<<ADSC);          // ADC starten
    while (ADCSRA & (1<<ADSC))     // auf das Ergebnis warten
      ;
    x = ADC;                       // Ergbnis abholen

    if( x < 128 )
      1 LED einschalten, alle anderen aus

    else if( x < 256 )
      2 LED einschalten, alle anderen aus

    else if( ....

  }

Das vemeidet dann auch Probleme, dass du ADCSRA genau zum richtigen 
Zeitpunkt abfragen musst. Nämlich genau in der zeitlichen Lücke, in der 
der ADC gerade fertig geworden ist und eigenständig den nächsten 
Messzyklus anfängt. Denn nur exakt in dieser zeitlichen Lücke ist das 
ADSC Bit dann auch tatsächlich auf 0.

[Michael (Gast)]

Huch, danke erstmal für die vielen Antworten!

Ich möchte tatsächlich, dass das Licht immer den aktuellsten 
Helligkeitswerten angepasst ist!

Dh. sehr dunkel alle LED's, mittel 4 LED's z.B.

Danke für den Tipp mit der Intialisierung, werd ich mir nochmal 
anschauen!

lg

[Karl H. (kbuchegg)]

Michael schrieb:
> Danke für den Tipp mit der Intialisierung, werd ich mir nochmal
> anschauen!

Du solltest dir vor allen Dingen das
AVR-GCC-Tutorial
anschauen und ganz konkret in diesem Tut nach "ADC" suchen. Da gibt es 
eine Verlinkung zu einer Subseite, auf der du alles über den ADC 
findest. Inklusive fertiger C-Funktionen, in denen du nur noch die 
Referenzspannung einstellen musst, die du aber ansonsten so verwenden 
kannst, wie sie sind.

[Michael (Gast)]

Ich hab das Programm jetzt "etwas" angepasst, und nun funktioniert es 
einwandfrei.

Danke für eure Hilfe, und unten nochmals den Code, falls es jemanden 
interessiert ;)

lg Michael

#include <avr/io.h>
#include <util/delay.h>



int main(void)
{
  unsigned int x;
  
  DDRA = 0xFE;
  DDRB = 0xFF;
  
  ADCSRA |= ((1<<ADPS2)|(1<<ADPS1)|(1<<ADPS0));
  //ADMUX = (1<<REFS0);                
  ADCSRA |= (1<<ADEN);               

    
  while (1)
  {
    ADCSRA |= (1<<ADSC);
    while (ADCSRA & (1<<ADSC))
    {
    }
    x = ADC;
    
    if( (x >= 0) && (x < 36))
    {
    PORTA = 0x01; 
    }
    
    else if( (x >= 36) && (x < 72))
    {
    PORTA = 0x03;
    }
    
    else if( (x >= 72) && (x < 108))
    {
    PORTA = 0x07;
    }
    
    else if( (x >= 108) && (x < 144))
    {
    PORTA = 0x0F;
    }
    
    else if( (x >= 144) && (x < 180))
    {
    PORTA = 0x1F;
    }
    
    else if( (x >= 180) && (x < 216))
    {
    PORTA = 0x3F;
    }
    
    else if( (x >= 216) && (x < 252))
    {
    PORTA = 0x7F;
    }
    
    else if( (x >= 252) && (x < 288))
    {
    PORTB |= (1<<PB2);
    PORTA = 0x7F;
    }
  }        
}

[Karl H. (kbuchegg)]

Wenn das Programm hier ....

    
    else if( (x >= 36) && (x < 72))
    {
    PORTA = 0x03;
    }

... jemals zur Abfrage kommt, dann steht bereits fest, dass x 
großer/gleich 36 sein MUSS. Denn wenn x kleiner wäre, dann würde das 
Programm gar nicht bis hier her kommen, sondern wegen

    if( (x >= 0) && (x < 36))
    {
    PORTA = 0x01; 
    }
    
    else if( (x >= 36) && (x < 72))
    {
    PORTA = 0x03;
    }
...

bereits im ersten if hängen bleiben. Dann wird aber alles nachfolgende 
nach dem else gar nicht mehr betrachtet.

D.h.
* du bürdest deinem µC eine Fleissaufgabe auf, indem du etwas testest, 
was gar nicht mehr möglich ist (genauso wie x als unsigned Wert nicht 
kleiner als 0 sein KANN).

* schlimmer aber: du baust dir eine potentielle Falle, indem du an 2 
Stellen im Programm denselben Zahlenwert hinschreiben musst. Und das ist 
ein 'Muss', das der Compiler für dich nicht überwachen kann. D.h. du 
hast dir da in den Code etwas eingebaut, was nur darauf angewiesen ist, 
das du als Programmierer sorgfältig bist. Und das geht dann eben öfter 
schied als einem lieb ist.

So

    if( x < 36 )
    {
      PORTA = 0x01; 
    }
    
    else if( x < 72 )
    {
      PORTA = 0x03;
    }
    
    else if( x < 108 )
    {
      PORTA = 0x07;
    }
    
    else if( x < 144 )
    {
      PORTA = 0x0F;
    }
    
    else if( x < 180 )
    {
      PORTA = 0x1F;
    }
    
    else if( x < 216 )
    {
      PORTA = 0x3F;
    }
    
    else if( x < 252 )
    {
      PORTA = 0x7F;
    }
    
    else if( x < 288 )
    {
      PORTB |= (1<<PB2);
      PORTA = 0x7F;
    }

... ist das haarscharf dieselbe Auswertung. Mit dem einen Unterschied, 
dass jede Grenze nur einmal in Form einer Zahl im Programm vorkommt. 
D.h. hier kann es dir per Definition nicht passieren, dass die Werte in 
2 aufeinanderfolgenden if -Abfragen nicht zusammenpassen.

[Karl H. (kbuchegg)]

  DDRA = 0xFE;

...    
    PORTA = 0x01;

bist du dir da ganz sicher?
Bit 0 vom Port A ist nicht auf Ausgang geschaltet.