From 783c77531c35e62dd754c510c4f2beefe6df4a9d Mon Sep 17 00:00:00 2001 From: Michael Sevakis Date: Sat, 21 Jan 2017 08:04:43 -0500 Subject: AMS: Return ascodec to interrupt-based I2C2 driver 1. Slightly revised and regularized internal interface. Callback is used for read and write to provide completion signal instead of having two mechanisms. 2. Lower overhead for asynchronous or alterate completion callbacks. We now only init what is required by the transfer. A couple unneeded structure members were also nixed. 3. Fixes a bug that would neglect a semaphore wait if pumping the I2C interrupts in a loop when not in thread state or interrupts are masked. 4. Corrects broken initialization order by defining KDEV_INIT, which makes kernel_init() call kernel_device_init() to initialize additional devices _after_ the kernel, threading and synchronization objects are safe to use. 5. Locking set_cpu_frequency has to be done at the highest level in system.c to ensure the boost counter and the frequency are both set in agreement. Reconcile the locking inteface between PP and AMS (the only two currently using locking there) to keep it clean. Now works fine with voltages in GIT HEAD on my Fuze v2, type 0. Previously, everything crashed and died instantly. action.c calling set_cpu_frequency from a tick was part of it. The rest may have been related to 3. and 4. Honestly, I'm not certain! Testing by Mihail Zenkov indicates it solves our problems. This will get the developer builds running again after the kernel assert code push. Change-Id: Ie245994fb3e318dd5ef48e383ce61fdd977224d4 --- apps/debug_menu.c | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'apps') diff --git a/apps/debug_menu.c b/apps/debug_menu.c index d35f40229e..000dbdc4fc 100644 --- a/apps/debug_menu.c +++ b/apps/debug_menu.c @@ -796,6 +796,12 @@ static bool dbg_cpufreq(void) lcd_putsf(0, line++, "Frequency: %ld.%ld MHz", temp, (FREQ-temp*1000000)/100000); lcd_putsf(0, line++, "boost_counter: %d", get_cpu_boost_counter()); +#ifdef HAVE_ADJUSTABLE_CPU_VOLTAGE + extern int get_cpu_voltage_setting(void); + temp = get_cpu_voltage_setting(); + lcd_putsf(0, line++, "CPU voltage: %d.%03dV", temp / 1000, temp % 1000); +#endif + lcd_update(); button = get_action(CONTEXT_STD,HZ/10); -- cgit v1.2.3