From 6f0320a9535bc1aa81d83fa879ac14d5ee603658 Mon Sep 17 00:00:00 2001 From: William Wilgus Date: Fri, 27 Jul 2018 23:56:32 +0200 Subject: As3525 v1/v2 Add power savings menu Allow user to select cpu undervolt There have been quite a few issues across the SANSA AMS line related to CPU undervolting while most players show greatly increased runtime some crash. Rather than constanly upping the voltage we now have a setting with a safe value for all players and the option for lower voltages I plan to add a few other options here later such as disk timings and maybe some other clocks/experimental settings Added: Disk Low speed option for AS3525v2 devices cuts frequency to 12 MHz from 24 MHz Added: Disk Low speed option for AS3525v1 devices cuts frequency to 15.5 MHz from 31 MHz Added: I2c Low Speed AS3525 devices, should be bigger improvement for v1 devices Fixed: Debug menu for AS3525v2 No SDSLOT frequency, Showed IDE freq though it is unused Added: DBOP and SSP underclocking affects display on v1/v2 respectively Fixed: debug menu now has SSP frequency, and SSP_CPSR Update: made settings menu more generic Update: cleaned up code Added: Clip v1 & Fuze v1 didn't have HAVE_ADJUSTABLE_CPU_VOLTAGE. not sure why but, waiting on testing to confirm Added: C200v2 and E200v2 devices and HAVE_ADJUSTABLE_CPU_VOLTAGE. Fixed: v1 devices don't like display timing set lower (dbop) v1 devices don't have a divider set for ssp (causes divide by 0) Fixed: ClipZip display lags with Max SSP divider changed from 0xFE to 0x32 Fixed: v1 devices didn't work properly with highspeed sd cards Added code from http://gerrit.rockbox.org/r/#/c/1704/ Added powersave and IDE interface enable/disable Added: V2 devices now have powersave enabled on sd interface Update: cleaned up code, lang defines, added manual entries Update ssp clock mechanism added calculated ssp divider to clipzip Update turn display clock off when clip+ turns off display Fixed: clipzip wrong register for SSP clock Change-Id: I04137682243be92f0f8d8bf1cfa54fbb1965559b TODO: add other players? --- firmware/target/arm/as3525/system-as3525.c | 70 +++++++++++++++++++++++++++++- 1 file changed, 69 insertions(+), 1 deletion(-) (limited to 'firmware/target/arm/as3525/system-as3525.c') diff --git a/firmware/target/arm/as3525/system-as3525.c b/firmware/target/arm/as3525/system-as3525.c index d630ef38ec..c11c90f9f3 100644 --- a/firmware/target/arm/as3525/system-as3525.c +++ b/firmware/target/arm/as3525/system-as3525.c @@ -52,6 +52,29 @@ struct mutex cpufreq_mtx; #define default_interrupt(name) \ extern __attribute__((weak,alias("UIRQ"))) void name (void) +#ifdef CONFIG_POWER_SAVING +/* Powersave functions either manipulate the system directly + or pass enabled flag on to these specific functions + dis/enabling powersaving for the selected subsystem +*/ +#if (CONFIG_POWER_SAVING & POWERSV_CPU) +/*cpu_set_powersave*/ +#include "settings.h" +#endif +#if (CONFIG_POWER_SAVING & POWERSV_DISP) +/*disp_set_powersave*/ +void ams_ssp_set_low_speed(bool slow); /*lcd-clip-plus.c & lcd-clipzip.c*/ +#endif +#if (CONFIG_POWER_SAVING & POWERSV_DISK) +/*disk_set_powersave*/ +void ams_sd_set_low_speed(bool slow); /* sd-as3525.c & sd-as3525v2.c */ +#endif +#if (CONFIG_POWER_SAVING & POWERSV_I2C) +/*i2c_set_powersave*/ +void ams_i2c_set_low_speed(bool slow); /* ascodec-as3525.c*/ +#endif +#endif /*CONFIG_POWER_SAVING*/ + #if CONFIG_USBOTG != USBOTG_DESIGNWARE static void UIRQ (void) __attribute__((interrupt ("IRQ"))); #endif @@ -422,6 +445,39 @@ void udelay(unsigned usecs) ); } +#ifdef CONFIG_POWER_SAVING +#if (CONFIG_POWER_SAVING & POWERSV_CPU) +void cpu_set_powersave(bool enabled) +{ + /*global_settings.cpu_powersave*/ + /*handled in: set_cpu_frequency()*/ + (void) enabled; +} +#endif +#if (CONFIG_POWER_SAVING & POWERSV_DISK) +void disk_set_powersave(bool enabled) +{ + /*global_settings.disk_powersave*/ + ams_sd_set_low_speed(enabled); +} +#endif +#if (CONFIG_POWER_SAVING & POWERSV_DISP) +void disp_set_powersave(bool enabled) +{ + /*global_settings.disp_powersave*/ + ams_ssp_set_low_speed(enabled); +} +#endif +#if (CONFIG_POWER_SAVING & POWERSV_I2C) +void i2c_set_powersave(bool enabled) +{ + /*global_settings.i2c_powersave*/ + ams_i2c_set_low_speed(enabled); +} +#endif +#endif /*defined(CONFIG_POWER_SAVING)*/ + + #ifndef BOOTLOADER #ifdef HAVE_ADJUSTABLE_CPU_FREQ bool set_cpu_frequency__lock(void) @@ -481,7 +537,12 @@ void set_cpu_frequency(long frequency) CGU_PROC = ((0xf << 4) | (0x3 << 2) | AS3525_CLK_MAIN); #ifdef HAVE_ADJUSTABLE_CPU_VOLTAGE - /* Decreasing frequency so reduce voltage after change */ + /* Decreasing frequency so reduce voltage after change */ +#if defined(CONFIG_POWER_SAVING) && (CONFIG_POWER_SAVING & POWERSV_CPU) + if (!global_settings.cpu_powersave) + ascodec_write(AS3514_CVDD_DCDC3, (AS314_CP_DCDC3_SETTING | CVDD_1_15)); + else +#endif ascodec_write(AS3514_CVDD_DCDC3, (AS314_CP_DCDC3_SETTING | CVDD_1_10)); #endif /* HAVE_ADJUSTABLE_CPU_VOLTAGE */ @@ -519,6 +580,13 @@ void set_cpu_frequency(long frequency) /* Set CVDD1 power supply */ #ifdef HAVE_ADJUSTABLE_CPU_VOLTAGE +#if defined(CONFIG_POWER_SAVING) && (CONFIG_POWER_SAVING & POWERSV_CPU) + if (!global_settings.cpu_powersave) + { + ascodec_write_pmu(0x17, 1, 0x80 | 26); + return; + } +#endif #if defined(SANSA_CLIPZIP) ascodec_write_pmu(0x17, 1, 0x80 | 20); #elif defined(SANSA_CLIPPLUS) -- cgit v1.2.3