device: Include Reason in Disconnect Skip Warning
[framework/connectivity/connman.git] / src / device.c
index 2745893..a9f4330 100644 (file)
@@ -2,7 +2,7 @@
  *
  *  Connection Manager
  *
- *  Copyright (C) 2007-2009  Intel Corporation. All rights reserved.
+ *  Copyright (C) 2007-2010  Intel Corporation. All rights reserved.
  *
  *  This program is free software; you can redistribute it and/or modify
  *  it under the terms of the GNU General Public License version 2 as
 #include <errno.h>
 #include <string.h>
 
-#include <gdbus.h>
-
 #include "connman.h"
 
-static DBusConnection *connection = NULL;
-
 struct connman_device {
        struct connman_element element;
        enum connman_device_type type;
-       enum connman_device_mode mode;
-       connman_bool_t secondary;
        connman_bool_t offlinemode;
        connman_bool_t blocked;
        connman_bool_t powered;
        connman_bool_t powered_pending;
        connman_bool_t powered_persistent;
-       connman_bool_t carrier;
        connman_bool_t scanning;
        connman_bool_t disconnected;
+       connman_bool_t reconnect;
        connman_uint16_t scan_interval;
+       connman_uint16_t backoff_interval;
        char *name;
        char *node;
        char *address;
        char *interface;
-       char *control;
        char *ident;
+       char *path;
        int phyindex;
        unsigned int connections;
        guint scan_timeout;
-       struct connman_ipconfig *ipconfig;
 
        struct connman_device_driver *driver;
        void *driver_data;
 
-       connman_bool_t registered;
-
        char *last_network;
        struct connman_network *network;
        GHashTable *networks;
-
-       DBusMessage *pending;
-       guint timeout;
 };
 
+#define SCAN_INITIAL_DELAY 10
+
 static gboolean device_scan_trigger(gpointer user_data)
 {
        struct connman_device *device = user_data;
@@ -100,12 +91,39 @@ static void reset_scan_trigger(struct connman_device *device)
        clear_scan_trigger(device);
 
        if (device->scan_interval > 0) {
-               guint interval = device->scan_interval;
+               guint interval;
+
+               if (g_hash_table_size(device->networks) == 0) {
+                       if (device->backoff_interval >= device->scan_interval)
+                               device->backoff_interval = SCAN_INITIAL_DELAY;
+                       interval = device->backoff_interval;
+               } else
+                       interval = device->scan_interval;
+
+               DBG("interval %d", interval);
+
                device->scan_timeout = g_timeout_add_seconds(interval,
                                        device_scan_trigger, device);
+
+               device->backoff_interval *= 2;
+               if (device->backoff_interval > device->scan_interval)
+                       device->backoff_interval = device->scan_interval;
        }
 }
 
+static void force_scan_trigger(struct connman_device *device)
+{
+       clear_scan_trigger(device);
+
+       device->scan_timeout = g_timeout_add_seconds(5,
+                                       device_scan_trigger, device);
+}
+
+void connman_device_schedule_scan(struct connman_device *device)
+{
+       reset_scan_trigger(device);
+}
+
 static const char *type2description(enum connman_device_type type)
 {
        switch (type) {
@@ -122,12 +140,11 @@ static const char *type2description(enum connman_device_type type)
                return "Bluetooth";
        case CONNMAN_DEVICE_TYPE_GPS:
                return "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_DEVICE_TYPE_CELLULAR:
                return "Cellular";
+       case CONNMAN_DEVICE_TYPE_GADGET:
+               return "Gadget";
+
        }
 
        return NULL;
@@ -149,12 +166,11 @@ static const char *type2string(enum connman_device_type type)
                return "bluetooth";
        case CONNMAN_DEVICE_TYPE_GPS:
                return "gps";
-       case CONNMAN_DEVICE_TYPE_MBM:
-       case CONNMAN_DEVICE_TYPE_HSO:
-       case CONNMAN_DEVICE_TYPE_HUAWEI:
-       case CONNMAN_DEVICE_TYPE_NOZOMI:
-       case CONNMAN_DEVICE_TYPE_NOVATEL:
+       case CONNMAN_DEVICE_TYPE_CELLULAR:
                return "cellular";
+       case CONNMAN_DEVICE_TYPE_GADGET:
+               return "gadget";
+
        }
 
        return NULL;
@@ -168,9 +184,6 @@ enum connman_service_type __connman_device_get_service_type(struct connman_devic
        case CONNMAN_DEVICE_TYPE_UNKNOWN:
        case CONNMAN_DEVICE_TYPE_VENDOR:
        case CONNMAN_DEVICE_TYPE_GPS:
-       case CONNMAN_DEVICE_TYPE_NOZOMI:
-       case CONNMAN_DEVICE_TYPE_HUAWEI:
-       case CONNMAN_DEVICE_TYPE_NOVATEL:
                break;
        case CONNMAN_DEVICE_TYPE_ETHERNET:
                return CONNMAN_SERVICE_TYPE_ETHERNET;
@@ -180,528 +193,113 @@ enum connman_service_type __connman_device_get_service_type(struct connman_devic
                return CONNMAN_SERVICE_TYPE_WIMAX;
        case CONNMAN_DEVICE_TYPE_BLUETOOTH:
                return CONNMAN_SERVICE_TYPE_BLUETOOTH;
-       case CONNMAN_DEVICE_TYPE_MBM:
-       case CONNMAN_DEVICE_TYPE_HSO:
+       case CONNMAN_DEVICE_TYPE_CELLULAR:
                return CONNMAN_SERVICE_TYPE_CELLULAR;
+       case CONNMAN_DEVICE_TYPE_GADGET:
+               return CONNMAN_SERVICE_TYPE_GADGET;
+
        }
 
        return CONNMAN_SERVICE_TYPE_UNKNOWN;
 }
 
-static int set_connected(struct connman_device *device,
-                                               connman_bool_t connected)
+int __connman_device_enable(struct connman_device *device)
 {
-       if (connected == TRUE) {
-               enum connman_element_type type = CONNMAN_ELEMENT_TYPE_UNKNOWN;
-               struct connman_element *element;
+       int err;
+       enum connman_service_type type;
 
-               device->disconnected = TRUE;
+       DBG("device %p %d", device, device->blocked);
 
-               switch (device->element.ipv4.method) {
-               case CONNMAN_IPCONFIG_METHOD_UNKNOWN:
-               case CONNMAN_IPCONFIG_METHOD_IGNORE:
-                       return 0;
-               case CONNMAN_IPCONFIG_METHOD_STATIC:
-                       type = CONNMAN_ELEMENT_TYPE_IPV4;
-                       break;
-               case CONNMAN_IPCONFIG_METHOD_DHCP:
-                       type = CONNMAN_ELEMENT_TYPE_DHCP;
-                       break;
-               }
-
-               element = connman_element_create(NULL);
-               if (element != NULL) {
-                       struct connman_service *service;
+       if (!device->driver || !device->driver->enable)
+               return -EOPNOTSUPP;
 
-                       element->type  = type;
-                       element->index = device->element.index;
+       if (device->powered_pending == TRUE)
+               return -EALREADY;
 
-                       if (connman_element_register(element,
-                                                       &device->element) < 0)
-                               connman_element_unref(element);
+       if (device->blocked == TRUE)
+               return -ENOLINK;
 
-                       device->disconnected = FALSE;
+       connman_device_set_disconnected(device, FALSE);
+       device->scanning = FALSE;
 
-                       service = __connman_service_lookup_from_device(device);
-                       __connman_service_indicate_state(service,
-                                       CONNMAN_SERVICE_STATE_CONFIGURATION);
+       err = device->driver->enable(device);
+       if (err < 0 && err != -EALREADY) {
+               if (err == -EINPROGRESS) {
+                       device->powered_pending = TRUE;
+                       device->offlinemode = FALSE;
+                       if (__connman_profile_get_offlinemode() == TRUE)
+                               __connman_profile_set_offlinemode(FALSE, FALSE);
                }
-       } else {
-               struct connman_service *service;
-
-               connman_element_unregister_children(&device->element);
-
-               device->disconnected = TRUE;
-
-               service = __connman_service_lookup_from_device(device);
-               __connman_service_indicate_state(service,
-                                       CONNMAN_SERVICE_STATE_IDLE);
-       }
-
-       if (connected == TRUE) {
-               enum connman_service_type type;
-
-               type = __connman_device_get_service_type(device);
-               __connman_notifier_connect(type);
-       } else {
-               enum connman_service_type type;
-
-               type = __connman_device_get_service_type(device);
-               __connman_notifier_disconnect(type);
+               return err;
        }
 
-       return 0;
-}
-
-static int set_carrier(struct connman_device *device, connman_bool_t carrier)
-{
-       if (carrier == TRUE)
-               __connman_profile_add_device(device);
-       else
-               __connman_profile_remove_device(device);
-
-       return set_connected(device, carrier);
-}
-
-static int powered_changed(struct connman_device *device)
-{
-       DBusMessage *signal;
-       DBusMessageIter entry, value;
-       const char *key = "Powered";
-
-       signal = dbus_message_new_signal(device->element.path,
-                               CONNMAN_DEVICE_INTERFACE, "PropertyChanged");
-       if (signal == NULL)
-               return -ENOMEM;
-
-       dbus_message_iter_init_append(signal, &entry);
-
-       dbus_message_iter_append_basic(&entry, DBUS_TYPE_STRING, &key);
-
-       dbus_message_iter_open_container(&entry, DBUS_TYPE_VARIANT,
-                                       DBUS_TYPE_BOOLEAN_AS_STRING, &value);
-       dbus_message_iter_append_basic(&value, DBUS_TYPE_BOOLEAN,
-                                                       &device->powered);
-       dbus_message_iter_close_container(&entry, &value);
+       device->powered_pending = TRUE;
+       device->powered = TRUE;
+       device->offlinemode = FALSE;
+       if (__connman_profile_get_offlinemode() == TRUE)
+               __connman_profile_set_offlinemode(FALSE, FALSE);
 
-       g_dbus_send_message(connection, signal);
+       type = __connman_device_get_service_type(device);
+       __connman_technology_enable(type);
 
        return 0;
 }
 
-static int set_powered(struct connman_device *device, connman_bool_t powered)
+int __connman_device_disable(struct connman_device *device)
 {
-       struct connman_device_driver *driver = device->driver;
-       enum connman_service_type type;
        int err;
-
-       DBG("device %p powered %d", device, powered);
-
-       if (device->powered_pending == powered)
-               return -EALREADY;
-
-       device->powered_pending = powered;
-
-       if (!driver)
-               return -EINVAL;
-
-       type = __connman_device_get_service_type(device);
-
-       if (powered == TRUE) {
-               if (driver->enable) {
-                       err = driver->enable(device);
-                       if (err == 0)
-                               __connman_notifier_enable(type);
-               } else
-                       err = -EINVAL;
-       } else {
-               clear_scan_trigger(device);
-
-               g_hash_table_remove_all(device->networks);
-
-               set_carrier(device, FALSE);
-
-               if (driver->disable) {
-                       err = driver->disable(device);
-                       if (err == 0)
-                               __connman_notifier_disable(type);
-               } else
-                       err = -EINVAL;
-       }
-
-       if (err == 0) {
-               device->powered = powered;
-
-               if (device->registered == TRUE)
-                       powered_changed(device);
-       }
-
-       return err;
-}
-
-static void append_path(gpointer key, gpointer value, gpointer user_data)
-{
-       struct connman_element *element = value;
-       DBusMessageIter *iter = user_data;
-
-       dbus_message_iter_append_basic(iter, DBUS_TYPE_OBJECT_PATH,
-                                                       &element->path);
-}
-
-static void append_networks(struct connman_device *device,
-                                               DBusMessageIter *entry)
-{
-       DBusMessageIter value, iter;
-       const char *key = "Networks";
-
-       dbus_message_iter_append_basic(entry, DBUS_TYPE_STRING, &key);
-
-       dbus_message_iter_open_container(entry, DBUS_TYPE_VARIANT,
-               DBUS_TYPE_ARRAY_AS_STRING DBUS_TYPE_OBJECT_PATH_AS_STRING,
-                                                               &value);
-
-       dbus_message_iter_open_container(&value, DBUS_TYPE_ARRAY,
-                               DBUS_TYPE_OBJECT_PATH_AS_STRING, &iter);
-       g_hash_table_foreach(device->networks, append_path, &iter);
-       dbus_message_iter_close_container(&value, &iter);
-
-       dbus_message_iter_close_container(entry, &value);
-}
-
-static DBusMessage *get_properties(DBusConnection *conn,
-                                       DBusMessage *msg, void *data)
-{
-       struct connman_device *device = data;
-       DBusMessage *reply;
-       DBusMessageIter array, dict, entry;
-       const char *str;
-
-       DBG("conn %p", conn);
-
-       if (__connman_security_check_privilege(msg,
-                                       CONNMAN_SECURITY_PRIVILEGE_PUBLIC) < 0)
-               return __connman_error_permission_denied(msg);
-
-       reply = dbus_message_new_method_return(msg);
-       if (reply == NULL)
-               return NULL;
-
-       dbus_message_iter_init_append(reply, &array);
-
-       dbus_message_iter_open_container(&array, DBUS_TYPE_ARRAY,
-                       DBUS_DICT_ENTRY_BEGIN_CHAR_AS_STRING
-                       DBUS_TYPE_STRING_AS_STRING DBUS_TYPE_VARIANT_AS_STRING
-                       DBUS_DICT_ENTRY_END_CHAR_AS_STRING, &dict);
-
-       if (device->name != NULL)
-               connman_dbus_dict_append_variant(&dict, "Name",
-                                       DBUS_TYPE_STRING, &device->name);
-
-       str = type2string(device->type);
-       if (str != NULL)
-               connman_dbus_dict_append_variant(&dict, "Type",
-                                               DBUS_TYPE_STRING, &str);
-
-       if (device->address != NULL)
-               connman_dbus_dict_append_variant(&dict, "Address",
-                                       DBUS_TYPE_STRING, &device->address);
-
-       if (device->interface != NULL)
-               connman_dbus_dict_append_variant(&dict, "Interface",
-                                       DBUS_TYPE_STRING, &device->interface);
-
-       connman_dbus_dict_append_variant(&dict, "Powered",
-                                       DBUS_TYPE_BOOLEAN, &device->powered);
-
-       if (device->driver && device->driver->scan)
-               connman_dbus_dict_append_variant(&dict, "Scanning",
-                                       DBUS_TYPE_BOOLEAN, &device->scanning);
-
-       switch (device->mode) {
-       case CONNMAN_DEVICE_MODE_UNKNOWN:
-               break;
-       case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-               __connman_element_append_ipv4(&device->element, &dict);
-               break;
-       case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-       case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-               if (device->scan_interval > 0)
-                       connman_dbus_dict_append_variant(&dict, "ScanInterval",
-                               DBUS_TYPE_UINT16, &device->scan_interval);
-
-               dbus_message_iter_open_container(&dict, DBUS_TYPE_DICT_ENTRY,
-                                                               NULL, &entry);
-               append_networks(device, &entry);
-               dbus_message_iter_close_container(&dict, &entry);
-               break;
-       }
-
-       dbus_message_iter_close_container(&array, &dict);
-
-       return reply;
-}
-
-static gboolean powered_timeout(gpointer user_data)
-{
-       struct connman_device *device = user_data;
+       enum connman_service_type type;
 
        DBG("device %p", device);
 
-       device->timeout = 0;
-
-       if (device->pending != NULL) {
-               DBusMessage *reply;
-
-               reply = __connman_error_operation_timeout(device->pending);
-               if (reply != NULL)
-                       g_dbus_send_message(connection, reply);
-
-               dbus_message_unref(device->pending);
-               device->pending = NULL;
-       }
-
-       return FALSE;
-}
-
-static DBusMessage *set_property(DBusConnection *conn,
-                                       DBusMessage *msg, void *data)
-{
-       struct connman_device *device = data;
-       DBusMessageIter iter, value;
-       const char *name;
-       int type;
-
-       DBG("conn %p", conn);
-
-       if (dbus_message_iter_init(msg, &iter) == FALSE)
-               return __connman_error_invalid_arguments(msg);
-
-       dbus_message_iter_get_basic(&iter, &name);
-       dbus_message_iter_next(&iter);
-       dbus_message_iter_recurse(&iter, &value);
-
-       if (__connman_security_check_privilege(msg,
-                                       CONNMAN_SECURITY_PRIVILEGE_MODIFY) < 0)
-               return __connman_error_permission_denied(msg);
-
-       type = dbus_message_iter_get_arg_type(&value);
-
-       if (g_str_equal(name, "Powered") == TRUE) {
-               connman_bool_t powered;
-               int err;
-
-               if (type != DBUS_TYPE_BOOLEAN)
-                       return __connman_error_invalid_arguments(msg);
-
-               dbus_message_iter_get_basic(&value, &powered);
-
-               device->powered_persistent = powered;
-
-               __connman_storage_save_device(device);
-
-               if (device->powered == powered)
-                       return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
-
-               if (device->pending != NULL)
-                       return __connman_error_in_progress(msg);
-
-               err = set_powered(device, powered);
-               if (err < 0) {
-                       if (err != -EINPROGRESS)
-                               return __connman_error_failed(msg, -err);
-
-                       device->pending = dbus_message_ref(msg);
-
-                       device->timeout = g_timeout_add_seconds(15,
-                                               powered_timeout, device);
-
-                       return NULL;
-               }
-       } else if (g_str_equal(name, "ScanInterval") == TRUE) {
-               connman_uint16_t interval;
-
-               switch (device->mode) {
-               case CONNMAN_DEVICE_MODE_UNKNOWN:
-               case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-                       return __connman_error_invalid_arguments(msg);
-               case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-               case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-                       break;
-               }
-
-               if (type != DBUS_TYPE_UINT16)
-                       return __connman_error_invalid_arguments(msg);
-
-               dbus_message_iter_get_basic(&value, &interval);
-
-               if (device->scan_interval != interval) {
-                       device->scan_interval = interval;
-
-                       __connman_storage_save_device(device);
+       if (!device->driver || !device->driver->disable)
+               return -EOPNOTSUPP;
 
-                       reset_scan_trigger(device);
-               }
-       } else if (g_str_has_prefix(name, "IPv4.") == TRUE) {
-               int err;
-
-               if (device->ipconfig == NULL)
-                       return __connman_error_invalid_property(msg);
-
-               switch (device->mode) {
-               case CONNMAN_DEVICE_MODE_UNKNOWN:
-               case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-               case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-                       return __connman_error_invalid_arguments(msg);
-               case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-                       break;
-               }
+       if (device->powered == FALSE)
+               return -ENOLINK;
 
-               err = __connman_ipconfig_set_ipv4(device->ipconfig,
-                                                       name + 5, &value);
-               if (err < 0)
-                       return __connman_error_failed(msg, -err);
-       } else
-               return __connman_error_invalid_property(msg);
+       if (device->powered_pending == FALSE)
+               return -EALREADY;
 
-       return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
-}
+       device->reconnect = FALSE;
 
-static DBusMessage *propose_scan(DBusConnection *conn,
-                                       DBusMessage *msg, void *data)
-{
-       struct connman_device *device = data;
-       int err;
+       clear_scan_trigger(device);
 
-       DBG("conn %p", conn);
+       g_hash_table_remove_all(device->networks);
 
-       switch (device->mode) {
-       case CONNMAN_DEVICE_MODE_UNKNOWN:
-       case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-               return __connman_error_not_supported(msg);
-       case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-       case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-               break;
+       err = device->driver->disable(device);
+       if (err < 0 && err != -EALREADY) {
+               if (err == -EINPROGRESS)
+                       device->powered_pending = FALSE;
+               return err;
        }
 
-       err = __connman_device_scan(device);
-       if (err < 0)
-               return __connman_error_failed(msg, -err);
-
-       return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
-}
-
-static GDBusMethodTable device_methods[] = {
-       { "GetProperties", "",      "a{sv}", get_properties },
-       { "SetProperty",   "sv",    "",      set_property,
-                                               G_DBUS_METHOD_FLAG_ASYNC },
-       { "ProposeScan",   "",      "",      propose_scan   },
-       { },
-};
-
-static GDBusSignalTable device_signals[] = {
-       { "PropertyChanged", "sv" },
-       { },
-};
-
-static void append_devices(DBusMessageIter *entry)
-{
-       DBusMessageIter value, iter;
-       const char *key = "Devices";
-
-       dbus_message_iter_append_basic(entry, DBUS_TYPE_STRING, &key);
-
-       dbus_message_iter_open_container(entry, DBUS_TYPE_VARIANT,
-               DBUS_TYPE_ARRAY_AS_STRING DBUS_TYPE_OBJECT_PATH_AS_STRING,
-                                                               &value);
-
-       dbus_message_iter_open_container(&value, DBUS_TYPE_ARRAY,
-                               DBUS_TYPE_OBJECT_PATH_AS_STRING, &iter);
-       __connman_element_list(NULL, CONNMAN_ELEMENT_TYPE_DEVICE, &iter);
-       dbus_message_iter_close_container(&value, &iter);
-
-       dbus_message_iter_close_container(entry, &value);
-}
-
-static void emit_devices_signal(void)
-{
-       DBusMessage *signal;
-       DBusMessageIter entry;
-
-       signal = dbus_message_new_signal(CONNMAN_MANAGER_PATH,
-                               CONNMAN_MANAGER_INTERFACE, "PropertyChanged");
-       if (signal == NULL)
-               return;
-
-       dbus_message_iter_init_append(signal, &entry);
+       device->connections = 0;
 
-       append_devices(&entry);
-
-       g_dbus_send_message(connection, signal);
-}
-
-static int register_interface(struct connman_element *element)
-{
-       struct connman_device *device = element->device;
-
-       DBG("element %p name %s", element, element->name);
-
-       if (g_dbus_register_interface(connection, element->path,
-                                       CONNMAN_DEVICE_INTERFACE,
-                                       device_methods, device_signals,
-                                       NULL, device, NULL) == FALSE) {
-               connman_error("Failed to register %s device", element->path);
-               return -EIO;
-       }
-
-       device->registered = TRUE;
+       device->powered_pending = FALSE;
+       device->powered = FALSE;
 
-       emit_devices_signal();
+       type = __connman_device_get_service_type(device);
+       __connman_technology_disable(type);
 
        return 0;
 }
 
-static void unregister_interface(struct connman_element *element)
+static int set_powered(struct connman_device *device, connman_bool_t powered)
 {
-       struct connman_device *device = element->device;
-
-       DBG("element %p name %s", element, element->name);
-
-       device->registered = FALSE;
-
-       emit_devices_signal();
+       DBG("device %p powered %d", device, powered);
 
-       g_dbus_unregister_interface(connection, element->path,
-                                               CONNMAN_DEVICE_INTERFACE);
+       if (powered == TRUE)
+               return __connman_device_enable(device);
+       else
+               return __connman_device_disable(device);
 }
 
 static int setup_device(struct connman_device *device)
 {
-       enum connman_service_type type;
-       int err;
-
        DBG("device %p", device);
 
-       err = register_interface(&device->element);
-       if (err < 0) {
-               if (device->driver->remove)
-                       device->driver->remove(device);
-               device->driver = NULL;
-               return err;
-       }
-
-       type = __connman_device_get_service_type(device);
-       __connman_notifier_register(type);
-
-       switch (device->mode) {
-       case CONNMAN_DEVICE_MODE_UNKNOWN:
-       case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-       case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-               break;
-       case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-               if (device->carrier == TRUE && device->secondary == FALSE)
-                       __connman_profile_add_device(device);
-               break;
-       }
+       __connman_technology_add_device(device);
 
        if (device->offlinemode == FALSE &&
                                device->powered_persistent == TRUE)
@@ -730,32 +328,18 @@ static void probe_driver(struct connman_element *element, gpointer user_data)
 
        element->device->driver = driver;
 
+       __connman_element_set_driver(element);
+
        setup_device(element->device);
 }
 
 static void remove_device(struct connman_device *device)
 {
-       enum connman_service_type type;
-
        DBG("device %p", device);
 
        __connman_device_disable(device);
 
-       switch (device->mode) {
-       case CONNMAN_DEVICE_MODE_UNKNOWN:
-       case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-       case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-               break;
-       case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-               if (device->secondary == FALSE)
-                       __connman_profile_remove_device(device);
-               break;
-       }
-
-       type = __connman_device_get_service_type(device);
-       __connman_notifier_unregister(type);
-
-       unregister_interface(&device->element);
+       __connman_technology_remove_device(device);
 
        if (device->driver->remove)
                device->driver->remove(device);
@@ -781,7 +365,7 @@ connman_bool_t __connman_device_has_driver(struct connman_device *device)
        if (device == NULL || device->driver == NULL)
                return FALSE;
 
-       return device->registered;
+       return TRUE;
 }
 
 static GSList *driver_list = NULL;
@@ -839,6 +423,8 @@ static void unregister_network(gpointer data)
 
        connman_element_unregister((struct connman_element *) network);
 
+       __connman_network_set_device(network, NULL);
+
        connman_network_unref(network);
 }
 
@@ -848,27 +434,14 @@ static void device_destruct(struct connman_element *element)
 
        DBG("element %p name %s", element, element->name);
 
-       if (device->timeout > 0) {
-               g_source_remove(device->timeout);
-               device->timeout = 0;
-       }
-
-       if (device->pending != NULL) {
-               dbus_message_unref(device->pending);
-               device->pending = NULL;
-       }
+       clear_scan_trigger(device);
 
        g_free(device->ident);
        g_free(device->node);
        g_free(device->name);
        g_free(device->address);
-       g_free(device->control);
        g_free(device->interface);
-
-       if (device->ipconfig != NULL) {
-               connman_ipconfig_unref(device->ipconfig);
-               device->ipconfig = NULL;
-       }
+       g_free(device->path);
 
        g_free(device->last_network);
 
@@ -890,6 +463,8 @@ struct connman_device *connman_device_create(const char *node,
 {
        struct connman_device *device;
        const char *str;
+       enum connman_service_type service_type;
+       connman_bool_t bg_scan;
 
        DBG("node %s type %d", node, type);
 
@@ -899,6 +474,8 @@ struct connman_device *connman_device_create(const char *node,
 
        DBG("device %p", device);
 
+       bg_scan = connman_setting_get_bool("BackgroundScanning");
+
        __connman_element_initialize(&device->element);
 
        device->element.name = g_strdup(node);
@@ -912,41 +489,33 @@ struct connman_device *connman_device_create(const char *node,
                connman_element_set_string(&device->element,
                                        CONNMAN_PROPERTY_ID_TYPE, str);
 
-       device->element.ipv4.method = CONNMAN_IPCONFIG_METHOD_DHCP;
-
-       device->type      = type;
-       device->name      = g_strdup(type2description(device->type));
-       device->mode      = CONNMAN_DEVICE_MODE_UNKNOWN;
-       device->secondary = FALSE;
+       device->type = type;
+       device->name = g_strdup(type2description(device->type));
 
        device->powered_persistent = TRUE;
 
        device->phyindex = -1;
 
+       service_type = __connman_device_get_service_type(device);
+       device->blocked = __connman_technology_get_blocked(service_type);
+       device->backoff_interval = SCAN_INITIAL_DELAY;
+
        switch (type) {
        case CONNMAN_DEVICE_TYPE_UNKNOWN:
-       case CONNMAN_DEVICE_TYPE_VENDOR:
-               device->scan_interval = 0;
-               break;
        case CONNMAN_DEVICE_TYPE_ETHERNET:
-       case CONNMAN_DEVICE_TYPE_WIFI:
-               device->scan_interval = 300;
-               break;
        case CONNMAN_DEVICE_TYPE_WIMAX:
-               device->scan_interval = 0;
-               break;
        case CONNMAN_DEVICE_TYPE_BLUETOOTH:
-               device->scan_interval = 0;
-               break;
+       case CONNMAN_DEVICE_TYPE_CELLULAR:
        case CONNMAN_DEVICE_TYPE_GPS:
+       case CONNMAN_DEVICE_TYPE_GADGET:
+       case CONNMAN_DEVICE_TYPE_VENDOR:
                device->scan_interval = 0;
                break;
-       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:
-               device->scan_interval = 0;
+       case CONNMAN_DEVICE_TYPE_WIFI:
+               if (bg_scan == TRUE)
+                       device->scan_interval = 300;
+               else
+                       device->scan_interval = 0;
                break;
        }
 
@@ -998,28 +567,6 @@ enum connman_device_type connman_device_get_type(struct connman_device *device)
 }
 
 /**
- * connman_device_get_name:
- * @device: device structure
- *
- * Get unique name of device
- */
-const char *connman_device_get_name(struct connman_device *device)
-{
-       return device->element.name;
-}
-
-/**
- * connman_device_get_path:
- * @device: device structure
- *
- * Get path name of device
- */
-const char *connman_device_get_path(struct connman_device *device)
-{
-       return device->element.path;
-}
-
-/**
  * connman_device_set_index:
  * @device: device structure
  * @index: index number
@@ -1029,15 +576,6 @@ const char *connman_device_get_path(struct connman_device *device)
 void connman_device_set_index(struct connman_device *device, int index)
 {
        device->element.index = index;
-
-       if (device->ipconfig != NULL) {
-               if (index == connman_ipconfig_get_index(device->ipconfig))
-                       return;
-
-               connman_ipconfig_unref(device->ipconfig);
-       }
-
-       device->ipconfig = connman_ipconfig_create(index);
 }
 
 /**
@@ -1066,12 +604,11 @@ void __connman_device_set_phyindex(struct connman_device *device,
  * connman_device_set_interface:
  * @device: device structure
  * @interface: interface name
- * @control: control interface
  *
  * Set interface name of device
  */
 void connman_device_set_interface(struct connman_device *device,
-                               const char *interface, const char *control)
+                                               const char *interface)
 {
        g_free(device->element.devname);
        device->element.devname = g_strdup(interface);
@@ -1079,9 +616,6 @@ void connman_device_set_interface(struct connman_device *device,
        g_free(device->interface);
        device->interface = g_strdup(interface);
 
-       g_free(device->control);
-       device->control = g_strdup(control);
-
        if (device->name == NULL) {
                const char *str = type2description(device->type);
                if (str != NULL && device->interface != NULL)
@@ -1090,11 +624,6 @@ void connman_device_set_interface(struct connman_device *device,
        }
 }
 
-const char *connman_device_get_control(struct connman_device *device)
-{
-       return device->control;
-}
-
 /**
  * connman_device_set_ident:
  * @device: device structure
@@ -1109,60 +638,12 @@ void connman_device_set_ident(struct connman_device *device,
        device->ident = g_strdup(ident);
 }
 
-const char *__connman_device_get_ident(struct connman_device *device)
+const char *connman_device_get_ident(struct connman_device *device)
 {
        return device->ident;
 }
 
 /**
- * connman_device_set_mode:
- * @device: device structure
- * @mode: network mode
- *
- * Change network mode of device
- */
-void connman_device_set_mode(struct connman_device *device,
-                                               enum connman_device_mode mode)
-{
-       device->mode = mode;
-}
-
-/**
- * connman_device_get_mode:
- * @device: device structure
- *
- * Get network mode of device
- */
-enum connman_device_mode connman_device_get_mode(struct connman_device *device)
-{
-       return device->mode;
-}
-
-/**
- * connman_device_set_secondary:
- * @device: device structure
- * @secondary: secondary value
- *
- * Change secondary value of device
- */
-void connman_device_set_secondary(struct connman_device *device,
-                                               connman_bool_t secondary)
-{
-       device->secondary = secondary;
-}
-
-/**
- * connman_device_get_secondary:
- * @device: device structure
- *
- * Get secondary value of device
- */
-connman_bool_t connman_device_get_secondary(struct connman_device *device)
-{
-       return device->secondary;
-}
-
-/**
  * connman_device_set_powered:
  * @device: device structure
  * @powered: powered state
@@ -1172,25 +653,23 @@ connman_bool_t connman_device_get_secondary(struct connman_device *device)
 int connman_device_set_powered(struct connman_device *device,
                                                connman_bool_t powered)
 {
+       int err;
        enum connman_service_type type;
 
        DBG("driver %p powered %d", device, powered);
 
-       if (device->timeout > 0) {
-               g_source_remove(device->timeout);
-               device->timeout = 0;
+       if (device->powered == powered) {
+               device->powered_pending = powered;
+               return -EALREADY;
        }
 
-       if (device->pending != NULL) {
-               g_dbus_send_reply(connection, device->pending,
-                                                       DBUS_TYPE_INVALID);
-
-               dbus_message_unref(device->pending);
-               device->pending = NULL;
-       }
+       if (powered == TRUE)
+               err = __connman_device_enable(device);
+       else
+               err = __connman_device_disable(device);
 
-       if (device->powered == powered)
-               return -EALREADY;
+       if (err < 0 && err != -EINPROGRESS && err != -EALREADY)
+               return err;
 
        device->powered = powered;
        device->powered_pending = powered;
@@ -1198,21 +677,19 @@ int connman_device_set_powered(struct connman_device *device,
        type = __connman_device_get_service_type(device);
 
        if (device->powered == TRUE)
-               __connman_notifier_enable(type);
+               __connman_technology_enable(type);
        else
-               __connman_notifier_disable(type);
-
-       if (device->registered == FALSE)
-               return 0;
+               __connman_technology_disable(type);
 
-       powered_changed(device);
+       if (device->offlinemode == TRUE && powered == TRUE)
+               return connman_device_set_powered(device, FALSE);
 
        if (powered == FALSE)
                return 0;
 
        reset_scan_trigger(device);
 
-       if (device->driver->scan)
+       if (device->driver && device->driver->scan)
                device->driver->scan(device);
 
        return 0;
@@ -1230,6 +707,8 @@ int __connman_device_set_blocked(struct connman_device *device,
        if (device->offlinemode == TRUE)
                return 0;
 
+       connman_info("%s {rfkill} blocked %d", device->interface, blocked);
+
        if (blocked == FALSE)
                powered = device->powered_persistent;
        else
@@ -1238,33 +717,9 @@ int __connman_device_set_blocked(struct connman_device *device,
        return set_powered(device, powered);
 }
 
-/**
- * connman_device_set_carrier:
- * @device: device structure
- * @carrier: carrier state
- *
- * Change carrier state of device (only for device without scanning)
- */
-int connman_device_set_carrier(struct connman_device *device,
-                                               connman_bool_t carrier)
+connman_bool_t __connman_device_get_blocked(struct connman_device *device)
 {
-       DBG("device %p carrier %d", device, carrier);
-
-       switch (device->mode) {
-       case CONNMAN_DEVICE_MODE_UNKNOWN:
-       case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-       case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-               return -EINVAL;
-       case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-               break;
-       }
-
-       if (device->carrier == carrier)
-               return -EALREADY;
-
-       device->carrier = carrier;
-
-       return set_carrier(device, device->carrier);
+       return device->blocked;
 }
 
 int __connman_device_scan(struct connman_device *device)
@@ -1275,79 +730,32 @@ int __connman_device_scan(struct connman_device *device)
        if (device->powered == FALSE)
                return -ENOLINK;
 
+       reset_scan_trigger(device);
+
        return device->driver->scan(device);
 }
 
-int __connman_device_enable(struct connman_device *device)
+int __connman_device_enable_persistent(struct connman_device *device)
 {
-       enum connman_service_type type;
        int err;
 
        DBG("device %p", device);
 
-       if (!device->driver || !device->driver->enable)
-               return -EOPNOTSUPP;
-
-       if (device->powered_pending == TRUE)
-               return -EALREADY;
-
-       device->powered_pending = TRUE;
-
-       err = device->driver->enable(device);
-       if (err < 0)
-               return err;
-
-       device->powered = TRUE;
-
-       type = __connman_device_get_service_type(device);
-       __connman_notifier_enable(type);
-
-       return 0;
-}
-
-int __connman_device_enable_persistent(struct connman_device *device)
-{
-       DBG("device %p", device);
-
        device->powered_persistent = TRUE;
 
        __connman_storage_save_device(device);
 
-       return __connman_device_enable(device);
-}
+       err = __connman_device_enable(device);
+       if (err == 0 || err == -EINPROGRESS) {
+               device->offlinemode = FALSE;
+               if (__connman_profile_get_offlinemode() == TRUE) {
+                       __connman_profile_set_offlinemode(FALSE, FALSE);
 
-int __connman_device_disable(struct connman_device *device)
-{
-       enum connman_service_type type;
-       int err;
-
-       DBG("device %p", device);
-
-       if (!device->driver || !device->driver->disable)
-               return -EOPNOTSUPP;
-
-       if (device->powered == FALSE)
-               return -ENOLINK;
-
-       if (device->powered_pending == FALSE)
-               return -EALREADY;
-
-       device->powered_pending = FALSE;
-
-       clear_scan_trigger(device);
-
-       g_hash_table_remove_all(device->networks);
-
-       err = device->driver->disable(device);
-       if (err < 0)
-               return err;
-
-       device->powered = FALSE;
-
-       type = __connman_device_get_service_type(device);
-       __connman_notifier_disable(type);
+                       __connman_profile_save_default();
+               }
+       }
 
-       return 0;
+       return err;
 }
 
 int __connman_device_disable_persistent(struct connman_device *device)
@@ -1361,19 +769,6 @@ int __connman_device_disable_persistent(struct connman_device *device)
        return __connman_device_disable(device);
 }
 
-int __connman_device_connect(struct connman_device *device)
-{
-       DBG("device %p", device);
-
-       if (device->disconnected == FALSE)
-               return -EINVAL;
-
-       if (device->driver && device->driver->connect)
-               device->driver->connect(device);
-
-       return 0;
-}
-
 int __connman_device_disconnect(struct connman_device *device)
 {
        GHashTableIter iter;
@@ -1388,15 +783,36 @@ int __connman_device_disconnect(struct connman_device *device)
        while (g_hash_table_iter_next(&iter, &key, &value) == TRUE) {
                struct connman_network *network = value;
 
+               if (connman_network_get_connecting(network) == TRUE) {
+                       /*
+                        * Skip network in the process of connecting.
+                        * This is a workaround for WiFi networks serviced
+                        * by the supplicant plugin that hold a reference
+                        * to the network.  If we disconnect the network
+                        * here then the referenced object will not be
+                        * registered and usage (like launching DHCP client)
+                        * will fail.  There is nothing to be gained by
+                        * removing the network here anyway.
+                        */
+                       connman_warn("Skipping disconnect of %s, network is connecting.",
+                               connman_network_get_identifier(network));
+                       continue;
+               }
+
                __connman_network_disconnect(network);
        }
 
-       if (device->driver && device->driver->disconnect)
-               device->driver->disconnect(device);
-
        return 0;
 }
 
+static void mark_network_available(gpointer key, gpointer value,
+                                                        gpointer user_data)
+{
+       struct connman_network *network = value;
+
+       connman_network_set_available(network, TRUE);
+}
+
 static void mark_network_unavailable(gpointer key, gpointer value,
                                                        gpointer user_data)
 {
@@ -1428,6 +844,20 @@ void __connman_device_cleanup_networks(struct connman_device *device)
                                        remove_unavailable_network, NULL);
 }
 
+connman_bool_t __connman_device_scanning(struct connman_device *device)
+{
+       return device->scanning;
+}
+
+void connman_device_reset_scanning(struct connman_device *device)
+{
+       device->scanning = FALSE;
+
+       g_hash_table_foreach(device->networks,
+                               mark_network_available, NULL);
+
+}
+
 /**
  * connman_device_set_scanning:
  * @device: device structure
@@ -1438,10 +868,6 @@ void __connman_device_cleanup_networks(struct connman_device *device)
 int connman_device_set_scanning(struct connman_device *device,
                                                connman_bool_t scanning)
 {
-       DBusMessage *signal;
-       DBusMessageIter entry, value;
-       const char *key = "Scanning";
-
        DBG("device %p scanning %d", device, scanning);
 
        if (!device->driver || !device->driver->scan)
@@ -1452,22 +878,6 @@ int connman_device_set_scanning(struct connman_device *device,
 
        device->scanning = scanning;
 
-       signal = dbus_message_new_signal(device->element.path,
-                               CONNMAN_DEVICE_INTERFACE, "PropertyChanged");
-       if (signal == NULL)
-               return 0;
-
-       dbus_message_iter_init_append(signal, &entry);
-
-       dbus_message_iter_append_basic(&entry, DBUS_TYPE_STRING, &key);
-
-       dbus_message_iter_open_container(&entry, DBUS_TYPE_VARIANT,
-                                       DBUS_TYPE_BOOLEAN_AS_STRING, &value);
-       dbus_message_iter_append_basic(&value, DBUS_TYPE_BOOLEAN, &scanning);
-       dbus_message_iter_close_container(&entry, &value);
-
-       g_dbus_send_message(connection, signal);
-
        if (scanning == TRUE) {
                reset_scan_trigger(device);
 
@@ -1482,9 +892,6 @@ int connman_device_set_scanning(struct connman_device *device,
        if (device->connections > 0)
                return 0;
 
-       if (device->disconnected == TRUE)
-               return 0;
-
        __connman_service_auto_connect();
 
        return 0;
@@ -1502,20 +909,14 @@ int connman_device_set_disconnected(struct connman_device *device,
 {
        DBG("device %p disconnected %d", device, disconnected);
 
-       switch (device->mode) {
-       case CONNMAN_DEVICE_MODE_UNKNOWN:
-       case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-               return -EINVAL;
-       case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-       case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-               break;
-       }
-
        if (device->disconnected == disconnected)
                return -EALREADY;
 
        device->disconnected = disconnected;
 
+       if (disconnected == TRUE)
+               force_scan_trigger(device);
+
        return 0;
 }
 
@@ -1531,33 +932,6 @@ connman_bool_t connman_device_get_disconnected(struct connman_device *device)
 }
 
 /**
- * connman_device_set_connected:
- * @device: device structure
- * @connected: connected state
- *
- * Change connected state of device (for Ethernet like devices)
- */
-int connman_device_set_connected(struct connman_device *device,
-                                               connman_bool_t connected)
-{
-       DBG("device %p connected %d", device, connected);
-
-       switch (device->mode) {
-       case CONNMAN_DEVICE_MODE_UNKNOWN:
-       case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-       case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-               return -EINVAL;
-       case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-               break;
-       }
-
-       if (device->carrier == FALSE)
-               return -ENOTCONN;
-
-       return set_connected(device, connected);
-}
-
-/**
  * connman_device_set_string:
  * @device: device structure
  * @key: unique identifier
@@ -1579,9 +953,14 @@ int connman_device_set_string(struct connman_device *device,
        } else if (g_str_equal(key, "Node") == TRUE) {
                g_free(device->node);
                device->node = g_strdup(value);
+       } else if (g_str_equal(key, "Path") == TRUE) {
+               g_free(device->path);
+               device->path = g_strdup(value);
+       } else {
+               return -EINVAL;
        }
 
-       return connman_element_set_string(&device->element, key, value);
+       return 0;
 }
 
 /**
@@ -1602,8 +981,12 @@ const char *connman_device_get_string(struct connman_device *device,
                return device->name;
        else if (g_str_equal(key, "Node") == TRUE)
                return device->node;
+       else if (g_str_equal(key, "Interface") == TRUE)
+               return device->interface;
+       else if (g_str_equal(key, "Path") == TRUE)
+               return device->path;
 
-       return connman_element_get_string(&device->element, key);
+       return NULL;
 }
 
 static void set_offlinemode(struct connman_element *element, gpointer user_data)
@@ -1619,6 +1002,9 @@ static void set_offlinemode(struct connman_element *element, gpointer user_data)
 
        device->offlinemode = offlinemode;
 
+       if (device->blocked == TRUE)
+               return;
+
        powered = (offlinemode == TRUE) ? FALSE : TRUE;
 
        if (device->powered == powered)
@@ -1644,12 +1030,21 @@ int __connman_device_set_offlinemode(connman_bool_t offlinemode)
 
 void __connman_device_increase_connections(struct connman_device *device)
 {
+       if (device == NULL)
+               return;
+
        device->connections++;
 }
 
 void __connman_device_decrease_connections(struct connman_device *device)
 {
+       if (device == NULL)
+               return;
+
        device->connections--;
+
+       if (device->connections == 0)
+               device->backoff_interval = SCAN_INITIAL_DELAY;
 }
 
 /**
@@ -1667,14 +1062,8 @@ int connman_device_add_network(struct connman_device *device,
 
        DBG("device %p network %p", device, network);
 
-       switch (device->mode) {
-       case CONNMAN_DEVICE_MODE_UNKNOWN:
-       case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
+       if (identifier == NULL)
                return -EINVAL;
-       case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-       case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-               break;
-       }
 
        __connman_network_set_device(network, device);
 
@@ -1723,11 +1112,19 @@ int connman_device_remove_network(struct connman_device *device,
        return 0;
 }
 
+void connman_device_remove_all_networks(struct connman_device *device)
+{
+       g_hash_table_remove_all(device->networks);
+}
+
 void __connman_device_set_network(struct connman_device *device,
                                        struct connman_network *network)
 {
        const char *name;
 
+       if (device == NULL)
+               return;
+
        if (device->network == network)
                return;
 
@@ -1749,6 +1146,18 @@ void __connman_device_set_network(struct connman_device *device,
        }
 }
 
+void __connman_device_set_reconnect(struct connman_device *device,
+                                               connman_bool_t reconnect)
+{
+       device->reconnect = reconnect;
+}
+
+connman_bool_t  __connman_device_get_reconnect(
+                               struct connman_device *device)
+{
+       return device->reconnect;
+}
+
 /**
  * connman_device_register:
  * @device: device structure
@@ -1873,11 +1282,10 @@ static int device_load(struct connman_device *device)
        GError *error = NULL;
        gchar *identifier;
        connman_bool_t powered;
-       int val;
 
        DBG("device %p", device);
 
-       keyfile = __connman_storage_open(ident);
+       keyfile = __connman_storage_open_profile(ident);
        if (keyfile == NULL)
                return 0;
 
@@ -1891,24 +1299,10 @@ static int device_load(struct connman_device *device)
                device->powered_persistent = powered;
        g_clear_error(&error);
 
-       switch (device->mode) {
-       case CONNMAN_DEVICE_MODE_UNKNOWN:
-       case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-               break;
-       case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-       case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-               val = g_key_file_get_integer(keyfile, identifier,
-                                               "ScanInterval", &error);
-               if (error == NULL && val > 0)
-                       device->scan_interval = val;
-               g_clear_error(&error);
-               break;
-       }
-
 done:
        g_free(identifier);
 
-       __connman_storage_close(ident, keyfile, FALSE);
+       __connman_storage_close_profile(ident, keyfile, FALSE);
 
        return 0;
 }
@@ -1921,7 +1315,7 @@ static int device_save(struct connman_device *device)
 
        DBG("device %p", device);
 
-       keyfile = __connman_storage_open(ident);
+       keyfile = __connman_storage_open_profile(ident);
        if (keyfile == NULL)
                return 0;
 
@@ -1932,22 +1326,10 @@ static int device_save(struct connman_device *device)
        g_key_file_set_boolean(keyfile, identifier,
                                        "Powered", device->powered_persistent);
 
-       switch (device->mode) {
-       case CONNMAN_DEVICE_MODE_UNKNOWN:
-       case CONNMAN_DEVICE_MODE_TRANSPORT_IP:
-               break;
-       case CONNMAN_DEVICE_MODE_NETWORK_SINGLE:
-       case CONNMAN_DEVICE_MODE_NETWORK_MULTIPLE:
-               if (device->scan_interval > 0)
-                       g_key_file_set_integer(keyfile, identifier,
-                                       "ScanInterval", device->scan_interval);
-               break;
-       }
-
 done:
        g_free(identifier);
 
-       __connman_storage_close(ident, keyfile, TRUE);
+       __connman_storage_close_profile(ident, keyfile, TRUE);
 
        return 0;
 }
@@ -1963,8 +1345,6 @@ int __connman_device_init(void)
 {
        DBG("");
 
-       connection = connman_dbus_get_connection();
-
        if (connman_storage_register(&device_storage) < 0)
                connman_error("Failed to register device storage");
 
@@ -1978,6 +1358,4 @@ void __connman_device_cleanup(void)
        connman_driver_unregister(&device_driver);
 
        connman_storage_unregister(&device_storage);
-
-       dbus_connection_unref(connection);
 }