From 850c4689e0ec222f2739ec75425e4f6a2e1f490e Mon Sep 17 00:00:00 2001
From: Andrew Dodd <atd7@cornell.edu>
Date: Sat, 23 Mar 2013 15:57:30 -0700
Subject: Revert "Update power profile to match stock values"

This should not have been merged.  Samsung is NOTORIOUS for screwing up power_profile - we've seen tuna power_profile values on Exynos devices as an example.

Among the obvious problems with the commit that is being reverted:
Missing 1.4 and 1.6 GHz entries on a device that supports operation at those frequencies.
Missing cpu.awake value to handle wakelock power calculation (this was added a long time ago by one of the CM maintainers, prior to the file being forked and copied to multiple repos).
Power consumption values for frequencies between 200 and 1200 MHz that match the profile entries for Exynos 4210 EXACTLY, despite the fact that the Exynos 4210 is built on a 45nm process and the 4412 was built using a 32nm process (consumes less power).

This reverts commit d76e08813ff4cefbc7b00b7c077d3b29804f1cd2

Change-Id: I4b42df70593ff4a7175c0e5acb022da2934c7b7e
---
 .../base/core/res/res/xml/power_profile.xml        | 81 +++++++++++++---------
 1 file changed, 48 insertions(+), 33 deletions(-)

(limited to 'overlay')

diff --git a/overlay/frameworks/base/core/res/res/xml/power_profile.xml b/overlay/frameworks/base/core/res/res/xml/power_profile.xml
index bb61130..9d1a88b 100644
--- a/overlay/frameworks/base/core/res/res/xml/power_profile.xml
+++ b/overlay/frameworks/base/core/res/res/xml/power_profile.xml
@@ -18,37 +18,52 @@
 -->
 
 <device name="Android">
-    <item name="none">0</item>
-    <item name="screen.on">71</item>
-    <item name="bluetooth.active">17</item>
-    <item name="bluetooth.on">0.3</item>
-    <item name="screen.full">380</item>
-    <item name="wifi.on">0.3</item>
-    <item name="wifi.active">96</item>
-    <item name="wifi.scan">70</item>
-    <item name="dsp.audio">44</item>
-    <item name="dsp.video">280</item>
-    <item name="radio.active">250</item>
-    <item name="radio.scanning">82</item>
-    <item name="gps.on">1</item>
-    <array name="radio.on">
-        <value>3.4</value>
-        <value>3.4</value>
-    </array>
-    <array name="cpu.speeds">
-        <value>12000000</value>
-        <value>10000000</value>
-        <value>800000</value>
-        <value>500000</value>
-        <value>200000</value>
-    </array>
-    <item name="cpu.idle">4</item>
-    <array name="cpu.active">
-        <value>577</value>
-        <value>408</value>
-        <value>249</value>
-        <value>148</value>
-        <value>55</value>
-    </array>
-    <item name="battery.capacity">2100</item>
+  <!-- All values are in mAh except as noted -->
+  <item name="none">0</item>
+  <item name="screen.on">71</item>
+  <item name="bluetooth.active">17</item>
+  <item name="bluetooth.on">0.3</item>
+  <item name="bluetooth.at">35690</item>
+  <item name="screen.full">380</item>
+  <item name="wifi.on">0.3</item>
+  <item name="wifi.active">96</item>
+  <item name="wifi.scan">70</item>
+  <item name="dsp.audio">44</item>
+  <item name="dsp.video">280</item>
+  <item name="radio.active">250</item>
+  <!-- The current consumed by the radio when it is scanning for a signal -->
+  <item name="radio.scanning">82</item>
+  <item name="gps.on">50</item>
+  <!-- Current consumed by the radio at different signal strengths, when paging -->
+  <array name="radio.on"> <!-- Strength 0 to BINS-1 -->
+      <value>3.4</value>
+      <value>3.4</value>
+  </array>
+  <!-- Different CPU speeds as reported in
+       /sys/devices/system/cpu/cpu0/cpufreq/stats/time_in_state -->
+  <array name="cpu.speeds">
+      <value>200000</value>
+      <value>500000</value>
+      <value>800000</value>
+      <value>1000000</value>
+      <value>1200000</value>
+      <value>1400000</value>
+      <value>1600000</value>
+  </array>
+  <!-- Power consumption when CPU is idle -->
+  <item name="cpu.idle">4</item>
+  <!-- Power consumption due to wake lock held -->
+  <item name="cpu.awake">44</item>
+  <!-- Power consumption at different speeds -->
+  <array name="cpu.active">
+      <value>55.4</value>
+      <value>82.1</value>
+      <value>113.7</value>
+      <value>140.0</value>
+      <value>170.0</value>
+      <value>200.0</value>
+      <value>230.0</value>
+  </array>
+  <!-- This is the battery capacity in mAh -->
+  <item name="battery.capacity">2100</item>
 </device>
-- 
cgit v1.1