STM32 ADC liefert Wert der Vorherigen Wandlung statt Aktuellen

[STM32 Einarbeiter (Gast)]

Hallo,

ich hab ein kleines Problem mit dem ADC des STM32F407VG  (Discovery 
Board).
Und zwar klappt es zwar die Wandlungen des ADC anzustoßen und ich 
bekommen auch ein Ergebnis, aber den Wert den ich erhalte ist immer von 
der Wandlung zuvor, also veraltet.
Es wäre schön, wenn jemand kurz über den Code schauen könnte, damit ich 
weißt was ich falsch mach.

Viele Grüße

#include <stm32f4xx.h>
#include <stm32f4xx_gpio.h>
#include <stm32f4xx_adc.h>
#include <stm32f4xx_conf.h>


uint16_t ADC_Val;                         //Stores the calculated ADC value
ADC_InitTypeDef       ADC_InitStructure;
ADC_CommonInitTypeDef ADC_CommonInitStructure;
GPIO_InitTypeDef      GPIO_InitStructure;
 
void ADC_Configuration(void)
{
/******************************************************************************/
/*               ADCs interface clock, pin and DMA configuration              */
/******************************************************************************/
        
  /* Enable peripheral clocks */
  RCC_APB2PeriphClockCmd(RCC_APB2Periph_ADC1 | RCC_APB2Periph_ADC2 | RCC_APB2Periph_ADC3 , ENABLE);
  
  RCC_AHB1PeriphClockCmd(RCC_AHB1ENR_GPIOCEN,ENABLE);//Clock for the ADC port!! Do not forget about this one ;)
 
  /* Configure ADC Channel 12 pin as analog input */
  GPIO_InitStructure.GPIO_Pin = GPIO_Pin_0 | GPIO_Pin_1 | GPIO_Pin_2;
  GPIO_InitStructure.GPIO_Mode = GPIO_Mode_AN;
  GPIO_InitStructure.GPIO_PuPd = GPIO_PuPd_NOPULL ;
  GPIO_Init(GPIOC, &GPIO_InitStructure);
 
  /* ADC Common configuration *************************************************/
  ADC_CommonInitStructure.ADC_Mode = ADC_Mode_Independent;
  ADC_CommonInit(&ADC_CommonInitStructure);
 
  /* ADC1 regular channel 10 to 15 configuration ************************************/
  ADC_InitStructure.ADC_Resolution = ADC_Resolution_12b;
  ADC_InitStructure.ADC_ScanConvMode = DISABLE;
  ADC_InitStructure.ADC_ContinuousConvMode = DISABLE;
  
//  ADC_InitStructure.ADC_DiscModeChannelCountConfig(1);
  ADC_InitStructure.ADC_ExternalTrigConvEdge = ADC_ExternalTrigConvEdge_None;
  ADC_InitStructure.ADC_DataAlign = ADC_DataAlign_Right;
  ADC_Init(ADC1, &ADC_InitStructure);
  ADC_Init(ADC2, &ADC_InitStructure);
  ADC_Init(ADC3, &ADC_InitStructure);
  

 
  ADC_RegularChannelConfig(ADC1, ADC_Channel_10, 1, ADC_SampleTime_480Cycles);
  ADC_RegularChannelConfig(ADC2, ADC_Channel_11, 1, ADC_SampleTime_480Cycles);
 ADC_RegularChannelConfig(ADC3, ADC_Channel_12, 1, ADC_SampleTime_480Cycles);
  
  
//    ADC_DiscModeChannelCountConfig(ADC3, 1);
//   ADC_DiscModeCmd(ADC3, ENABLE);
  
 
/* Enable ADC1 to ADC3*/
  ADC_Cmd(ADC1, ENABLE);
  ADC_Cmd(ADC2, ENABLE);
  ADC_Cmd(ADC3, ENABLE);
 
}//end ADC_Configuration
 
uint16_t Get_ADC_Value(uint8_t Channel)
{
    switch(Channel)
    {
        case 10:
            ADC_SoftwareStartConv(ADC1);
            while(ADC_GetSoftwareStartConvStatus(ADC1) != RESET){ADC_Val = 0;}
            ADC_Val = ADC_GetConversionValue(ADC1);
            break;
        case 11:
            ADC_SoftwareStartConv(ADC2);
            while(ADC_GetSoftwareStartConvStatus(ADC2) != RESET){ADC_Val = 0;}
            ADC_Val = ADC_GetConversionValue(ADC2);
            break;
        case 12:
            ADC_SoftwareStartConv(ADC3);
            while(ADC_GetSoftwareStartConvStatus(ADC3) != RESET){ADC_Val = 0;}
            ADC_Val = ADC_GetConversionValue(ADC3);
            ADC_Val = ADC_GetConversionValue(ADC3);
            break;
        default:
            ADC_Val = 0;
            break;
    }
    return ADC_Val;
}

int main(void){
  volatile  uint16_t aaa=0;
  ADC_Configuration();


  
  while(1==1){
  aaa=Get_ADC_Value(12);
    
    
    
    
  }
}

[Stefan (Gast)]

Wo ist der Code von ADC_GetConversionValue()?

Ansonsten würde ich sagen: it is not a bug, it is a feature ...:

Beim Aufruf von ADC_GetConversionValue() liest Du wahrscheinlich erst 
den letzten Wert aus dem Wandler aus und startest ihn dann neu. Das 
hätte den Vorteil, daß Du nicht auf den ADC warten musst. In dem Fall 
wäre der richtige Name nicht
   ADC_GetConversionValue()
sondern
   ADC_GetLastConversionValueAndStartNextConversion()


Gruß Stefan

[Dennis X. (Gast)]

Stefan schrieb:
> ADC_GetLastConversionValueAndStartNextConversion()

:D Sehr gut!

[STM32 Einarbeiter (Gast)]

Hallo,

ADC_GetConversionValue() kommt aus der Standard Peripheral Library...

Fehler war, man muss scheinbar

auf while (ADC_GetFlagStatus(ADC3, ADC_FLAG_EOC) == RESET) abfragen,

statt auf

while(ADC_GetSoftwareStartConvStatus(ADC3) != RESET)