diff --git a/src/modules/cpufreq/e_mod_main.c b/src/modules/cpufreq/e_mod_main.c index 33154999a..7acd8d59d 100644 --- a/src/modules/cpufreq/e_mod_main.c +++ b/src/modules/cpufreq/e_mod_main.c @@ -8,18 +8,18 @@ /* gadcon requirements */ static E_Gadcon_Client *_gc_init(E_Gadcon *gc, const char *name, const char *id, const char *style); -static void _gc_shutdown(E_Gadcon_Client *gcc); -static void _gc_orient(E_Gadcon_Client *gcc, E_Gadcon_Orient orient); -static const char *_gc_label(const E_Gadcon_Client_Class *client_class); -static Evas_Object *_gc_icon(const E_Gadcon_Client_Class *client_class, Evas *evas); -static const char *_gc_id_new(const E_Gadcon_Client_Class *client_class); +static void _gc_shutdown(E_Gadcon_Client *gcc); +static void _gc_orient(E_Gadcon_Client *gcc, E_Gadcon_Orient orient); +static const char *_gc_label(const E_Gadcon_Client_Class *client_class); +static Evas_Object *_gc_icon(const E_Gadcon_Client_Class *client_class, Evas *evas); +static const char *_gc_id_new(const E_Gadcon_Client_Class *client_class); /* and actually define the gadcon class that this module provides (just 1) */ static const E_Gadcon_Client_Class _gadcon_class = { GADCON_CLIENT_CLASS_VERSION, "cpufreq", - { - _gc_init, _gc_shutdown, _gc_orient, _gc_label, _gc_icon, _gc_id_new, NULL, NULL - }, + { + _gc_init, _gc_shutdown, _gc_orient, _gc_label, _gc_icon, _gc_id_new, NULL, NULL + }, E_GADCON_CLIENT_STYLE_PLAIN }; @@ -29,36 +29,36 @@ typedef struct _Instance Instance; struct _Instance { E_Gadcon_Client *gcc; - Evas_Object *o_cpu; + Evas_Object *o_cpu; }; -static void _button_cb_mouse_down(void *data, Evas *e, Evas_Object *obj, void *event_info); -static void _menu_cb_post(void *data, E_Menu *m); -static void _cpufreq_set_governor(const char *governor); -static void _cpufreq_set_frequency(int frequency); +static void _button_cb_mouse_down(void *data, Evas *e, Evas_Object *obj, void *event_info); +static void _menu_cb_post(void *data, E_Menu *m); +static void _cpufreq_set_governor(const char *governor); +static void _cpufreq_set_frequency(int frequency); static Eina_Bool _cpufreq_cb_check(void *data); -static Status *_cpufreq_status_new(void); -static void _cpufreq_status_free(Status *s); -static int _cpufreq_status_check_available(Status *s); -static int _cpufreq_status_check_current(Status *s); -static int _cpufreq_cb_sort(const void *item1, const void *item2); -static void _cpufreq_face_update_available(Instance *inst); -static void _cpufreq_face_update_current(Instance *inst); -static void _cpufreq_face_cb_set_frequency(void *data, Evas_Object *o, const char *emission, const char *source); -static void _cpufreq_face_cb_set_governor(void *data, Evas_Object *o, const char *emission, const char *source); +static Status *_cpufreq_status_new(void); +static void _cpufreq_status_free(Status *s); +static int _cpufreq_status_check_available(Status *s); +static int _cpufreq_status_check_current(Status *s); +static int _cpufreq_cb_sort(const void *item1, const void *item2); +static void _cpufreq_face_update_available(Instance *inst); +static void _cpufreq_face_update_current(Instance *inst); +static void _cpufreq_face_cb_set_frequency(void *data, Evas_Object *o, const char *emission, const char *source); +static void _cpufreq_face_cb_set_governor(void *data, Evas_Object *o, const char *emission, const char *source); static Eina_Bool _cpufreq_event_cb_powersave(void *data __UNUSED__, int type, void *event); -static void _cpufreq_menu_fast(void *data, E_Menu *m, E_Menu_Item *mi); -static void _cpufreq_menu_medium(void *data, E_Menu *m, E_Menu_Item *mi); -static void _cpufreq_menu_normal(void *data, E_Menu *m, E_Menu_Item *mi); -static void _cpufreq_menu_slow(void *data, E_Menu *m, E_Menu_Item *mi); -static void _cpufreq_menu_very_slow(void *data, E_Menu *m, E_Menu_Item *mi); -static void _cpufreq_menu_restore_governor(void *data, E_Menu *m, E_Menu_Item *mi); -static void _cpufreq_menu_auto_powersave(void *data, E_Menu *m, E_Menu_Item *mi); -static void _cpufreq_menu_governor(void *data, E_Menu *m, E_Menu_Item *mi); -static void _cpufreq_menu_powersave_governor(void *data, E_Menu *m, E_Menu_Item *mi); -static void _cpufreq_menu_frequency(void *data, E_Menu *m, E_Menu_Item *mi); -static void _cpufreq_poll_interval_update(void); +static void _cpufreq_menu_fast(void *data, E_Menu *m, E_Menu_Item *mi); +static void _cpufreq_menu_medium(void *data, E_Menu *m, E_Menu_Item *mi); +static void _cpufreq_menu_normal(void *data, E_Menu *m, E_Menu_Item *mi); +static void _cpufreq_menu_slow(void *data, E_Menu *m, E_Menu_Item *mi); +static void _cpufreq_menu_very_slow(void *data, E_Menu *m, E_Menu_Item *mi); +static void _cpufreq_menu_restore_governor(void *data, E_Menu *m, E_Menu_Item *mi); +static void _cpufreq_menu_auto_powersave(void *data, E_Menu *m, E_Menu_Item *mi); +static void _cpufreq_menu_governor(void *data, E_Menu *m, E_Menu_Item *mi); +static void _cpufreq_menu_powersave_governor(void *data, E_Menu *m, E_Menu_Item *mi); +static void _cpufreq_menu_frequency(void *data, E_Menu *m, E_Menu_Item *mi); +static void _cpufreq_poll_interval_update(void); static E_Config_DD *conf_edd = NULL; @@ -75,7 +75,7 @@ _gc_init(E_Gadcon *gc, const char *name, const char *id, const char *style) o = edje_object_add(gc->evas); e_theme_edje_object_set(o, "base/theme/modules/cpufreq", - "e/modules/cpufreq/main"); + "e/modules/cpufreq/main"); edje_object_signal_callback_add(o, "e,action,governor,next", "*", _cpufreq_face_cb_set_governor, NULL); edje_object_signal_callback_add(o, "e,action,frequency,increase", "*", @@ -90,7 +90,7 @@ _gc_init(E_Gadcon *gc, const char *name, const char *id, const char *style) inst->o_cpu = o; evas_object_event_callback_add(o, EVAS_CALLBACK_MOUSE_DOWN, - _button_cb_mouse_down, inst); + _button_cb_mouse_down, inst); cpufreq_config->instances = eina_list_append(cpufreq_config->instances, inst); if (cpufreq_config->status) _cpufreq_status_free(cpufreq_config->status); @@ -140,7 +140,7 @@ _gc_icon(const E_Gadcon_Client_Class *client_class __UNUSED__, Evas *evas) o = edje_object_add(evas); snprintf(buf, sizeof(buf), "%s/e-module-cpufreq.edj", - e_module_dir_get(cpufreq_config->module)); + e_module_dir_get(cpufreq_config->module)); edje_object_file_set(o, buf, "icon"); return o; } @@ -168,200 +168,200 @@ _button_cb_mouse_down(void *data, Evas *e __UNUSED__, Evas_Object *obj __UNUSED_ if (ev->button == 1) { - E_Menu *mg, *mo; - E_Menu_Item *mi; - Eina_List *l; - int cx, cy; - char buf[256]; + E_Menu *mg, *mo; + E_Menu_Item *mi; + Eina_List *l; + int cx, cy; + char buf[256]; - mo = e_menu_new(); - cpufreq_config->menu_poll = mo; + mo = e_menu_new(); + cpufreq_config->menu_poll = mo; - mi = e_menu_item_new(mo); - e_menu_item_label_set(mi, _("Fast (4 ticks)")); - e_menu_item_radio_set(mi, 1); - e_menu_item_radio_group_set(mi, 1); - if (cpufreq_config->poll_interval <= 4) e_menu_item_toggle_set(mi, 1); - e_menu_item_callback_set(mi, _cpufreq_menu_fast, NULL); + mi = e_menu_item_new(mo); + e_menu_item_label_set(mi, _("Fast (4 ticks)")); + e_menu_item_radio_set(mi, 1); + e_menu_item_radio_group_set(mi, 1); + if (cpufreq_config->poll_interval <= 4) e_menu_item_toggle_set(mi, 1); + e_menu_item_callback_set(mi, _cpufreq_menu_fast, NULL); - mi = e_menu_item_new(mo); - e_menu_item_label_set(mi, _("Medium (8 ticks)")); - e_menu_item_radio_set(mi, 1); - e_menu_item_radio_group_set(mi, 1); - if (cpufreq_config->poll_interval > 4) e_menu_item_toggle_set(mi, 1); - e_menu_item_callback_set(mi, _cpufreq_menu_medium, NULL); + mi = e_menu_item_new(mo); + e_menu_item_label_set(mi, _("Medium (8 ticks)")); + e_menu_item_radio_set(mi, 1); + e_menu_item_radio_group_set(mi, 1); + if (cpufreq_config->poll_interval > 4) e_menu_item_toggle_set(mi, 1); + e_menu_item_callback_set(mi, _cpufreq_menu_medium, NULL); - mi = e_menu_item_new(mo); - e_menu_item_label_set(mi, _("Normal (32 ticks)")); - e_menu_item_radio_set(mi, 1); - e_menu_item_radio_group_set(mi, 1); - if (cpufreq_config->poll_interval >= 32) e_menu_item_toggle_set(mi, 1); - e_menu_item_callback_set(mi, _cpufreq_menu_normal, NULL); + mi = e_menu_item_new(mo); + e_menu_item_label_set(mi, _("Normal (32 ticks)")); + e_menu_item_radio_set(mi, 1); + e_menu_item_radio_group_set(mi, 1); + if (cpufreq_config->poll_interval >= 32) e_menu_item_toggle_set(mi, 1); + e_menu_item_callback_set(mi, _cpufreq_menu_normal, NULL); - mi = e_menu_item_new(mo); - e_menu_item_label_set(mi, _("Slow (64 ticks)")); - e_menu_item_radio_set(mi, 1); - e_menu_item_radio_group_set(mi, 1); - if (cpufreq_config->poll_interval >= 64) e_menu_item_toggle_set(mi, 1); - e_menu_item_callback_set(mi, _cpufreq_menu_slow, NULL); + mi = e_menu_item_new(mo); + e_menu_item_label_set(mi, _("Slow (64 ticks)")); + e_menu_item_radio_set(mi, 1); + e_menu_item_radio_group_set(mi, 1); + if (cpufreq_config->poll_interval >= 64) e_menu_item_toggle_set(mi, 1); + e_menu_item_callback_set(mi, _cpufreq_menu_slow, NULL); - mi = e_menu_item_new(mo); - e_menu_item_label_set(mi, _("Very Slow (256 ticks)")); - e_menu_item_radio_set(mi, 1); - e_menu_item_radio_group_set(mi, 1); - if (cpufreq_config->poll_interval >= 128) + mi = e_menu_item_new(mo); + e_menu_item_label_set(mi, _("Very Slow (256 ticks)")); + e_menu_item_radio_set(mi, 1); + e_menu_item_radio_group_set(mi, 1); + if (cpufreq_config->poll_interval >= 128) e_menu_item_toggle_set(mi, 1); - e_menu_item_callback_set(mi, _cpufreq_menu_very_slow, NULL); + e_menu_item_callback_set(mi, _cpufreq_menu_very_slow, NULL); - if (cpufreq_config->status->governors) - { - mo = e_menu_new(); - cpufreq_config->menu_governor = mo; + if (cpufreq_config->status->governors) + { + mo = e_menu_new(); + cpufreq_config->menu_governor = mo; - for (l = cpufreq_config->status->governors; l; l = l->next) - { - mi = e_menu_item_new(mo); - if (!strcmp(l->data, "userspace")) - e_menu_item_label_set(mi, _("Manual")); - else if (!strcmp(l->data, "ondemand")) - e_menu_item_label_set(mi, _("Automatic")); - else if (!strcmp(l->data, "conservative")) - e_menu_item_label_set(mi, _("Lower Power Automatic")); - else if (!strcmp(l->data, "powersave")) - e_menu_item_label_set(mi, _("Minimum Speed")); - else if (!strcmp(l->data, "performance")) - e_menu_item_label_set(mi, _("Maximum Speed")); - e_menu_item_radio_set(mi, 1); - e_menu_item_radio_group_set(mi, 1); - if (!strcmp(cpufreq_config->status->cur_governor, l->data)) - e_menu_item_toggle_set(mi, 1); - e_menu_item_callback_set(mi, _cpufreq_menu_governor, l->data); - } + for (l = cpufreq_config->status->governors; l; l = l->next) + { + mi = e_menu_item_new(mo); + if (!strcmp(l->data, "userspace")) + e_menu_item_label_set(mi, _("Manual")); + else if (!strcmp(l->data, "ondemand")) + e_menu_item_label_set(mi, _("Automatic")); + else if (!strcmp(l->data, "conservative")) + e_menu_item_label_set(mi, _("Lower Power Automatic")); + else if (!strcmp(l->data, "powersave")) + e_menu_item_label_set(mi, _("Minimum Speed")); + else if (!strcmp(l->data, "performance")) + e_menu_item_label_set(mi, _("Maximum Speed")); + e_menu_item_radio_set(mi, 1); + e_menu_item_radio_group_set(mi, 1); + if (!strcmp(cpufreq_config->status->cur_governor, l->data)) + e_menu_item_toggle_set(mi, 1); + e_menu_item_callback_set(mi, _cpufreq_menu_governor, l->data); + } - e_menu_item_separator_set(e_menu_item_new(mo), 1); + e_menu_item_separator_set(e_menu_item_new(mo), 1); - mi = e_menu_item_new(mo); - e_menu_item_label_set(mi, _("Restore CPU Power Policy")); - e_menu_item_check_set(mi, 1); - e_menu_item_toggle_set(mi, cpufreq_config->restore_governor); - e_menu_item_callback_set(mi, _cpufreq_menu_restore_governor, NULL); + mi = e_menu_item_new(mo); + e_menu_item_label_set(mi, _("Restore CPU Power Policy")); + e_menu_item_check_set(mi, 1); + e_menu_item_toggle_set(mi, cpufreq_config->restore_governor); + e_menu_item_callback_set(mi, _cpufreq_menu_restore_governor, NULL); - mo = e_menu_new(); - cpufreq_config->menu_powersave = mo; + mo = e_menu_new(); + cpufreq_config->menu_powersave = mo; - for (l = cpufreq_config->status->governors; l; l = l->next) - { - if (!strcmp(l->data, "userspace")) - continue; + for (l = cpufreq_config->status->governors; l; l = l->next) + { + if (!strcmp(l->data, "userspace")) + continue; - mi = e_menu_item_new(mo); - if (!strcmp(l->data, "ondemand")) - e_menu_item_label_set(mi, _("Automatic")); - else if (!strcmp(l->data, "conservative")) - e_menu_item_label_set(mi, _("Lower Power Automatic")); - else if (!strcmp(l->data, "powersave")) - e_menu_item_label_set(mi, _("Minimum Speed")); - else if (!strcmp(l->data, "performance")) - e_menu_item_label_set(mi, _("Maximum Speed")); + mi = e_menu_item_new(mo); + if (!strcmp(l->data, "ondemand")) + e_menu_item_label_set(mi, _("Automatic")); + else if (!strcmp(l->data, "conservative")) + e_menu_item_label_set(mi, _("Lower Power Automatic")); + else if (!strcmp(l->data, "powersave")) + e_menu_item_label_set(mi, _("Minimum Speed")); + else if (!strcmp(l->data, "performance")) + e_menu_item_label_set(mi, _("Maximum Speed")); - e_menu_item_radio_set(mi, 1); - e_menu_item_radio_group_set(mi, 1); - if (cpufreq_config->powersave_governor - && !strcmp(cpufreq_config->powersave_governor, l->data)) - e_menu_item_toggle_set(mi, 1); - e_menu_item_callback_set(mi, _cpufreq_menu_powersave_governor, l->data); - } + e_menu_item_radio_set(mi, 1); + e_menu_item_radio_group_set(mi, 1); + if (cpufreq_config->powersave_governor + && !strcmp(cpufreq_config->powersave_governor, l->data)) + e_menu_item_toggle_set(mi, 1); + e_menu_item_callback_set(mi, _cpufreq_menu_powersave_governor, l->data); + } - e_menu_item_separator_set(e_menu_item_new(mo), 1); + e_menu_item_separator_set(e_menu_item_new(mo), 1); - mi = e_menu_item_new(mo); - e_menu_item_label_set(mi, _("Automatic powersaving")); - e_menu_item_check_set(mi, 1); - e_menu_item_toggle_set(mi, cpufreq_config->auto_powersave); - e_menu_item_callback_set(mi, _cpufreq_menu_auto_powersave, NULL); - } + mi = e_menu_item_new(mo); + e_menu_item_label_set(mi, _("Automatic powersaving")); + e_menu_item_check_set(mi, 1); + e_menu_item_toggle_set(mi, cpufreq_config->auto_powersave); + e_menu_item_callback_set(mi, _cpufreq_menu_auto_powersave, NULL); + } - if ((cpufreq_config->status->frequencies) && - (cpufreq_config->status->can_set_frequency)) - { - mo = e_menu_new(); - cpufreq_config->menu_frequency = mo; + if ((cpufreq_config->status->frequencies) && + (cpufreq_config->status->can_set_frequency)) + { + mo = e_menu_new(); + cpufreq_config->menu_frequency = mo; - for (l = cpufreq_config->status->frequencies; l; l = l->next) - { - int frequency; + for (l = cpufreq_config->status->frequencies; l; l = l->next) + { + int frequency; - frequency = (long)l->data; - mi = e_menu_item_new(mo); - if (frequency < 1000000) - snprintf(buf, sizeof(buf), _("%i MHz"), frequency / 1000); - else - snprintf(buf, sizeof(buf), _("%'.1f GHz"), - frequency / 1000000.); - buf[sizeof(buf) - 1] = 0; - e_menu_item_label_set(mi, buf); - e_menu_item_radio_set(mi, 1); - e_menu_item_radio_group_set(mi, 1); - if (cpufreq_config->status->cur_frequency == frequency) - e_menu_item_toggle_set(mi, 1); - e_menu_item_callback_set(mi, _cpufreq_menu_frequency, l->data); - } - } + frequency = (long)l->data; + mi = e_menu_item_new(mo); + if (frequency < 1000000) + snprintf(buf, sizeof(buf), _("%i MHz"), frequency / 1000); + else + snprintf(buf, sizeof(buf), _("%'.1f GHz"), + frequency / 1000000.); + buf[sizeof(buf) - 1] = 0; + e_menu_item_label_set(mi, buf); + e_menu_item_radio_set(mi, 1); + e_menu_item_radio_group_set(mi, 1); + if (cpufreq_config->status->cur_frequency == frequency) + e_menu_item_toggle_set(mi, 1); + e_menu_item_callback_set(mi, _cpufreq_menu_frequency, l->data); + } + } - mg = e_menu_new(); - mi = e_menu_item_new(mg); - e_menu_item_label_set(mi, _("Time Between Updates")); - e_menu_item_submenu_set(mi, cpufreq_config->menu_poll); + mg = e_menu_new(); + mi = e_menu_item_new(mg); + e_menu_item_label_set(mi, _("Time Between Updates")); + e_menu_item_submenu_set(mi, cpufreq_config->menu_poll); - if (cpufreq_config->menu_governor) - { - mi = e_menu_item_new(mg); - e_menu_item_label_set(mi, _("Set CPU Power Policy")); - e_menu_item_submenu_set(mi, cpufreq_config->menu_governor); - } + if (cpufreq_config->menu_governor) + { + mi = e_menu_item_new(mg); + e_menu_item_label_set(mi, _("Set CPU Power Policy")); + e_menu_item_submenu_set(mi, cpufreq_config->menu_governor); + } - if (cpufreq_config->menu_frequency) - { - mi = e_menu_item_new(mg); - e_menu_item_label_set(mi, _("Set CPU Speed")); - e_menu_item_submenu_set(mi, cpufreq_config->menu_frequency); - } - if (cpufreq_config->menu_powersave) - { - mi = e_menu_item_new(mg); - e_menu_item_label_set(mi, _("Powersaving behavior")); - e_menu_item_submenu_set(mi, cpufreq_config->menu_powersave); - } + if (cpufreq_config->menu_frequency) + { + mi = e_menu_item_new(mg); + e_menu_item_label_set(mi, _("Set CPU Speed")); + e_menu_item_submenu_set(mi, cpufreq_config->menu_frequency); + } + if (cpufreq_config->menu_powersave) + { + mi = e_menu_item_new(mg); + e_menu_item_label_set(mi, _("Powersaving behavior")); + e_menu_item_submenu_set(mi, cpufreq_config->menu_powersave); + } - e_gadcon_canvas_zone_geometry_get(inst->gcc->gadcon, - &cx, &cy, NULL, NULL); - cpufreq_config->menu = mg; - e_menu_post_deactivate_callback_set(mg, _menu_cb_post, inst); + e_gadcon_canvas_zone_geometry_get(inst->gcc->gadcon, + &cx, &cy, NULL, NULL); + cpufreq_config->menu = mg; + e_menu_post_deactivate_callback_set(mg, _menu_cb_post, inst); - e_gadcon_locked_set(inst->gcc->gadcon, 1); + e_gadcon_locked_set(inst->gcc->gadcon, 1); - e_menu_activate_mouse(mg, - e_util_zone_current_get(e_manager_current_get()), - cx + ev->output.x, cy + ev->output.y, 1, 1, - E_MENU_POP_DIRECTION_AUTO, ev->timestamp); + e_menu_activate_mouse(mg, + e_util_zone_current_get(e_manager_current_get()), + cx + ev->output.x, cy + ev->output.y, 1, 1, + E_MENU_POP_DIRECTION_AUTO, ev->timestamp); } else if (ev->button == 3) { - E_Menu *m; - int cx, cy; + E_Menu *m; + int cx, cy; - m = e_menu_new(); - m = e_gadcon_client_util_menu_items_append(inst->gcc, m, 0); - cpufreq_config->menu = m; - e_menu_post_deactivate_callback_set(m, _menu_cb_post, NULL); + m = e_menu_new(); + m = e_gadcon_client_util_menu_items_append(inst->gcc, m, 0); + cpufreq_config->menu = m; + e_menu_post_deactivate_callback_set(m, _menu_cb_post, NULL); - e_gadcon_canvas_zone_geometry_get(inst->gcc->gadcon, - &cx, &cy, NULL, NULL); + e_gadcon_canvas_zone_geometry_get(inst->gcc->gadcon, + &cx, &cy, NULL, NULL); - e_menu_activate_mouse(m, - e_util_zone_current_get(e_manager_current_get()), - cx + ev->output.x, cy + ev->output.y, 1, 1, - E_MENU_POP_DIRECTION_AUTO, ev->timestamp); + e_menu_activate_mouse(m, + e_util_zone_current_get(e_manager_current_get()), + cx + ev->output.x, cy + ev->output.y, 1, 1, + E_MENU_POP_DIRECTION_AUTO, ev->timestamp); } } @@ -397,24 +397,24 @@ _cpufreq_set_governor(const char *governor) int ret; snprintf(buf, sizeof(buf), - "%s %s %s", cpufreq_config->set_exe_path, "governor", governor); + "%s %s %s", cpufreq_config->set_exe_path, "governor", governor); ret = system(buf); if (ret != 0) { - E_Dialog *dia; + E_Dialog *dia; E_Container *con; con = e_container_current_get(e_manager_current_get()); - if (!(dia = e_dialog_new(con, "E", "_e_mod_cpufreq_error_setfreq"))) + if (!(dia = e_dialog_new(con, "E", "_e_mod_cpufreq_error_setfreq"))) return; - e_dialog_title_set(dia, "Enlightenment Cpufreq Module"); - e_dialog_icon_set(dia, "enlightenment", 64); - e_dialog_text_set(dia, _("There was an error trying to set the
" - "cpu frequency governor via the module's
" - "setfreq utility.")); - e_dialog_button_add(dia, _("OK"), NULL, NULL, NULL); - e_win_centered_set(dia->win, 1); - e_dialog_show(dia); + e_dialog_title_set(dia, "Enlightenment Cpufreq Module"); + e_dialog_icon_set(dia, "enlightenment", 64); + e_dialog_text_set(dia, _("There was an error trying to set the
" + "cpu frequency governor via the module's
" + "setfreq utility.")); + e_dialog_button_add(dia, _("OK"), NULL, NULL, NULL); + e_win_centered_set(dia->win, 1); + e_dialog_show(dia); } } @@ -429,46 +429,46 @@ _cpufreq_set_frequency(int frequency) #endif if (!cpufreq_config->status->can_set_frequency) { - E_Dialog *dia; + E_Dialog *dia; E_Container *con; con = e_container_current_get(e_manager_current_get()); - if (!(dia = e_dialog_new(con, "E", "_e_mod_cpufreq_error_setfreq"))) + if (!(dia = e_dialog_new(con, "E", "_e_mod_cpufreq_error_setfreq"))) return; - e_dialog_title_set(dia, "Enlightenment Cpufreq Module"); - e_dialog_icon_set(dia, "enlightenment", 64); - e_dialog_text_set(dia, _("Your kernel does not support setting the
" - "CPU frequency at all. You may be missing
" - "Kernel modules or features, or your CPU
" - "simply does not support this feature.")); - e_dialog_button_add(dia, _("OK"), NULL, NULL, NULL); - e_win_centered_set(dia->win, 1); - e_dialog_show(dia); - return; + e_dialog_title_set(dia, "Enlightenment Cpufreq Module"); + e_dialog_icon_set(dia, "enlightenment", 64); + e_dialog_text_set(dia, _("Your kernel does not support setting the
" + "CPU frequency at all. You may be missing
" + "Kernel modules or features, or your CPU
" + "simply does not support this feature.")); + e_dialog_button_add(dia, _("OK"), NULL, NULL, NULL); + e_win_centered_set(dia->win, 1); + e_dialog_show(dia); + return; } // change it to "userspace" _cpufreq_set_governor("userspace"); snprintf(buf, sizeof(buf), - "%s %s %i", cpufreq_config->set_exe_path, "frequency", frequency); + "%s %s %i", cpufreq_config->set_exe_path, "frequency", frequency); ret = system(buf); if (ret != 0) { - E_Dialog *dia; + E_Dialog *dia; E_Container *con; con = e_container_current_get(e_manager_current_get()); - if (!(dia = e_dialog_new(con, "E", "_e_mod_cpufreq_error_setfreq"))) + if (!(dia = e_dialog_new(con, "E", "_e_mod_cpufreq_error_setfreq"))) return; - e_dialog_title_set(dia, "Enlightenment Cpufreq Module"); - e_dialog_icon_set(dia, "enlightenment", 64); - e_dialog_text_set(dia, _("There was an error trying to set the
" - "cpu frequency setting via the module's
" - "setfreq utility.")); - e_dialog_button_add(dia, _("OK"), NULL, NULL, NULL); - e_win_centered_set(dia->win, 1); - e_dialog_show(dia); + e_dialog_title_set(dia, "Enlightenment Cpufreq Module"); + e_dialog_icon_set(dia, "enlightenment", 64); + e_dialog_text_set(dia, _("There was an error trying to set the
" + "cpu frequency setting via the module's
" + "setfreq utility.")); + e_dialog_button_add(dia, _("OK"), NULL, NULL, NULL); + e_win_centered_set(dia->win, 1); + e_dialog_show(dia); } } @@ -483,22 +483,22 @@ _cpufreq_cb_check(void *data __UNUSED__) active = cpufreq_config->status->active; if (_cpufreq_status_check_current(cpufreq_config->status)) { - for (l = cpufreq_config->instances; l; l = l->next) - { - inst = l->data; - _cpufreq_face_update_current(inst); - } + for (l = cpufreq_config->instances; l; l = l->next) + { + inst = l->data; + _cpufreq_face_update_current(inst); + } } if (active != cpufreq_config->status->active) { - for (l = cpufreq_config->instances; l; l = l->next) - { - inst = l->data; - if (cpufreq_config->status->active == 0) - edje_object_signal_emit(inst->o_cpu, "e,state,disabled", "e"); - else if (cpufreq_config->status->active == 1) - edje_object_signal_emit(inst->o_cpu, "e,state,enabled", "e"); - } + for (l = cpufreq_config->instances; l; l = l->next) + { + inst = l->data; + if (cpufreq_config->status->active == 0) + edje_object_signal_emit(inst->o_cpu, "e,state,disabled", "e"); + else if (cpufreq_config->status->active == 1) + edje_object_signal_emit(inst->o_cpu, "e,state,enabled", "e"); + } } return ECORE_CALLBACK_RENEW; @@ -523,8 +523,9 @@ _cpufreq_status_free(Status *s) if (s->frequencies) eina_list_free(s->frequencies); if (s->governors) { - for (l = s->governors; l; l = l->next) free(l->data); - eina_list_free(s->governors); + for (l = s->governors; l; l = l->next) + free(l->data); + eina_list_free(s->governors); } if (s->cur_governor) free(s->cur_governor); if (s->orig_governor) eina_stringshare_del(s->orig_governor); @@ -539,7 +540,8 @@ _cpufreq_cb_sort(const void *item1, const void *item2) a = (long)item1; b = (long)item2; if (a < b) return -1; - else if (a > b) return 1; + else if (a > b) + return 1; return 0; } @@ -548,7 +550,7 @@ _cpufreq_status_check_available(Status *s) { char buf[PATH_MAX]; Eina_List *l; - // FIXME: this sssumes all cores accept the same freqs/ might be wrong + // FIXME: this sssumes all cores accept the same freqs/ might be wrong #ifdef __FreeBSD__ int freq, i; size_t len = 0; @@ -558,37 +560,38 @@ _cpufreq_status_check_available(Status *s) len = sizeof(buf); if (sysctlbyname("dev.cpu.0.freq_levels", buf, &len, NULL, 0) == 0) { - /* sysctl returns 0 on success */ - if (s->frequencies) - { - eina_list_free(s->frequencies); - s->frequencies = NULL; - } + /* sysctl returns 0 on success */ + if (s->frequencies) + { + eina_list_free(s->frequencies); + s->frequencies = NULL; + } - /* parse freqs and store the frequencies in s->frequencies */ - pos = buf; - while (pos) - { - q = strchr(pos, '/'); - if (!q) break; + /* parse freqs and store the frequencies in s->frequencies */ + pos = buf; + while (pos) + { + q = strchr(pos, '/'); + if (!q) break; - *q = '\0'; - freq = atoi(pos); - freq *= 1000; - s->frequencies = eina_list_append(s->frequencies, (void *)freq); + *q = '\0'; + freq = atoi(pos); + freq *= 1000; + s->frequencies = eina_list_append(s->frequencies, (void *)freq); - pos = q + 1; - pos = strchr(pos, ' '); - } + pos = q + 1; + pos = strchr(pos, ' '); + } } /* sort is not necessary because freq_levels is already sorted */ /* freebsd doesn't have governors */ if (s->governors) { - for (l = s->governors; l; l = l->next) free(l->data); - eina_list_free(s->governors); - s->governors = NULL; + for (l = s->governors; l; l = l->next) + free(l->data); + eina_list_free(s->governors); + s->governors = NULL; } #else FILE *f; @@ -596,63 +599,64 @@ _cpufreq_status_check_available(Status *s) f = fopen("/sys/devices/system/cpu/cpu0/cpufreq/scaling_available_frequencies", "r"); if (f) { - char *freq; + char *freq; - if (s->frequencies) - { - eina_list_free(s->frequencies); - s->frequencies = NULL; - } + if (s->frequencies) + { + eina_list_free(s->frequencies); + s->frequencies = NULL; + } - if (fgets(buf, sizeof(buf), f) == NULL) return 1; - buf[sizeof(buf) - 1] = 0; - fclose(f); + if (fgets(buf, sizeof(buf), f) == NULL) return 1; + buf[sizeof(buf) - 1] = 0; + fclose(f); - freq = strtok(buf, " "); - do - { - if (atoi(freq) != 0) - { - s->frequencies = eina_list_append(s->frequencies, - (void *) (long)atoi(freq)); - } - freq = strtok(NULL, " "); - } - while (freq); + freq = strtok(buf, " "); + do + { + if (atoi(freq) != 0) + { + s->frequencies = eina_list_append(s->frequencies, + (void *)(long)atoi(freq)); + } + freq = strtok(NULL, " "); + } + while (freq); - s->frequencies = eina_list_sort(s->frequencies, - eina_list_count(s->frequencies), - _cpufreq_cb_sort); + s->frequencies = eina_list_sort(s->frequencies, + eina_list_count(s->frequencies), + _cpufreq_cb_sort); } f = fopen("/sys/devices/system/cpu/cpu0/cpufreq/scaling_available_governors", "r"); if (f) { - char *gov; + char *gov; - if (s->governors) - { - for (l = s->governors; l; l = l->next) - free(l->data); - eina_list_free(s->governors); - s->governors = NULL; - } + if (s->governors) + { + for (l = s->governors; l; l = l->next) + free(l->data); + eina_list_free(s->governors); + s->governors = NULL; + } - if (fgets(buf, sizeof(buf), f) == NULL) return 1; - buf[sizeof(buf) - 1] = 0; - fclose(f); + if (fgets(buf, sizeof(buf), f) == NULL) return 1; + buf[sizeof(buf) - 1] = 0; + fclose(f); - gov = strtok(buf, " "); - do - { - while ((*gov) && (isspace(*gov))) gov++; - if (strlen(gov) != 0) - s->governors = eina_list_append(s->governors, strdup(gov)); - gov = strtok(NULL, " "); - } - while (gov); + gov = strtok(buf, " "); + do + { + while ((*gov) && (isspace(*gov))) + gov++; + if (strlen(gov) != 0) + s->governors = eina_list_append(s->governors, strdup(gov)); + gov = strtok(NULL, " "); + } + while (gov); - s->governors = + s->governors = eina_list_sort(s->governors, eina_list_count(s->governors), (int (*)(const void *, const void *))strcmp); } @@ -678,10 +682,10 @@ _cpufreq_status_check_current(Status *s) /* frequency is stored in dev.cpu.0.freq */ if (sysctlbyname("dev.cpu.0.freq", &frequency, &len, NULL, 0) == 0) { - frequency *= 1000; - if (frequency != s->cur_frequency) ret = 1; - s->cur_frequency = frequency; - s->active = 1; + frequency *= 1000; + if (frequency != s->cur_frequency) ret = 1; + s->cur_frequency = frequency; + s->active = 1; } /* hardcoded for testing */ @@ -693,45 +697,45 @@ _cpufreq_status_check_current(Status *s) _cpufreq_status_check_available(s); // average out frequencies of all cores for (i = 0; i < 64; i++) - { - snprintf(buf, sizeof(buf), "/sys/devices/system/cpu/cpu%i/cpufreq/scaling_cur_freq", i); - f = fopen(buf, "r"); - if (f) - { - if (fgets(buf, sizeof(buf), f) == NULL) continue; - buf[sizeof(buf) - 1] = 0; - fclose(f); + { + snprintf(buf, sizeof(buf), "/sys/devices/system/cpu/cpu%i/cpufreq/scaling_cur_freq", i); + f = fopen(buf, "r"); + if (f) + { + if (fgets(buf, sizeof(buf), f) == NULL) continue; + buf[sizeof(buf) - 1] = 0; + fclose(f); - frequency = atoi(buf); - if (frequency > frequency_max) frequency_max = frequency; - if (frequency < frequency_min) frequency_min = frequency; - freqtot += frequency; - s->active = 1; - } - else - break; - } - if (i < 1) i = 1; - frequency = freqtot / i; - if (frequency != s->cur_frequency) ret = 1; - if (frequency_min != s->cur_min_frequency) ret = 1; - if (frequency_max != s->cur_max_frequency) ret = 1; - s->cur_frequency = frequency; - s->cur_min_frequency = frequency_min; - s->cur_max_frequency = frequency_max; + frequency = atoi(buf); + if (frequency > frequency_max) frequency_max = frequency; + if (frequency < frequency_min) frequency_min = frequency; + freqtot += frequency; + s->active = 1; + } + else + break; + } + if (i < 1) i = 1; + frequency = freqtot / i; + if (frequency != s->cur_frequency) ret = 1; + if (frequency_min != s->cur_min_frequency) ret = 1; + if (frequency_max != s->cur_max_frequency) ret = 1; + s->cur_frequency = frequency; + s->cur_min_frequency = frequency_min; + s->cur_max_frequency = frequency_max; // printf("%i | %i %i\n", frequency, frequency_min, frequency_max); - // FIXME: this sssumes all cores are on the same governor + // FIXME: this sssumes all cores are on the same governor f = fopen("/sys/devices/system/cpu/cpu0/cpufreq/scaling_setspeed", "r"); if (f) { - s->can_set_frequency = 1; - fclose(f); + s->can_set_frequency = 1; + fclose(f); } else { - s->can_set_frequency = 0; + s->can_set_frequency = 0; } f = fopen("/sys/devices/system/cpu/cpu0/cpufreq/scaling_governor", "r"); @@ -740,27 +744,27 @@ _cpufreq_status_check_current(Status *s) char *p; buf[0] = 0; - if (fgets(buf, sizeof(buf), f) == NULL) return ret;; - buf[sizeof(buf) - 1] = 0; - fclose(f); - for (p = buf; (*p != 0) && (isalnum(*p)); p++); + if (fgets(buf, sizeof(buf), f) == NULL) return ret; ; + buf[sizeof(buf) - 1] = 0; + fclose(f); + for (p = buf; (*p != 0) && (isalnum(*p)); p++) ; *p = 0; - if ((!s->cur_governor) || (strcmp(buf, s->cur_governor))) - { - ret = 1; + if ((!s->cur_governor) || (strcmp(buf, s->cur_governor))) + { + ret = 1; - if (s->cur_governor) free(s->cur_governor); - s->cur_governor = strdup(buf); + if (s->cur_governor) free(s->cur_governor); + s->cur_governor = strdup(buf); - for (i = strlen(s->cur_governor) - 1; i >= 0; i--) - { - if (isspace(s->cur_governor[i])) - s->cur_governor[i] = 0; - else - break; - } - } + for (i = strlen(s->cur_governor) - 1; i >= 0; i--) + { + if (isspace(s->cur_governor[i])) + s->cur_governor[i] = 0; + else + break; + } + } } #endif return ret; @@ -808,16 +812,16 @@ _cpufreq_face_update_current(Instance *inst) frequency_msg->val[3] = cpufreq_config->status->cur_max_frequency; frequency_msg->val[4] = 0; // pad edje_object_message_send(inst->o_cpu, EDJE_MESSAGE_INT_SET, 3, - frequency_msg); + frequency_msg); free(frequency_msg); /* BSD crashes here without the if-condition * since it has no governors (yet) */ if (cpufreq_config->status->cur_governor) { - governor_msg.str = cpufreq_config->status->cur_governor; - edje_object_message_send(inst->o_cpu, EDJE_MESSAGE_STRING, 4, - &governor_msg); + governor_msg.str = cpufreq_config->status->cur_governor; + edje_object_message_send(inst->o_cpu, EDJE_MESSAGE_STRING, 4, + &governor_msg); } } @@ -829,21 +833,21 @@ _cpufreq_face_cb_set_frequency(void *data __UNUSED__, Evas_Object *obj __UNUSED_ for (l = cpufreq_config->status->frequencies; l; l = l->next) { - if (cpufreq_config->status->cur_frequency == (long)l->data) - { - if (!strcmp(emission, "e,action,frequency,increase")) - { - if (l->next) next_frequency = (long)l->next->data; - break; - } - else if (!strcmp(emission, "e,action,frequency,decrease")) - { - if (l->prev) next_frequency = (long)l->prev->data; - break; - } - else - break; - } + if (cpufreq_config->status->cur_frequency == (long)l->data) + { + if (!strcmp(emission, "e,action,frequency,increase")) + { + if (l->next) next_frequency = (long)l->next->data; + break; + } + else if (!strcmp(emission, "e,action,frequency,decrease")) + { + if (l->prev) next_frequency = (long)l->prev->data; + break; + } + else + break; + } } if (next_frequency != 0) _cpufreq_set_frequency(next_frequency); } @@ -856,14 +860,14 @@ _cpufreq_face_cb_set_governor(void *data __UNUSED__, Evas_Object *obj __UNUSED__ for (l = cpufreq_config->status->governors; l; l = l->next) { - if (!strcmp(l->data, cpufreq_config->status->cur_governor)) - { - if (l->next) - next_governor = l->next->data; - else - next_governor = cpufreq_config->status->governors->data; - break; - } + if (!strcmp(l->data, cpufreq_config->status->cur_governor)) + { + if (l->next) + next_governor = l->next->data; + else + next_governor = cpufreq_config->status->governors->data; + break; + } } if (next_governor) _cpufreq_set_governor(next_governor); } @@ -885,34 +889,36 @@ _cpufreq_event_cb_powersave(void *data __UNUSED__, int type, void *event) for (l = cpufreq_config->status->governors; l; l = l->next) { - if (!strcmp(l->data, "conservative")) - has_conservative = EINA_TRUE; - else if (!strcmp(l->data, "powersave")) - has_powersave = EINA_TRUE; + if (!strcmp(l->data, "conservative")) + has_conservative = EINA_TRUE; + else if (!strcmp(l->data, "powersave")) + has_powersave = EINA_TRUE; } - switch(ev->mode) + switch (ev->mode) { case E_POWERSAVE_MODE_NONE: case E_POWERSAVE_MODE_LOW: - _cpufreq_set_governor(cpufreq_config->status->orig_governor); - eina_stringshare_del(cpufreq_config->status->orig_governor); - cpufreq_config->status->orig_governor = NULL; - break; + _cpufreq_set_governor(cpufreq_config->status->orig_governor); + eina_stringshare_del(cpufreq_config->status->orig_governor); + cpufreq_config->status->orig_governor = NULL; + break; + case E_POWERSAVE_MODE_MEDIUM: case E_POWERSAVE_MODE_HIGH: - if ((cpufreq_config->powersave_governor) || (has_conservative)) - { - if (cpufreq_config->powersave_governor) - _cpufreq_set_governor(cpufreq_config->powersave_governor); - else - _cpufreq_set_governor("conservative"); - break; - } + if ((cpufreq_config->powersave_governor) || (has_conservative)) + { + if (cpufreq_config->powersave_governor) + _cpufreq_set_governor(cpufreq_config->powersave_governor); + else + _cpufreq_set_governor("conservative"); + break; + } + case E_POWERSAVE_MODE_EXTREME: - if (has_powersave) - _cpufreq_set_governor("powersave"); - break; + if (has_powersave) + _cpufreq_set_governor("powersave"); + break; } return ECORE_CALLBACK_PASS_ON; @@ -981,7 +987,7 @@ _cpufreq_menu_governor(void *data, E_Menu *m __UNUSED__, E_Menu_Item *mi __UNUSE governor = data; if (governor) { - _cpufreq_set_governor(governor); + _cpufreq_set_governor(governor); eina_stringshare_replace(&cpufreq_config->governor, governor); } e_config_save_queue(); @@ -999,7 +1005,7 @@ _cpufreq_menu_powersave_governor(void *data, E_Menu *m __UNUSED__, E_Menu_Item * } static void -_cpufreq_menu_frequency(void * data, E_Menu *m __UNUSED__, E_Menu_Item *mi __UNUSED__) +_cpufreq_menu_frequency(void *data, E_Menu *m __UNUSED__, E_Menu_Item *mi __UNUSED__) { int frequency; @@ -1014,7 +1020,7 @@ _cpufreq_poll_interval_update(void) ecore_poller_del(cpufreq_config->frequency_check_poller); cpufreq_config->frequency_check_poller = ecore_poller_add(ECORE_POLLER_CORE, cpufreq_config->poll_interval, - _cpufreq_cb_check, NULL); + _cpufreq_cb_check, NULL); e_config_save_queue(); } @@ -1049,13 +1055,13 @@ e_modapi_init(E_Module *m) if (!cpufreq_config) { - cpufreq_config = E_NEW(Config, 1); - cpufreq_config->config_version = CPUFREQ_CONFIG_VERSION; - cpufreq_config->poll_interval = 32; - cpufreq_config->restore_governor = 0; - cpufreq_config->auto_powersave = 1; - cpufreq_config->powersave_governor = NULL; - cpufreq_config->governor = NULL; + cpufreq_config = E_NEW(Config, 1); + cpufreq_config->config_version = CPUFREQ_CONFIG_VERSION; + cpufreq_config->poll_interval = 32; + cpufreq_config->restore_governor = 0; + cpufreq_config->auto_powersave = 1; + cpufreq_config->powersave_governor = NULL; + cpufreq_config->governor = NULL; } E_CONFIG_LIMIT(cpufreq_config->poll_interval, 1, 1024); @@ -1064,21 +1070,21 @@ e_modapi_init(E_Module *m) cpufreq_config->set_exe_path = strdup(buf); cpufreq_config->frequency_check_poller = ecore_poller_add(ECORE_POLLER_CORE, cpufreq_config->poll_interval, - _cpufreq_cb_check, NULL); + _cpufreq_cb_check, NULL); cpufreq_config->status = _cpufreq_status_new(); _cpufreq_status_check_available(cpufreq_config->status); if ((cpufreq_config->restore_governor) && (cpufreq_config->governor)) { - /* If the governor is available, restore it */ - for (l = cpufreq_config->status->governors; l; l = l->next) - { - if (!strcmp(l->data, cpufreq_config->governor)) - { - _cpufreq_set_governor(cpufreq_config->governor); - break; - } - } + /* If the governor is available, restore it */ + for (l = cpufreq_config->status->governors; l; l = l->next) + { + if (!strcmp(l->data, cpufreq_config->governor)) + { + _cpufreq_set_governor(cpufreq_config->governor); + break; + } + } } cpufreq_config->module = m; @@ -1096,33 +1102,33 @@ e_modapi_shutdown(E_Module *m __UNUSED__) ecore_poller_del(cpufreq_config->frequency_check_poller); if (cpufreq_config->menu) { - e_menu_post_deactivate_callback_set(cpufreq_config->menu, NULL, NULL); - e_object_del(E_OBJECT(cpufreq_config->menu)); - cpufreq_config->menu = NULL; + e_menu_post_deactivate_callback_set(cpufreq_config->menu, NULL, NULL); + e_object_del(E_OBJECT(cpufreq_config->menu)); + cpufreq_config->menu = NULL; } if (cpufreq_config->menu_poll) { - e_menu_post_deactivate_callback_set(cpufreq_config->menu_poll, NULL, NULL); - e_object_del(E_OBJECT(cpufreq_config->menu_poll)); - cpufreq_config->menu_poll = NULL; + e_menu_post_deactivate_callback_set(cpufreq_config->menu_poll, NULL, NULL); + e_object_del(E_OBJECT(cpufreq_config->menu_poll)); + cpufreq_config->menu_poll = NULL; } if (cpufreq_config->menu_governor) { - e_menu_post_deactivate_callback_set(cpufreq_config->menu_governor, NULL, NULL); - e_object_del(E_OBJECT(cpufreq_config->menu_governor)); - cpufreq_config->menu_governor = NULL; + e_menu_post_deactivate_callback_set(cpufreq_config->menu_governor, NULL, NULL); + e_object_del(E_OBJECT(cpufreq_config->menu_governor)); + cpufreq_config->menu_governor = NULL; } if (cpufreq_config->menu_frequency) { - e_menu_post_deactivate_callback_set(cpufreq_config->menu_frequency, NULL, NULL); - e_object_del(E_OBJECT(cpufreq_config->menu_frequency)); - cpufreq_config->menu_frequency = NULL; + e_menu_post_deactivate_callback_set(cpufreq_config->menu_frequency, NULL, NULL); + e_object_del(E_OBJECT(cpufreq_config->menu_frequency)); + cpufreq_config->menu_frequency = NULL; } if (cpufreq_config->menu_powersave) { - e_menu_post_deactivate_callback_set(cpufreq_config->menu_powersave, NULL, NULL); - e_object_del(E_OBJECT(cpufreq_config->menu_powersave)); - cpufreq_config->menu_powersave = NULL; + e_menu_post_deactivate_callback_set(cpufreq_config->menu_powersave, NULL, NULL); + e_object_del(E_OBJECT(cpufreq_config->menu_powersave)); + cpufreq_config->menu_powersave = NULL; } if (cpufreq_config->governor) eina_stringshare_del(cpufreq_config->governor); @@ -1140,3 +1146,4 @@ e_modapi_save(E_Module *m __UNUSED__) e_config_domain_save("module.cpufreq", conf_edd, cpufreq_config); return 1; } + diff --git a/src/modules/cpufreq/freqset.c b/src/modules/cpufreq/freqset.c index ce397634e..86e71151a 100644 --- a/src/modules/cpufreq/freqset.c +++ b/src/modules/cpufreq/freqset.c @@ -21,7 +21,7 @@ main(int argc, char *argv[]) fprintf(stderr, "\tfreqset \n"); return 1; } - + if (seteuid(0)) { fprintf(stderr, "Unable to assume root privileges\n"); @@ -31,13 +31,13 @@ main(int argc, char *argv[]) if (!strcmp(argv[1], "frequency")) { int new_frequency = atoi(argv[2]); - int len = 4; + int len = 4; if (sysctlbyname("dev.cpu.0.freq", NULL, 0, &new_frequency, &len) == -1) { fprintf(stderr, "Unable to open frequency interface for writing.\n"); return 1; } - + return 0; } else if (!strcmp(argv[1], "governor")) @@ -50,20 +50,20 @@ main(int argc, char *argv[]) fprintf(stderr, "Unknown command.\n"); return 1; } -#else +#else if (!strcmp(argv[1], "frequency")) { - if (sys_cpu_setall("scaling_setspeed", argv[2]) == 0) + if (sys_cpu_setall("scaling_setspeed", argv[2]) == 0) { fprintf(stderr, "Unable to open frequency interface for writing.\n"); return 1; } - + return 0; } else if (!strcmp(argv[1], "governor")) { - if (sys_cpu_setall("scaling_governor", argv[2]) == 0) + if (sys_cpu_setall("scaling_governor", argv[2]) == 0) { fprintf(stderr, "Unable to open governor interface for writing.\n"); return 1; @@ -88,22 +88,22 @@ main(int argc, char *argv[]) static int sys_cpu_setall(const char *control, const char *value) { - int num = 0; - char filename[4096]; - FILE *f; + int num = 0; + char filename[4096]; + FILE *f; while (1) { - snprintf(filename, sizeof(filename), "/sys/devices/system/cpu/cpu%i/cpufreq/%s", num, control); - f = fopen(filename, "w"); + snprintf(filename, sizeof(filename), "/sys/devices/system/cpu/cpu%i/cpufreq/%s", num, control); + f = fopen(filename, "w"); - if (!f) - { - return(num); - } - fprintf(f, "%s\n", value); - fclose(f); - num++; + if (!f) + { + return num; + } + fprintf(f, "%s\n", value); + fclose(f); + num++; } return -1; } @@ -119,10 +119,10 @@ sys_cpufreq_set(const char *control, const char *value) if (!f) { - if (sys_cpu_setall(control, value) > 0) - return 1; - else - return -1; + if (sys_cpu_setall(control, value) > 0) + return 1; + else + return -1; } fprintf(f, "%s\n", value); @@ -130,3 +130,4 @@ sys_cpufreq_set(const char *control, const char *value) return 1; } +