Atmega8 Timer1 Interrupt Routine wird nicht angesprungen

Auf ADC0 ist am Mittelabgriff eines Potentiometer mit 5 V Versorgung 
angeschlossen.

Wenn ich den Code aus der Interrupt-Routine in der Hauptschleife des 
Hauptprogramms ausführe, dann wird die PWM mit 2kHz Frequenz ausgeführt. 
Über das Potentiometer läßt sich dann die Pulsbreite der PWM stufenlos 
einstellen. Nur der Atmega8 führt dann die AD-Wandlung permanent aus.

Ich möchte allerdings, dass die AD-Wandlung nur einmal pro Interrupt 
Zyklus ausgeführt wird.

Warum genau lässt Du nicht den ADC frei laufen und machst das PWM-Update 
in der ISR "ADC Conversion complete"?

Vielen Dank für die Kommentare!

Högli schrieb:
> In der ISR die ADC Messung starten. Dann in der Hauptschleife "// Warten
> bis die AD-Wandlung abgeschloßen ist" oder den ADC Interrupt nutzen. In
> der ISR auf Abschluß der Wandlung warten ist selten sinnvoll.

Ich habe den Kommentar umgesetzt, und siehe da es funktioniert. Dabei 
habe ich der Übersicht halber noch einzelne Änderungen durchgeführt. Der 
neue C-Code lautet folgendermassen:

1

#define F_CPU 160000000

2

#include <avr/io.h>

3

#include <stdint.h>

4

#include <avr/interrupt.h>

5

6

7

volatile uint16_t buffer;

8

volatile int8_t flag;

9

10

ISR(TIMER1_COMPA_vect)

11

{

12

  flag=1;

13

}

14

15

16

17

int main()

18

{  

19

  //Alle Interrupts ausschalten

20

  cli();

21

22

  //Hilfsport als Ausgang definieren und zuruecksetzen

23

  DDRD = DDRB | (1 << PD0);

24

  PORTD = 0;

25

26

27

  // Timer für PWM initialisieren

28

  // OC1A auf Ausgang

29

  DDRB = DDRB | (1 << PB1 );

30

  TCCR1A = TCCR1A | (1 << COM1A1);

31

  TCCR1A = TCCR1A | (0 << COM1A0);

32

  TCCR1B = TCCR1B | (1 << WGM13);

33

  TCCR1B = TCCR1B | (0 << WGM12);

34

  TCCR1B = TCCR1B | (1 << CS10);

35

  TCCR1B = TCCR1B | (0 << CS11);

36

  TCCR1B = TCCR1B | (0 << CS12);

37

  TCCR1A = TCCR1A | (0 << WGM10);

38

  TCCR1A = TCCR1A | (0 << WGM11);

39

40

  // PWM Frequenz auf 1953 Hz stellen bei 16 Mhz Oscillator

41

  ICR1 = 4095;

42

43

  //Timer Interrupt initialisieren und globale Interrupts enable  

44

  TIMSK = TIMSK | (1 << OCIE1A);

45

  sei();

46

47

  // AD-Wandler initialisieren und auf 10 Bit stellen

48

  ADMUX = (1<<REFS0);

49

  ADCSRA = (1<<ADEN);

50

51

  //Startwert für PWM festlegen

52

  OCR1A = 1000;

53

54

  while( 1 )

55

  {  if (flag==1)

56

    {

57

        //Hilfsport setzen

58

        PORTD |= 1<<PD0;

59

60

        // Wandlung vom analogen Eingang starten

61

        ADCSRA |= (1<<ADSC);

62

63

        // Warten bis die AD-Wandlung abgeschloßen ist

64

        while ( !(ADCSRA & (1<<ADIF)) );

65

66

        //Hilfsport zurücksetzen

67

        //  PORTD &= ~(1 << PD0);

68

        PORTD = 0;

69

70

        buffer = ADC;

71

//        OCR1A = buffer * 4;

72

        OCR1A = (buffer<<2);

73

        flag = 0;

74

    }

75

76

  }

77

}

Den ADC möchte ich nicht frei laufen lassen, um für spätere Anwendungen 
Rechenkapazität zu sparen.

??? Du willst Rechenzeit sparen indem Du durch den Timer ein Flag setzt, 
in der Mainschleife und dann wartest bis der ADC fertig ist???

So ist es! Vielleicht mache ich noch einen Denkfehler.

Was ist denn aus Deiner Sicht der Vorteil, wenn man den ADC frei laufen 
läßt ?

Guten Tag,

es gibt noch ein weiteres Problem mit diesem Codeabschnitt:

1

// Wandlung vom analogen Eingang starten

2

ADCSRA |= (1<<ADSC);

3

4

// Warten bis die AD-Wandlung abgeschloßen ist

5

while ( !(ADCSRA & (1<<ADIF)) );

Wenn man nach einen atMega Datenblatt geht, dann ist es so richtig:

1

// Wandlung vom analogen Eingang starten

2

ADCSRA |= (1<<ADSC);

3

4

// Warten bis die AD-Wandlung abgeschloßen ist

5

while ( (ADCSRA & (1<<ADSC)) ) { }

>
>

1

2

> // Warten bis die AD-Wandlung abgeschloßen ist

3

> while ( (ADCSRA & (1<<ADSC)) ) { }

Uwe S. schrieb:
> // Warten bis die AD-Wandlung abgeschloßen ist
> while ( (ADCSRA & (1<<ADSC)) ) { }

Und was passiert, wenn man die Zeile ganz weg läßt ?

Funktionieren tut es auch ohne dieser Zeile.

Danke spess53,

für die Klarstellung, wenn man nur das Datenblatt zum  Verständnis lesen 
würden, wenn hätte man ein Problem weniger.

Wenn ich das Datenblatt nicht lesen würde, dann wäre ich nicht soweit 
gekommen!

Eine bessere Frage wäre wohl gewesen, warum der ATMEGA8 die ADC Wandlung 
noch zu Ende führt:
> while ( (ADCSRA & (1<<ADSC)) ) { }

Meine Erwartung wäre gewesen, dass der ATMEGA8 im Code fortschreitet 
(wenn die Zeile nicht vorhanden ist)und die ADC Wandlung abbricht.

Händeringer schrieb:
> In Anbetracht der Tatsache, daß es sich um DATENBÜCHER mit mehreren
> hundert Seiten handelt, in denen

die Kapitel mir der Beschreibung zu den einzelnen Modulen immer nur 
wenige Seiten umfassen, ist das durch aus machbar.

Oliver