From 85e00bf2ce6aa080a42ad6f8dd47733b514cceac Mon Sep 17 00:00:00 2001 From: Zhao Wei Liew Date: Sun, 26 Jun 2016 21:29:34 +0800 Subject: power: Simplify display_hint_sent Check for whether the display hint has been sent at the start of the set_interactive() function in the common powerHAL. This gets rid of the need to replicate the same variable in every target-specific powerHAL that implements the set_interactive_override() function. Change-Id: If7dd11fcb578211f4f83847f9257232c4138ce53 --- power-8992.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) (limited to 'power-8992.c') diff --git a/power-8992.c b/power-8992.c index 49e73c6..f41d537 100644 --- a/power-8992.c +++ b/power-8992.c @@ -47,8 +47,6 @@ #include "power-common.h" #include "utils.h" -static int display_hint_sent; - static int process_video_encode_hint(void* metadata) { char governor[80]; struct video_encode_metadata_t video_encode_metadata; @@ -121,18 +119,14 @@ int set_interactive_override(int on) { /* Display off */ if (is_interactive_governor(governor)) { int resource_values[] = {0x777}; /* 4+0 core config in display off */ - if (!display_hint_sent) { - perform_hint_action(DISPLAY_STATE_HINT_ID, resource_values, - sizeof(resource_values) / sizeof(resource_values[0])); - display_hint_sent = 1; - return HINT_HANDLED; - } + perform_hint_action(DISPLAY_STATE_HINT_ID, resource_values, + sizeof(resource_values) / sizeof(resource_values[0])); + return HINT_HANDLED; } } else { /* Display on */ if (is_interactive_governor(governor)) { undo_hint_action(DISPLAY_STATE_HINT_ID); - display_hint_sent = 0; return HINT_HANDLED; } } -- cgit v1.2.3