From 689414dbd0c18379144f96e3bde393b259d541cd Mon Sep 17 00:00:00 2001 From: Ethan Chen Date: Thu, 1 Mar 2018 21:41:04 -0800 Subject: power: Use is_interactive_governor/is_schedutil_governor * Don't copy paste the same governor check code multiple times Change-Id: I2258553deb02a5ae149b0a96aca5fe5584f0ea14 --- power-8996.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) (limited to 'power-8996.c') diff --git a/power-8996.c b/power-8996.c index 440e9be..2cd8339 100644 --- a/power-8996.c +++ b/power-8996.c @@ -75,8 +75,7 @@ static int process_video_encode_hint(void* metadata) { } if (video_encode_metadata.state == 1) { - if ((strncmp(governor, INTERACTIVE_GOVERNOR, strlen(INTERACTIVE_GOVERNOR)) == 0) && - (strlen(governor) == strlen(INTERACTIVE_GOVERNOR))) { + if (is_interactive_governor(governor)) { /* 1. cpufreq params * -above_hispeed_delay for LVT - 40ms * -go hispeed load for LVT - 95 @@ -108,8 +107,7 @@ static int process_video_encode_hint(void* metadata) { return HINT_HANDLED; } } else if (video_encode_metadata.state == 0) { - if ((strncmp(governor, INTERACTIVE_GOVERNOR, strlen(INTERACTIVE_GOVERNOR)) == 0) && - (strlen(governor) == strlen(INTERACTIVE_GOVERNOR))) { + if (is_interactive_governor(governor)) { camera_hint_ref_count--; if (!camera_hint_ref_count) { undo_hint_action(video_encode_metadata.hint_id); @@ -146,8 +144,7 @@ int set_interactive_override(int on) { if (!on) { /* Display off */ - if ((strncmp(governor, INTERACTIVE_GOVERNOR, strlen(INTERACTIVE_GOVERNOR)) == 0) && - (strlen(governor) == strlen(INTERACTIVE_GOVERNOR))) { + if (is_interactive_governor(governor)) { int resource_values[] = {}; /* dummy node */ if (!display_hint_sent) { perform_hint_action(DISPLAY_STATE_HINT_ID, resource_values, @@ -159,8 +156,7 @@ int set_interactive_override(int on) { } } else { /* Display on */ - if ((strncmp(governor, INTERACTIVE_GOVERNOR, strlen(INTERACTIVE_GOVERNOR)) == 0) && - (strlen(governor) == strlen(INTERACTIVE_GOVERNOR))) { + if (is_interactive_governor(governor)) { undo_hint_action(DISPLAY_STATE_HINT_ID); display_hint_sent = 0; ALOGI("Display Off hint stop"); -- cgit v1.2.3