ADC: LED blinkt nur

[Andre S. (pipesmoker)]

Hallo Leute,
ich habe erst vor kurzem mit uC-Programmierung begonnen und bin 
praktisch noch ganz am Anfang.
Zur Zeit versuche ich, ein Bisschen mit dem ADC herumzuspielen und je 
nach Spannungsbereich verschiedene LEDs schalten zu lassen.
Allerdings blinken die LEDs bei mir immer nur und schalten nie auf 
'Dauerlicht'.

Ich bin dankbar fuer jeden Hinweis...
Gruss,
Pipesmoker

#define F_CPU 4000000UL  /* 4 MHz Interner Oszillator */
#include <avr/io.h>
#include <inttypes.h>
#include <util/delay.h>

/* Funktion fuer Verzoegerung um eine ms (4 Takte) */
void delay_ms(uint16_t ms) {
        while ( ms ) 
        {
                delay_ms(1);
                ms--;
    ms--;
    ms--;
    ms--;
        }
}


/* ADC initialisieren */
void ADC_Init(void) {
  uint16_t result;
  ADMUX = (1 << REFS1) | (1 << REFS0); //interne Referenzspannung nutzen
  ADCSRA = (1 << ADPS2) | (1 << ADPS1) | (1 << ADPS0); //Frequenzvorteiler nutzen
  ADCSRA |= (1 << ADEN); //ADC aktivieren
  ADCSRA |= (1 << ADSC); //eine ADC-Probewandlung
  while (ADCSRA & (1 << ADSC) ) {} //Auf Abschluss der Probewandlung warten
  result = ADCW;
}


/* Mehrfachmessung mit Mittelwert */
uint16_t ADC_Read_Avg(uint8_t channel, uint8_t average)
{
  uint32_t result = 0; //Ergebnis initialisieren

  uint8_t i;
  for (i=0; i < average; ++i)
  {
    result += ADC_Read(channel); //average Messungen ausfuehren
  }

  return (uint16_t)(result / average); //Mittelwert bilden
}


int main (void)
{
  uint16_t adcval;
  ADC_Init(); //Initialisierungsfunktion mit Probemessung

//  11 Ausgaenge definieren
  DDRB = 0xff; //alle PB-Pins als Ausgang definieren (=8 Stueck)
  DDRC |= (1 << PC3) | (1 << PC4) | (1 << PC5); //Ausgang PC3, 4 und 5

  DDRC &= ~((1 << PC6) | (1 << PC2) | (1 << PC1) | (1 << PC0));
        
        while (1) //Endlosschleife
        {

    adcval = ADC_Read_Avg(0, 10); //Mehrfachmessung, Kanal 0, 10 Messungen

    if (adcval > 512)
    {
      PORTC &= ~(1 << PC3);

    }

    else if (adcval <= 512) 
    {
      PORTC &= ~(1 << PC5);
    }
                
        }
}

Hoffe, ihr koennt mir weiterhelfen...

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Andre S. schrieb:
> /* Funktion fuer Verzoegerung um eine ms (4 Takte) */
> void delay_ms(uint16_t ms) {
>         while ( ms )
>         {
>                 delay_ms(1);
>                 ms--;
>                 ms--;
>                 ms--;
>                 ms--;
>         }
> }
Dir ist klar, dass diese Funktion, wenn sie z.B. mit dem Wert 3  oder 
auch 5 oder auch 7 (usw... eben alles, was nicht durch 4 teilbar ist) 
aufgerufen wird, recht lang zum Beenden baucht?

Zudem ist das Ganze rekursiv:
> void delay_ms(uint16_t ms) {
>         while ( ms )
>         {
>                 delay_ms(1);
Ein Glück nur, dass die Funktion niemals aufgerufen wird...  ;-)

> Zur Zeit versuche ich, ein Bisschen mit dem ADC herumzuspielen und je
> nach Spannungsbereich verschiedene LEDs schalten zu lassen.
> Allerdings blinken die LEDs bei mir immer nur und schalten nie auf
> 'Dauerlicht'.
Wie schnell blinken die LEDs?
Welche LEDs blinken?
Wie sind die LEDs angeschlossen?

[reverse (Gast)]

wenn das der ganze Code ist und die Hartware richtig angeschlossen ist, 
dann versteh ich nicht wieso es bei dir blinkt, außer wenn der uC immer 
resetet

[Andre S. (pipesmoker)]

Hallo,
> Dir ist klar, dass diese Funktion, wenn sie z.B. mit dem Wert 3  oder
> auch 5 oder auch 7 (usw... eben alles, was nicht durch 4 teilbar ist)
> aufgerufen wird, recht lang zum Beenden baucht?

darueber habe ich mir noch keine Gedanken gemacht, aber danke fuer den 
Hinweis!

> Ein Glück nur, dass die Funktion niemals aufgerufen wird...  ;-)

Naja, das war praktisch mein erstes Projekt - eine blinkende LED. Und 
die habe ich mit dieser Funktion verzoegert...

Die LEDs blinken nicht mehr, ich habe einfach den Programmer abgezogen 
weil ich mir irgendwann nicht mehr erklaeren konnte, was das eigentlich 
soll - und nach einer Stunde (und einem Reboot) habe ich alles wieder 
angesteckt, den Code nochmal auf den Controller geschrieben und siehe 
da, es hat funktioniert :-).

Trotzdem danke!

[Dennis H. (c-logic)]

Dieses hier ist doppelt gemoppelt

if (adcval > 512)
    {
      PORTC &= ~(1 << PC3);
    }
    else if (adcval <= 512)
    {
      PORTC &= ~(1 << PC5);
    }


if (adcval > 512)
{
  PORTC &= ~(1 << PC3);
}
else
{ //wenn adcval nicht größer als 512 ist,kann es nur kleiner oder gleich 
sein
  PORTC &= ~(1 << PC5);
}

Zum Problem:

Sind die Werte vielleicht Schaltungsbedingt einfach nur zu instabil ?
Welcher Controller ist da gemeinst ?

Wieso laut deines Programmes die LED's überhaupt blinken ist schon 
merkwürdig.
Denn laut Programm müßten die LED's am Ende nur ausgeschalten werden.
Allerdings sind sie schon nach dem RESET aus, per default.

[Andre S. (pipesmoker)]

Ja, die Grenzen hatte ich hier fuers Forum so gesetzt. Ich habe ein paar 
mehr LEDs rangehangen und die Bereiche entsprechend fein aufgeteilt.

Die LEDs haben scheinbar geblinkt, weil ich vorher ein entsprechendes 
Programm dafuer geschrieben hatte, das 'neue' ADC-Programm wurde dann 
aber irgendwie nicht draufgeschoben, sondern ich muss (wie bereits 
geschrieben) den Programmer abziehen, habe irgendwann mal einen Reboot 
gemacht und dann lief alles wieder.
Trotzdem danke fuer deine Antwort.

Gruss,
pipesmoker