Problem ADC10 MSP430 interner Tempsensor

[Jonas E. (joenten)]

Hallo zusammen,

ich habe folgendes Problem, ich habe eine Programm geschrieben, dass den 
interen Temperatursensor zur Temperaturmessung benutzt und mir diese 
Temperatur auf einem LCD-Display ausgibt. Das glaubt auch alles 
wunderbar.

Nur wenn ich jetzt den oberen Teil meiner While-Schleife

 
char ch_repeat = 0;
    float f_messureSum = 0;
    float f_messure = 0;
        
    for ( ch_repeat = 5;  ch_repeat > 0; ch_repeat--)
    {
      ADC10_start();
      f_messure = ADC10MEM;                  // Messergeniss von ADC10 speichern  
      f_messureSum += f_messure;
    }
    f_messureAverage = f_messureSum/5;

in eine Funktion einbauen will und dann nur noch die Funktion in der 
While-Schleife aufrufen möchte, geht das irgendwie nicht mehr. Er misst 
dann erst gar nichts mehr und ruft auch nicht die Mess-Funktionen auf.
Weiß jemand an was das liegen könnte?
Das ist der komplette  Code:

#include <msp430g2452.h>
#include "init.h"
#include "lcdroutine.h"
#include <string.h>
#include <stdlib.h>
#include <stdio.h>



/************************************************************
* declarations
************************************************************/

void ADC10_start(void);        // ADC10 start
float f_messureAverage;
float f_messure;
long l_temperature;
char ch_outTemperature[];
char ch_repeat = 0;
  float f_messureSum = 0;
  float f_messure = 0;

void TA_start(void);
void ADC10_start(void);
//void ADC10_messure(void);

/************************************************************
* main
************************************************************/

int main(void)
{
  INIT_msp();              // Initialisierung MSP  
  LCD_Init();                    // Initialisierung LCD
  LCD_CursorOn(0);          // LCD Cursor off   
  
  LCD_SetCursorPos(1,1);        // Cursor Postion setzen  
  LCD_OutString("Temperatur:");     // Ausgabe
  
  while(1)
    {
    char ch_repeat = 0;
    float f_messureSum = 0;
    float f_messure = 0;
        
    for ( ch_repeat = 5;  ch_repeat > 0; ch_repeat--)
    {
      ADC10_start();
      f_messure = ADC10MEM;                  // Messergeniss von ADC10 speichern  
      f_messureSum += f_messure;
    }
    f_messureAverage = f_messureSum/5;  
        
    l_temperature = ((((f_messureAverage/1024)*1.5)-0.986)/0.00355);
    ltoa (l_temperature, ch_outTemperature);
    LCD_SetCursorPos(2,8);        // Cursor Postion setzen  
    LCD_OutString(ch_outTemperature);   // Ausgabe
    LCD_OutString(" Grad   ");        // Ausgabe
    }
}

/************************************************************
* functions
************************************************************/
void TA_start(void)
{
  TACTL   |= TACLR        // Clear Timer
       + MC_1;        // UP-Mode/Timer startet
}
void ADC10_start(void)
{  
  TA_start();
  ADC10CTL0 |= ENC           // ADC10 Enable Conversion
        + ADC10SC;        // ADC10 Start Conversion
  __bis_SR_register(CPUOFF + GIE);    // LPM0, ADC10_ISR will force exit
}

//void ADC10_messure(void)
//{
//
//}

/************************************************************
* Interrupt Service Routines
************************************************************/

#pragma vector=TIMER0_A0_VECTOR
__interrupt void ta0_isr(void)
{
  TACCR0 &= ~CCIFG;
  P1OUT ^= BIT6;
}

#pragma vector=ADC10_VECTOR
__interrupt void ADC10_ISR(void)
{
   __bic_SR_register_on_exit(CPUOFF);        // Clear CPUOFF bit from 0(SR) wake up MSP
   ADC10CTL0 &= ~ADC10IFG;          // ADC10 interupt flag reset
   ADC10CTL0 &= ~ENC;            // rest ENC
   TACTL   &= ~(MC0 | MC1);        // Timer stoppt
 }

danke für eure Hilfe :-)

[Rufus Τ. F. (rufus)]

Jonas Enderlin schrieb:
> Das ist der komplette  Code:

Nö. Das ist nicht der Code, in dem Du uns zeigst, wie Du versucht hast, 
Deine Messroutine in eine Funktion auszulagern, das ist nicht der Code, 
der "irgendwie" nicht funktioniert, sondern das ist der Code, der bei 
Dir (wohl) funktioniert.

Das Wort "messure" solltest Du noch mal durch Deinen spell-checker 
jagen.

[Jonas E. (joenten)]

Ja das ist der Code, nachdem ich meine Änderung vorgenommen habe und die 
Funktion nicht aufgerufen wird.

#include <msp430g2452.h>
#include "init.h"
#include "lcdroutine.h"
#include <string.h>
#include <stdlib.h>
#include <stdio.h>



/************************************************************
* declarations
************************************************************/

void ADC10_start(void);        // ADC10 start
float f_measureAverage;
float f_measure;
long l_temperature;
char ch_outTemperature[];
char ch_repeat = 0;
float f_measureSum = 0;
float f_measure = 0;

void TA_start(void);
void ADC10_start(void);
void ADC10_measure(void);

/************************************************************
* main
************************************************************/

int main(void)
{
  INIT_msp();              // Initialisierung MSP  
  LCD_Init();                    // Initialisierung LCD
  LCD_CursorOn(0);          // LCD Cursor off   
  
  LCD_SetCursorPos(1,1);        // Cursor Postion setzen  
  LCD_OutString("Temperatur:");     // Ausgabe
  
  while(1)
    {
    void ADC10_measure(void);    
      
    l_temperature = ((((f_measureAverage/1024)*1.5)-0.986)/0.00355);
    ltoa (l_temperature, ch_outTemperature);
    LCD_SetCursorPos(2,8);        // Cursor Postion setzen  
    LCD_OutString(ch_outTemperature);   // Ausgabe
    LCD_OutString(" Grad   ");        // Ausgabe
    }
}

/************************************************************
* functions
************************************************************/
void TA_start(void)
{
  TACTL   |= TACLR        // Clear Timer
       + MC_1;        // UP-Mode/Timer startet
}
void ADC10_start(void)
{  
  TA_start();
  ADC10CTL0 |= ENC           // ADC10 Enable Conversion
        + ADC10SC;        // ADC10 Start Conversion
  __bis_SR_register(CPUOFF + GIE);    // LPM0, ADC10_ISR will force exit
}

void ADC10_measure(void)
{
  char ch_repeat = 0;
  float f_measureSum = 0;
  float f_measure = 0;
      
  for ( ch_repeat = 4;  ch_repeat > 0; ch_repeat--)
  {
    ADC10_start();
    f_measure = ADC10MEM;                  // Messergeniss von ADC10 speichern  
    f_measureSum += f_measure;
  }
  f_measureAverage = f_measureSum/4;  
}

/************************************************************
* Interrupt Service Routines
************************************************************/

#pragma vector=TIMER0_A0_VECTOR
__interrupt void ta0_isr(void)
{
  TACCR0 &= ~CCIFG;
  P1OUT ^= BIT6;
}

#pragma vector=ADC10_VECTOR
__interrupt void ADC10_ISR(void)
{
   __bic_SR_register_on_exit(CPUOFF);        // Clear CPUOFF bit from 0(SR) wake up MSP
   ADC10CTL0 &= ~ADC10IFG;          // ADC10 interupt flag reset
   ADC10CTL0 &= ~ENC;            // rest ENC
   TACTL   &= ~(MC0 | MC1);        // Timer stoppt
 }

[Rufus Τ. F. (rufus)]

Jonas Enderlin schrieb:
> while(1)
>     {
>     void ADC10_measure(void);

Das ist kein Funktionsaufruf. Das ist ein Funktionsprototyp.

Wenn Du beide void entfernst, wird ein Funktionsaufruf daraus.


Wie wäre es mit einem C-Buch?

Nach wie vor zu empfehlen:
Brian Kernighan & Dennis Ritchie, "Programmieren in C", 2. Ausgabe, 
Hanser-Verlag.