MSP430 - Problem mit ADC und DTC, mehrere Channels wiederholt lesen

[Markus X. (markusx)]

Hallo,

ich schreibe mit Energia an einem kleinen Programm zur Spannungs-/ 
Strommessung.
Ziel ist es mittels eines Timer-Interrupts alle 100ms (funktioniert) die 
Werte aufzunehmen und alle 1s den Mittelwert zu bilden und diesen dann 
auszugeben, bzw. aufzuaddieren.

Der Interrupt funktioniert und ich kann mir auf den Display testweise 
die Zeit anzeigen lassen.

Zum ADC: Ich habe auf den Eingängen A0, A1, A4 und A5 die Analogwerte 
anliegen. Diese wollte ich mit dem ADC continuierlich sampeln und 
mittels DTC auf ein Array schreiben.
Aus diesem Array hätte ich dann bei einem Interrupt den gerade 
darinstehenden Wert genommen.

Soweit der Plan... Leider funktionert das ganze noch nicht. Scheinbar 
wird nichts in das Array geschrieben (?).

Danke für die Hilfe!

MfG
Markus

#include <LiquidCrystal.h>
#include <msp430.h>



LiquidCrystal lcd(19, 18, 14, 13, 12, 11);
const int buttonPin = PUSH2;

int spannung = 0, mwspannung=0;
int strom = 0, mwstrom =0;
int zaehler=0;
int verbrauch=0;
int gesamt=0;
unsigned int results[6];



void setup() {
  lcd.begin(16, 2);
   
 WDTCTL = WDTPW + WDTHOLD;                       // Disable watchdog reset
  
  ADC10CTL0 &= ~ENC;                             
  ADC10CTL1 |= CONSEQ_3+ INCH_5; 
  ADC10CTL0 |= ADC10SHT_1 + ADC10ON + MSC; 
  ADC10AE0 |= 0x33; // select channel A0,A1,A4,A5
  ADC10SA = (int)results;
  ADC10DTC0 |= ADC10CT;
  ADC10DTC1 = 0x06;
  ADC10CTL0 |= ADC10SC + ENC; // start conversions



    
    TA0CTL &=~MC_1;            
    
    TA0CCR0 = 0x9C34;   //rechnerisch: 9C40, aber Korrektur, da zu langsam
    TA0CCTL0 = CCIE; 
    TA0CTL = TASSEL_2 + ID_3 + MC_1 ;         
    _EINT();


                 
  
 
}

void loop() {
  //lcd.clear();
 

  lcd.setCursor(3, 0);
  lcd.print(results[0]);   //Test ob Array beschrieben wird
 
 lcd.setCursor(12, 0);
   lcd.print(results[1]); 
    
}

//Interrupt Routine entfernt

[Go MSP (Gast)]

Probier doch einmal den TI Code:
Beitrag "Re: MSP430 ADC10 mehrere Kanäle"

[Markus X. (markusx)]

Hallo,

danke, aber der TI Code, bzw. die hier im Forum liegenden Codes sind mir
bekannt. (denke zumindest alles gefunden zu haben)

TI-Init:

  WDTCTL = WDTPW + WDTHOLD;                 // Stop WDT
  ADC10CTL1 = INCH_2 + CONSEQ_1;            // A2/A1/A0, single sequence
  ADC10CTL0 = ADC10SHT_2 + MSC + ADC10ON + ADC10IE;
  ADC10DTC1 = 0x03;                         // 3 conversions
  ADC10AE0 |= 0x07;                         // P2.2,1,0 ADC10 option
select
  P1DIR |= 0x01;                            // Set P1.0 output



Mein Init:

  WDTCTL = WDTPW + WDTHOLD;                       // Disable watchdog
reset
  ADC10CTL0 &= ~ENC;                              // ADC aus
  ADC10CTL1 |= CONSEQ_3+ INCH_5;                  // Kont. Samplen,
A5-A0
  ADC10CTL0 |= ADC10SHT_1 + ADC10ON + MSC;        //
  ADC10AE0 |= 0x33; // select channel A0,A1,A4,A5
  ADC10SA = (int)results;
  ADC10DTC0 |= ADC10CT;
  ADC10DTC1 = 0x06;
  ADC10CTL0 |= ADC10SC + ENC; // start conversions


Meine Frage geht gerade in die Richtung, ob die Zeile:

ADC10SA = (int)results;

So funktioniert. Ansonsten gibt es ja nicht viele Ansätze a) ADC geht /
geht nicht -> init falsch
b) Arrayzuweisung geht nicht ->keine Werte da

[Rufus Τ. F. (rufus)]

Vergleich mal ganz genau, was hier passiert:


TI:

> ADC10CTL0 = ADC10SHT_2 + MSC + ADC10ON + ADC10IE;


Du:

> ADC10CTL0 &= ~ENC;                              // ADC aus
> ADC10CTL0 |= ADC10SHT_1 + ADC10ON + MSC;        //

Das TI-Beispiel initialisiert das komplette ADC10CTL0-Register, Du aber 
belässt es im Reset-Zustand und änderst nur einige Bits darin.

Das muss nicht die Ursache Deines Problems sein, kann aber.

[Go MSP (Gast)]

Bei der Initialisierung muss nach dem disablen aufs Busy gewartet 
werden.

ADC10CTL0 &= ~ENC;
while(ADC10CTL1 & BUSY);

Rufus Τ. Firefly schrieb:
> Das TI-Beispiel initialisiert das komplette ADC10CTL0-Register, Du aber
> belässt es im Reset-Zustand und änderst nur einige Bits darin.

Das sollte natürlich auch geprüft werden.

[Markus X. (markusx)]

Hallo,

danke für die Hilfe, das mit dem verodern ... da wollte ich einfach auf 
Nummer sicher gehen nichts wichtiges zu überschreiben. Ich habs jetzt 
mit Blick aufs Handbuch geändert, leider will er immer noch nicht :(

Hier ein hoffentlich verständliches Testprogramm

LiquidCrystal lcd(19, 18, 14, 13, 12, 11);


int zaehler=1;
int spannung;
unsigned int results[6];
int rtc=0;



void setup() {
  // set up the LCD's number of columns and rows: 
  lcd.begin(16, 2);
  
 
 WDTCTL = WDTPW + WDTHOLD;                       // Disable watchdog reset

  ADC10CTL0 = 0x00;
  while(ADC10CTL1 & BUSY);
  ADC10CTL1 = CONSEQ_1+ INCH_5;             //continuous sample mode, 
  ADC10CTL0 = ADC10SHT_2 + ADC10ON + MSC;   //sample and hold time, adc on, cont. sample
  ADC10AE0  = 0x33;                         // select channel A0,A1,A4,A5
  ADC10SA   = results[0];
  ADC10DTC0 = ADC10CT;
  ADC10DTC1 = 0x06;
  ADC10CTL0 |= ADC10SC + ENC;                // start conversions


  TA0CTL &=~MC_1;            
  TA0CCR0 = 0x9BB2;   //rechnerisch: 9C40, aber Korrektur, da zu langsam
  TA0CCTL0 = CCIE; // Capture/compare interrupt flag
  TA0CTL = TASSEL_2 + ID_3 + MC_1 ;         // Timer A config: SMCLK, /8, count up, overflow interrupt enabled
  _EINT();

}

void loop() {
  //lcd.clear();
 lcd.setCursor(0, 0); lcd.print("V:");
 lcd.setCursor(0, 1); lcd.print("t:");
 
  lcd.setCursor(3, 0);
  lcd.print(spannung);
  lcd.setCursor(3, 1);
  lcd.print(rtc);
 
}

#pragma vector = TIMER0_A0_VECTOR                   // Timer A overflow (TAIFG) interrupt
__interrupt void TIMER0_A0_ISR (void)                //
{   zaehler++;
   
  if (zaehler==50){
  rtc++;  
   spannung = results[1];
   zaehler=0;
       };
}

[Tapp (Gast)]

Die Adresse ist nicht

ADC10SA   = results[0];

sondern

ADC10SA   =  &(results[0]);

[Markus X. (markusx)]

Oha,

beim ganzen probieren vergessen... Danke


aber &(results[0]) führt zur Fehlermeldung

und (int)&(results[0]) führt auch nicht zur gewünschten Funktion

[Markus X. (markusx)]

Hallo,


Hier die Lösung:

 ADC10DTC0 = ADC10CT + ADC10B1;

ADC10B1 muss scheinbar auch gesetzt werden.