ATtiny vco (voltage controlled oscillator) mit timer

[Alexander M. (alex1983m)]

Guten Abend,
zu meiner Hardware:
ATtiny13, Dragon, AtmelStudio 6

vorgeschichte:
- AD-Wandlung an PB3 & PB4 (ohne Timer-Interrupt)funktioniert Wunderbar
- Timer-Interrupt (ohne AD-Wandlung) funktioniert Wunderbar

zu meinem Problem:
ich würde gerne beides kombinieren um daraus 2 vco's (z.B PB0 & PB1) zu 
realisieren, zur not tuts auch nur ein vco.

Hat irgendjemand ideen?

Hier mein Code der so nicht funktioniert:

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

int portb_out();
int init_timer();
int config_adc(int x);


volatile int a = 0;            // Zählvariable
volatile int b = 0;            // Zählvariable


unsigned int adc3h = 0;
unsigned int adc3l = 0;
unsigned int adc2h = 0;
unsigned int adc2l = 0;

ISR(TIM0_OVF_vect) 
{
  ++a;
  ++b;        
}


int main(void) 
{
  portb_out();            // aufruf des Prototypen zur Initialisierung
  init_timer();           // aufruf des Prototypen zur Initialisierung
  
  
  while(1) 
  {
    config_adc(2);
    adc2l = ADCL;                  // schreibt die untersten 2 Bits
    adc2h = ADCH;                  // schreibt die obersten 8 Bits 
    ADCSRA = ADCSRA & (~(1<<ADEN));                  // stoppt den ADC
    
    if (a == adc2h)
    {
      PORTB ^= 0b00000001;    // ^= toggelt PB0
      a = 0;
    } 
    
    config_adc(3);                          
    adc3l = ADCL;                  // schreibt die untersten 2 Bits 
    adc3h = ADCH;                  // schreibt die obersten 8 Bits 
    ADCSRA = ADCSRA & (~(1<<ADEN));                  // stoppt den ADC
    if (b == adc3h)
    {
      PORTB ^= 0b00000010;    // ^= toggelt PB1
      b = 0;
    }
    
  }
}



int portb_out()
{
  DDRB = DDRB | 0b00000011;                    
  PORTB = 0;       
}

int init_timer()
{
  TCCR0B=0x01;
  TIMSK0 |=1<<TOIE0;          // enable timer overflow interrupt
  sei();
}

int config_adc(x)
{
  ADMUX = x;                          // Auswahl des ADC-Kanals:
  DIDR0 = 0b00011000;                      
  ADMUX |= (0 << REFS0);                    
  ADMUX |= (1 << ADLAR);                  
  ADCSRA |= (1 << ADPS2) | (1 << ADPS1) | (1 << ADPS0);     
  ADCSRB |= (0 << ADTS2) | (0 << ADTS1) | (0 << ADTS1);   
  ADCSRA |= (1 << ADEN);                    // aktiviert den ADC
  ADCSRA |= (1 << ADSC);                    
  while(ADCSRA & (1<<ADSC))                  
  {
    asm volatile("nop");                  // NOP
  }
}

[Alexander M. (alex1983m)]

Hat jmd. eine Idee?

[Jonas B. (jibi)]

Moin,

warum konfigurierst du die Adc's in deiner Hauptschleife ständig neu? Es 
reicht wenn man das einmal macht.

> while(1)
>  {
>    config_adc(2);
>  ...

Sonst sind noch ein paar weitere Denkfehler drin.
Schau mal dieser Code wartet wohl bis der Adc einen Wert gesampled hat:

>while(ADCSRA & (1<<ADSC))
>  {
>    asm volatile("nop");                  // NOP
>  }

So und danach solltest du denn Wert auch speichern, fehlt bei dir. 
Ansonsten musst du einiges umstellen, damit dein Code Sinn ergibt.

Gruß Jonas

[Alexander M. (alex1983m)]

Beim ständigen konfigurieren der ADC'c übergebe ich immer den Wert 2 und 
danach 3, dadurch wähle ich zu erst den ADC von Port PB4 aus lese den 
AD-Wert ein, dann wird der Port PB3-ADC gewählt und der AD-Wert 
eingelessen.

Funktioniert das mit der while-schleife so nicht? Ich speicher doch den 
Wert des ADC in der main-routine.

...
> config_adc(2);
>     adc2l = ADCL;                  // schreibt die untersten 2 Bits
>     adc2h = ADCH;                  // schreibt die obersten 8 Bits
...


Und was müsste man sonst am Code ändern damit er Sinn ergibt, versteh 
ich irgendwie nicht ganz?!

Gruß Alex

[Karl H. (kbuchegg)]

Alexander M. schrieb:

> Und was müsste man sonst am Code ändern damit er Sinn ergibt, versteh
> ich irgendwie nicht ganz?!

Schau dir an, wie man den ADC sauber benutzt.

http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Analoge_Ein-_und_Ausgabe#Nutzung_des_ADC

Da findest du auch saubere Routinen, die du mehr oder weniger direkt 
benutzen kannst. Wenn du ADLAR haben willst, dann musst du sie ein wenig 
verändern, aber im Großen und Ganzen haben die Funktionen eine sinnvolle 
Aufteilung in Initialisierung und 'Mache_eine_Messung'

Und die Signalgenerierung verlagerst du als ganzes in die Timer ISR. 
Denn das ist wichtiger das dieser Teil absolut regelmässig als ganzes 
abläuft, als das der ADC-Wert ein paar µ-Sekunden früher oder später zur 
Verfügung steht.