ADC Interrupt beim stm32f4

[Philipp B. (phill4563)]

Hallo liebe Community,
ich habe eine Frage bezüglich meines Programms.
Es geht zunächst darum Werte vom ADC über einen Interrupt in eine 
Variable zu schreiben.

Der Eingangskanal PA1 wird von mir extern über einen Potti beschaltet.
In der interrupt routine sollte eigentlich der Wert an "x" übergeben 
werden,  er springt aber nicht in diese Funktion hinein.
Später sollen einmal 2 channels jeweils in 2 variabeln gespeichert 
werden, dass ist momentan aber noch nicht relevant.

Hier mal mein bsp Programm

#include "stm32f4xx.h"
#include "stm32f4xx_conf.h"
 
GPIO_InitTypeDef  GPIO_InitStructure;
ADC_InitTypeDef ADC_InitStructure;
ADC_CommonInitTypeDef ADC_CommonInitStructure;
NVIC_InitTypeDef    NVIC_adc;

uint16_t x;
uint16_t y;

void Delay(__IO uint32_t nCount) {
  while(nCount--) {
  }
}
 

void ADC_IRQHandler()  
 {
   x= ADC_GetConversionValue(ADC1);
   GPIO_ToggleBits (GPIOD, GPIO_Pin_14|GPIO_Pin_13|GPIO_Pin_12);
   ADC_ClearITPendingBit(ADC1,ADC_IT_EOC);
 }
 
 
int main(void) {
  /* GPIOD Periph clock enable */
  RCC_AHB1PeriphClockCmd(RCC_AHB1Periph_GPIOD, ENABLE);
  RCC_AHB1PeriphClockCmd(RCC_AHB1Periph_GPIOA, ENABLE);
  RCC_APB2PeriphClockCmd(RCC_APB2Periph_ADC1, ENABLE);
 
  /* Configure PD12, 13, 14 and PD15 in output pushpull mode */
    /* Nur zur Darstellung*/
  GPIO_InitStructure.GPIO_Pin = GPIO_Pin_12 | GPIO_Pin_13 | GPIO_Pin_14 | GPIO_Pin_15;
  GPIO_InitStructure.GPIO_Mode = GPIO_Mode_OUT;
  GPIO_InitStructure.GPIO_OType = GPIO_OType_PP;
  GPIO_InitStructure.GPIO_Speed = GPIO_Speed_100MHz;
  GPIO_InitStructure.GPIO_PuPd = GPIO_PuPd_NOPULL;
  GPIO_Init(GPIOD, &GPIO_InitStructure);
  
  //  GPIO_InitStructure.GPIO_Pin = GPIO_Pin_0;
  //GPIO_InitStructure.GPIO_Mode = GPIO_Mode_IN;
  //  GPIO_Init(GPIOA,&GPIO_InitStructure);
  
  
 

 
GPIO_InitStructure.GPIO_Pin =  GPIO_Pin_1|GPIO_Pin_2;
GPIO_InitStructure.GPIO_Mode = GPIO_Mode_AIN;
GPIO_InitStructure.GPIO_Speed = GPIO_Speed_50MHz;
GPIO_Init(GPIOA, &GPIO_InitStructure);


  ADC_CommonInitStructure.ADC_Mode = ADC_Mode_Independent;
  ADC_CommonInitStructure.ADC_Prescaler = ADC_Prescaler_Div2;
  ADC_CommonInitStructure.ADC_DMAAccessMode = ADC_DMAAccessMode_Disabled;
  ADC_CommonInitStructure.ADC_TwoSamplingDelay = ADC_TwoSamplingDelay_5Cycles;
  ADC_CommonInit(&ADC_CommonInitStructure);


  ADC_InitStructure.ADC_Resolution = ADC_Resolution_12b;
  ADC_InitStructure.ADC_ScanConvMode = DISABLE;
  ADC_InitStructure.ADC_ContinuousConvMode = ENABLE;
  ADC_InitStructure.ADC_ExternalTrigConvEdge = ADC_ExternalTrigConvEdge_None;
  ADC_InitStructure.ADC_ExternalTrigConv = ADC_ExternalTrigConv_T1_CC1  ;
  ADC_InitStructure.ADC_DataAlign = ADC_DataAlign_Right;
  ADC_InitStructure.ADC_NbrOfConversion = 1;
  ADC_Init(ADC1, &ADC_InitStructure);
  ADC_ITConfig(ADC1, ADC_IT_EOC, ENABLE);

  ADC_RegularChannelConfig(ADC1, ADC_Channel_1, 1, ADC_SampleTime_144Cycles);
 //ADC_RegularChannelConfig(ADC1, ADC_Channel_12, 1, ADC_SampleTime_3Cycles);
  ADC_Cmd(ADC1, ENABLE);
  ADC_SoftwareStartConv(ADC1);
  
  
  NVIC_adc.NVIC_IRQChannel = ADC_IRQn;
  NVIC_adc.NVIC_IRQChannelCmd = ENABLE;
  NVIC_adc.NVIC_IRQChannelPreemptionPriority = 0x0F;
  NVIC_adc.NVIC_IRQChannelSubPriority = 0x01;
  NVIC_Init(&NVIC_adc);
  
 
  while (1) 
    {

    }
}

Wobei ich mir absolut nicht sicher bin ist der Befehl:

ADC_InitStructure.ADC_ExternalTrigConv = ADC_ExternalTrigConv_T1_CC1  ;

da ich ja eigentlich ein internes Ereignis triggere.
Ich hoffe ihr könnt mich auf meine Fehler hinweisen und bedanke mich 
schon einmal im voraus.
Viele Grüße
Phil

[Nils P. (ert)]

Ich nehme mal an die von ST meinten damit außerhalb des ADCs...

Ich hatte damals auch Problem mit diesem Interrupt. Habe ein paar Sachen 
ausprobiert. Da ich aber noch einen Timer über hatte habe ich in dessen 
Interrupt die ADC-Umwandlung gestartet.

Nutze den DMA der ist super, dann kannste du dir den ADC-Int komplett 
sparen und hast Zeit für schöne Sachen (zb im delay amzuhängen ;) )

Gruß Ert

[Philipp B. (phill4563)]

Das mit dem Timer habe ich mir auch schon überlegt, jedoch kann ich so 
ja nur einen channel des adc's auslesen.
Da ich aber später auf mehrere Channel zuggreifen möchte, wird es so 
nicht funktionieren denke ich.
Viele Grüße
Phil

[Nils P. (ert)]

Dann hast du den ADC falsch verstanden ;)

Einfach hier:
ADC_InitStructure.ADC_NbrOfConversion = x;

und hier:
ADC_RegularChannelConfig(ADC1, ADC_Channel_1, y, 
ADC_SampleTime_144Cycles);

erweitern.

Dann wandelt der ADC alle angegebenen Kanäle mit
ADC_SoftwareStartConv(ADC1);

Oder wenn du nichts regelt willst, du nur dauerhaft Werte benötigst und 
der Samplezeitpunkt nicht so wichtig ist, stell den ADC gleich in 
Continuous Mode ein

Gruß Ert

[Philipp B. (phill4563)]

hi ;)
also ich habe das bisher so verstanden, dass ich bei

ADC_InitStructure.ADC_NbrOfConversion = x;

die Anzahl der Wandlungen pro Zyklus eintragen muss. Also bei 2 Channel 
des ADC1 muss dort eine 2 stehen.

Un mit der Funktion:

ADC_RegularChannelConfig(ADC1, ADC_Channel_1, y, 
ADC_SampleTime_144Cycles);

stelle ich konkret die Kanäle ein, welche ich benutzen möchte wobei der 
parameter y sagt in welcher reihenfolge diese gewandelt werden.

Deswegen wollte ich die Ergebnisse ja auch über einen interrupt den 
beiden variablen zuweisen, weil ich ja nicht weiss welchen Channel der 
ADC1 gerade gewandelt hat. (über das EOC flag)

würde mich freuen falls mein Denkfehler hier berichtigt werden würde ;)
grüße
phil

[Nils P. (ert)]

Beide Angaben sind so korrekt.

Da ich immer mit dem DMA arbeite und da bei der richtigen Wahl der Größe 
(n*Channelanzahl) keine Missverständnisse auftreten hatte ich dein 
Problem nicht... Die Injeceted-Funktion ist auch ganz nett, da steht der 
ADC-Wert auch immer im gleichen Register.

Du müsstes eine Zählvariable nutzen welche du beim Timer Int null setzt. 
und beim ADC Int inkrementierst.

Greez Ert

[Pete K. (pete77)]

Fehlt da nicht irgendwo ein SystemInit(); ?

[Philipp B. (phill4563)]

Ich habe jetzt noch in einer Schleife folgende Abfrage hinzugefügt:

ADC_InitStructure.ADC_ContinuousConvMode = ENABLE;
ADC_InitStructure.ADC_DataAlign = ADC_DataAlign_Right;
ADC_InitStructure.ADC_ExternalTrigConv = ADC_ExternalTrigConv_T1_CC1;
ADC_InitStructure.ADC_NbrOfConversion = 2;
ADC_InitStructure.ADC_ScanConvMode = DISABLE;
ADC_Init(ADC1, &ADC_InitStructure);


ADC_CommonInitStructure.ADC_Mode =ADC_Mode_Independent;
ADC_CommonInitStructure.ADC_Prescaler = ADC_Prescaler_Div8  ;
ADC_CommonInitStructure.ADC_TwoSamplingDelay = ADC_TwoSamplingDelay_20Cycles ;
ADC_CommonInit(&ADC_CommonInitStructure);

ADC_RegularChannelConfig(ADC1, ADC_Channel_1, 1, ADC_SampleTime_56Cycles);
ADC_RegularChannelConfig(ADC1, ADC_Channel_2, 2, ADC_SampleTime_56Cycles);
ADC_Cmd(ADC1, ENABLE);
ADC_EOCOnEachRegularChannelCmd  ( ADC1,ENABLE ) ;
ADC_SoftwareStartConv(ADC1);
  
 
  while (1) {

    if (ADC_GetFlagStatus(ADC1,ADC_FLAG_EOC) && k==1)
    {
  
      k=2;
      x=ADC_GetConversionValue(ADC1);
    }  
      if (ADC_GetFlagStatus(ADC1,ADC_FLAG_EOC) && k==2)
      {
      y=ADC_GetConversionValue(ADC1);
        
      k=1;
      
      }

normalerweise sollte er doch das EOC Flag nach jeder Wandlung eines 
Channels setzen, so habe ich das zumindest aus den ref manus. 
verstanden.

Weiterhin sollte doch jetzt jeder variable (x,y) der jeweilige Wert 
übergeben werden, dass ist aber nicht der fall. (in beiden steht der 
gleiche wert)

Deine Idee mit den Injeceted Channels habe ich mir auch mal angeschaut. 
Mit denen sollte es eigentlich auch gehen, da in der STD-Lib eine 
Funktion

"uint16_t ADC_GetInjectedConversionValue  ( ADC_TypeDef *  ADCx,
  uint8_t  ADC_InjectedChannel
 )
"
vorhanden ist. Diese gibt es allerdings nicht für die Regular Channel, 
was ich nicht verstehe...
Am besten werde ich es mit 2 Wandlern machen, oder doch die DMA benutzen

Grüße Phil

[Philipp B. (phill4563)]

Als kurzen Nachtrag:
bei dem obigen Beispiel habe den Code nicht im Interrupt sondern in der 
"while1" ausgeführt

Grüße
Phil