ADC: LED blinkt nur

Andre S. (pipesmoker) OP 11.01.2012 08:37 #2495732

Lesenswert?

•

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...

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

Lothar M. (lkmiller) Moderator (Firma: Titel) 11.01.2012 08:49 #2495752

Lesenswert?

•

▲
▼

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?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

reverse Gast 11.01.2012 09:10 #2495780

Lesenswert?

•

▲
▼

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Andre S. (pipesmoker) OP 11.01.2012 19:43 #2496838

Lesenswert?

•

▲
▼

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!

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Dennis H. (c-logic) 11.01.2012 20:46 #2496958

Lesenswert?

•

▲
▼

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.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Andre S. (pipesmoker) OP 11.01.2012 23:00 #2497242

Lesenswert?

•

▲
▼

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

ADC: LED blinkt nur

Antwort schreiben