Atmega 128 und ADC

[franki (Gast)]

Hi,

ich brauche etwas Hilfe. Könnte jemand sagen ob der Folgende Code so 
einigermaßen in Ordnung ist?
Ich bin noch Neuling in diesem Bereich.

Aufgabenstellung
:http://mit.eit.h-da.de/Labor/400_und_500_ADCInterruptTimer.pdf
Ab seite 7.

#include "ledsw.h"
#include "LabIncludes.h"
#include "adc.h"
#include <avr/interrupt.h>



int main(void)
{

  main_ag1();
  
  
}

int main_ag1(void) 
{  
  initPort_AInBOut(); //Ports initialisieren
  init_ADC(1);  //Free running
  start_ADC();
  while(1)     //Polling
    {
    wait_adc();
    showBar(read_adc(2)); //höchstwertige 8 bit ausgeben
    }

  return 1;

}

int main_ag2(void)
{
  initPort_AInBOut(); //Ports initialisieren
  init_ADC(0);  //Single Shot
  
    while(1)     //Polling
    {
    start_ADC();
    wait_adc();
    showBar(read_adc(3));  //10 Bit ausgeben
    }
  return 1;
}

int main_ag3(void)
{
  initPort_AInBOut(); //Ports initialisieren
  init_ADC(1); //Single Shot
  while(1)     //Polling
    {
    start_ADC();  //ADC starten
    wait_adc();    //Warten bis Wandlung abgeschlossen  
    showBarBlink(read_adc(3)); //LED 7 wird zusätzlich zum Blinken gebracht.
    }

  return 1;

}



ISR( ADC_vect)

{ //ISR = Interrupt Service Routine
         
    TIMSK = 0x00; //output compare interrupt ausschalten.
                       //Initial value ist überall 0.  
    putByteLED_B(read_adc(2)); //die höchstwertigen 8 Bit ausgeben.
}


int main_ag4(void)
{
  initPort_AInBOut(); //Ports initialisieren
  init_ADC(1); //Free Runnning
  initPort_AInBOut(); //Ports initialisieren
  start_ADC();  //ADC starten

  //erst das OCIE-Bit (Output Compare Interrupt Enable) im Register TIMSK und
    //das ADIF(ADc Interrupt Flag) eins ist wird der Interrupt ausgelöst
  // Interrupt bei Output Compare Match 
  TIMSK|= (1<<OCIE0);
  sei(); //Im Status-Register wird das Global Interrupt Enable Bit gesetzt, d.h. Interrupts werden zugelassen
  
    
  while(1); //muss auf jeden fall sein. da sonst das prog 
       //beendet wird und die ISR nicht ausgeführt wird
}
////////////////////////////////////////

ISR( ADC_vect1)

{ //ISR = Interrupt Service Routine
         
    TIMSK = 0x00; //output compare interrupt ausschalten.
                       //Initial value ist überall 0.  
    showBar(read_adc(3)); //10 Bit ausgeben als Balken anzeigen.
}

[Krapao (Gast)]

Schrott.

Es sind tote Funktionen im Programm. Unschön.

Es fehlen Funktionen im Programm. Programm nicht kontrollierbar.

Es fehlt eine ISR im Programm. Programm stürzt ab.

Es ist ein ISR Vektorname falsch. Programm stürzt ab.

[franki (Gast)]

die Funktionen wie showBar usw sind in einer anderen .c Datei definiert 
und sollen als Funktionstüchtig angenommen werden. Ist das Verständnis 
so möglich?

[Alexander S. (esko)]

Doppelpost: Beitrag "Atmega 128 und ADC"

[Noname (Gast)]

>die Funktionen wie showBar usw sind in einer anderen .c Datei definiert
>und sollen als Funktionstüchtig angenommen werden. Ist das Verständnis
>so möglich?

Bei nem Kollegen, von dem ich weiss oder zumindest glauben kann, das er 
wenigstens schon zwei, drei Jahre programmiert, würde ich davon 
ausgehen.

Hier, wo sich so allerhand unterschiedliche Vorkenntnisse und 
Erfahrungen tummeln und wenn ich auf den ersten Blick sehe, das die 
Initialisierung(en) fehl(t/en) und wenn ich sowas wie "main_ag1" lese, 
tue ich mir den Rest nicht an.