STM32 - Fehler in CMSIS system_stm32l0xx.c

[Mehmet K. (mkmk)]

Nachdem ich bei einem STM32L051 die HSI Frequenz heraufgesetzt hatte, 
rief ich SystemCoreClockUpdate() auf. Und meine Kontroll-LED spielte 
verrückt.

system_stm32l0xx.c Zeile 249:

    case 0x0C:  /* PLL used as system clock */
      (...)
      if (pllsource == 0x00)
      {
        /* HSI oscillator clock selected as PLL clock entry */
        SystemCoreClock = (((HSI_VALUE) * pllmul) / plldiv);    <--- Fehler
      }
      else
      {
        /* HSE selected as PLL clock entry */
        SystemCoreClock = (((HSE_VALUE) * pllmul) / plldiv);
      }
      break;
      (...)

HSI_VALUE ist mit einem define auf 16MHz gesetzt.
Es wird dabei nicht nachgefragt, ob der HSI nicht vielleicht mit 
HSI16DIVF gestartet worden war.

Korrekt sollte es sein:

uint32_t hsi_value = (RCC->CR & RCC_CR_HSIDIVEN) ? HSI_VALUE / 4 : HSI_VALUE;
SystemCoreClock = ((hsi_value * pllmul) / plldiv);

Muss vielleicht erwaehen, dass ich zum ersten Mal mit HSI arbeite.
Könnte also sein, dass dieser Fehler auch in anderen system_stm32xxx 
Files vorhanden ist.

[... (Gast)]

Das HSI16DIVF haben nur die STM32L0.

[Bernd K. (prof7bit)]

Bei mir sieht die so aus:

void SystemCoreClockUpdate() {
}

Und käme ich auf die Idee die Taktfrequenz im laufenden Betrieb zu 
ändern dann würde SystemCoreClock genau dort geupdatet wo es 
vernünftigerweise hingehört, nämlich dort wo der Takt geändert wurde und 
wo man nicht anhand der Registerwerte umständlich herumraten muß was man 
eben gerade explizit getan hat.