stm32f103 dma-spi

[grundschüler (Gast)]

versuche meinen vs1052 code auf dma umzustellen.

Das Beschicken des mp3-Modulds funktioniert jetz - als Vorbereitung zur 
dma - ohne Warteschleife per Interrupt:

void SysTick_Handler(){//1ms


    /* Increment performance counter */
  //disk_timerproc();  /* Disk timer process */

  if(Timer_fat)Timer_fat--;

  if(Timer_down)
    Timer_down--;

  timer_smh++;
  timer_timeout_up_ms++;


  if(timer_smh%5==0){

    if ((vs_status & (1<<0))){
        if ((GPIOB->IDR & (1<<4))) {
        GPIOA->ODR |=(1<<9);//xdcs
          GPIOA->ODR &=~(1<<9);//xdcs

          uint8_t i;
          for (i=0;i<32;i++){
            SPI2->DR = *vs_pt++;
            while (!(SPI2->SR & SPI_SR_RXNE));//0_rxne 1_txe 7_bsy// ok
            (SPI2->DR);//return
            zlisr++;
          }
          if (zlisr > vs_buffer_size-1){
            vs_status &= ~(1<<0);
          }
        }
      }
#endif
  }//%5



Ein Problem habe ich mit strcpy:

[c]/**/
u16 j;
for(j=0;j<1965;j++){
  vs_buffer[1965+j]=hello_mp3[+j];
  vs_buffer[j]=hello_mp3[+j];
}

strcpy(vs_buffer,hello_mp3);

//vs_pt = hello_mp3; // Point "p" to the beginning of array
vs_pt = vs_buffer; // Point "p" to the beginning of array
vs_status |= (1<<0);
xdcs_h;
zlisr=0;

Wenn ich die for-Schleife auskommentiere, kommt im mp3-modul nichts mehr 
an owohl der Speicherinhalt nach dem Debugger der gleiche ist.

Woran könnte das liegen?

[bittereMedizin (Gast)]

du hast doch offensichtlich ein Problem mit deinem spi-code. Bevor du
jetzt auf dma umstellst, wäre Fehlersuche und Fehlerbehebung angebracht.

[grundschüler (Gast)]

bittereMedizin schrieb:
> du hast doch offensichtlich ein Problem mit deinem spi-code.

??? Ich will es ja nicht völlig ausschließen, es ist aber eigentlich 
problemloser code:

kk_init_spi2(
    u8 modus // 0: ph0/pol0 1:ph1/pol1
        ){

    //pinout
    RCC->APB2ENR |= RCC_APB2Periph_GPIOB;
    setGPIO_CRHL(B,12,GPIO_OUT_PP);//nss
    setGPIO_CRHL(B,13,GPIO_AF_PP);//clk
    setGPIO_CRHL(B,14,GPIO_AF_OPD);//miso
    pinSET(B,14);//pullup
    setGPIO_CRHL(B,15,GPIO_AF_PP);//mosi

    //spi_enable
    RCC->APB1ENR |= RCC_APB1Periph_SPI2;
    SPI2->CR1 = SPI_CR1_SSM|SPI_CR1_SSI|SPI_CR1_MSTR|SPI_CR1_SPE;

    //modus
    if(modus)SPI2->CR1 |= SPI_CR1_CPHA|SPI_CR1_CPOL;

    //dma
  #define TXDMAEN (1<<1)
//  SPI2->CR2 |= (TXDMAEN);
    }

[Jim M. (turboj)]

grundschüler schrieb:
> strcpy(vs_buffer,hello_mp3);

MP3 Daten können 0x00 enthalten und werden nicht mit 0x00 abgeschlossen.

Daher immer memcpy() und nicht strcpy() verwenden.

[grundschüler (Gast)]

Jim M. schrieb:
> grundschüler schrieb:
>> strcpy(vs_buffer,hello_mp3);
>
> MP3 Daten können 0x00 enthalten und werden nicht mit 0x00 abgeschlossen.
>
> Daher immer memcpy() und nicht strcpy() verwenden.

strcpy hört bei 0 auf?
Danke, das leuchtet ein.

[dasrotemopped (Gast)]

https://learn.adafruit.com/adafruit-music-maker-shield-vs1053-mp3-wav-wave-ogg-vorbis-player/downloads

wartest du darauf, das der DMA Transfer beendet wurde bevor du das 2. 
Device an SPI ansprichst ? Und wie kannst du DMA benutzen wenn du 
zwischen 2 CS umschalten musst ?

[Robin S. (der_r)]

grundschüler schrieb:
> strcpy hört bei 0 auf?
> Danke, das leuchtet ein.

Ja,
Strings sind in C nichts weiter als ein Char-Array, welches am Ende noch 
ein Terminator-Zeichen hat. Da die Arraygröße aus Sicht der meisten 
Funktionen unbekannt ist, arbeiten die einfach so lange, bis dieses 
Zeichen erreicht wird.

Das Zeichen wird '\0' geschrieben, wobei '\0' der 0 (siehe 
ASCII-Tabelle) entspricht.

[grundschüler (Gast)]

dasrotemopped schrieb:
> wartest du darauf, das der DMA Transfer beendet wurde bevor du das 2.
> Device an SPI ansprichst ? Und wie kannst du DMA benutzen wenn du
> zwischen 2 CS umschalten musst ?

Nur dieser Teil

          uint8_t i;
          for (i=0;i<32;i++){
            SPI2->DR = *vs_pt++;
            while (!(SPI2->SR & SPI_SR_RXNE));//0_rxne 1_txe 7_bsy// ok
            (SPI2->DR);//return
            zlisr++;
          }

wird durch den dma-Aufruf ersetzt. Da ist - bis auf RXNE - kein 
Warteteil und kein CS drin. Müsste also so per dma klappen...

[dasrotemopped (Gast)]

stimmt meine Vermutung, Nucleo-F103RB + Adafruit Music Maker Shield  ?
Wenn nicht, wie ist deine aktuelle Pinbelegung / Doppelbelegung für 
SDcard und VS1052 ?


Gruß,

dasrotemopped.

[grundschüler (Gast)]

dasrotemopped schrieb:
> Nucleo-F103RB

nein, 103c8, läuft alles erstaunlich gut. Wenn  die dma läuft, stell ich 
den code ein.

[grundschüler (Gast)]

bei der dma hängt es noch:

void dma1chan5_m_to_spi2(void){
    DMA_InitTypeDef DMA_InitStructure;

    #define TXDMAEN (1<<1)
  SPI2->CR2 |= (TXDMAEN);

  DMA_DeInit(DMA1_Channel5);
  //1. periph adr
     DMA_InitStructure.DMA_PeripheralBaseAddr = &SPI2->DR;
     DMA_InitStructure.DMA_MemoryBaseAddr = vs_buffer;//xHelloMP3;
     DMA_InitStructure.DMA_DIR = DMA_DIR_PeripheralDST;
     DMA_InitStructure.DMA_BufferSize = 32;//1695;//cndtr Byte????
     DMA_InitStructure.DMA_PeripheralInc = DMA_PeripheralInc_Disable;
     DMA_InitStructure.DMA_MemoryInc = DMA_MemoryInc_Enable;
     DMA_InitStructure.DMA_PeripheralDataSize = DMA_PeripheralDataSize_Byte;
     DMA_InitStructure.DMA_MemoryDataSize = DMA_MemoryDataSize_Byte;
     DMA_InitStructure.DMA_Mode = DMA_Mode_Normal;//circular
     DMA_InitStructure.DMA_Priority = DMA_Priority_Medium;
     DMA_InitStructure.DMA_M2M = DMA_M2M_Disable;
     DMA_Init(DMA1_Channel5, &DMA_InitStructure);

     DMA_ITConfig(DMA1_Channel5,DMA_IT_TC,ENABLE);

     // Enable the SPI DMA TX Interrupt
     NVIC_InitTypeDef NVIC_InitStructure;
     NVIC_InitStructure.NVIC_IRQChannel = DMA1_Channel5_IRQn;
     NVIC_InitStructure.NVIC_IRQChannelPreemptionPriority = 1;
     NVIC_InitStructure.NVIC_IRQChannelSubPriority = 1;
     NVIC_InitStructure.NVIC_IRQChannelCmd = ENABLE;
     NVIC_Init(&NVIC_InitStructure);

}
//==========================================================
void DMA1_Channel5_IRQHandler(){
GPIOA->ODR |=(1<<9);//xdcs
DMA_ClearFlag(DMA1_FLAG_TC5);
}

//==========================================================

im systick;
  if(timer_smh%5==0){

//kein DMA
    if ((vs_status & (1<<0))){
        if ((GPIOB->IDR & (1<<4))) {
          GPIOA->ODR &=~(1<<9);//xdcs
          uint8_t i;
          for (i=0;i<32;i++){
            SPI2->DR = *vs_pt++;
            while (!(SPI2->SR & SPI_SR_RXNE));//0_rxne 1_txe 7_bsy// ok
            (SPI2->DR);//return
            vs_zl_buffer++;
          }
          if (vs_zl_buffer > vs_buffer_size-1){
            vs_status &= ~(1<<0);
          }
             GPIOA->ODR |=(1<<9);//xdcs
         }
      }

//DMA
    if ((vs_status & (1<<1))){
           GPIOA->ODR &=~(1<<9);//xdcs
       if (vs_zl_buffer < vs_buffer_size){
           DMA1_Channel5->CMAR=vs_zl_buffer+=32;
           DMA_Cmd(DMA1_Channel5, ENABLE);
         }else{
           vs_status &= ~(1<<1);
            GPIOA->ODR |=(1<<9);//xdcs
        }
    }

  }//%5

irgendwie geht der dma-trigger nicht. Jemand eine Idee?

[grundschüler (Gast)]

habe jetzt das Diller DMA-example gefunden. Der Startcode für eine 
einzelne DMA-übertragung ist sehr umständlich:

         SPI2->DR;//fifo leeren von mir
        DMA_Cmd(DMA1_Channel4, DISABLE);
        DMA_Cmd(DMA1_Channel5, DISABLE);
        DMA_SetCurrDataCounter(DMA1_Channel4, 2);
        DMA_SetCurrDataCounter(DMA1_Channel5, 2);
        GPIO_WriteBit(GPIOB, GPIO_Pin_12, RESET);
        DMA_Cmd(DMA1_Channel4, ENABLE);
        DMA_Cmd(DMA1_Channel5, ENABLE);

versuche das jetzt auf mein Projekt umzubasteln.

[grundschüler (Gast)]

noch nicht alltagstauglich, aber man hört jetzt was per dma im vs-modul:

  if(timer_smh%3==0){

//kein DMA
    if ((vs_status & (1<<0))){
        if ((GPIOB->IDR & (1<<4))) {
          GPIOA->ODR &=~(1<<9);//xdcs
          uint8_t i;
          for (i=0;i<32;i++){
            SPI2->DR = *vs_pt++;
            while (!(SPI2->SR & SPI_SR_RXNE));//0_rxne 1_txe 7_bsy// ok
            (SPI2->DR);//return
            vs_zl_buffer++;
          }
          if (vs_zl_buffer > vs_buffer_size-1){
            vs_status &= ~(1<<0);
          }
             GPIOA->ODR |=(1<<9);//xdcs
         }
      }

//DMA
    if ((vs_status & (1<<1))){
           GPIOA->ODR &=~(1<<9);//xdcs
       if (vs_zl_buffer < vs_buffer_size){
 //        u8 *adr=vs_buffer+vs_zl_buffer;
         DMA1_Channel5->CMAR=vs_buffer+vs_zl_buffer;//;
         vs_zl_buffer+=32;
      DMA_Cmd(DMA1_Channel4, DISABLE);
      DMA_Cmd(DMA1_Channel5, DISABLE);
      DMA_SetCurrDataCounter(DMA1_Channel4, 32);
      DMA_SetCurrDataCounter(DMA1_Channel5, 32);
      GPIO_WriteBit(GPIOB, GPIO_Pin_12, RESET);
      DMA_Cmd(DMA1_Channel4, ENABLE);
      DMA_Cmd(DMA1_Channel5, ENABLE);
         }else{
           vs_status &= ~(1<<1);
            GPIOA->ODR |=(1<<9);//xdcs
        }
    }

  }//%5

das Zeitintervall musste ich von 5 auf 3ms verkürzen. Außerdem fällt das 
lcd-modul nach Einsatz der dma aus. Aber grundsätzlich geht die DMA