diff options
| author | Yuuki NAGAO <wf.yn386@gmail.com> | 2025-09-21 10:07:02 +0900 |
|---|---|---|
| committer | Damien George <damien@micropython.org> | 2025-09-23 14:48:16 +1000 |
| commit | 7ba82219f3d40689479704edfcc97fcf5b914b2c (patch) | |
| tree | 586532153a2235c4343e4ed3e6a48a17b167a5be | |
| parent | 65b2dc255b2415e7afd6960587a1857297f6913a (diff) | |
stm32/adc: Fix pyb.ADC issue for STM32G0.
Fixes are:
- The internal sensors of STM32G0 are connected channel 12, 13, 14.
- Update adc_refcor before reading vbat and tempsensor because reference
values are at VDDA=3.0V.
- The method of initialize of sampling time is different with other MCUs:
STM32G0 should initialize SamplingTimeCommon1/2 first. When reading
value from ADC, ADC_ChannelConfTypeDef.SamplingTime should be specified
SamplingTimeCommon1 or SamplingTimeCommon2. In this patch,
SamplingTimeCommon2 is used for internal sensors and SamplingTimeCommon1
is used for external ADC channels.
Signed-off-by: Yuuki NAGAO <wf.yn386@gmail.com>
| -rw-r--r-- | ports/stm32/adc.c | 24 |
1 files changed, 18 insertions, 6 deletions
diff --git a/ports/stm32/adc.c b/ports/stm32/adc.c index f47e9eaad..07368f473 100644 --- a/ports/stm32/adc.c +++ b/ports/stm32/adc.c @@ -212,6 +212,14 @@ static inline uint32_t adc_get_internal_channel(uint32_t channel) { if (channel == 16) { channel = ADC_CHANNEL_TEMPSENSOR; } + #elif defined(STM32G0) + if (channel == 12) { + channel = ADC_CHANNEL_TEMPSENSOR; + } else if (channel == 13) { + channel = ADC_CHANNEL_VREFINT; + } else if (channel == 14) { + channel = ADC_CHANNEL_VBAT; + } #elif defined(STM32G4) if (channel == 16) { channel = ADC_CHANNEL_TEMPSENSOR_ADC1; @@ -344,6 +352,10 @@ static void adcx_init_periph(ADC_HandleTypeDef *adch, uint32_t resolution) { adch->Init.DataAlign = ADC_DATAALIGN_RIGHT; adch->Init.DMAContinuousRequests = DISABLE; #elif defined(STM32G0) || defined(STM32G4) || defined(STM32H5) || defined(STM32L4) || defined(STM32WB) + #if defined(STM32G0) + adch->Init.SamplingTimeCommon1 = ADC_SAMPLETIME_12CYCLES_5; + adch->Init.SamplingTimeCommon2 = ADC_SAMPLETIME_160CYCLES_5; + #endif #if defined(STM32G4) || defined(STM32H5) adch->Init.ClockPrescaler = ADC_CLOCK_ASYNC_DIV16; #else @@ -432,9 +444,9 @@ static void adc_config_channel(ADC_HandleTypeDef *adc_handle, uint32_t channel) } #elif defined(STM32G0) if (__HAL_ADC_IS_CHANNEL_INTERNAL(channel)) { - sConfig.SamplingTime = ADC_SAMPLETIME_160CYCLES_5; + sConfig.SamplingTime = ADC_SAMPLINGTIME_COMMON_2; } else { - sConfig.SamplingTime = ADC_SAMPLETIME_12CYCLES_5; + sConfig.SamplingTime = ADC_SAMPLINGTIME_COMMON_1; } #elif defined(STM32G4) || defined(STM32H5) || defined(STM32L4) || defined(STM32WB) if (__HAL_ADC_IS_CHANNEL_INTERNAL(channel)) { @@ -939,9 +951,9 @@ float adc_read_core_temp_float(ADC_HandleTypeDef *adcHandle) { return 0.0f; // TODO #else - #if defined(STM32G4) || defined(STM32L1) || defined(STM32L4) + #if defined(STM32G0) || defined(STM32G4) || defined(STM32L1) || defined(STM32L4) // Update the reference correction factor before reading tempsensor - // because TS_CAL1 and TS_CAL2 of STM32G4,L1/L4 are at VDDA=3.0V + // because TS_CAL1 and TS_CAL2 of STM32G0,G4,L1,L4 are at VDDA=3.0V adc_read_core_vref(adcHandle); #endif @@ -966,9 +978,9 @@ float adc_read_core_temp_float(ADC_HandleTypeDef *adcHandle) { } float adc_read_core_vbat(ADC_HandleTypeDef *adcHandle) { - #if defined(STM32G4) || defined(STM32L4) + #if defined(STM32G0) || defined(STM32G4) || defined(STM32L4) // Update the reference correction factor before reading tempsensor - // because VREFINT of STM32G4,L4 is at VDDA=3.0V + // because VREFINT of STM32G0,G4,L4 is at VDDA=3.0V adc_read_core_vref(adcHandle); #endif |
