From 43b9e3524f90a59e59f4d6a1f0e7f6629e4d029f Mon Sep 17 00:00:00 2001 From: Michael Bestas Date: Sun, 6 Oct 2019 01:34:17 +0300 Subject: power: Clean up and fix set_interactive_override * Remove debug logs and useless comments * Consistent return values * Add back display hints for legacy platforms, they used to live in power-common.c but removed in commit 561cffcbfa66192e839e040d2da3bff2ecf8dcac * Check if set_interactive_override handled hints and print info message if it didn't Change-Id: Iff9f7327ba71c282e34ff3143c45a3f40f72fc92 --- power-8084.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) (limited to 'power-8084.c') diff --git a/power-8084.c b/power-8084.c index 73176c1..8612769 100644 --- a/power-8084.c +++ b/power-8084.c @@ -145,12 +145,11 @@ int set_interactive_override(int on) { if (get_scaling_governor(governor, sizeof(governor)) == -1) { ALOGE("Can't obtain scaling governor."); - return HINT_NONE; } if (!on) { - /* Display off. */ + /* Display off */ /* * We need to be able to identify the first display off hint * and release the current lock holder @@ -161,12 +160,19 @@ int set_interactive_override(int on) { } /* Used for all subsequent toggles to the display */ undo_hint_action(DISPLAY_STATE_HINT_ID_2); + if (is_interactive_governor(governor)) { + int resource_values[] = {TR_MS_50, THREAD_MIGRATION_SYNC_OFF}; + perform_hint_action(DISPLAY_STATE_HINT_ID, resource_values, + ARRAY_SIZE(resource_values)); + } } else { /* Display on */ int resource_values2[] = {CPUS_ONLINE_MIN_2}; perform_hint_action(DISPLAY_STATE_HINT_ID_2, resource_values2, ARRAY_SIZE(resource_values2)); + if (is_interactive_governor(governor)) { + undo_hint_action(DISPLAY_STATE_HINT_ID); + } } - - return HINT_NONE; + return HINT_HANDLED; } -- cgit v1.2.3