diff options
| author | Ethan Chen <intervigil@gmail.com> | 2018-03-01 21:41:04 -0800 |
|---|---|---|
| committer | Michael Bestas <mkbestas@lineageos.org> | 2019-10-23 01:12:56 +0300 |
| commit | 689414dbd0c18379144f96e3bde393b259d541cd (patch) | |
| tree | 8f1b68bdcf50591afd962b8a4aa73f5cd6e7ff2d /power-8916.c | |
| parent | bd869d265f52e4d9e8f96fc625c185b9b762a0b5 (diff) | |
| download | vendor_qcom_opensource_power-689414dbd0c18379144f96e3bde393b259d541cd.tar.gz vendor_qcom_opensource_power-689414dbd0c18379144f96e3bde393b259d541cd.tar.bz2 vendor_qcom_opensource_power-689414dbd0c18379144f96e3bde393b259d541cd.zip | |
power: Use is_interactive_governor/is_schedutil_governor
* Don't copy paste the same governor check code multiple times
Change-Id: I2258553deb02a5ae149b0a96aca5fe5584f0ea14
Diffstat (limited to 'power-8916.c')
| -rw-r--r-- | power-8916.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/power-8916.c b/power-8916.c index 0c59ab3..873c51d 100644 --- a/power-8916.c +++ b/power-8916.c @@ -125,8 +125,7 @@ int set_interactive_override(int on) { /* Display off. */ switch (is_target_8916()) { case 8916: { - if ((strncmp(governor, INTERACTIVE_GOVERNOR, strlen(INTERACTIVE_GOVERNOR)) == 0) && - (strlen(governor) == strlen(INTERACTIVE_GOVERNOR))) { + if (is_interactive_governor(governor)) { int resource_values[] = {TR_MS_50, THREAD_MIGRATION_SYNC_OFF}; if (!display_hint_sent) { @@ -139,8 +138,7 @@ int set_interactive_override(int on) { break; default: { - if ((strncmp(governor, INTERACTIVE_GOVERNOR, strlen(INTERACTIVE_GOVERNOR)) == 0) && - (strlen(governor) == strlen(INTERACTIVE_GOVERNOR))) { + if (is_interactive_governor(governor)) { int resource_values[] = {TR_MS_CPU0_50, TR_MS_CPU4_50, THREAD_MIGRATION_SYNC_OFF}; @@ -174,15 +172,13 @@ int set_interactive_override(int on) { /* Display on. */ switch (is_target_8916()) { case 8916: { - 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; } } break; default: { - if ((strncmp(governor, INTERACTIVE_GOVERNOR, strlen(INTERACTIVE_GOVERNOR)) == 0) && - (strlen(governor) == strlen(INTERACTIVE_GOVERNOR))) { + if (is_interactive_governor(governor)) { /* Recovering MIN_FREQ in display ON case */ snprintf(tmp_str, NODE_MAX, "%d", MIN_FREQ_CPU0_DISP_ON); if (sysfs_write(scaling_min_freq[0], tmp_str) != 0) { |