From: Keith Packard <keithp@keithp.com>
Date: Mon, 9 Dec 2013 05:08:36 +0000 (-0800)
Subject: altos: Oops. Was only filling out part of the TeleMetrum ADC record
X-Git-Tag: 1.3~52
X-Git-Url: https://git.gag.com/?p=fw%2Faltos;a=commitdiff_plain;h=dd91a5d5069ff940e07b8817a934ee65d4e8e235

altos: Oops. Was only filling out part of the TeleMetrum ADC record

Because it's missing a return, we'd end up filling out one element of
the ADC record per interrupt, and rotating through which one was set,
hitting all of the even offsets within the struct. Yikes!

Signed-off-by: Keith Packard <keithp@keithp.com>
---

diff --git a/src/cc1111/ao_adc.c b/src/cc1111/ao_adc.c
index 15429677..1689ebef 100644
--- a/src/cc1111/ao_adc.c
+++ b/src/cc1111/ao_adc.c
@@ -85,6 +85,7 @@ ao_adc_isr(void) __interrupt 1
 		else
 #endif
 			ADCCON3 = ADCCON3_EREF_VDD | ADCCON3_EDIV_512 | sequence;
+		return;
 	}
 #endif