Atmega48 ADC zu PWM

[Ranjid (Gast)]

Hallo, ich versuche gerade mittels ADC über PWM eine LED zu dimmen, das 
heißt die Helligkeit der LED hängt von der Helligkeit des Umgebungslicht 
ab.
Aber irgendwie wie will der ADC nicht so wie ich will. Der PWM 
funktioniert, den habe ich schon so getestet, aber der ADC will 
anscheinden nicht so.
AREF habe ich mit 100nF nach GND gelegt. Alles dürfte richtig beschaltet 
sein. Ich habe auch schon versucht die interne Spannung zu benutzen, hat 
aber auch nicht geklappt.

Hier mein Code

#define F_CPU 14745600UL

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

int main(void)
{
  uint16_t adc_value = 0;
  uint16_t pwm_calc_value = 0;
  
  DDRB |= (1<<PB1);
  DDRC &= ~(1<<PC0);
  PORTC |= (1<<PC0);
  
  TCCR1A |= ((1<<COM1A1) | (1<<WGM10));  //non inverting, Prescaler 256
  TCCR1B |= ((1<<WGM12) | (1<<CS12));  // Prescaler 256, Fast PWM
  OCR1A = 200;
  
  ADMUX |= (1<<REFS0);  //extern rev, adc0
  ADCSRA |= ((1<<ADEN) | (1<<ADIE) | (1<<ADPS0) | (1<<ADPS1) | (1<<ADPS2));  //adc enable, interrupt, prescaler 128
  //free running mode ADCSRB
  
  sei();
    while(1)
    {
        adc_value = ADCW;
        pwm_calc_value = adc_value / 5;
        OCR1A = pwm_calc_value;
        _delay_ms(50);
    }
}

ADCW wird erstmal durch 5 geteilt, damit er klein genug ist, um ihn in 
OCR1A zu schreiben. Meine Frequenz beträgt 14745600Hz, wenn ich sie 
durch 200000Hz teile kommt ~73 raus, wenn ich sie durch 50000 teile, 
kommt ~294 raus, da heißt ich kann nur einen Prescaler von 128 benutzen, 
oder ?

[spess53 (Gast)]

Hi

>ADCW wird erstmal durch 5 geteilt, damit er klein genug ist, um ihn in
>OCR1A zu schreiben.

Wieso durch 5? Du hast eine 8 Bit PWM. Damit der ADC-Wert reinpasst must 
du durch 4 teilen. Besser noch ADLAR setzen. Dann kannst du ADCH direkt 
in OCR1AL laden.

> Meine Frequenz beträgt 14745600Hz, wenn ich sie
>durch 200000Hz teile kommt ~73 raus, wenn ich sie durch 50000 teile,
>kommt ~294 raus, da heißt ich kann nur einen Prescaler von 128 benutzen,
>oder ?

Du kannst jeden Prescaler benutzen. Mit Vorteile 128 bekommst du bei 
8-Bit-PWM ein PWM-Frequenz von 450Hz.

MfG Spess

[Thomas E. (thomase)]

Da liesst du ADCW natürlich aus, ohne zu prüfen, ob er fertig ist mit 
der Wandlung.

pwm_calc_value = adc_value / 5;
Warum /5? Solche krummen Zahlen sind immer schlecht.

Fast-PWM 10 bit ist dein Freund. Da kannst du deinen ADC-Wert direkt 
reinschreiben.
Dann setzt du noch ADATE, stellst den Trigger auf Timer1-Overflow, liest 
in der Timer-Isr den ADC-Wert und schreibst ihn ins OCR-Register.
Fertig. Ein bisschen konfigurieren, eine Zeile Code(OCR1A = ADCW;).
Den Rest macht der Controller alleine und dein Programm kann Primzahlen 
berechnen.

mfg.

[Ranjid (Gast)]

Thomas Eckmann schrieb:

> Fast-PWM 10 bit ist dein Freund. Da kannst du deinen ADC-Wert direkt
> reinschreiben.
> Dann setzt du noch ADATE, stellst den Trigger auf Timer1-Overflow, liest
> in der Timer-Isr den ADC-Wert und schreibst ihn ins OCR-Register.
> Fertig. Ein bisschen konfigurieren, eine Zeile Code(OCR1A = ADCW;).
> Den Rest macht der Controller alleine und dein Programm kann Primzahlen
> berechnen.
>
> mfg.

Würde auch der Timer0 oder 2 gehen, denn der timer1 (16bit) ist ja schon 
für die PWM verbraucht ?

[Ranjid (Gast)]

Okay habe jetzt ADATE in TCCR1A gesetzt und in TCCR1B den Trigger auf 
Timer1 overflow aktiviert, zeigt aber keine Wirkung

[Thomas E. (thomase)]

Ranjid schrieb:
> Thomas Eckmann schrieb:
>
>> Fast-PWM 10 bit ist dein Freund. Da kannst du deinen ADC-Wert direkt
>> reinschreiben.
>> Dann setzt du noch ADATE, stellst den Trigger auf Timer1-Overflow, liest
>> in der Timer-Isr den ADC-Wert und schreibst ihn ins OCR-Register.
>> Fertig. Ein bisschen konfigurieren, eine Zeile Code(OCR1A = ADCW;).
>> Den Rest macht der Controller alleine und dein Programm kann Primzahlen
>> berechnen.
>>
>> mfg.
>
> Würde auch der Timer0 oder 2 gehen, denn der timer1 (16bit) ist ja schon
> für die PWM verbraucht ?
Du kannst den Timer1 dafür nehmen. Den Overflow erzeugt er auch mit Pwm. 
Die Pwm stört das nicht. Die läuft davon unabhängig weiter.

mfg.

[Ranjid (Gast)]

Hier mal mein Code mit der Timer Variante, zeigt leider keine Wirkung, 
was mache ich falsch ?

#define F_CPU 14745600UL

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

ISR(TIMER1_OVF_vect)
{
  OCR1A = ADCW;
}

int main(void)
{

  DDRB |= (1<<PB1);
  DDRC &= ~(1<<PC0);
  PORTC |= (1<<PC0);
  
  TCCR1A |= ((1<<COM1A1) | (1<<WGM10) | (1<<WGM11));  //non inverting, Prescaler 256
  TCCR1B |= ((1<<WGM12) | (1<<CS12));  // Prescaler 256, Fast PWM
  OCR1A = 40;
  
  ADMUX |= ((1<<REFS0) | (1<<REFS1));  //extern rev, adc0
  ADCSRA |= ((1<<ADEN) | (1<<ADIE) | (1<<ADPS0) | (1<<ADPS1) | (1<<ADPS2));  //adc enable, interrupt, prescaler 128, 
  //free running mode ADCSRB
  //ADCSRB |= ((1<<ADTS2) | (1<<ADTS1));
  
  sei();
    while(1)
    {
    //something code !?!
    }
}

[Thomas E. (thomase)]

Ranjid schrieb:
> Okay habe jetzt ADATE in TCCR1A gesetzt und in TCCR1B den Trigger auf
> Timer1 overflow aktiviert, zeigt aber keine Wirkung
Huuuch?
ADATE ist nicht in TCCR1A und in TCCR1B gibt es kein Overflow.
Davon, daß der Timer zum Triggern des ADC benutzt wird, kriegt der gar 
nichts mit.
Gesetzt wird das im ADCSRA-Register(ADATE) und im ADCSRB wird die 
Trigger Source eingestellt(ADTS0 - 2).

mfg.

[Dietrich L. (dietrichl)]

Ranjid schrieb:
> ADCSRA |= ((1<<ADEN) | (1<<ADIE) | (1<<ADPS0) | (1<<ADPS1) | ...
                             ^^^^
Du hast den Interrupt enabled, aber kein ISR!

Gruß Dietrich

[mysterion (Gast)]

Thomas Eckmann schrieb:

> Gesetzt wird das im ADCSRA-Register(ADATE) und im ADCSRB wird die
> Trigger Source eingestellt(ADTS0 - 2).
>
> mfg.

Sry, das meinte ich, habe nur falsch geschrieben, in meinem Source den 
ich gepostet habe, hatte ich es richtig, geht aber trotzdem nicht. Werde 
mir nochmal das Dateblatt unter die Lupe nehmen.

[ADC_vect (Gast)]

Statt ISR(TIMER1_OVF_vect) folgendes benutzen : ISR(ADC_vect). Wenn ich 
das Vorhaben richtig verstanden habe ( ADC messen und in OCRA laden 
sobald der Wert ermittelt ist).

[mysterion (Gast)]

ADC_vect schrieb:
> Statt ISR(TIMER1_OVF_vect) folgendes benutzen : ISR(ADC_vect).
> Wenn ich
> das Vorhaben richtig verstanden habe ( ADC messen und in OCRA laden
> sobald der Wert ermittelt ist).

Habe ich auch schon versucht, jedoch bleibt die LED immer in der 
gleichen Helligkeit. Am PWM liegt es nicht, wenn ich ihn "manuel" 
verstelle, kann man sie die led steuern, der Fehler ist irgendwo beim 
ADC. Ist vielleicht meine Frequenz zu hoch ? Sie liegt nicht in dem 
optimalen Bereich von 50 und 200Khz wie Atmel empfielht. Habe leider 
keinen anderen Quarz zu hand und zum internet möchte ich ungern 
wechseln.

[Thomas E. (thomase)]

#ifndef F_CPU
  #error F_CPU not defined.
#endif

//#include <avr/io.h>
#include <avr/interrupt.h>
/*
#include <stdlib.h>
#include <stdio.h>
#include <stdbool.h>
#include <util\delay.h>
#include <avr/eeprom.h>
#include <avr/boot.h>
#include <avr/pgmspace.h>
#include <avr/signature.h>
#include <avr/wdt.h>
*/

/*
#define TIMER1_MODE TIMER1_FAST_PWM_10
#define TIMER1_PRESCALER 64
#define TIMER1_OCR1A 0
#include "timer.h"
*/

void Initialize(void);

int main(void)
{
  Initialize();

  while(1)
  {

  }
}

void Initialize(void)
{
  ADMUX = (1 << REFS0) | (1 << MUX1);  //Ref = AVcc, Mux = Adc2
  ADCSRA = (1 << ADEN) | (1 << ADATE) | (1 << ADPS2)  
                  | (1 << ADPS1);
  ADCSRB = (1 << ADTS2)  | (1 << ADTS1);//

//  InitTimer();

  TCCR1A = (1 << COM1A1) | (1 << WGM11) | (1 << WGM10);
  TCCR1B = (1 << WGM12) | (1 << CS11) | (1 << CS10);
  DDRB |= (1 << 1);
  TIMSK1 |= (1 << TOIE1);

  sei();
}

ISR(TIMER1_OVF_vect)
{
  OCR1A = ADC;
}

Getestet mit F_CPU = 18,432 MHz und einem 10K-Poti.

Beim Timing ist zu beachten, daß der Timer den ADC nicht triggert, bevor 
die Wandlung fertig ist. Sonst fängt er mittendrin wieder von vorne an 
und liefert natürlich Unsinn.

mfg.

[Thomas E. (thomase)]

ADC_vect schrieb:
> Statt ISR(TIMER1_OVF_vect) folgendes benutzen : ISR(ADC_vect). Wenn ich
> das Vorhaben richtig verstanden habe ( ADC messen und in OCRA laden
> sobald der Wert ermittelt ist).
Man kann auch den ADC-Interrupt verwenden. Aber den Timer-Interrupt 
braucht man trotzdem, da der Adc nicht durch das Overflow-Flag sondern 
durch den Aufruf des Overflow-Interrupts getriggert wird.

mfg.

[ADC_vect (Gast)]

Schönen Dank, Thomas.
Die "ADC Auto Trigger Source" kannte ich bisher noch nicht.
Eine elegantere Möglichkeit wird es wohl nicht geben.

mfg