Merge pull request #458 from jeremiahrose/patch-1

runger1101001 · web-flow · commit 1e1ace5c5fb9 · 2025-03-14T13:01:39.000+01:00
Fix `Pin is not configured as analog channel` error
diff --git a/src/current_sense/hardware_specific/esp32/esp32_adc_driver.cpp b/src/current_sense/hardware_specific/esp32/esp32_adc_driver.cpp
@@ -162,8 +162,8 @@ bool IRAM_ATTR adcInit(uint8_t pin){
         analogReadResolution(SIMPLEFOC_ADC_RES);
     }   
     pinMode(pin, ANALOG);
-    analogSetPinAttenuation(pin, SIMPLEFOC_ADC_ATTEN);
     analogRead(pin);
+    analogSetPinAttenuation(pin, SIMPLEFOC_ADC_ATTEN);
 
 #if CONFIG_IDF_TARGET_ESP32 // if esp32 variant
     __configFastADCs();

Original file line number	Diff line number	Diff line change
`@@ -162,8 +162,8 @@ bool IRAM_ATTR adcInit(uint8_t pin){`
`162`	`162`	`analogReadResolution(SIMPLEFOC_ADC_RES);`
`163`	`163`	`}`
`164`	`164`	`pinMode(pin, ANALOG);`
`165`		`- analogSetPinAttenuation(pin, SIMPLEFOC_ADC_ATTEN);`
`166`	`165`	`analogRead(pin);`
	`166`	`+ analogSetPinAttenuation(pin, SIMPLEFOC_ADC_ATTEN);`
`167`	`167`
`168`	`168`	`#if CONFIG_IDF_TARGET_ESP32 // if esp32 variant`
`169`	`169`	`__configFastADCs();`