STM32 USB CDC Objetkt in Kompilat mehrmals vorhanden

[Benny (Gast)]

Hallo,

ich habe mir per CubeMX ein Projekt zusammengeklickt und habe mich 
aufgrund der geringen Ressourcen des STM32F042K6 mal ein wenig mit der 
Analyse des Maps-Files gechäftigt. Dabei ist mir aufgefallen, dass der 
Compiler (GCC 5.3) ein Objekt im .data-Setment mehrmals anlegt. Ich habe 
die entsprechende Stelle vom Programm AMAP angehängt. Auch mein Projekt 
ist anbei. Falls jemand noch irgendwelche Kompilate benötigt, kann ich 
sie gern hochladen.

Von der Größe her handelt es sich anscheinend um das Objekt hpcd_USB_FS 
in der Datei src/gen/src/usbd_conf.c. Dieses wird in 
src\gen\src\usbd_conf.c definiert/genutzt und in 
rc\gen\src\stm32f0xx_it.c noch einmal genutzt. Warum wird es zweimal 
angelegt? Hat jemand eine Idee?

VG Benny

[Der Andere (Gast)]

Benny schrieb:
> Dieses wird in
> src\gen\src\usbd_conf.c definiert/genutzt und in
> rc\gen\src\stm32f0xx_it.c noch einmal genutzt

Wie ist es in den beiden C Dateien definiert?

[Benny (Gast)]

Denkbar einfach, oder? In der stm32f0xx_it.c:

extern PCD_HandleTypeDef hpcd_USB_FS;

void USB_IRQHandler(void)
{
  /* USER CODE BEGIN USB_IRQn 0 */

  /* USER CODE END USB_IRQn 0 */
  HAL_PCD_IRQHandler(&hpcd_USB_FS);
  /* USER CODE BEGIN USB_IRQn 1 */

  /* USER CODE END USB_IRQn 1 */
}

In der usbd_conf.c:

PCD_HandleTypeDef hpcd_USB_FS;

USBD_StatusTypeDef  USBD_LL_Init (USBD_HandleTypeDef *pdev)
{ 
  /* Init USB_IP */
  /* Link The driver to the stack */
  hpcd_USB_FS.pData = pdev;
  pdev->pData = &hpcd_USB_FS;

  hpcd_USB_FS.Instance = USB;
  hpcd_USB_FS.Init.dev_endpoints = 8;
  hpcd_USB_FS.Init.speed = PCD_SPEED_FULL;
  hpcd_USB_FS.Init.ep0_mps = DEP0CTL_MPS_64;
  hpcd_USB_FS.Init.phy_itface = PCD_PHY_EMBEDDED;
  hpcd_USB_FS.Init.low_power_enable = DISABLE;
  hpcd_USB_FS.Init.lpm_enable = DISABLE;
  hpcd_USB_FS.Init.battery_charging_enable = DISABLE;
  if (HAL_PCD_Init(&hpcd_USB_FS) != HAL_OK)
  {
    Error_Handler();
  }

  HAL_PCDEx_PMAConfig((PCD_HandleTypeDef*)pdev->pData , 0x00 , PCD_SNG_BUF, 0x18);
  HAL_PCDEx_PMAConfig((PCD_HandleTypeDef*)pdev->pData , 0x80 , PCD_SNG_BUF, 0x58);
  HAL_PCDEx_PMAConfig((PCD_HandleTypeDef*)pdev->pData , 0x81 , PCD_SNG_BUF, 0xC0);  
  HAL_PCDEx_PMAConfig((PCD_HandleTypeDef*)pdev->pData , 0x01 , PCD_SNG_BUF, 0x110);
  HAL_PCDEx_PMAConfig((PCD_HandleTypeDef*)pdev->pData , 0x82 , PCD_SNG_BUF, 0x100);  
  return USBD_OK;
}

[Benny (Gast)]

Ich habe den "Fehler" gefunden!

Mir ist aufgefallen, dass das gesuchte Objekt im Map-File keinen Namen, 
sondern eine Nummer besitzt. Ich konnte mich daran erinnern, dass dies 
vor allem bei lokalen statischen Variablen vorgenommen. Also habe ich 
mich in der Datei noch einmal umgesehen. Uns siehe da:

/**
  * @brief  static single allocation.
  * @param  size: size of allocated memory
  * @retval None
  */
void *USBD_static_malloc(uint32_t size)
{
  static uint32_t mem[(sizeof(USBD_CDC_HandleTypeDef)/4)+1];/* On 32-bit boundary */
  return mem;
}

Diese Funktion wird aufgerufen um Speicher für den USB-CDC-IF Stack zu 
alloziieren.

...
  pdev->pClassData = USBD_malloc(sizeof (USBD_CDC_HandleTypeDef));
  
  if(pdev->pClassData == NULL)
...

So und jetzt Hosen runter, wer würde so etwas programmieren? Ich finde 
das mehr als umständlich. Klar kann man so eine Integration in eine 
Speicherverwaltung hinbekommen, aber für einen Mikrocontroller ist das 
ja mehr als überflüssig. Es würde doch ausreichen (wie überall) einfach 
diese Variable zu instanziieren und in die Init zu geben.

[Rufus Τ. F. (rufus)]

Benny schrieb:
> So und jetzt Hosen runter, wer würde so etwas programmieren?

Vermutlich jemand, der dem USB-Stack von einem System, auf dem malloc 
verwendet werden kann, auf eines portiert hat, auf dem er es vermeiden 
möchte, malloc zu verwenden.

Ziemlich übler Hack, und eine interessante Quelle für die lustigsten 
Fehler (man muss nur die Funktion mehrfach aufrufen und annehmen, jeder 
Aufruf gäbe einen unterschiedlichen Pointer zurück ...)