problems about sequence ADC using DMA

bo han

Hi all, i am doing sequence ADC and read out it using DMA. First i close HAL_DMA and HAL_AES_DMA to ensure no access to DMA registers. Then i write code according to the sample code from

http://www.ti.com/litv/zip/swrc257

before using the code, the bluetooth function is pretty good. But when i call dmaInit in my own code, the ble just broadcrast 1 package. There seems something wrong but i don't know why. Can someone help me?

/***********************************************************************************
  File name:    adcDma.c

***********************************************************************************/
#include "adcdma.h"

void initDmaa(uint16 __xdata *ADCresult_adr){
  
    /****************************************************************************
    * I/O-Port configuration
    * PIN0_0 to P0_3 is configured as ADC input pins.
	* APCFG[7:0] select P0.7–P0.0 as analog I/O.
    */  
    
    APCFG = 0x0F;
    
    
    // Set the DMA channel 0 configuration address registers.
    DMA0CFGL = ((uint16)&dma_channel[0]) & 0x00FF;
    DMA0CFGH = ((uint16)&dma_channel[0] >> 8);

    // Set the DMA channel 1-4 configuration address registers.
    DMA1CFGL = ((uint16)&dma_channel[1]) & 0x00FF;
    DMA1CFGH = ((uint16)&dma_channel[1] >> 8);
    
        /****************************************************************************
    * DMA configuration:
    * Set up DMA channel 1 and 2 for transfer from ADCL and ADCH to
    * the ADC result table. The ADC will generate the DMA trigger
    * on end of conversion.
    */
	
    // Configure DMA channel 0.
    dma_channel_init(&dma_channel[0], ADCresult_adr, 1, 21); //ch 0  21

    // Configure DMA channel 1.
    dma_channel_init(&dma_channel[1], &ADCresult_adr[1], 1, 22); // ch 1 22 .. 23 34

	// Configure DMA channel 2.
    dma_channel_init(&dma_channel[2], &ADCresult_adr[2], 1, 23);

    // Configure DMA channel 3.
    dma_channel_init(&dma_channel[3], &ADCresult_adr[3], 1, 24);
    
}


void startADCDma(void)
{
    // Arm DMA channels, takes 36 clock cycles.
    DMAARM = 0x0F;
    ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP; // 9 NOPs
    ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP; // 9 NOPs
    ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP; // 9 NOPs
    ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP;ASM_NOP; // 9 NOPs


    /****************************************************************************
    * ADC configuration :
    *  - Continous mode
    *  - Single-ended sequence conversion
    *  - Reference Voltage is VDD on the AVDD pin
    *  - 9 bit resolution (128 dec rate)
    */

    /* ADC conversion :
	 * reference voltage ACDD5 10
	 * 9 bits ENOB settings    01
	 * SCH from channel AIN0 to AIN3  0011
	 */
    ADCCON2 = 0x93;

    // FULL SPEED continous ADC
    ADCCON1 = 0x13;

  
    /* 
    * Await all data transfers being completed. from dma channel 0 to 3
    */
  
    while( DMAIRQ != 0x0F );
	
    // reset the interrupt
    DMAIRQ = 0x00;
    // unarm
    DMAARM = 0x00;
      
    // Stop continuous mode.(ADC)
    ADCCON1 |= 0x40;
  


}



void dma_channel_init ( DMA_DESC __xdata *dma_p,
                        uint16 __xdata *dest_adr,
                        uint8 lenl,
                        uint8 trig )
{
    // Setup DMA configuration.-
    dma_p->SRCADDRH  = (uint16)(&X_ADCL) >> 8;
    dma_p->SRCADDRL  = (uint16)(&X_ADCL) & 0x00FF;
    dma_p->DESTADDRH = ((uint16)dest_adr) >> 8;
    dma_p->DESTADDRL = ((uint16)dest_adr);
    dma_p->VLEN      = 0x00;
    dma_p->LENH      = 0;
    dma_p->LENL      = lenl;                   // Tranfer Count
    dma_p->WORDSIZE  = 0x00;
    dma_p->TMODE     = 0x01;
    dma_p->TRIG      = trig;                   // Channel trigger
    dma_p->SRCINC    = 0x01;
    dma_p->DESTINC   = 0x01;
    dma_p->IRQMASK   = 0x00;
    dma_p->M8        = 0x01;
    dma_p->PRIORITY  = 0x02;
}

/******************************************************************************
**************************   DMA structures / macros  *************************
*******************************************************************************

The macros and structs in this section simplify setup and usage of DMA.

******************************************************************************/

#ifndef _adcdma_H_
#define _adcdma_H_


#include <ioCC2540.h>
#include <hal_types.h>


// Place the bitfield members from the most significant bit to the least significant bit.
#pragma bitfields=reversed
typedef struct {
   uint8 SRCADDRH;
   uint8 SRCADDRL;
   uint8 DESTADDRH;
   uint8 DESTADDRL;
   uint8 VLEN      : 3;
   uint8 LENH      : 5;
   uint8 LENL      : 8;
   uint8 WORDSIZE  : 1;
   uint8 TMODE     : 2;
   uint8 TRIG      : 5;
   uint8 SRCINC    : 2;
   uint8 DESTINC   : 2;
   uint8 IRQMASK   : 1;
   uint8 M8        : 1;
   uint8 PRIORITY  : 2;
} DMA_DESC;
#pragma bitfields=default


// DMA descriptor n -> DMA channel n.
static __xdata DMA_DESC dma_channel[4];

/*******************************************************************************
 * FUNCTIONS
 */
void initDmaa(uint16 __xdata *ADCresult_adr);
void startADCDma(void);
void dma_channel_init ( DMA_DESC __xdata *dma_p,
                        uint16 __xdata *dest_adr,
                        uint8 lenl,
                        uint8 trig );


#endif

over 11 years ago

0 Greenja over 11 years ago

Guru 22270 points

Hello Bo,

When you say the BLE only broadcast 1 packet, do you mean that only one packet is sent and then the BLE connection is terminated?

If other packets are being sent from the device then perhaps you have not set up a Timer for getting a new reading and only the first one is sent.

If you are only getting one ADC channel reading then you are only reading the first value stored in the first DMA destination address.

Thanks,

0 Peter Borenstein over 11 years ago

Mastermind 8695 points

According to the TI software developer's guide, HAL_DMA and HAL_AES_DMA are required to be true.

0 bo han over 11 years ago in reply to Peter Borenstein

Prodigy 240 points

Hi Peter, thanks for your remind otherwise i would make a big mistake. But the problem still exists.

0 bo han over 11 years ago in reply to Greenja

Prodigy 240 points

Hi greenja,

I set the device to advertise once on power, how ever, when i use the init comand, every time i reset my device, i can only find 1 package at package sniffer. Then nothing, no package, no connection. I think that maybe the dma problem, but i don't know where is it.

BR.

0 Greenja over 11 years ago in reply to bo han

Guru 22270 points

In your code, comment out line 82 or 83, the while loop for the DMA to complete and try it again. Or stop the debugger to see where the program is stuck. If you are losing the connection, then your code isn't returning control to the OSAL.

0 Peter Borenstein over 11 years ago in reply to Greenja

Mastermind 8695 points

I thought OP says he only has problems when he runs the init? I was under the impression he never actually calls startADCDMA(). OP is now saying something about only seeing one packet when he resets. I don't appreciate the about of details being given for this nich problem

0 bo han over 11 years ago in reply to Peter Borenstein

Prodigy 240 points

Hi piter, Here is more detial about the problem.

I use sample code simpleBLEPeripheral to send the packages. In the code, i make the device to advertise immediately after the power is on. And every 10ms i will send notifications to the central device. I test it, advertising and connection and sending data packages are all working. After i test all this, ADC using DMA is added which is the code i used above. I called initDmaa in the simpleBLEPeripheralInit so the initDMA will only run once. now i donot use startAMCDMA yet. Then i download the code to my chip, when i press the reset buttom on the board, it is supposed to to advertise all the time, but i only see 1 broadcast package. But the only thing i have done is add this dmainit to the code. In the code i only set all the ADC pins to input and the dma channels to the ADC registers.

0 ke fan over 11 years ago in reply to bo han

Expert 1890 points

it seems that DMA channels 1,2 are preoccupied by BLE, so that application can not use these resources without conflict with BLE. channels 3,4 are available for ADC if UART does not use these channels or the application does not use UART at all.

http://e2e.ti.com/support/low_power_rf/f/538/t/321486.aspx

Bluetooth®︎

Bluetooth forum

problems about sequence ADC using DMA