From d4bc101866c09188bc90d68e3cd9f72d294c3606 Mon Sep 17 00:00:00 2001 From: Stephen 'Okra' Houston Date: Wed, 12 Jul 2017 13:32:56 -0500 Subject: [PATCH] Cpuclock: Clean up some governor handling code. --- src/modules/sysinfo/cpuclock/cpuclock.c | 3 ++- src/modules/sysinfo/cpuclock/cpuclock_config.c | 8 +++++--- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/src/modules/sysinfo/cpuclock/cpuclock.c b/src/modules/sysinfo/cpuclock/cpuclock.c index 71dab70f6..0172a5e79 100644 --- a/src/modules/sysinfo/cpuclock/cpuclock.c +++ b/src/modules/sysinfo/cpuclock/cpuclock.c @@ -67,7 +67,8 @@ _cpuclock_set_thread_governor(void *data, Ecore_Thread *th EINA_UNUSED) { const char *governor = data; - _cpuclock_sysfs_setall("scaling_governor", governor); + if (_cpuclock_sysfs_setall("scaling_governor", governor) == 0) + return; if (!strcmp(governor, "ondemand")) _cpuclock_sysfs_set("ondemand/ignore_nice_load", "0"); else if (!strcmp(governor, "conservative")) diff --git a/src/modules/sysinfo/cpuclock/cpuclock_config.c b/src/modules/sysinfo/cpuclock/cpuclock_config.c index 995258370..255b10104 100644 --- a/src/modules/sysinfo/cpuclock/cpuclock_config.c +++ b/src/modules/sysinfo/cpuclock/cpuclock_config.c @@ -154,7 +154,7 @@ _powersave_changed(void *data, Evas_Object *obj, void *event_info EINA_UNUSED) { Cpuclock_Config *cc = data; Instance *inst = cc->inst; - const char *value = elm_object_text_get(obj); + const char *value = evas_object_data_get(obj, "governor"); if (value) eina_stringshare_replace(&inst->cfg->cpuclock.powersave_governor, value); @@ -167,7 +167,7 @@ _governor_changed(void *data, Evas_Object *obj, void *event_info EINA_UNUSED) { Cpuclock_Config *cc = data; Instance *inst = cc->inst; - const char *value = elm_object_text_get(obj); + const char *value = evas_object_data_get(obj, "governor"); if (value) { @@ -403,7 +403,8 @@ cpuclock_configure(Instance *inst) elm_object_text_set(o, _("Maximum Speed")); else elm_object_text_set(o, l->data); - elm_box_pack_end(box, o); + evas_object_data_set(o, "governor", (const char *)l->data); + elm_box_pack_end(box, o); evas_object_smart_callback_add(o, "changed", _governor_changed, cc); evas_object_show(o); @@ -472,6 +473,7 @@ cpuclock_configure(Instance *inst) else elm_object_text_set(o, l->data); elm_object_disabled_set(o, inst->cfg->cpuclock.auto_powersave); + evas_object_data_set(o, "governor", l->data); elm_box_pack_end(box, o); evas_object_smart_callback_add(o, "changed", _powersave_changed, cc); evas_object_show(o);