From 1245ddcc21c7cf89c25cc5be8d137dd139d95351 Mon Sep 17 00:00:00 2001
From: Matthew Kennedy <matthewkennedy@outlook.com>
Date: Sun, 21 Jul 2024 23:23:52 -0700
Subject: [PATCH] adc clarity

---
 .../config/boards/prometheus/efifeatures.h    |  4 --
 firmware/hw_layer/adc/adc_inputs.cpp          | 56 +++++++------------
 2 files changed, 19 insertions(+), 41 deletions(-)

diff --git a/firmware/config/boards/prometheus/efifeatures.h b/firmware/config/boards/prometheus/efifeatures.h
index 4b16efe98c..fa9a80a077 100644
--- a/firmware/config/boards/prometheus/efifeatures.h
+++ b/firmware/config/boards/prometheus/efifeatures.h
@@ -67,10 +67,6 @@
 
 #undef CONSOLE_MODE_SWITCH_PORT
 
-//!!!!!!!!!!!!!!!
-//#undef EFI_INTERNAL_ADC
-//#define EFI_INTERNAL_ADC FALSE
-
 #undef ADC_VCC
 #define ADC_VCC 3.275f
 
diff --git a/firmware/hw_layer/adc/adc_inputs.cpp b/firmware/hw_layer/adc/adc_inputs.cpp
index 872ae12670..373fe67ac4 100644
--- a/firmware/hw_layer/adc/adc_inputs.cpp
+++ b/firmware/hw_layer/adc/adc_inputs.cpp
@@ -244,30 +244,22 @@ void AdcDevice::enableChannel(adc_channel_e hwChannel) {
 		return;
 	}
 
-	int logicChannel = channelCount++;
+	// hwChannel = which external pin are we using
+	// adcChannelIndex = which ADC channel are we using
+	// adcIndex = which index does that get in sampling order
+	size_t adcChannelIndex = hwChannel - EFI_ADC_0;
+	size_t adcIndex = channelCount++;
 
-	/* TODO: following is correct for STM32 ADC1/2.
-	 * ADC3 has another input to gpio mapping
-	 * and should be handled separately */
-	size_t channelAdcIndex = hwChannel - EFI_ADC_0;
+	internalAdcIndexByHardwareIndex[hwChannel] = adcIndex;
+	hardwareIndexByIndernalAdcIndex[adcIndex] = hwChannel;
 
-	internalAdcIndexByHardwareIndex[hwChannel] = logicChannel;
-	hardwareIndexByIndernalAdcIndex[logicChannel] = hwChannel;
-	if (logicChannel < 6) {
-		m_hwConfig->sqr3 |= channelAdcIndex << (5 * logicChannel);
-	} else if (logicChannel < 12) {
-		m_hwConfig->sqr2 |= channelAdcIndex << (5 * (logicChannel - 6));
-	} else if (logicChannel < 18) {
-		m_hwConfig->sqr1 |= channelAdcIndex << (5 * (logicChannel - 12));
+	if (adcIndex < 6) {
+		m_hwConfig->sqr3 |= adcChannelIndex << (5 * adcIndex);
+	} else if (adcIndex < 12) {
+		m_hwConfig->sqr2 |= adcChannelIndex << (5 * (adcIndex - 6));
+	} else if (adcIndex < 18) {
+		m_hwConfig->sqr1 |= adcChannelIndex << (5 * (adcIndex - 12));
 	}
-#if ADC_MAX_CHANNELS_COUNT > 16
-	else if (logicChannel < 24) {
-		m_hwConfig->sqr4 |= channelAdcIndex << (5 * (logicChannel - 18));
-	}
-	else if (logicChannel < 30) {
-		m_hwConfig->sqr5 |= channelAdcIndex << (5 * (logicChannel - 24));
-	}
-#endif /* ADC_MAX_CHANNELS_COUNT */
 }
 
 adc_channel_e AdcDevice::getAdcHardwareIndexByInternalIndex(int index) const {
@@ -350,27 +342,17 @@ void removeFastAdcChannel(const char *name, adc_channel_e setting) {
 // Weak link a stub so that every board doesn't have to implement this function
 __attribute__((weak)) void setAdcChannelOverrides() { }
 
-static void configureInputs() {
-	memset(adcHwChannelEnabled, 0, sizeof(adcHwChannelEnabled));
-
-	/**
-	 * order of analog channels here is totally random and has no meaning
-	 * we also have some weird implementation with internal indices - that all has no meaning, it's just a random implementation
-	 * which does not mean anything.
-	 */
-
-	addFastAdcChannel("MAP", engineConfiguration->map.sensor.hwChannel);
-	addFastAdcChannel("AUXF#1", engineConfiguration->auxFastSensor1_adcChannel);
-
-	setAdcChannelOverrides();
-}
-
 static CCM_OPTIONAL SlowAdcController slowAdcController;
 
 void initAdcInputs() {
 	efiPrintf("initAdcInputs()");
 
-	configureInputs();
+	memset(adcHwChannelEnabled, 0, sizeof(adcHwChannelEnabled));
+
+	addFastAdcChannel("MAP", engineConfiguration->map.sensor.hwChannel);
+	addFastAdcChannel("AUXF#1", engineConfiguration->auxFastSensor1_adcChannel);
+
+	setAdcChannelOverrides();
 
 #if EFI_INTERNAL_ADC
 	portInitAdc();