MSP430 ADC10 bei Taster-Interrupt

[Dominik R. (vision)]

Hallo,

ich habe mal wieder ein kleines Problemmit dem MSP430. Und zwar möchte 
ich mittels ADC10 die Spannung am P1.0 ermitteln. So lange ich das 
normal in der Endlosschleife mache ist das auch kein Problem. Nun möchte 
ich die Messung aber durch einen Taster-Intterrupt auslösen und dann 
funktioniert es nicht mehr. Hat irgend jemand eine Idee?

#include <msp430.h>
#include "stdint.h"

#define BUTTON_T   BIT6
#define BUTTON_Z   BIT7
#define LED_GREEN  BIT5
#define LED_YELLOW  BIT6
#define LED_RED    BIT7

volatile uint16_t value;

void configureADC(void){
  ADC10CTL0 = ADC10SHT_2 + ADC10ON + ADC10IE; 
  ADC10CTL1 = INCH_0;             // Channel 0
  ADC10AE0 |= BIT0;
}

void sampleADC(void){
  //start sampling
  ADC10CTL0 |= ENC + ADC10SC;
  __bis_SR_register(CPUOFF + GIE);
  value = ADC10MEM;

  if(value < 341){
    P1OUT ^= LED_GREEN;
  }
  else if(value < 682){
    P1OUT ^= LED_YELLOW;
  }
  else{
    P1OUT ^= LED_RED;
  }
}


void init(void){
  WDTCTL = WDTPW + WDTHOLD;            // Watchdog Timer ausschalten
  P1DIR |= (LED_GREEN + LED_YELLOW + LED_RED);  // Set 1.5, 1.6, 1.7 to output direction
  P2SEL &= ~(BUTTON_T + BUTTON_Z);        // 2.6 and 2.7 as GPIO
  P1OUT |= (LED_GREEN + LED_YELLOW + LED_RED);  // turn off all LEDs
  __enable_interrupt();
}

void buttonTinit(void){
  P2REN |= (BUTTON_T + BUTTON_Z);  // Buttons
  P2OUT |= (BUTTON_T + BUTTON_Z);  // with PullUP

  P2IE |= (BUTTON_T + BUTTON_Z);  // Button_T interrupt enabled
  P2IFG &= ~(BUTTON_T + BUTTON_Z);// Interrupt-Flag clear
}


int main(void) {

  init();
  configureADC();
  buttonTinit();
  __low_power_mode_1();
    
  for(;;) {
  }

  //für Compiler
  //return 0;
}

//Button ISR
#pragma vector=PORT2_VECTOR
__interrupt void Port_2 (void){
  __delay_cycles(20);
  sampleADC();
  P2IFG &= ~(BUTTON_T + BUTTON_Z);
}

//ADC10 ISR
#pragma vector=ADC10_VECTOR
__interrupt void ADC10_ISR (void){
  __bic_SR_register_on_exit(CPUOFF);
}

[Dominik R. (vision)]

So,ich habe nun herausgefunden, das die A/D-Wandlung zwar erfolgt, aber 
der Inhalt des ADC10MEM-Registers nicht in die Variable "value" 
übernommen wird.
Kann mir jemand sagen, wo der Fehler ist?

[Family User Guide ist Spitze (Gast)]

Dominik R. schrieb:
> __bis_SR_register(CPUOFF + GIE);

was bewirkt die Anweisung?

Dominik R. schrieb:
> __bic_SR_register_on_exit(CPUOFF);

und was passiert danach?

Du hast es programmiert, du musst es wissen.

[Vision (Gast)]

Stimmt auffallend. Erst Variable zuweisen, dann in LPM... Ist schon 
mist, wenn an den Fingern immer die Copy-Paste klebt...

Danke für den Hinweis

[Family User Guide ist Spitze (Gast)]

Vision schrieb:
> Erst Variable zuweisen, dann in LPM... Ist schon
> mist, wenn an den Fingern immer die Copy-Paste klebt...

Das ist aber nur ein Teil des Übels.

delay in ISR, mitten in der ISR in den LPM wechseln, ...

Da fehlt grundsätzliches Wissen. Wenn man den LPM nicht versteht, sollte 
man ohne programmieren.