STM32 VCP über USB Problem

[Patrick L. (crashdemon)]

Hallo Leute,

ich nutze das STM32 Discovery Board 
(http://www.st.com/web/en/catalog/tools/FM116/SC959/SS1532/PF254044) um 
mir die Daten der darauf befindlichen Sensoren per User USB an den PC zu 
senden. Dafür habe ich das "VirtualComport_Loopback" Beispiel aus der 
STM32_USB-FS-Device_Lib_V4.0.0 auf meine Problemstellung adaptiert. 
Leider will der Code aber nicht wie gedacht funktionieren.

main:

int main(void)
{ 

    ...
    
    Gyro_Init(); // Gyro initialisieren
    Acc_Init(); // Acc initialisieren
    
    SysTick_init(); // Initialisiert die SysTicks
    fDt = FDT; // Timebase (CycleTime)
    
    while(1)
    {         
        if(sample)
        {
            GyroReadAngRate(raw_gyro); 
            AccReadData(raw_acc);
      
            if(bDeviceState == CONFIGURED)
            {
                //CDC_Send_DATA((unsigned char*)buffer, strlen(buffer));

                sprintf(buffer, "%+.2f,", raw_acc[0]);
                USB_SendData(buffer);

                sprintf(buffer, "%+.2f,", raw_acc[1]);
                USB_SendData(buffer);

                sprintf(buffer, "%+.2f\n\n", raw_acc[2]);
                USB_SendData(buffer);                
            }        
            
            sample = 0;
        }
    }
}

USB_SendData

void USB_SendData(char *data)
{ 
    if(packet_sent == 1)
    {
        CDC_Send_DATA((unsigned char*) data, strlen(data));    
    }
}

Ich habe das Beispiel so abgeändert das immer nur die Sensordaten 
rausgesendet werden. Allerdings werden diese teilweise falsch 
ausgegeben:

Empfangene Daten:

-329.00,+10.00,-328.00,+12.00,-334.00,+17.00,-329.00,+11.00,-329.00,+13. 
00,+1014.00<\n>
<\n>
+15.00,+1009.00<\n>
<\n>
+12.00,+1012.00<\n>
<\n>
+14.00,+1008.00<\n>

Es scheint mir nach einem Timing Problem, kennt jemand so ein Problem 
oder hat jemand einen Code der meine Problemstellung erfüllt?

[holger (Gast)]

>Allerdings werden diese teilweise falsch ausgegeben:

Woher denkst du soll jemand wissen welche Ausgabe du
ERWARTET hast? Was ist denn jetzt falsch an der Ausgabe?

[Patrick L. (crashdemon)]

Wird eig. aus dem Code ersichtlich. Ich würde z.B. immer Zeilen in der 
folgenden Form erwarten:

0.20,102.00,0.39\n\n

Siehe Codeteil:

sprintf(buffer, "%+.2f,", raw_acc[0]);
USB_SendData(buffer);

sprintf(buffer, "%+.2f,", raw_acc[1]);
USB_SendData(buffer);

sprintf(buffer, "%+.2f\n\n", raw_acc[2]);
USB_SendData(buffer);

[holger (Gast)]

void USB_SendData(char *data)
{ 
    if(packet_sent == 1)
    {
        CDC_Send_DATA((unsigned char*) data, strlen(data));    
    }
}

Jetzt überleg mal was passiert wenn packet_send gleich Null ist.
Richtig: Deine Ausgabe wird nicht gemacht. Das dürfte des Rätsels
Lösung sein;)

[Patrick L. (crashdemon)]

...nein das ist nicht die Lösung. Vorallem sehr kurzsichtig gedacht. Am 
besten wäre es wenn sich hier Leute äussern die die 
STM32_USB-FS-Device_Lib_V4.0.0 einsetzen und sich damit auskennen.

[holger (Gast)]

>...nein das ist nicht die Lösung.

Schade.

> Vorallem sehr kurzsichtig gedacht.

Nö, logische Schlussfolgerung. Wenn packet_sent Null ist
wird nichts gesendet. Aus, Basta.

[Uwe B. (derexponent)]

Patrick L. schrieb:
> Am
> besten wäre es wenn sich hier Leute äussern die die
> STM32_USB-FS-Device_Lib_V4.0.0 einsetzen und sich damit auskennen

mit solchen Äuserungen wäre ich vorsichtig

und holger hat sehr wahrscheinlich recht


versuch mal das hier :

void USB_SendData(char *data)
{ 
    // warte bis letztes Packet komplett gesendet ist
    while(packet_sent != 1);

    // sende neues Packet
    CDC_Send_DATA((unsigned char*) data, strlen(data));    

}

und schreib ob das besser funktioniert

[Patrick L. (crashdemon)]

Hab die oben angegebene Codeänderung mal ausgetestet. Resultat ist das 
dann gar nichts mehr empfangen wird. Ich hab habe packet_sent auch 
pauschal ausgeschlossen, da ich ja mitunter Daten empfange, diese dann 
aber nicht zwingend vollständig sind (siehe oben).

Für mich sieht das nach einem Timing Problem aus. Vllt. kann man die 
Sachen einfach nicht so schnell hintereinander raussenden und man müsste 
eine Wartezeit zwischen den USB_SendData Aufrufen einfügen?

Der ursprüngliche Code von STM sieht so aus:

int main(void)
{
  Set_System();
  Set_USBClock();
  USB_Interrupts_Config();
  USB_Init();
  
  while (1)
  {
    if (bDeviceState == CONFIGURED)
    {
      CDC_Receive_DATA();
      /*Check to see if we have data yet */
      if (Receive_length  != 0)
      {
        if (packet_sent == 1)
          CDC_Send_DATA ((unsigned char*)Receive_Buffer,Receive_length);
        Receive_length = 0;
      }
    }
  }
}

Dort werden die Daten die man vom PC an das Eval Board sendet direkt 
wieder zurück gesendet. Was auch ohne Probleme Funktioniert hat.

Die CDC_Send_DATA Funktion habe ich komnplett von STM übernommen.

uint32_t CDC_Send_DATA (uint8_t *ptrBuffer, uint8_t Send_length)
{
  /*if max buffer is Not reached*/
  if(Send_length < VIRTUAL_COM_PORT_DATA_SIZE)     
  {
    /*Sent flag*/
    packet_sent = 0;
    /* send  packet to PMA*/
    UserToPMABufferCopy((unsigned char*)ptrBuffer, ENDP1_TXADDR, Send_length);
    SetEPTxCount(ENDP1, Send_length);
    SetEPTxValid(ENDP1);
  }
  else
  {
    return 0;
  } 
  return 1;
}

[Uwe B. (derexponent)]

Patrick L. schrieb:
> Ich hab habe packet_sent auch
> pauschal ausgeschlossen, da ich ja mitunter Daten empfange, diese dann
> aber nicht zwingend vollständig sind

und da liegt halt dein Denkfehler

du hast eine IF-Abfrage vor dem eigentlichen senden

if(packet_sent == 1)

solange diese nicht erfüllt ist, kannst du die Funktion "USB_SendData" 
aufrufen sooft du willst...am PC wird nichts ankommen
(das sieht dann halt so aus als ob etwas fehlen würde)

hast du den Quellcode von ST 1:1 übernommen ?
dann müsste im Main.c das hier stehen :

uint32_t packet_sent=1;

was du auch probieren kannst ist alles in einem Rutsch zu senden :

sprintf(buffer, "%+.2f,%+.2f,%+.2f\n\n", raw_acc[0], raw_acc[1], raw_acc[2]);
USB_SendData(buffer);

dann wird halt u.U. der komplett String einmal nicht gesendet
(Vorsicht "bufffer" muss groß genug sein)

oder du machst (wie du schon geschrieben hast)
eine kleine Pause zwischen jedem Sendeaufruf

[Patrick L. (crashdemon)]

Es klappt jetzt, dazu habe ich jetzt einfach die if(packet_sent == 1) 
Abfrage aus void USB_SendData(char *data) komplett entfernt.

Durch einfügen einer Wartezeit von 1ms nach USB_SendData() klappt auch 
das raussenden ohne Probleme.