stm32 DMA läuft nur in Simulation!?

[stef b. (joe_)]

Hallo!

Habe auf dem STM32 einen ADC mit DMA programmiert. Der ADC alleine läuft 
problemlos. Mit DMA läuft das ganze aber nur in der Simulation 
(KEIL-µVision) ... und auf dem Board passiert nix! Hat jemand ne Idee 
woran das liegen kann? Bin langsam am verzweifeln!

Wird das transfer complete flag (TCIF1) gesetzt geht die LED an ... aber 
halt nur in der Simulation, nicht aber auf dem STM32!

#include "stm32f10x.h"
uint32_t DMA;  // DMA goal

int main(void)
{
// enable clock
RCC->APB2ENR |= RCC_APB2ENR_IOPCEN;   // GPIOC
RCC->APB2ENR |= RCC_APB2ENR_AFIOEN;  // Alternate Function
RCC->APB2ENR |= RCC_APB2ENR_ADC1EN;  // ADC
/* Input @ PC0 */
GPIOC->CRL  &= ~GPIO_CRL_MODE0;  // Input
GPIOC->CRL  &= ~GPIO_CRL_CNF0;   // Analog
/* LED @ PC8 */
GPIOC->CRH |= GPIO_CRH_MODE8_1;  // Output 2MHz     
GPIOC->CRH &= ~GPIO_CRH_CNF8;    // General push-pull

ADC1->CR2 |= ADC_CR2_ADON | ADC_CR2_CONT;  // ADC wakeup / continuous 
ADC1->CR1 |= ADC_CR1_AWDIE;   // enable interrupt: analog watchdog
ADC1->CR1 |= ADC_CR1_AWDEN;   // enable analog watchdog for regular channel
ADC1->CR1 |= ADC_CR1_AWDSGL;      // watchdog enabled on a single channel
ADC1->CR1 |= ADC_CR1_AWDCH_1 | ADC_CR1_AWDCH_3;  // watchdog on channel 10
ADC1->CR1 |= ADC_CR1_SCAN;        // enable scan mode
ADC1->CR2 |= ADC_CR2_DMA;         // enable dma mode
ADC1->SMPR1 &= ~ADC_SMPR1_SMP10;  // set sample time for channel 10 to 1.5 
ADC1->SQR1 &= ~ADC_SQR1_L;        // 1 conversions
ADC1->SQR3|= ADC_SQR3_SQ1_1 | ADC_SQR3_SQ1_3;//channel 10 first    
ADC1->HTR |= (uint16_t)(0x0FF0);  // set watchdog high threshold
ADC1->LTR |= (uint16_t)(0x000F);  // set watchdog low threshold

DMA1_Channel1->CPAR  = (uint32_t)(&(ADC1->DR)); // peripheral address
DMA1_Channel1->CNDTR = (uint16_t)(0x0001);  // 1 transfers
DMA1_Channel1->CCR |= DMA_CCR1_CIRC;        // circular mode enable
DMA1_Channel1->CCR |= DMA_CCR1_MSIZE_0;     // memory size 16 bits
DMA1_Channel1->CCR |= DMA_CCR1_PSIZE_0;     // peripheral size 16 bits
DMA1_Channel1->CMAR = (uint32_t)(&(DMA));   // set DMA goal for ADC
DMA1_Channel1->CCR |= DMA_CCR1_TCIE;  //enable interrupt transfer complete
DMA1_Channel1->CCR |= DMA_CCR1_EN;   // DMA on
ADC1->SR |= ADC_CR1_EOCIE;      // enable interrupt End of conversation
ADC1->CR2|= ADC_CR2_CAL;        // start ADC calibration
while(ADC1->CR2 & ADC_CR2_CAL){}// wait during calibration 
ADC1->CR2   |= ADC_CR2_ADON;    // turn ADC on
  
while (1)
{
   if(DMA1->ISR & DMA_ISR_TCIF1)    //<- TCIF1 wird nur in Software gesetzt
   {
     DMA1->IFCR |= DMA_IFCR_CTCIF1; // clear interrupt
     GPIOC->BSRR |= GPIO_BSRR_BS8;  // LED on

   }        

}
}

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Ich seh nirgends, das du die Clock für die DMA auch anschaltest?

stef ba schrieb:
> DMA1_Channel1->CCR |= DMA_CCR1_PSIZE_0;     // peripheral size 16 bits
> DMA1_Channel1->CMAR = (uint32_t)(&(DMA));   // set DMA goal for ADC

Das sieht auch nicht richtig aus. Mach dir lieber ein struct mit 2 
16-bit Zielen anstatt einem 32-bit target. Könnte sein, das der Compiler 
dir sonst einen Strich durch die Rechnung macht. Leider benutze ich die 
Std. Peri Lib, dewegen kann ich deine Registerbeschreiberei nicht im 
einzelnen nachvollziehen. Auf Wunsch poste ich aber mal meinen Code.

[stef b. (joe_)]

Matthias Sch. schrieb:
> Ich seh nirgends, das du die Clock für die DMA auch anschaltest?

das stimmt, hab ich auch nicht ...mal schauen ob es damit geht! aber 
komisch dass die simulation trotzdem geht!

ich probier nochmal aus und meld mich wieder! vielen dank erstmal!

[Star K. (starkeeper)]

Du bindest den Header von der STM32-Lib ein aber nutzt sie nicht? Oder 
was ist der Grund dafür, dass du jedes Register von Hand beschreiben 
möchtest?

Von ST gibt es eine Peripherals Library die dir einiges an Arbeit 
abnehmen kann und so schon mal die Fehlermöglichkeiten minimiert.

[stef b. (joe_)]

Star Keeper schrieb:
> Von ST gibt es eine Peripherals Library

das wusste ich noch nicht ... aber gut zu wissen!

aber wieso benutze ich sie nicht? dort stehen doch eh nur die adressen 
der Ports und Register drin, so z.B. RCC_APB2ENR_IOPCEN ... ist aus der 
stm32f10x.h

[Star K. (starkeeper)]

stef ba schrieb:
> dort stehen doch eh nur die adressen
> der Ports und Register drin

mmmmh, nein.
Dort stehen für jede Peripherie Funktionen zum Einrichten und Benutzen 
zur Verfügung. Dies vereinfacht den Einsatz der Peripherie ungemein.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Hier ist mal ein Beispel mit Std. Peri.Lib. für den ADC mit 4 Kanälen:

// defines for readability
#define SPAREINPUT_CHANNEL          ADC_Channel_0
#define SPEEDINPUT_CHANNEL          ADC_Channel_1
#define PHASE_CURRENTS_CHANNEL          ADC_Channel_2
#define TEMP_CHANNEL                    ADC_Channel_3

typedef struct ADCValues
{
  uint16_t spare1;
  uint16_t speedInput;
  uint16_t current;
  uint16_t temp;
  uint16_t spare2;
} ADCValues_t;

volatile ADCValues_t ADCRead;
/* Init the ADC converter to free-run with 4 channels
 *
 */
void ADCInit(void) {
ADC_InitTypeDef       ADC_InitStructure;
DMA_InitTypeDef    DMA_InitStructure;
RCC_APB2PeriphClockCmd(RCC_APB2Periph_ADC1,ENABLE);
RCC_ADCCLKConfig(RCC_PCLK2_Div8);
RCC_AHBPeriphClockCmd(RCC_AHBPeriph_DMA1, ENABLE);
/* DMA1 channel1 configuration ----------------------------------------------*/
  DMA_DeInit(DMA1_Channel1);    // Clear everything we had 
  DMA_InitStructure.DMA_PeripheralBaseAddr =  (uint32_t)&(ADC1->DR); // source address is the ADC result register
  DMA_InitStructure.DMA_MemoryBaseAddr = (u32)&ADCRead;      // Its the target struct
  DMA_InitStructure.DMA_DIR = DMA_DIR_PeripheralSRC;   // from Peripheral to RAM
  DMA_InitStructure.DMA_BufferSize = 4;    /// Note that we have 4 Channels
  DMA_InitStructure.DMA_PeripheralInc = DMA_PeripheralInc_Disable;  // Source stays same address
  DMA_InitStructure.DMA_MemoryInc = DMA_MemoryInc_Enable;     //destination will be incremented
  DMA_InitStructure.DMA_PeripheralDataSize = DMA_PeripheralDataSize_HalfWord;  // 16 bit results
  DMA_InitStructure.DMA_MemoryDataSize = DMA_MemoryDataSize_HalfWord;   // yeah same size -> 16 bit
  DMA_InitStructure.DMA_Mode = DMA_Mode_Circular;  // Wrap around the destination address
  DMA_InitStructure.DMA_Priority = DMA_Priority_VeryHigh;   // why not? its the only DMA thingy in this program
  DMA_InitStructure.DMA_M2M = DMA_M2M_Disable;   // we don't want RAM to RAM
  DMA_Init(DMA1_Channel1, &DMA_InitStructure);   // Init the stuff finally 
  /* Enable DMA1 channel1 */
  DMA_Cmd(DMA1_Channel1, ENABLE);  // Run, but we wait for the ADC init
/* ADC1 configuration
 * 4 channel continuous
 */
  ADC_DeInit(ADC1);

ADC_StructInit(&ADC_InitStructure);
ADC_InitStructure.ADC_Mode = ADC_Mode_Independent;
ADC_InitStructure.ADC_ScanConvMode = ENABLE;
ADC_InitStructure.ADC_ContinuousConvMode = ENABLE;
ADC_InitStructure.ADC_NbrOfChannel = 4;
ADC_InitStructure.ADC_DataAlign = ADC_DataAlign_Right;
ADC_InitStructure.ADC_ExternalTrigConv = ADC_ExternalTrigConv_None;
/* Now do the setup */
ADC_DMACmd(ADC1, ENABLE);
ADC_Init(ADC1, &ADC_InitStructure);
/* Enable ADC1 */
ADC_RegularChannelConfig(ADC1,SPAREINPUT_CHANNEL,1,ADC_SampleTime_239Cycles5);
ADC_RegularChannelConfig(ADC1,SPEEDINPUT_CHANNEL,2,ADC_SampleTime_239Cycles5);
ADC_RegularChannelConfig(ADC1,PHASE_CURRENTS_CHANNEL,3,ADC_SampleTime_239Cycles5);
ADC_RegularChannelConfig(ADC1,TEMP_CHANNEL,4,ADC_SampleTime_239Cycles5);
ADC_Cmd(ADC1, ENABLE);
/* Enable ADC1 reset calibaration register */
ADC_ResetCalibration(ADC1);
/* Check the end of ADC1 reset calibration register */
while(ADC_GetResetCalibrationStatus(ADC1));
/* Start ADC1 calibaration */
ADC_StartCalibration(ADC1);
/* Check the end of ADC1 calibration */
while(ADC_GetCalibrationStatus(ADC1));

ADC_SoftwareStartConvCmd(ADC1,ENABLE);
Delay(200);
}