Merge pull request #8462 from tannewt/fix_adc_pin_test_idf5

gamblor21 · web-flow · commit f14ca95df48a · 2023-10-06T18:22:56.000-05:00
Fix pin adc unit check
diff --git a/ports/espressif/common-hal/analogio/AnalogIn.c b/ports/espressif/common-hal/analogio/AnalogIn.c
@@ -63,7 +63,7 @@
 
 void common_hal_analogio_analogin_construct(analogio_analogin_obj_t *self,
     const mcu_pin_obj_t *pin) {
-    if (pin->adc_index == 0 || pin->adc_channel == NO_ADC_CHANNEL) {
+    if (pin->adc_index == NO_ADC || pin->adc_channel == NO_ADC_CHANNEL) {
         raise_ValueError_invalid_pin();
     }
     common_hal_mcu_pin_claim(pin);

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@`
`63`	`63`
`64`	`64`	`void common_hal_analogio_analogin_construct(analogio_analogin_obj_t *self,`
`65`	`65`	`const mcu_pin_obj_t *pin) {`
`66`		`- if (pin->adc_index == 0 \|\| pin->adc_channel == NO_ADC_CHANNEL) {`
	`66`	`+ if (pin->adc_index == NO_ADC \|\| pin->adc_channel == NO_ADC_CHANNEL) {`
`67`	`67`	`raise_ValueError_invalid_pin();`
`68`	`68`	`}`
`69`	`69`	`common_hal_mcu_pin_claim(pin);`