scottk/OpenCellular
1
0
Fork 0
mirror of https://github.com/Telecominfraproject/OpenCellular.git synced 2026-01-03 21:49:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove fast PWM tweak mode

Browse Source 
Fast mode was originally introduced to prevent the user seeing battery
assist mode. Now that we don't report battery assist mode to the user,
let's remove this mode to improve stability.

BUG=None
TEST=Charge with various chargers. See stable PWM duty cycles.
BRANCH=Spring

Original-Change-Id: I4a2a66e03d50dea7a28fc826369dcf2b3452007a
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/60754
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
(cherry picked from commit 46a2073437f9a150656d37e8bd9219b01b1b3ac1)

Change-Id: I8c5b24946385576ada9f08e342beb1a70c55c974
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/61399
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
This commit is contained in:
Vic Yang
2013-07-02 17:23:59 +08:00
committed by ChromeBot
parent 0d362effd7
commit 913f227e36
1 changed files with 4 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
common/extpower_usb.c
View File

@@ -75,7 +75,6 @@ enum ilim_config {
#define PWM_CTRL_OC_RETRY 2
#define PWM_CTRL_STEP_DOWN 3
#define PWM_CTRL_STEP_UP 5
#define PWM_CTRL_STEP_FAST_DOWN 15
#define PWM_CTRL_VBUS_HARD_LOW 4400
#define PWM_CTRL_VBUS_LOW 4500
#define PWM_CTRL_VBUS_HIGH 4700 /* Must be higher than 4.5V */
@@ -99,7 +98,6 @@ static int current_dev_type = TSU6721_TYPE_NONE;
static int nominal_pwm_duty;
static int current_pwm_duty;
static int user_pwm_duty = -1;
static int pwm_fast_mode;
static int pending_tsu6721_reset;
static int pending_adc_watchdog_disable;
@@ -371,13 +369,7 @@ static void set_pwm_duty_cycle(int percent)
*/
static int pwm_get_next_lower(void)
{
int fast_next = current_pwm_duty - PWM_CTRL_STEP_FAST_DOWN;
if (current_limit_mode == LIMIT_AGGRESSIVE) {
if (pwm_fast_mode &&
fast_next > nominal_pwm_duty - PWM_CTRL_OC_MARGIN &&
fast_next > over_current_pwm_duty)
return MAX(fast_next, 0);
if (current_pwm_duty > nominal_pwm_duty -
PWM_CTRL_OC_MARGIN &&
current_pwm_duty > over_current_pwm_duty &&
@@ -425,7 +417,6 @@ static void pwm_nominal_duty_cycle(int percent)
set_pwm_duty_cycle(new_percent);
nominal_pwm_duty = percent;
pwm_fast_mode = 1;
}
static void adc_watch_vbus(int high, int low)
@@ -503,11 +494,15 @@ static void usb_detect_overcurrent(int dev_type)
}
if (power_removed_type[idx] == dev_type) {
if (oc_detect_retry[idx] > 0) {
CPRINTF("[%T USB overcurrent: Retry (%d)]\n",
oc_detect_retry[idx]);
oc_detect_retry[idx]--;
return;
}
over_current_pwm_duty = power_removed_pwm_duty[idx] +
PWM_CTRL_OC_BACK_OFF;
CPRINTF("[%T USB overcurrent: Limited to %d%%]\n",
over_current_pwm_duty);
}
}
}
@@ -824,7 +819,6 @@ static void pwm_tweak(void)
*/
if (pwm_check_vbus_low(vbus, current)) {
set_pwm_duty_cycle(current_pwm_duty + PWM_CTRL_STEP_UP);
pwm_fast_mode = 0;
CPRINTF("[%T PWM duty up %d%%]\n", current_pwm_duty);
} else if (pwm_check_vbus_high(vbus)) {
next = pwm_get_next_lower();