From 8bdd2e999bd2466908224a2e53b0b9ef77c52c90 Mon Sep 17 00:00:00 2001 From: Marcel Holtmann Date: Fri, 24 Jul 2009 00:44:08 +0200 Subject: [PATCH] Fix notification to use service types instead of device types --- include/notifier.h | 4 ++-- include/service.h | 2 ++ plugins/iospm.c | 22 ++++++++-------------- src/notifier.c | 6 +++--- 4 files changed, 15 insertions(+), 19 deletions(-) diff --git a/include/notifier.h b/include/notifier.h index fdbc1b9..1addb55 100644 --- a/include/notifier.h +++ b/include/notifier.h @@ -22,7 +22,7 @@ #ifndef __CONNMAN_NOTIFIER_H #define __CONNMAN_NOTIFIER_H -#include +#include #ifdef __cplusplus extern "C" { @@ -41,7 +41,7 @@ extern "C" { struct connman_notifier { const char *name; int priority; - void (*device_enabled) (enum connman_device_type type, + void (*service_enabled) (enum connman_service_type type, connman_bool_t enabled); void (*offline_mode) (connman_bool_t enabled); }; diff --git a/include/service.h b/include/service.h index 6138f2c..b19a531 100644 --- a/include/service.h +++ b/include/service.h @@ -22,6 +22,8 @@ #ifndef __CONNMAN_SERVICE_H #define __CONNMAN_SERVICE_H +#include + #ifdef __cplusplus extern "C" { #endif diff --git a/plugins/iospm.c b/plugins/iospm.c index e644369..02e532e 100644 --- a/plugins/iospm.c +++ b/plugins/iospm.c @@ -61,23 +61,17 @@ static void send_indication(const char *path, connman_bool_t enabled) dbus_message_unref(message); } -static void iospm_device_enabled(enum connman_device_type type, +static void iospm_service_enabled(enum connman_service_type type, connman_bool_t enabled) { switch (type) { - case CONNMAN_DEVICE_TYPE_UNKNOWN: - case CONNMAN_DEVICE_TYPE_VENDOR: - case CONNMAN_DEVICE_TYPE_ETHERNET: - case CONNMAN_DEVICE_TYPE_WIFI: - case CONNMAN_DEVICE_TYPE_WIMAX: - case CONNMAN_DEVICE_TYPE_GPS: - case CONNMAN_DEVICE_TYPE_MBM: - case CONNMAN_DEVICE_TYPE_HSO: - case CONNMAN_DEVICE_TYPE_NOZOMI: - case CONNMAN_DEVICE_TYPE_HUAWEI: - case CONNMAN_DEVICE_TYPE_NOVATEL: + case CONNMAN_SERVICE_TYPE_UNKNOWN: + case CONNMAN_SERVICE_TYPE_ETHERNET: + case CONNMAN_SERVICE_TYPE_WIFI: + case CONNMAN_SERVICE_TYPE_WIMAX: + case CONNMAN_SERVICE_TYPE_CELLULAR: break; - case CONNMAN_DEVICE_TYPE_BLUETOOTH: + case CONNMAN_SERVICE_TYPE_BLUETOOTH: send_indication(IOSPM_BLUETOOTH, enabled); break; } @@ -91,7 +85,7 @@ static void iospm_offline_mode(connman_bool_t enabled) static struct connman_notifier iospm_notifier = { .name = "iospm", .priority = CONNMAN_NOTIFIER_PRIORITY_DEFAULT, - .device_enabled = iospm_device_enabled, + .service_enabled= iospm_service_enabled, .offline_mode = iospm_offline_mode, }; diff --git a/src/notifier.c b/src/notifier.c index cfd6750..2403e91 100644 --- a/src/notifier.c +++ b/src/notifier.c @@ -176,7 +176,7 @@ static void technology_registered(enum connman_service_type type, g_dbus_send_message(connection, signal); } -static void technology_enabled(enum connman_device_type type, +static void technology_enabled(enum connman_service_type type, connman_bool_t enabled) { GSList *list; @@ -212,8 +212,8 @@ done: for (list = notifier_list; list; list = list->next) { struct connman_notifier *notifier = list->data; - if (notifier->device_enabled) - notifier->device_enabled(type, enabled); + if (notifier->service_enabled) + notifier->service_enabled(type, enabled); } } -- 2.7.4