ATmega328p-pu / TIMER bzw. OCR0A-Register

[Felix (Gast)]

Hallo,

Ich habe ein Problem und hoffe, Ihr könnt mir helfen,

Ich habe gerade versucht, einen Timer für den ATmega328p zu 
programmieren.
Ich verwende, (weil ich einen Motor ansteuern will, den 
PWM-phasenrichtigen Modus)
Laut Datenblatt S.108 ist das Mode 1.

Wenn ich aber das OCR0A-Register beschreibe, dass ja ständig mit dem 
aktuellen Zählwert (TCNT0-Register) verglichen wird, habe ich in manchen 
Fällen ein Problem.
Einerseits, wenn der Wert des OCR0A-Registers zu klein wird (also in die 
nähe von Null kommt) oder, wenn er zu groß wird (also in die nähe von 
255 kommt). Wenn mein OCR0A Register einen so einen kritischen Wert 
erhält, spielt bei mir alles verrückt. Es verändert sich die Frequenz 
(wird halb so langsam) und das Puls-Pausenverhältnis hüpft auf rund 50 
%.
Weiß jemand von euch, warum man dem OCR0A-Register keine besonders hohen 
oder niedrigen Werte geben darf ?
Danke schon mal im Voraus,

LG Felix

[Sinnlos (Gast)]

Felix schrieb:
> Weiß jemand von euch, warum man dem OCR0A-Register keine besonders hohen
> oder niedrigen Werte geben darf ?

Klar darf man. Warum hast du dein Programm nicht angehangen?

[Felix (Gast)]

Ok, hier is mein Programm..ich lese von einem Potentiometer den Wert 
über einen AD-Wandler ein, und will darüber das Puls-Pausenverhältnis 
regeln.
Aber wenn der Wert zu gering ist (Der Wert wird dann insr OCR0A-Register 
übernommen) fängt meine Pulsweite zu spinnen an.. Hier mal mein Code:

//ATmega328p-pu
#define F_CPU 8000000UL

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

void adc_init();
void timer_init();

int main(void)
{
  
  DDRB |= (1<<PINB0); //Steuerausgang für PWM
  adc_init();
  timer_init();
    
    while (1);
  return 0;
}

ISR(ADC_vect)
{
  cli();
  OCR0A = (ADC/4); //Weil ADC-Max-Value 1024 (10 bit) und TIMER-Max-Value 256 (8-bit) ist
  ADCSRA |= (1<<ADSC); //Jede Konvertierung wird nur durch dieses Bit gestartet ("Single Conversion Modus")
  sei();
}

ISR(TIMER0_COMPA_vect)
{  
  cli();
  PORTB ^= (1<<PINB0); //Toggeln  
  sei();
}

void timer_init()
{
  //Beim Initialisieren sollte OCR0A einen Wert haben
  //Bei ORRA <= 1 gibt es aber Probleme!
  OCR0A = 2;
  
  //Start Timer0 (PWM->Phasenrichtig) mit TOP von 0xFF
  //im nichtinvertierenden phasenrichtigen Modus
  TCCR0A |= (1<<CS00)|(1<<WGM00);
  
  //CPU-Clock und Prescaller von 0
  TCCR0B |= (1<<CS00);
  
  //Enable Timer Compare Match-Interrupt
  TIMSK0 |= (1<<OCIE0A);
  
  //Set the I-bit in SREG -> start Interrupts
  sei();
}

void adc_init()
{
  ADMUX |= (1<<REFS0)|(1<<MUX2)|(1<<MUX0);
  // Referenzspannung-> AVCC intern, ADC5 als AD-Wandler
  
  ADCSRA |= (1<<ADEN)|(1<<ADPS1)|(1<<ADPS2)|(1<<ADIE);
  //AD_enable, und Prescaller auf 64 für f=125KHz, AD-Interrupt-enable
  
  sei();
  //schaltet Interrupts ein
  
  ADCSRA |= (1<<ADSC);
  //Startet die ERSTE Konvertierung... (Im "Single-Conversion-Mode" muss das Bit
  //ADSC jedesmal manuell beschrieben werden, um die Konvertierung zu starten  
}

[Felix (Gast)]

PS: oder wenn der Wert zu Groß ist, fängt es genauso zu spinnen an, hier 
mal ein Foto von "Spinnen".
Danke,

[Sinnlos (Gast)]

Ich würde versuchen einen HW-Ausgang des Timers zu nutzen. Also zB PD6 
(OC0A) oder so. Toggeln in der ISR verbraucht Zeit um in die ISR und 
zurück zu springen. Da könnte der TCNT0 schon einen Wert haben, der 
nicht mehr zum bisherigen Ablauf bei sehr kleinen OCR0A paßt.

Die cli, sei in den ISRs sind witzlos, da sowieso keine anderen ISR 
parallel laufen können.

[Felix Adam (Gast)]

Um den HW-PWM-Ausgang zu nutzen, ändere wie folgt ab:

TCCR0A = (1<<COM0A1) | (1<<COM0A0) | (1<<WGM00);

[Dieter F. (Gast)]

?

 TCCR0A |= (1<<CS00) ...

CS00 gibt es da nicht :-)

[spess53 (Gast)]

Hi

Die CLI/SEI sind mehr als flüssig-> überflüssig.

Das Beschreiben des OCR0A-Registers sollte im entsprechenden Interrupt 
erfolgen. Zeit dazu hast du. Die Timerfrequenz kann niedriger gewählt 
werden.

Die Angaben des 'Oszis' in Bezug auf Dutycycle, Periode und Frequenz 
sind mehr als merkwürdig.

MfG Spess

[spess53 (Gast)]

Hi

Nachtrag:

Zum Synchronisieren von ADC und Timer ist das Autotrigger-Register 
ADCSRB geeignet.

OCR0A = (ADC/4) lässt sich durch Setzen von ADLAR einsparen.

MfG Spess

[Felix (Gast)]

Ja, Ihr seid genial!
Danke an Euch alle. hab jetzt insofern abgeändert, dass ich gleich den 
TimerPin PIND6 als Output konfiguriert habe, und den, statt des 
Ursprünglichen PINB0 verwende, die cli(); und sei(); hab ich auch 
weggelassen und das TCCR0A Register hab ich entsprechend für den PIND6 
konfiguriert.
Hier noch mein funktionierender Code:

//ATmega328p-pu
#define F_CPU 8000000UL

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

void adc_init();
void timer_init();

int main(void)
{
  
  //DDRB |= (1<<PINB0); //Steuerausgang für PWM
  DDRD |= (1<<PIND6);
  adc_init();
  timer_init();
    
    while (1);
  return 0;
}

ISR(ADC_vect)
{
  //cli();
  OCR0A = (ADC/4); //Weil ADC-Max-Value 1023 (10 bit) und TIMER-Max-Value 255 (8-bit) ist
  ADCSRA |= (1<<ADSC); //Jede Konvertierung wird nur durch dieses Bit gestartet ("Single Conversion Modus")
  //sei();
}

ISR(TIMER0_COMPA_vect)
{  
  //cli();
  //PORTB ^= (1<<PINB0); //Toggeln  
  //sei();
}

void timer_init()
{
  //Beim Initialisieren sollte OCR0A einen Wert haben
  //Bei ORRA <= 1 gibt es aber Probleme!
  OCR0A = 2;
  
  //Start Timer0 (PWM->Phasenrichtig) mit TOP von 0xFF
  //im nichtinvertierenden phasenrichtigen Modus
  TCCR0A |= (1<<WGM00)|(1<<COM0A1)|(1<<COM0A0);
  
  //CPU-Clock und Prescaller von 0
  TCCR0B |= (1<<CS00);
  
  //Enable Timer Compare Match-Interrupt
  TIMSK0 |= (1<<OCIE0A);
  
  //Set the I-bit in SREG -> start Interrupts
  sei();
}

void adc_init()
{
  ADMUX |= (1<<REFS0)|(1<<MUX2)|(1<<MUX0);
  // Referenzspannung-> AVCC intern, ADC5 als AD-Wandler
  
  ADCSRA |= (1<<ADEN)|(1<<ADPS1)|(1<<ADPS2)|(1<<ADIE);
  //AD_enable, und Prescaller auf 64 für f=125KHz, AD-Interrupt-enable
  
  sei();
  //schaltet Interrupts ein
  
  ADCSRA |= (1<<ADSC);
  //Startet die ERSTE Konvertierung... (Im "Single-Conversion-Mode" muss das Bit
  //ADSC jedesmal manuell beschrieben werden, um die Konvertierung zu starten  
}