Merge branch 'tech' into 'master'

Add "technologies" configuration option

See merge request !111

ofono/Makefile.am

@@ -593,20 +593,11 @@ builtin_modules += nettime
 builtin_sources += plugins/nettime.c
 endif
 
-if SAILFISH_PROVISION
-builtin_sources += plugins/sailfish_provision.c
-PROVISION = 1
-else
-if PROVISION
-builtin_sources += plugins/provision.c
-endif
-endif
-
 if PROVISION
 builtin_sources += plugins/mbpi.h plugins/mbpi.c
 
 builtin_modules += provision
-builtin_sources += plugins/provision.h
+builtin_sources += plugins/provision.h plugins/provision.c
 
 builtin_modules += cdma_provision
 builtin_sources += plugins/cdma-provision.c
@@ -929,9 +920,9 @@ unit_test_caif_LDADD = @GLIB_LIBS@
 unit_objects += $(unit_test_caif_OBJECTS)
 
 unit_test_provision_SOURCES = unit/test-provision.c \
-				plugins/provision.h plugins/mbpi.c \
-				plugins/sailfish_provision.c \
-				src/gprs-provision.c src/log.c
+				plugins/provision.h plugins/provision.c \
+				plugins/mbpi.c src/gprs-provision.c \
+				src/log.c
 unit_test_provision_LDADD = @GLIB_LIBS@ -ldl
 unit_objects += $(unit_test_provision_OBJECTS)
 

ofono/configure.ac

@@ -217,11 +217,6 @@ AC_ARG_ENABLE(sailfishos, AC_HELP_STRING([--enable-sailfishos],
                 [enable sailfishos plugin]), [enable_sailfishos=${enableval}])
 AM_CONDITIONAL(SAILFISHFOS, test "${enable_sailfishos}" = "yes")
 
-AC_ARG_ENABLE(sailfish-provision, AC_HELP_STRING([--enable-sailfish-provision],
-			[enable Sailfish OS provisioning plugin]),
-				[enable_sailfish_provision=${enableval}])
-AM_CONDITIONAL(SAILFISH_PROVISION, test "${enable_sailfish_provision=$}" = "yes")
-
 AC_ARG_ENABLE(nettime, AC_HELP_STRING([--disable-nettime],
                                 [disable Nettime plugin]),
                                         [enable_nettime=${enableval}])

ofono/drivers/atmodem/gprs-context.c

@@ -247,8 +247,6 @@ static void at_gprs_activate_primary(struct ofono_gprs_context *gc,
 
 	/* We only support CHAP and PAP */
 	switch (ctx->auth_method) {
-	case OFONO_GPRS_AUTH_METHOD_ANY:
-	case OFONO_GPRS_AUTH_METHOD_NONE:
 	case OFONO_GPRS_AUTH_METHOD_CHAP:
 		gcd->auth_method = G_AT_PPP_AUTH_METHOD_CHAP;
 		break;
@@ -296,8 +294,6 @@ static void at_gprs_activate_primary(struct ofono_gprs_context *gc,
 			 * prefix, this is the least invasive place to set it.
 			 */
 			switch (ctx->auth_method) {
-			case OFONO_GPRS_AUTH_METHOD_ANY:
-			case OFONO_GPRS_AUTH_METHOD_NONE:
 			case OFONO_GPRS_AUTH_METHOD_CHAP:
 				snprintf(buf + len, sizeof(buf) - len - 3,
 						",\"CHAP:%s\"", ctx->apn);

ofono/drivers/ril/ril_config.c

@@ -20,7 +20,7 @@
 
 /* Utilities for parsing ril_subscription.conf */
 
-char* ril_config_get_string(GKeyFile *file, const char *group, const char *key)
+char *ril_config_get_string(GKeyFile *file, const char *group, const char *key)
 {
 	char *val = g_key_file_get_string(file, group, key, NULL);
 
@@ -32,6 +32,31 @@ char* ril_config_get_string(GKeyFile *file, const char *group, const char *key)
 	return val;
 }
 
+char **ril_config_get_strings(GKeyFile *file, const char *group,
+					const char *key, char delimiter)
+{
+	char *str = ril_config_get_string(file, group, key);
+
+	if (str) {
+		char **strv, **p;
+		char delimiter_str[2];
+
+		delimiter_str[0] = delimiter;
+		delimiter_str[1] = 0;
+		strv = g_strsplit(str, delimiter_str, -1);
+
+		/* Strip whitespaces */
+		for (p = strv; *p; p++) {
+			*p = g_strstrip(*p);
+		}
+
+		g_free(str);
+		return strv;
+	}
+
+	return NULL;
+}
+
 gboolean ril_config_get_integer(GKeyFile *file, const char *group,
 					const char *key, int *out_value)
 {
@@ -112,12 +137,12 @@ gboolean ril_config_get_flag(GKeyFile *file, const char *group,
 GUtilInts *ril_config_get_ints(GKeyFile *file, const char *group,
 					const char *key)
 {
-	char* value = ril_config_get_string(file, group, key);
+	char *value = ril_config_get_string(file, group, key);
 
 	if (value) {
 		char **values = g_strsplit(value, ",", -1);
 		char **ptr = values;
-		GUtilIntArray* array = gutil_int_array_new();
+		GUtilIntArray *array = gutil_int_array_new();
 
 		while (*ptr) {
 			const char *str = *ptr++;
@@ -141,7 +166,7 @@ char *ril_config_ints_to_string(GUtilInts *ints, char separator)
 	if (ints) {
 		guint i, n;
 		const int *data = gutil_ints_get_data(ints, &n);
-		GString* buf = g_string_new(NULL);
+		GString *buf = g_string_new(NULL);
 
 		for (i=0; i<n; i++) {
 			if (buf->len > 0) {

ofono/drivers/ril/ril_config.h

@@ -22,7 +22,10 @@
 
 #define RILCONF_SETTINGS_GROUP      "Settings"
 
-char *ril_config_get_string(GKeyFile *file, const char *group, const char *key);
+char *ril_config_get_string(GKeyFile *file, const char *group,
+					const char *key);
+char **ril_config_get_strings(GKeyFile *file, const char *group,
+			      const char *key, char delimiter);
 gboolean ril_config_get_integer(GKeyFile *file, const char *group,
 					const char *key, int *value);
 gboolean ril_config_get_boolean(GKeyFile *file, const char *group,

ofono/drivers/ril/ril_data.c

@@ -788,7 +788,7 @@ static gboolean ril_data_call_setup_submit(struct ril_data_request *req)
 	struct ril_data_priv *priv = req->data->priv;
 	const char *proto_str = ril_data_ofono_protocol_to_ril(setup->proto);
 	GRilIoRequest* ioreq;
-	int tech, auth = RIL_AUTH_NONE;
+	int tech, auth;
 
 	GASSERT(proto_str);
 
@@ -811,22 +811,14 @@ static gboolean ril_data_call_setup_submit(struct ril_data_request *req)
 		tech = RADIO_TECH_HSPA;
 	}
 
-	if (setup->username && setup->username[0]) {
-		switch (setup->auth_method) {
-		case OFONO_GPRS_AUTH_METHOD_ANY:
-			auth = RIL_AUTH_BOTH;
-			break;
-		case OFONO_GPRS_AUTH_METHOD_NONE:
-			auth = RIL_AUTH_NONE;
-			break;
-		case OFONO_GPRS_AUTH_METHOD_CHAP:
-			auth = RIL_AUTH_CHAP;
-			break;
-		case OFONO_GPRS_AUTH_METHOD_PAP:
-			auth = RIL_AUTH_PAP;
-			break;
-		}
-	}
+        /*
+         * We do the same as in $AOSP/frameworks/opt/telephony/src/java/com/
+         * android/internal/telephony/dataconnection/DataConnection.java,
+         * onConnect(), and use authentication or not depending on whether
+         * the user field is empty or not.
+         */
+	auth = (setup->username && setup->username[0]) ?
+					RIL_AUTH_BOTH : RIL_AUTH_NONE;
 
 	/*
 	 * TODO: add comments about tethering, other non-public

ofono/drivers/ril/ril_devinfo.c

@@ -1,7 +1,7 @@
 /*
  *  oFono - Open Source Telephony - RIL-based devices
  *
- *  Copyright (C) 2015-2017 Jolla Ltd.
+ *  Copyright (C) 2015-2016 Jolla Ltd.
  *
  *  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
@@ -17,24 +17,16 @@
 #include "ril_util.h"
 #include "ril_log.h"
 
-#include <gutil_idlequeue.h>
-
 /*
  * TODO: No public RIL api to query manufacturer or model.
  * Check where to get, could /system/build.prop be updated to have good values?
  */
 
-enum ril_devinfo_cb_tag {
-	DEVINFO_QUERY_SERIAL = 1,
-	DEVINFO_QUERY_SVN
-};
-
 struct ril_devinfo {
 	struct ofono_devinfo *info;
 	GRilIoQueue *q;
-	GUtilIdleQueue *iq;
-	char *log_prefix;
-	char *imeisv;
+	guint register_id;
+	guint imei_id;
 	char *imei;
 };
 
@@ -44,7 +36,6 @@ struct ril_devinfo_cbd {
 	gpointer data;
 };
 
-#define DBG_(self,fmt,args...) DBG("%s" fmt, (self)->log_prefix, ##args)
 #define ril_devinfo_cbd_free g_free
 
 static inline struct ril_devinfo *ril_devinfo_get_data(
@@ -71,7 +62,7 @@ static void ril_devinfo_query_unsupported(struct ofono_devinfo *info,
 	cb(ril_error_failure(&error), "", data);
 }
 
-static void ril_devinfo_query_revision_cb(GRilIoChannel *io, int status,
+static void ril_devinfo_query_cb(GRilIoChannel *io, int status,
 			const void *data, guint len, void *user_data)
 {
 	struct ofono_error error;
@@ -82,7 +73,7 @@ static void ril_devinfo_query_revision_cb(GRilIoChannel *io, int status,
 		GRilIoParser rilp;
 		grilio_parser_init(&rilp, data, len);
 		res = grilio_parser_get_utf8(&rilp);
-		DBG_(cbd->di, "%s", res);
+		DBG("%s", res);
 		cbd->cb(ril_error_ok(&error), res ? res : "", cbd->data);
 		g_free(res);
 	} else {
@@ -95,46 +86,23 @@ static void ril_devinfo_query_revision(struct ofono_devinfo *info,
 {
 	struct ril_devinfo *di = ril_devinfo_get_data(info);
 
-	DBG_(di, "");
-	grilio_queue_send_request_full(di->q, NULL,
-				RIL_REQUEST_BASEBAND_VERSION,
-				ril_devinfo_query_revision_cb,
-				ril_devinfo_cbd_free,
+	DBG("");
+	grilio_queue_send_request_full(di->q, NULL, RIL_REQUEST_BASEBAND_VERSION,
+				ril_devinfo_query_cb, ril_devinfo_cbd_free,
 				ril_devinfo_cbd_new(di, cb, data));
 }
 
-static void ril_devinfo_query_serial_cb(gpointer user_data)
+static gboolean ril_devinfo_query_serial_cb(void *user_data)
 {
 	struct ril_devinfo_cbd *cbd = user_data;
 	struct ril_devinfo *di = cbd->di;
 	struct ofono_error error;
 
-	DBG_(di, "%s", di->imei);
+	GASSERT(di->imei_id);
+	di->imei_id = 0;
+
 	cbd->cb(ril_error_ok(&error), di->imei, cbd->data);
-}
-
-static void ril_devinfo_query_svn_cb(gpointer user_data)
-{
-	struct ril_devinfo_cbd *cbd = user_data;
-	struct ril_devinfo *di = cbd->di;
-	struct ofono_error error;
-
-	DBG_(di, "%s", di->imeisv);
-	if (di->imeisv && di->imeisv[0]) {
-		cbd->cb(ril_error_ok(&error), di->imeisv, cbd->data);
-	} else {
-		cbd->cb(ril_error_failure(&error), "", cbd->data);
-	}
-}
-
-static void ril_devinfo_query(struct ril_devinfo *di,
-			enum ril_devinfo_cb_tag tag, GUtilIdleFunc fn,
-			ofono_devinfo_query_cb_t cb, void *data)
-{
-	GVERIFY_FALSE(gutil_idle_queue_cancel_tag(di->iq, tag));
-	gutil_idle_queue_add_tag_full(di->iq, tag, fn,
-					ril_devinfo_cbd_new(di, cb, data),
-					ril_devinfo_cbd_free);
+	return FALSE;
 }
 
 static void ril_devinfo_query_serial(struct ofono_devinfo *info,
@@ -143,28 +111,29 @@ static void ril_devinfo_query_serial(struct ofono_devinfo *info,
 {
 	struct ril_devinfo *di = ril_devinfo_get_data(info);
 
-	DBG_(di, "");
-	ril_devinfo_query(di, DEVINFO_QUERY_SERIAL,
-				ril_devinfo_query_serial_cb, cb, data);
+	GASSERT(!di->imei_id);
+	if (di->imei_id) {
+		g_source_remove(di->imei_id);
+		di->imei_id = 0;
+	}
+
+	DBG("%s", di->imei);
+	di->imei_id = g_idle_add_full(G_PRIORITY_DEFAULT_IDLE,
+					ril_devinfo_query_serial_cb,
+					ril_devinfo_cbd_new(di, cb, data),
+					ril_devinfo_cbd_free);
 }
 
-static void ril_devinfo_query_svn(struct ofono_devinfo *info,
-				ofono_devinfo_query_cb_t cb,
-				void *data)
-{
-	struct ril_devinfo *di = ril_devinfo_get_data(info);
-
-	DBG_(di, "");
-	ril_devinfo_query(di, DEVINFO_QUERY_SVN,
-				ril_devinfo_query_svn_cb, cb, data);
-}
-
-static void ril_devinfo_register(gpointer user_data)
+static gboolean ril_devinfo_register(gpointer user_data)
 {
 	struct ril_devinfo *di = user_data;
 
-	DBG_(di, "");
+	DBG("");
+	di->register_id = 0;
 	ofono_devinfo_register(di->info);
+
+	/* This makes the timeout a single-shot */
+	return FALSE;
 }
 
 static int ril_devinfo_probe(struct ofono_devinfo *info, unsigned int vendor,
@@ -173,18 +142,13 @@ static int ril_devinfo_probe(struct ofono_devinfo *info, unsigned int vendor,
 	struct ril_modem *modem = data;
 	struct ril_devinfo *di = g_new0(struct ril_devinfo, 1);
 
-	di->log_prefix = (modem->log_prefix && modem->log_prefix[0]) ?
-		g_strconcat(modem->log_prefix, " ", NULL) : g_strdup("");
-
-	DBG_(di, "%s", modem->imei);
+	DBG("%s %s %p", ril_modem_get_path(modem), modem->imei, di);
 	GASSERT(modem->imei);
-
 	di->q = grilio_queue_new(ril_modem_io(modem));
 	di->info = info;
-	di->imeisv = g_strdup(modem->imeisv);
 	di->imei = g_strdup(modem->imei);
-	di->iq = gutil_idle_queue_new();
-	gutil_idle_queue_add(di->iq, ril_devinfo_register, di);
+
+	di->register_id = g_idle_add(ril_devinfo_register, di);
 	ofono_devinfo_set_data(info, di);
 	return 0;
 }
@@ -193,14 +157,19 @@ static void ril_devinfo_remove(struct ofono_devinfo *info)
 {
 	struct ril_devinfo *di = ril_devinfo_get_data(info);
 
-	DBG_(di, "");
+	DBG("%p", di);
 	ofono_devinfo_set_data(info, NULL);
-	gutil_idle_queue_cancel_all(di->iq);
-	gutil_idle_queue_unref(di->iq);
+
+	if (di->register_id > 0) {
+		g_source_remove(di->register_id);
+	}
+
+	if (di->imei_id > 0) {
+		g_source_remove(di->imei_id);
+	}
+
 	grilio_queue_cancel_all(di->q, FALSE);
 	grilio_queue_unref(di->q);
-	g_free(di->log_prefix);
-	g_free(di->imeisv);
 	g_free(di->imei);
 	g_free(di);
 }
@@ -209,11 +178,10 @@ const struct ofono_devinfo_driver ril_devinfo_driver = {
 	.name                   = RILMODEM_DRIVER,
 	.probe                  = ril_devinfo_probe,
 	.remove                 = ril_devinfo_remove,
-	/* query_revision won't be called if query_model is missing */
+	.query_manufacturer     = ril_devinfo_query_unsupported,
 	.query_model            = ril_devinfo_query_unsupported,
 	.query_revision         = ril_devinfo_query_revision,
-	.query_serial           = ril_devinfo_query_serial,
-	.query_svn              = ril_devinfo_query_svn
+	.query_serial           = ril_devinfo_query_serial
 };
 
 /*

ofono/drivers/ril/ril_modem.c

@@ -1,7 +1,7 @@
 /*
  *  oFono - Open Source Telephony - RIL-based devices
  *
- *  Copyright (C) 2015-2017 Jolla Ltd.
+ *  Copyright (C) 2015-2016 Jolla Ltd.
  *
  *  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
@@ -52,7 +52,6 @@ struct ril_modem_data {
 	struct ril_modem modem;
 	GRilIoQueue *q;
 	char *log_prefix;
-	char *imeisv;
 	char *imei;
 	char *ecclist_file;
 	gboolean pre_sim_done;
@@ -434,7 +433,6 @@ static void ril_modem_remove(struct ofono_modem *ofono)
 	grilio_queue_unref(md->q);
 	g_free(md->ecclist_file);
 	g_free(md->log_prefix);
-	g_free(md->imeisv);
 	g_free(md->imei);
 	g_free(md);
 }
@@ -462,7 +460,6 @@ struct ril_modem *ril_modem_create(GRilIoChannel *io, const char *log_prefix,
 		/* Copy config */
 		modem->config = *slot->config;
 		modem->imei = md->imei = g_strdup(slot->imei);
-		modem->imeisv = md->imeisv = g_strdup(slot->imeisv);
 		modem->log_prefix = log_prefix;
 		modem->ecclist_file =
 		md->ecclist_file = g_strdup(slot->ecclist_file);

ofono/drivers/ril/ril_network.c

@@ -1,7 +1,7 @@
 /*
  *  oFono - Open Source Telephony - RIL-based devices
  *
- *  Copyright (C) 2015-2016 Jolla Ltd.
+ *  Copyright (C) 2015-2017 Jolla Ltd.
  *
  *  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
@@ -420,13 +420,16 @@ static int ril_network_mode_to_rat(struct ril_network *self,
 	switch (mode) {
 	case OFONO_RADIO_ACCESS_MODE_ANY:
 	case OFONO_RADIO_ACCESS_MODE_LTE:
-		if (self->settings->enable_4g) {
+		if (self->settings->techs & OFONO_RADIO_ACCESS_MODE_LTE) {
 			return PREF_NET_TYPE_LTE_GSM_WCDMA;
 		}
 		/* no break */
 	default:
 	case OFONO_RADIO_ACCESS_MODE_UMTS:
-		return PREF_NET_TYPE_GSM_WCDMA_AUTO;
+		if (self->settings->techs & OFONO_RADIO_ACCESS_MODE_UMTS) {
+			return PREF_NET_TYPE_GSM_WCDMA_AUTO;
+		}
+		/* no break */
 	case OFONO_RADIO_ACCESS_MODE_GSM:
 		return PREF_NET_TYPE_GSM_ONLY;
 	}

ofono/drivers/ril/ril_plugin.c

@@ -41,7 +41,9 @@
 #include "ofono.h"
 #include "storage.h"
 
-#define RIL_DEVICE_IDENTITY_RETRIES_LAST 2
+#define OFONO_RADIO_ACCESS_MODE_ALL (OFONO_RADIO_ACCESS_MODE_GSM |\
+                                     OFONO_RADIO_ACCESS_MODE_UMTS |\
+                                     OFONO_RADIO_ACCESS_MODE_LTE)
 
 #define RADIO_GID                   1001
 #define RADIO_UID                   1001
@@ -51,7 +53,7 @@
 #define RILMODEM_DEFAULT_SOCK       "/dev/socket/rild"
 #define RILMODEM_DEFAULT_SOCK2      "/dev/socket/rild2"
 #define RILMODEM_DEFAULT_SUB        "SUB1"
-#define RILMODEM_DEFAULT_4G         TRUE /* 4G is on by default */
+#define RILMODEM_DEFAULT_TECHS      OFONO_RADIO_ACCESS_MODE_ALL
 #define RILMODEM_DEFAULT_SLOT       0xffffffff
 #define RILMODEM_DEFAULT_TIMEOUT    0 /* No timeout */
 #define RILMODEM_DEFAULT_SIM_FLAGS  RIL_SIM_CARD_V9_UICC_SUBSCRIPTION_WORKAROUND
@@ -72,7 +74,8 @@
 #define RILCONF_SLOT                "slot"
 #define RILCONF_SUB                 "sub"
 #define RILCONF_TIMEOUT             "timeout"
-#define RILCONF_4G                  "enable4G"
+#define RILCONF_4G                  "enable4G" /* Deprecated */
+#define RILCONF_TECHS               "technologies"
 #define RILCONF_UICC_WORKAROUND     "uiccWorkaround"
 #define RILCONF_ECCLIST_FILE        "ecclistFile"
 #define RILCONF_ALLOW_DATA_REQ      "allowDataReq"
@@ -132,7 +135,6 @@ struct ril_slot {
 	struct ril_slot_info pub;
 	char *path;
 	char *imei;
-	char *imeisv;
 	char *name;
 	char *sockpath;
 	char *sub;
@@ -160,7 +162,6 @@ struct ril_slot {
 	gulong io_event_id[IO_EVENT_COUNT];
 	gulong imei_req_id;
 	gulong sim_card_state_event_id;
-	gboolean received_sim_status;
 	guint trace_id;
 	guint dump_id;
 	guint retry_id;
@@ -179,7 +180,6 @@ static void ril_debug_grilio_notify(struct ofono_debug_desc *desc);
 static void ril_debug_mce_notify(struct ofono_debug_desc *desc);
 static void ril_plugin_debug_notify(struct ofono_debug_desc *desc);
 static void ril_plugin_retry_init_io(struct ril_slot *slot);
-static void ril_plugin_check_modem(struct ril_slot *slot);
 
 GLOG_MODULE_DEFINE("rilmodem");
 
@@ -252,7 +252,7 @@ static void ril_plugin_foreach_slot(struct ril_plugin_priv *plugin,
 
 static void ril_plugin_send_screen_state(struct ril_slot *slot)
 {
-	if (slot->io && slot->io->connected) {
+	if (slot->io) {
 		GRilIoRequest *req = grilio_request_sized_new(8);
 		grilio_request_append_int32(req, 1);    /* Number of params */
 		grilio_request_append_int32(req, slot->plugin->display_on);
@@ -358,7 +358,6 @@ static void ril_plugin_shutdown_slot(struct ril_slot *slot, gboolean kill_io)
 			ril_sim_card_unref(slot->sim_card);
 			slot->sim_card_state_event_id = 0;
 			slot->sim_card = NULL;
-			slot->received_sim_status = FALSE;
 		}
 
 		if (slot->io) {
@@ -490,23 +489,14 @@ static int ril_plugin_update_modem_paths(struct ril_plugin_priv *plugin)
 	if (plugin->default_data_imsi) {
 		slot = ril_plugin_find_slot_imsi(plugin->slots,
 				plugin->default_data_imsi);
-	} else if (!ril_plugin_multisim(plugin)) {
-		if (plugin->data_slot) {
-			/* Make sure that the slot is enabled and SIM is in */
-			slot = ril_plugin_find_slot_imsi(plugin->slots,
+	} else if (plugin->data_slot) {
+		/* Make sure that the slot is enabled and SIM is in */
+		slot = ril_plugin_find_slot_imsi(plugin->slots,
 				plugin->data_slot->modem ?
 				ofono_sim_get_imsi(plugin->data_slot->sim) :
 				NULL);
-		} else {
-			/* Check if anything is available */
-			slot = ril_plugin_find_slot_imsi(plugin->slots, NULL);
-		}
 	} else {
-		/*
-		 * Should we automatically select the default data sim
-		 * on a multisim phone that has only one sim inserted?
-		 */
-		slot = NULL;
+		slot = ril_plugin_find_slot_imsi(plugin->slots, NULL);
 	}
 
 	if (slot && !slot->radio->online) {
@@ -598,79 +588,14 @@ static void ril_plugin_update_ready(struct ril_plugin_priv *plugin)
 	}
 }
 
-static void ril_plugin_device_identity_cb(GRilIoChannel *io, int status,
-			const void *data, guint len, void *user_data)
-{
-	struct ril_slot *slot = user_data;
-	char *imei = NULL;
-	char *imeisv = NULL;
-
-	GASSERT(slot->imei_req_id);
-	slot->imei_req_id = 0;
-
-	if (status == RIL_E_SUCCESS) {
-		GRilIoParser rilp;
-		guint32 n;
-
-		/*
-		 * RIL_REQUEST_DEVICE_IDENTITY
-		 *
-		 * "response" is const char **
-		 * ((const char **)response)[0] is IMEI (for GSM)
-		 * ((const char **)response)[1] is IMEISV (for GSM)
-		 * ((const char **)response)[2] is ESN (for CDMA)
-		 * ((const char **)response)[3] is MEID (for CDMA)
-		 */
-		grilio_parser_init(&rilp, data, len);
-		if (grilio_parser_get_uint32(&rilp, &n) && n >= 2) {
-			imei = grilio_parser_get_utf8(&rilp);
-			imeisv = grilio_parser_get_utf8(&rilp);
-			DBG("%s %s", imei, imeisv);
-		} else {
-			DBG("parsing failure!");
-		}
-
-		/*
-		 * slot->imei should be either NULL (when we get connected
-		 * to rild the very first time) or match the already known
-		 * IMEI (if rild crashed and we have reconnected)
-		 */
-		if (slot->imei && imei && strcmp(slot->imei, imei)) {
-			ofono_warn("IMEI has changed \"%s\" -> \"%s\"",
-							slot->imei, imei);
-		}
-	} else {
-		ofono_error("Slot %u IMEI query error: %s", slot->config.slot,
-						ril_error_to_string(status));
-	}
-
-	if (slot->imei) {
-		/* We assume that IMEI never changes */
-		g_free(imei);
-	} else {
-		slot->pub.imei =
-		slot->imei = imei ? imei : g_strdup_printf("%d", slot->index);
-	}
-
-	if (slot->imeisv) {
-		g_free(imeisv);
-	} else {
-		slot->pub.imeisv =
-		slot->imeisv = (imeisv ? imeisv : g_strdup(""));
-	}
-
-	ril_plugin_check_modem(slot);
-	ril_plugin_update_ready(slot->plugin);
-}
-
 static void ril_plugin_sim_state_changed(struct ril_sim_card *card, void *data)
 {
 	struct ril_slot *slot = data;
 	struct ril_plugin_priv *plugin = slot->plugin;
-	const struct ril_sim_card_status *status = card->status;
 	gboolean present;
 
-	if (status && status->card_state == RIL_CARDSTATE_PRESENT) {
+	if (card && card->status &&
+			card->status->card_state == RIL_CARDSTATE_PRESENT) {
 		DBG("SIM found in slot %u", slot->config.slot);
 		present = TRUE;
 	} else {
@@ -678,36 +603,6 @@ static void ril_plugin_sim_state_changed(struct ril_sim_card *card, void *data)
 		present = FALSE;
 	}
 
-	if (status) {
-		if (!slot->received_sim_status && slot->imei_req_id) {
-			/*
-			 * We have received the SIM status but haven't yet
-			 * got IMEI from the modem. Some RILs behave this
-			 * way if the modem doesn't have IMEI initialized
-			 * yet. Cancel the current request (with unlimited
-			 * number of retries) and give a few more tries
-			 * (this time, limited number).
-			 *
-			 * Some RILs fail RIL_REQUEST_DEVICE_IDENTITY until
-			 * the modem hasn't been properly initialized.
-			 */
-			GRilIoRequest* req = grilio_request_new();
-
-			DBG("Giving slot %u last chance", slot->config.slot);
-			grilio_request_set_retry(req, RIL_RETRY_MS,
-					 RIL_DEVICE_IDENTITY_RETRIES_LAST);
-			grilio_channel_cancel_request(slot->io,
-						slot->imei_req_id, FALSE);
-			slot->imei_req_id =
-				grilio_channel_send_request_full(slot->io,
-					req, RIL_REQUEST_DEVICE_IDENTITY,
-					ril_plugin_device_identity_cb,
-					NULL, slot);
-			grilio_request_unref(req);
-		}
-		slot->received_sim_status = TRUE;
-	}
-
 	if (slot->pub.sim_present != present) {
 		slot->pub.sim_present = present;
 		ril_plugin_dbus_signal_sim(plugin->dbus, slot->index, present);
@@ -1022,6 +917,42 @@ static void ril_plugin_check_modem(struct ril_slot *slot)
 	}
 }
 
+
+static void ril_plugin_imei_cb(GRilIoChannel *io, int status,
+			const void *data, guint len, void *user_data)
+{
+	struct ril_slot *slot = user_data;
+	char *imei = NULL;
+
+	GASSERT(slot->imei_req_id);
+	slot->imei_req_id = 0;
+
+	if (status == RIL_E_SUCCESS) {
+		GRilIoParser rilp;
+
+		grilio_parser_init(&rilp, data, len);
+		imei = grilio_parser_get_utf8(&rilp);
+
+		DBG("%s", imei);
+
+		/*
+		 * slot->imei should be either NULL (when we get connected
+		 * to rild the very first time) or match the already known
+		 * IMEI (if rild crashed and we have reconnected)
+		 */
+		GASSERT(!slot->imei || !g_strcmp0(slot->imei, imei));
+	} else {
+		ofono_error("Slot %u IMEI query error: %s", slot->config.slot,
+						ril_error_to_string(status));
+	}
+
+	g_free(slot->imei);
+	slot->pub.imei = slot->imei = (imei ? imei : g_strdup("ERROR"));
+
+	ril_plugin_check_modem(slot);
+	ril_plugin_update_ready(slot->plugin);
+}
+
 /*
  * It seems to be necessary to kick (with RIL_REQUEST_RADIO_POWER) the
  * modems with power on after one of the modems has been powered off.
@@ -1057,19 +988,17 @@ static void ril_plugin_slot_connected(struct ril_slot *slot)
 	GASSERT(!slot->io_event_id[IO_EVENT_CONNECTED]);
 
 	/*
-	 * Modem will be registered after RIL_REQUEST_DEVICE_IDENTITY
-	 * successfully completes. By the time ofono starts, rild may
-	 * not be completely functional. Waiting until it responds to
-	 * RIL_REQUEST_DEVICE_IDENTITY (and retrying the request on
-	 * failure) gives rild time to finish whatever it's doing during
-	 * initialization.
+	 * Modem will be registered after RIL_REQUEST_GET_IMEI successfully
+	 * completes. By the time ofono starts, rild may not be completely
+	 * functional. Waiting until it responds to RIL_REQUEST_GET_IMEI
+	 * (and retrying the request on failure) gives rild time to finish
+	 * whatever it's doing during initialization.
 	 */
 	GASSERT(!slot->imei_req_id);
 	req = grilio_request_new();
 	grilio_request_set_retry(req, RIL_RETRY_MS, -1);
-	slot->imei_req_id = grilio_channel_send_request_full(slot->io,
-				req, RIL_REQUEST_DEVICE_IDENTITY,
-				ril_plugin_device_identity_cb, NULL, slot);
+	slot->imei_req_id = grilio_channel_send_request_full(slot->io, req,
+			RIL_REQUEST_GET_IMEI, ril_plugin_imei_cb, NULL, slot);
 	grilio_request_unref(req);
 
 	GASSERT(!slot->radio);
@@ -1086,10 +1015,6 @@ static void ril_plugin_slot_connected(struct ril_slot *slot)
 							slot->sim_flags);
 	slot->sim_card_state_event_id = ril_sim_card_add_state_changed_handler(
 			slot->sim_card, ril_plugin_sim_state_changed, slot);
-	/* ril_sim_card is expected to perform RIL_REQUEST_GET_SIM_STATUS
-	 * asynchronously and report back when request has completed: */
-	GASSERT(!slot->sim_card->status);
-	GASSERT(!slot->received_sim_status);
 
 	GASSERT(!slot->network);
 	slot->network = ril_network_new(slot->io, log_prefix, slot->radio,
@@ -1189,7 +1114,7 @@ static struct ril_slot *ril_plugin_slot_new(const char *sockpath,
 	slot->path = g_strdup(path);
 	slot->name = g_strdup(name);
 	slot->config.slot = slot_index;
-	slot->config.enable_4g = RILMODEM_DEFAULT_4G;
+	slot->config.techs = RILMODEM_DEFAULT_TECHS;
 	slot->config.empty_pin_query = RILMODEM_DEFAULT_EMPTY_PIN_QUERY;
 	slot->timeout = RILMODEM_DEFAULT_TIMEOUT;
 	slot->sim_flags = RILMODEM_DEFAULT_SIM_FLAGS;
@@ -1239,7 +1164,8 @@ static struct ril_slot *ril_plugin_parse_config_group(GKeyFile *file,
 	char *sock = g_key_file_get_string(file, group, RILCONF_SOCKET, NULL);
 	if (sock) {
 		int value;
-		char* strval;
+		char *strval;
+		char **strv;
 		char *sub = ril_config_get_string(file, group, RILCONF_SUB);
 
 		slot = ril_plugin_slot_new(NULL, NULL, NULL,
@@ -1267,9 +1193,52 @@ static struct ril_slot *ril_plugin_parse_config_group(GKeyFile *file,
 			DBG("%s: timeout %d", group, slot->timeout);
 		}
 
-		ril_config_get_boolean(file, group, RILCONF_4G,
-						&slot->config.enable_4g);
-		DBG("%s: 4G %s", group, slot->config.enable_4g ? "on" : "off");
+		strv = ril_config_get_strings(file, group, RILCONF_TECHS, ',');
+		if (strv) {
+			char **p;
+
+			slot->config.techs = 0;
+			for (p = strv; *p; p++) {
+				const char *s = *p;
+				enum ofono_radio_access_mode m;
+
+				if (!s[0]) {
+					continue;
+				}
+
+				if (!strcmp(s, "all")) {
+					slot->config.techs =
+						OFONO_RADIO_ACCESS_MODE_ALL;
+					break;
+				}
+
+				if (!ofono_radio_access_mode_from_string(s,
+									&m)) {
+					ofono_warn("Unknown technology %s "
+						"in [%s] section of %s", s,
+						group, RILMODEM_CONF_FILE);
+					continue;
+				}
+
+				if (m == OFONO_RADIO_ACCESS_MODE_ANY) {
+					slot->config.techs =
+						OFONO_RADIO_ACCESS_MODE_ALL;
+					break;
+				}
+
+				slot->config.techs |= m;
+			}
+			g_strfreev(strv);
+		}
+
+		/* "enable4G" is deprecated */
+		value = slot->config.techs;
+		if (ril_config_get_flag(file, group, RILCONF_4G,
+				OFONO_RADIO_ACCESS_MODE_LTE, &value)) {
+			slot->config.techs = value;
+		}
+
+		DBG("%s: technologies 0x%02x", group, slot->config.techs);
 
 		if (ril_config_get_boolean(file, group, RILCONF_EMPTY_PIN_QUERY,
 					&slot->config.empty_pin_query)) {
@@ -1398,7 +1367,6 @@ static void ril_plugin_delete_slot(struct ril_slot *slot)
 	g_hash_table_destroy(slot->pub.errors);
 	g_free(slot->path);
 	g_free(slot->imei);
-	g_free(slot->imeisv);
 	g_free(slot->name);
 	g_free(slot->sockpath);
 	g_free(slot->sub);

ofono/drivers/ril/ril_plugin.h

@@ -48,7 +48,6 @@ typedef struct ril_slot_info const *ril_slot_info_ptr;
 struct ril_slot_info {
 	const char *path;
 	const char *imei;
-	const char *imeisv;
 	const char *ecclist_file;
 	gboolean enabled;
 	gboolean sim_present;
@@ -70,7 +69,6 @@ struct ril_plugin {
 struct ril_modem {
 	GRilIoChannel *io;
 	const char *imei;
-	const char *imeisv;
 	const char *log_prefix;
 	const char *ecclist_file;
 	struct ofono_modem *ofono;

ofono/drivers/ril/ril_plugin_dbus.c

@@ -46,7 +46,7 @@ struct ril_plugin_dbus {
 
 #define RIL_DBUS_PATH               "/"
 #define RIL_DBUS_INTERFACE          "org.nemomobile.ofono.ModemManager"
-#define RIL_DBUS_INTERFACE_VERSION  (7)
+#define RIL_DBUS_INTERFACE_VERSION  (6)
 
 #define RIL_DBUS_SIGNAL_ENABLED_MODEMS_CHANGED      "EnabledModemsChanged"
 #define RIL_DBUS_SIGNAL_PRESENT_SIMS_CHANGED        "PresentSimsChanged"
@@ -77,11 +77,6 @@ static const char *ril_plugin_dbus_imei(const struct ril_slot_info *slot)
 	return slot->imei;
 }
 
-static const char *ril_plugin_dbus_imeisv(const struct ril_slot_info *slot)
-{
-	return slot->imeisv;
-}
-
 static void ril_plugin_dbus_append_path_array(DBusMessageIter *it,
 	struct ril_plugin_dbus *dbus, ril_plugin_dbus_slot_select_fn selector)
 {
@@ -445,13 +440,6 @@ static void ril_plugin_dbus_append_all6(DBusMessageIter *it,
 	ril_plugin_dbus_append_modem_errors(it, dbus);
 }
 
-static void ril_plugin_dbus_append_all7(DBusMessageIter *it,
-						struct ril_plugin_dbus *dbus)
-{
-	ril_plugin_dbus_append_all6(it, dbus);
-	ril_plugin_dbus_append_string_array(it, dbus, ril_plugin_dbus_imeisv);
-}
-
 static DBusMessage *ril_plugin_dbus_get_all(DBusConnection *conn,
 						DBusMessage *msg, void *data)
 {
@@ -494,13 +482,6 @@ static DBusMessage *ril_plugin_dbus_get_all6(DBusConnection *conn,
 						ril_plugin_dbus_append_all6);
 }
 
-static DBusMessage *ril_plugin_dbus_get_all7(DBusConnection *conn,
-						DBusMessage *msg, void *data)
-{
-	return ril_plugin_dbus_imei_reply(msg, (struct ril_plugin_dbus *)data,
-						ril_plugin_dbus_append_all7);
-}
-
 static DBusMessage *ril_plugin_dbus_get_interface_version(DBusConnection *conn,
 						DBusMessage *msg, void *data)
 {
@@ -548,19 +529,6 @@ static DBusMessage *ril_plugin_dbus_get_imei(DBusConnection *conn,
 					ril_plugin_dbus_append_imei_array);
 }
 
-static void ril_plugin_dbus_append_imeisv_array(DBusMessageIter *it,
-						struct ril_plugin_dbus *dbus)
-{
-	ril_plugin_dbus_append_string_array(it, dbus, ril_plugin_dbus_imeisv);
-}
-
-static DBusMessage *ril_plugin_dbus_get_imeisv(DBusConnection *conn,
-						DBusMessage *msg, void *data)
-{
-	return ril_plugin_dbus_imei_reply(msg, (struct ril_plugin_dbus *)data,
-					ril_plugin_dbus_append_imeisv_array);
-}
-
 static DBusMessage *ril_plugin_dbus_reply_with_string(DBusMessage *msg,
 							const char *str)
 {
@@ -822,7 +790,6 @@ static DBusMessage *ril_plugin_dbus_set_mms_sim(DBusConnection *conn,
 #define RIL_DBUS_READY_ARG               {"ready" , "b"}
 #define RIL_DBUS_MODEM_ERRORS_ARG        {"errors" , \
                                           "aa(" RIL_DBUS_ERROR_SIGNATURE ")"}
-#define RIL_DBUS_IMEISV_ARG              {"imeisv" , "as"}
 #define RIL_DBUS_GET_ALL_ARGS \
 	RIL_DBUS_VERSION_ARG, \
 	RIL_DBUS_AVAILABLE_MODEMS_ARG, \
@@ -847,9 +814,6 @@ static DBusMessage *ril_plugin_dbus_set_mms_sim(DBusConnection *conn,
 #define RIL_DBUS_GET_ALL6_ARGS \
 	RIL_DBUS_GET_ALL5_ARGS, \
 	RIL_DBUS_MODEM_ERRORS_ARG
-#define RIL_DBUS_GET_ALL7_ARGS \
-	RIL_DBUS_GET_ALL6_ARGS, \
-	RIL_DBUS_IMEISV_ARG
 static const GDBusMethodTable ril_plugin_dbus_methods[] = {
 	{ GDBUS_METHOD("GetAll",
 			NULL, GDBUS_ARGS(RIL_DBUS_GET_ALL_ARGS),
@@ -869,9 +833,6 @@ static const GDBusMethodTable ril_plugin_dbus_methods[] = {
 	{ GDBUS_ASYNC_METHOD("GetAll6",
 			NULL, GDBUS_ARGS(RIL_DBUS_GET_ALL6_ARGS),
 			ril_plugin_dbus_get_all6) },
-	{ GDBUS_ASYNC_METHOD("GetAll7",
-			NULL, GDBUS_ARGS(RIL_DBUS_GET_ALL7_ARGS),
-			ril_plugin_dbus_get_all7) },
 	{ GDBUS_METHOD("GetInterfaceVersion",
 			NULL, GDBUS_ARGS(RIL_DBUS_VERSION_ARG),
 			ril_plugin_dbus_get_interface_version) },
@@ -887,9 +848,6 @@ static const GDBusMethodTable ril_plugin_dbus_methods[] = {
 	{ GDBUS_ASYNC_METHOD("GetIMEI",
 			NULL, GDBUS_ARGS(RIL_DBUS_IMEI_ARG),
 			ril_plugin_dbus_get_imei) },
-	{ GDBUS_ASYNC_METHOD("GetIMEISV",
-			NULL, GDBUS_ARGS(RIL_DBUS_IMEISV_ARG),
-			ril_plugin_dbus_get_imeisv) },
 	{ GDBUS_METHOD("GetDefaultDataSim",
 			NULL, GDBUS_ARGS(RIL_DBUS_DEFAULT_DATA_SIM_ARG),
 			ril_plugin_dbus_get_default_data_sim) },

ofono/drivers/ril/ril_radio_settings.c

@@ -1,7 +1,7 @@
 /*
  *  oFono - Open Source Telephony - RIL-based devices
  *
- *  Copyright (C) 2015-2016 Jolla Ltd.
+ *  Copyright (C) 2015-2017 Jolla Ltd.
  *
  *  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
@@ -113,15 +113,11 @@ static gboolean ril_radio_settings_query_available_rats_cb(gpointer data)
 	struct ofono_error error;
 	struct ril_radio_settings_cbd *cbd = data;
 	struct ril_radio_settings *rsd = cbd->rsd;
-	guint rats = OFONO_RADIO_ACCESS_MODE_GSM | OFONO_RADIO_ACCESS_MODE_UMTS;
 
-	if (cbd->rsd->settings->enable_4g) {
-		rats |= OFONO_RADIO_ACCESS_MODE_LTE;
-	}
-
-	GASSERT(cbd->rsd->source_id);
+	GASSERT(rsd->source_id);
 	rsd->source_id = 0;
-	cbd->cb.available_rats(ril_error_ok(&error), rats, cbd->data);
+	cbd->cb.available_rats(ril_error_ok(&error), rsd->settings->techs,
+								cbd->data);
 	return G_SOURCE_REMOVE;
 }
 
@@ -132,8 +128,8 @@ static void ril_radio_settings_query_available_rats(
 	struct ril_radio_settings *rsd = ril_radio_settings_get_data(rs);
 
 	DBG_(rsd, "");
-	ril_radio_settings_later(rsd, ril_radio_settings_query_available_rats_cb,
-								cb, data);
+	ril_radio_settings_later(rsd,
+			ril_radio_settings_query_available_rats_cb, cb, data);
 }
 
 static gboolean ril_radio_settings_register(gpointer user_data)

ofono/drivers/ril/ril_sim.c

@@ -1,7 +1,7 @@
 /*
  *  oFono - Open Source Telephony - RIL-based devices
  *
- *  Copyright (C) 2015-2017 Jolla Ltd.
+ *  Copyright (C) 2015-2016 Jolla Ltd.
  *
  *  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
@@ -85,7 +85,6 @@ struct ril_sim {
 	ofono_sim_passwd_cb_t query_passwd_state_cb;
 	void *query_passwd_state_cb_data;
 	guint query_passwd_state_timeout_id;
-	gulong query_passwd_state_sim_status_refresh_id;
 };
 
 struct ril_sim_io_response {
@@ -692,12 +691,6 @@ static void ril_sim_finish_passwd_state_query(struct ril_sim *sd,
 		sd->query_passwd_state_timeout_id = 0;
 	}
 
-	if (sd->query_passwd_state_sim_status_refresh_id) {
-		ril_sim_card_remove_handler(sd->card,
-			sd->query_passwd_state_sim_status_refresh_id);
-		sd->query_passwd_state_sim_status_refresh_id = 0;
-	}
-
 	if (sd->query_passwd_state_cb) {
 		ofono_sim_passwd_cb_t cb = sd->query_passwd_state_cb;
 		void *data = sd->query_passwd_state_cb_data;
@@ -891,15 +884,6 @@ static void ril_sim_query_pin_retries(struct ofono_sim *sim,
 	cb(ril_error_ok(&error), sd->retries, data);
 }
 
-static void ril_sim_query_passwd_state_complete_cb(struct ril_sim_card *sc,
-							void *user_data)
-{
-	struct ril_sim *sd = user_data;
-
-	GASSERT(sd->query_passwd_state_sim_status_refresh_id);
-	ril_sim_finish_passwd_state_query(sd, ril_sim_passwd_state(sd));
-}
-
 static gboolean ril_sim_query_passwd_state_timeout_cb(gpointer user_data)
 {
 	struct ril_sim *sd = user_data;
@@ -915,41 +899,29 @@ static void ril_sim_query_passwd_state(struct ofono_sim *sim,
 					ofono_sim_passwd_cb_t cb, void *data)
 {
 	struct ril_sim *sd = ril_sim_get_data(sim);
+	enum ofono_sim_password_type passwd_state = ril_sim_passwd_state(sd);
+	struct ofono_error error;
 
 	if (sd->query_passwd_state_timeout_id) {
 		g_source_remove(sd->query_passwd_state_timeout_id);
 		sd->query_passwd_state_timeout_id = 0;
 	}
 
-	if (!sd->query_passwd_state_sim_status_refresh_id) {
-		ril_sim_card_remove_handler(sd->card,
-			sd->query_passwd_state_sim_status_refresh_id);
-		sd->query_passwd_state_sim_status_refresh_id = 0;
-	}
-
-	/* Always request fresh status, just in case. */
-	ril_sim_card_request_status(sd->card);
-	sd->query_passwd_state_cb = cb;
-	sd->query_passwd_state_cb_data = data;
-
-	if (ril_sim_passwd_state(sd) != OFONO_SIM_PASSWORD_INVALID) {
-		/* Just wait for GET_SIM_STATUS completion */
-		DBG_(sd, "waiting for SIM status query to complete");
-		sd->query_passwd_state_sim_status_refresh_id =
-			ril_sim_card_add_status_received_handler(sd->card,
-				ril_sim_query_passwd_state_complete_cb, sd);
+	if (passwd_state != OFONO_SIM_PASSWORD_INVALID) {
+		DBG_(sd, "%d", passwd_state);
+		sd->query_passwd_state_cb = NULL;
+		sd->query_passwd_state_cb_data = NULL;
+		sd->ofono_passwd_state = passwd_state;
+		cb(ril_error_ok(&error), passwd_state, data);
 	} else {
 		/* Wait for the state to change */
 		DBG_(sd, "waiting for the SIM state to change");
+		sd->query_passwd_state_cb = cb;
+		sd->query_passwd_state_cb_data = data;
+		sd->query_passwd_state_timeout_id =
+			g_timeout_add_seconds(SIM_STATE_CHANGE_TIMEOUT_SECS,
+				ril_sim_query_passwd_state_timeout_cb, sd);
 	}
-
-	/*
-	 * We still need to complete the request somehow, even if
-	 * GET_STATUS never completes or SIM status never changes.
-	 */
-	sd->query_passwd_state_timeout_id =
-		g_timeout_add_seconds(SIM_STATE_CHANGE_TIMEOUT_SECS,
-			ril_sim_query_passwd_state_timeout_cb, sd);
 }
 
 static gboolean ril_sim_pin_change_state_timeout_cb(gpointer user_data)
@@ -1308,11 +1280,6 @@ static void ril_sim_remove(struct ofono_sim *sim)
 		g_source_remove(sd->query_passwd_state_timeout_id);
 	}
 
-	if (sd->query_passwd_state_sim_status_refresh_id) {
-		ril_sim_card_remove_handler(sd->card,
-			sd->query_passwd_state_sim_status_refresh_id);
-	}
-
 	ril_sim_card_remove_handler(sd->card, sd->card_status_id);
 	ril_sim_card_unref(sd->card);
 

ofono/drivers/ril/ril_sim_card.c

@@ -1,7 +1,7 @@
 /*
  *  oFono - Open Source Telephony - RIL-based devices
  *
- *  Copyright (C) 2015-2017 Jolla Ltd.
+ *  Copyright (C) 2015-2016 Jolla Ltd.
  *
  *  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
@@ -64,6 +64,8 @@ G_DEFINE_TYPE(RilSimCard, ril_sim_card, G_TYPE_OBJECT)
 #define RIL_SIMCARD_STATE_CHANGED  (0x01)
 #define RIL_SIMCARD_STATUS_CHANGED (0x02)
 
+static void ril_sim_card_request_status(struct ril_sim_card *self);
+
 static gboolean ril_sim_card_app_equal(const struct ril_sim_card_app *a1,
 					const struct ril_sim_card_app *a2)
 {
@@ -363,7 +365,7 @@ static void ril_sim_card_status_cb(GRilIoChannel *io, int ril_status,
 	}
 }
 
-void ril_sim_card_request_status(struct ril_sim_card *self)
+static void ril_sim_card_request_status(struct ril_sim_card *self)
 {
 	struct ril_sim_card_priv *priv = self->priv;
 

ofono/drivers/ril/ril_sim_card.h

@@ -1,7 +1,7 @@
 /*
  *  oFono - Open Source Telephony - RIL-based devices
  *
- *  Copyright (C) 2015-2017 Jolla Ltd.
+ *  Copyright (C) 2015-2016 Jolla Ltd.
  *
  *  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
@@ -55,7 +55,6 @@ typedef void (*ril_sim_card_cb_t)(struct ril_sim_card *sc, void *arg);
 struct ril_sim_card *ril_sim_card_new(GRilIoChannel *io, guint slot, int flags);
 struct ril_sim_card *ril_sim_card_ref(struct ril_sim_card *sc);
 void ril_sim_card_unref(struct ril_sim_card *sc);
-void ril_sim_card_request_status(struct ril_sim_card *self);
 gboolean ril_sim_card_ready(struct ril_sim_card *sc);
 gulong ril_sim_card_add_status_received_handler(struct ril_sim_card *sc,
 					ril_sim_card_cb_t cb, void *arg);

ofono/drivers/ril/ril_sim_settings.c

@@ -1,7 +1,7 @@
 /*
  *  oFono - Open Source Telephony - RIL-based devices
  *
- *  Copyright (C) 2016 Jolla Ltd.
+ *  Copyright (C) 2016-2017 Jolla Ltd.
  *
  *  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
@@ -26,8 +26,12 @@
 #define RIL_SIM_STORE_GROUP             "Settings"
 #define RIL_SIM_STORE_PREF_MODE         "TechnologyPreference"
 
-#define RIL_SIM_STORE_PREF_MODE_DEFAULT(self) ((self)->enable_4g ? \
-	OFONO_RADIO_ACCESS_MODE_LTE : OFONO_RADIO_ACCESS_MODE_UMTS)
+#define RIL_SIM_STORE_PREF_MODE_DEFAULT(self)           (\
+	((self)->techs & OFONO_RADIO_ACCESS_MODE_LTE) ?  \
+	OFONO_RADIO_ACCESS_MODE_LTE :                    \
+	((self)->techs & OFONO_RADIO_ACCESS_MODE_UMTS) ? \
+	OFONO_RADIO_ACCESS_MODE_UMTS :                   \
+	OFONO_RADIO_ACCESS_MODE_GSM)
 
 typedef GObjectClass RilSimSettingsClass;
 typedef struct ril_sim_settings RilSimSettings;
@@ -84,8 +88,7 @@ static void ril_sim_settings_reload(struct ril_sim_settings *self)
 		mode_str = g_key_file_get_string(priv->storage,
 			RIL_SIM_STORE_GROUP, RIL_SIM_STORE_PREF_MODE, NULL);
 		if (ofono_radio_access_mode_from_string(mode_str, &mode)) {
-			if (!self->enable_4g &&
-					mode == OFONO_RADIO_ACCESS_MODE_LTE) {
+			if (!(self->techs & mode)) {
 				mode = OFONO_RADIO_ACCESS_MODE_ANY;
 			}
 		} else {
@@ -263,7 +266,7 @@ void ril_sim_settings_remove_handlers(struct ril_sim_settings *self,
 struct ril_sim_settings *ril_sim_settings_new(const struct ril_slot_config *sc)
 {
 	struct ril_sim_settings *self = g_object_new(RIL_SIM_SETTINGS_TYPE, 0);
-	self->enable_4g = sc->enable_4g;
+	self->techs = sc->techs;
 	self->slot = sc->slot;
 	self->pref_mode = RIL_SIM_STORE_PREF_MODE_DEFAULT(self);
 	return self;

ofono/drivers/ril/ril_sim_settings.h

@@ -1,7 +1,7 @@
 /*
  *  oFono - Open Source Telephony - RIL-based devices
  *
- *  Copyright (C) 2016 Jolla Ltd.
+ *  Copyright (C) 2016-2017 Jolla Ltd.
  *
  *  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
@@ -18,16 +18,14 @@
 
 #include "ril_types.h"
 
-#include <ofono/radio-settings.h>
-
 struct ril_sim_settings_priv;
 
 struct ril_sim_settings {
 	GObject object;
 	struct ril_sim_settings_priv *priv;
-	gboolean enable_4g;
 	guint slot;
 	const char *imsi;
+	enum ofono_radio_access_mode techs;
 	enum ofono_radio_access_mode pref_mode;
 };
 

ofono/drivers/ril/ril_subscription.conf

@@ -67,9 +67,15 @@ socket=/dev/socket/rild
 #
 #timeout=0
 
-# Setting this one to false would disable 4G technology selection.
+# Comma-separated list of radio technologies supported by the modem.
+# Valid technologies are "gsm", "umts" and "lte". The special value
+# "all" means that all technologies are supported.
 #
-# By default 4G is enabled
+# The default is all
+#
+#technologies=all
+
+# This one is deprecated, use the technologies entry instead (above).
 #
 #enable4G=true
 

ofono/drivers/ril/ril_types.h

@@ -23,11 +23,12 @@
 #include <grilio_types.h>
 #include <gutil_macros.h>
 
-#include <ofono/types.h>
-
 struct ofono_modem;
 struct ofono_sim;
 
+#include <ofono/types.h>
+#include <ofono/radio-settings.h>
+
 #include <stdio.h>
 #include <errno.h>
 #include <stdlib.h>
@@ -51,7 +52,7 @@ struct ril_cell_info;
 
 struct ril_slot_config {
 	guint slot;
-	gboolean enable_4g;
+	enum ofono_radio_access_mode techs;
 	gboolean empty_pin_query;
 	GUtilInts *local_hangup_reasons;
 	GUtilInts *remote_hangup_reasons;

ofono/drivers/ubloxmodem/gprs-context.c

@@ -261,7 +261,6 @@ static void ublox_send_uauthreq(struct ofono_gprs_context *gc,
 	case OFONO_GPRS_AUTH_METHOD_PAP:
 		auth = 1;
 		break;
-	case OFONO_GPRS_AUTH_METHOD_ANY:
 	case OFONO_GPRS_AUTH_METHOD_CHAP:
 		auth = 2;
 		break;

ofono/include/gprs-context.h

@@ -49,9 +49,7 @@ enum ofono_gprs_context_type {
 };
 
 enum ofono_gprs_auth_method {
-	OFONO_GPRS_AUTH_METHOD_ANY = 0,
-	OFONO_GPRS_AUTH_METHOD_NONE,
-	OFONO_GPRS_AUTH_METHOD_CHAP,
+	OFONO_GPRS_AUTH_METHOD_CHAP = 0,
 	OFONO_GPRS_AUTH_METHOD_PAP,
 };
 

ofono/plugins/mbpi.c

@@ -53,9 +53,6 @@ const char *mbpi_database = MBPI_DATABASE;
 enum ofono_gprs_proto mbpi_default_internet_proto = OFONO_GPRS_PROTO_IPV4V6;
 enum ofono_gprs_proto mbpi_default_mms_proto = OFONO_GPRS_PROTO_IP;
 enum ofono_gprs_proto mbpi_default_proto = OFONO_GPRS_PROTO_IP;
-enum ofono_gprs_auth_method mbpi_default_auth_method = OFONO_GPRS_AUTH_METHOD_ANY;
-
-#define OFONO_GPRS_AUTH_METHOD_UNSPECIFIED ((enum ofono_gprs_auth_method)(-1))
 
 #define _(x) case x: return (#x)
 
@@ -169,10 +166,6 @@ static void authentication_start(GMarkupParseContext *context,
 		*auth_method = OFONO_GPRS_AUTH_METHOD_CHAP;
 	else if (strcmp(text, "pap") == 0)
 		*auth_method = OFONO_GPRS_AUTH_METHOD_PAP;
-	else if (strcmp(text, "any") == 0)
-		*auth_method = OFONO_GPRS_AUTH_METHOD_ANY;
-	else if (strcmp(text, "none") == 0)
-		*auth_method = OFONO_GPRS_AUTH_METHOD_NONE;
 	else
 		mbpi_g_set_error(context, error, G_MARKUP_ERROR,
 					G_MARKUP_ERROR_UNKNOWN_ATTRIBUTE,
@@ -351,7 +344,7 @@ static void apn_handler(GMarkupParseContext *context, struct gsm_data *gsm,
 	ap->apn = g_strdup(apn);
 	ap->type = OFONO_GPRS_CONTEXT_TYPE_INTERNET;
 	ap->proto = mbpi_default_proto;
-	ap->auth_method = OFONO_GPRS_AUTH_METHOD_UNSPECIFIED;
+	ap->auth_method = OFONO_GPRS_AUTH_METHOD_CHAP;
 
 	g_markup_parse_context_push(context, &apn_parser, ap);
 }
@@ -421,17 +414,6 @@ static void gsm_end(GMarkupParseContext *context, const gchar *element_name,
 	if (ap == NULL)
 		return;
 
-	/* Fix the authentication method if none was specified */
-	if (ap->auth_method == OFONO_GPRS_AUTH_METHOD_UNSPECIFIED) {
-		if ((!ap->username || !ap->username[0]) &&
-				(!ap->password || !ap->password[0])) {
-			/* No username or password => no authentication */
-			ap->auth_method = OFONO_GPRS_AUTH_METHOD_NONE;
-		} else {
-			ap->auth_method = mbpi_default_auth_method;
-		}
-	}
-
 	if (gsm->allow_duplicates == FALSE) {
 		GSList *l;
 

ofono/plugins/mbpi.h

@@ -23,7 +23,6 @@ extern const char *mbpi_database;
 extern enum ofono_gprs_proto mbpi_default_internet_proto;
 extern enum ofono_gprs_proto mbpi_default_mms_proto;
 extern enum ofono_gprs_proto mbpi_default_proto;
-extern enum ofono_gprs_auth_method mbpi_default_auth_method;
 
 const char *mbpi_ap_type(enum ofono_gprs_context_type type);
 

ofono/plugins/provision.c

@@ -3,6 +3,7 @@
  *  oFono - Open Source Telephony
  *
  *  Copyright (C) 2008-2011  Intel Corporation. All rights reserved.
+ *  Copyright (C) 2013-2016  Jolla Ltd.
  *
  *  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
@@ -23,6 +24,7 @@
 #include <config.h>
 #endif
 
+#define _GNU_SOURCE
 #include <errno.h>
 #include <string.h>
 
@@ -35,9 +37,114 @@
 #include <ofono/modem.h>
 #include <ofono/gprs-provision.h>
 
+#include "provision.h"
 #include "mbpi.h"
 
-static int provision_get_settings(const char *mcc, const char *mnc,
+struct provision_ap_defaults {
+	enum ofono_gprs_context_type type;
+	const char *name;
+	const char *apn;
+};
+
+static gboolean provision_match_name(const struct ofono_gprs_provision_data *ap,
+							const char* spn)
+{
+	return (ap->provider_name && strcasestr(ap->provider_name, spn)) ||
+		(ap->name && strcasestr(ap->name, spn)) ||
+		(ap->apn && strcasestr(ap->apn, spn));
+}
+
+static void provision_free_ap(gpointer data)
+{
+	mbpi_ap_free(data);
+}
+
+static gint provision_compare_ap(gconstpointer a, gconstpointer b, gpointer data)
+{
+	const struct ofono_gprs_provision_data *ap1 = a;
+	const struct ofono_gprs_provision_data *ap2 = b;
+	const char* spn = data;
+
+	if (spn) {
+		const gboolean match1 = provision_match_name(ap1, spn);
+		const gboolean match2 = provision_match_name(ap2, spn);
+		if (match1 && !match2) {
+			return -1;
+		} else if (match2 && !match1) {
+			return 1;
+		}
+	}
+
+	if (ap1->provider_primary && !ap2->provider_primary) {
+		return -1;
+	} else if (ap2->provider_primary && !ap1->provider_primary) {
+		return 1;
+	} else {
+		return 0;
+	}
+}
+
+/* Picks best ap, deletes the rest. Creates one if necessary */
+static GSList *provision_pick_best_ap(GSList *list, const char* spn,
+	const enum ofono_gprs_proto default_proto,
+	const struct provision_ap_defaults *defaults)
+{
+	/* Sort the list */
+	list = g_slist_sort_with_data(list, provision_compare_ap, (void*)spn);
+	if (list) {
+		/* Pick the best one, delete the rest */
+		GSList *best = list;
+		g_slist_free_full(g_slist_remove_link(list, best),
+							provision_free_ap);
+		return best;
+	} else {
+		/* or create one from the default data */
+		struct ofono_gprs_provision_data *ap =
+			g_new0(struct ofono_gprs_provision_data, 1);
+
+		ap->proto = default_proto;
+		ap->type = defaults->type;
+		ap->name = g_strdup(defaults->name);
+		ap->apn = g_strdup(defaults->apn);
+		return g_slist_append(NULL, ap);
+	}
+}
+
+/* Returns the list containing exactly one INTERNET and one MMS access point */
+static GSList *provision_normalize_apn_list(GSList *apns, const char* spn)
+{
+	static const struct provision_ap_defaults internet_defaults =
+		{ OFONO_GPRS_CONTEXT_TYPE_INTERNET, "Internet", "internet" };
+	static const struct provision_ap_defaults mms_defaults =
+		{ OFONO_GPRS_CONTEXT_TYPE_MMS, "MMS", "mms" };
+
+	GSList *internet_apns = NULL;
+	GSList *mms_apns = NULL;
+
+	/* Split internet and mms apns, delete all others */
+	while (apns) {
+		GSList *link = apns;
+		struct ofono_gprs_provision_data *ap = link->data;
+
+		apns = g_slist_remove_link(apns, link);
+		if (ap->type == OFONO_GPRS_CONTEXT_TYPE_INTERNET) {
+			internet_apns = g_slist_concat(internet_apns, link);
+		} else if (ap->type == OFONO_GPRS_CONTEXT_TYPE_MMS) {
+			mms_apns = g_slist_concat(mms_apns, link);
+		} else {
+			g_slist_free_full(link, provision_free_ap);
+		}
+	}
+
+	/* Pick the best ap of each type and concatenate them */
+	return g_slist_concat(
+		provision_pick_best_ap(internet_apns, spn,
+			mbpi_default_internet_proto, &internet_defaults),
+		provision_pick_best_ap(mms_apns, spn,
+			mbpi_default_mms_proto, &mms_defaults));
+}
+
+int provision_get_settings(const char *mcc, const char *mnc,
 				const char *spn,
 				struct ofono_gprs_provision_data **settings,
 				int *count)
@@ -48,21 +155,26 @@ static int provision_get_settings(const char *mcc, const char *mnc,
 	int ap_count;
 	int i;
 
-	DBG("Provisioning for MCC %s, MNC %s, SPN '%s'", mcc, mnc, spn);
+	ofono_info("Provisioning for MCC %s, MNC %s, SPN '%s'", mcc, mnc, spn);
 
-	apns = mbpi_lookup_apn(mcc, mnc, FALSE, &error);
-	if (apns == NULL) {
-		if (error != NULL) {
-			ofono_error("%s", error->message);
-			g_error_free(error);
-		}
-
-		return -ENOENT;
+	/*
+	 * Passing FALSE to mbpi_lookup_apn() would return
+	 * an empty list if duplicates are found.
+	 */
+	apns = mbpi_lookup_apn(mcc, mnc, TRUE, &error);
+	if (error != NULL) {
+		ofono_error("%s", error->message);
+		g_error_free(error);
 	}
 
+	ofono_info("Found %d APs in MBPI", g_slist_length(apns));
+	apns = provision_normalize_apn_list(apns, spn);
+	if (apns == NULL)
+		return -ENOENT;
+
 	ap_count = g_slist_length(apns);
 
-	DBG("Found %d APs", ap_count);
+	ofono_info("Provisioning %d APs", ap_count);
 
 	*settings = g_try_new0(struct ofono_gprs_provision_data, ap_count);
 	if (*settings == NULL) {
@@ -81,11 +193,11 @@ static int provision_get_settings(const char *mcc, const char *mnc,
 	for (l = apns, i = 0; l; l = l->next, i++) {
 		struct ofono_gprs_provision_data *ap = l->data;
 
-		DBG("Name: '%s'", ap->name);
-		DBG("APN: '%s'", ap->apn);
-		DBG("Type: %s", mbpi_ap_type(ap->type));
-		DBG("Username: '%s'", ap->username);
-		DBG("Password: '%s'", ap->password);
+		ofono_info("Name: '%s'", ap->name);
+		ofono_info("APN: '%s'", ap->apn);
+		ofono_info("Type: %s", mbpi_ap_type(ap->type));
+		ofono_info("Username: '%s'", ap->username);
+		ofono_info("Password: '%s'", ap->password);
 
 		memcpy(*settings + i, ap,
 			sizeof(struct ofono_gprs_provision_data));

ofono/plugins/sailfish_provision.c

@@ -1,251 +0,0 @@
-/*
- *  oFono - Open Source Telephony
- *
- *  Copyright (C) 2008-2011  Intel Corporation. All rights reserved.
- *  Copyright (C) 2013-2017  Jolla Ltd.
- *
- *  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
- *  published by the Free Software Foundation.
- *
- *  This program is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- */
-
-#ifdef HAVE_CONFIG_H
-#include <config.h>
-#endif
-
-#define _GNU_SOURCE
-#include <errno.h>
-#include <string.h>
-
-#include <glib.h>
-
-#define OFONO_API_SUBJECT_TO_CHANGE
-#include <ofono/log.h>
-#include <ofono/plugin.h>
-#include <ofono/modem.h>
-#include <ofono/gprs-provision.h>
-
-#include "provision.h"
-#include "mbpi.h"
-
-struct provision_ap_defaults {
-	enum ofono_gprs_context_type type;
-	const char *name;
-	const char *apn;
-};
-
-static gint provision_match_strings(const char *s1, const char *s2)
-{
-	gint match = 0;
-
-	/* Caller checks s2 for NULL */
-	if (s1) {
-		const gssize len1 = strlen(s1);
-		const gssize len2 = strlen(s2);
-
-		if (len1 == len2 && !strcmp(s1, s2)) {
-			/* Best match ever */
-			match = 3;
-		} else if (g_utf8_validate(s1, len1, NULL) &&
-					g_utf8_validate(s2, len2, NULL)) {
-			char *d1 = g_utf8_strdown(s1, len1);
-			char *d2 = g_utf8_strdown(s2, len2);
-
-			if (len1 == len2 && !strcmp(d1, d2)) {
-				/* Case insensitive match */
-				match = 2;
-			} else if ((len1 > len2 && strstr(d1, d2)) ||
-					(len2 > len1 && strstr(d2, d1))) {
-				/* Partial case insensitive match */
-				match = 1;
-			}
-
-			g_free(d1);
-			g_free(d2);
-		}
-	}
-
-	return match;
-}
-static gint provision_match_spn(const struct ofono_gprs_provision_data *ap,
-							const char *spn)
-{
-	return provision_match_strings(ap->provider_name, spn) * 4 +
-				provision_match_strings(ap->name, spn);
-}
-
-static void provision_free_ap(gpointer data)
-{
-	mbpi_ap_free(data);
-}
-
-static gint provision_compare_ap(gconstpointer a, gconstpointer b,
-							gpointer data)
-{
-	const struct ofono_gprs_provision_data *ap1 = a;
-	const struct ofono_gprs_provision_data *ap2 = b;
-	const char *spn = data;
-
-	if (spn) {
-		const gint result = provision_match_spn(ap2, spn) -
-				provision_match_spn(ap1, spn);
-
-		if (result) {
-			return result;
-		}
-	}
-
-	if (ap1->provider_primary && !ap2->provider_primary) {
-		return -1;
-	} else if (ap2->provider_primary && !ap1->provider_primary) {
-		return 1;
-	}
-
-	return 0;
-}
-
-/* Picks best ap, deletes the rest. Creates one if necessary */
-static GSList *provision_pick_best_ap(GSList *list, const char *spn,
-	const enum ofono_gprs_proto default_proto,
-	const struct provision_ap_defaults *defaults)
-{
-	/* Sort the list */
-	list = g_slist_sort_with_data(list, provision_compare_ap, (void*)spn);
-	if (list) {
-		/* Pick the best one, delete the rest */
-		GSList *best = list;
-		g_slist_free_full(g_slist_remove_link(list, best),
-							provision_free_ap);
-		return best;
-	} else {
-		/* or create one from the default data */
-		struct ofono_gprs_provision_data *ap =
-			g_new0(struct ofono_gprs_provision_data, 1);
-
-		ap->proto = default_proto;
-		ap->type = defaults->type;
-		ap->name = g_strdup(defaults->name);
-		ap->apn = g_strdup(defaults->apn);
-		ap->auth_method = OFONO_GPRS_AUTH_METHOD_NONE;
-		return g_slist_append(NULL, ap);
-	}
-}
-
-/* Returns the list containing exactly one INTERNET and one MMS access point */
-static GSList *provision_normalize_apn_list(GSList *apns, const char *spn)
-{
-	static const struct provision_ap_defaults internet_defaults =
-		{ OFONO_GPRS_CONTEXT_TYPE_INTERNET, "Internet", "internet" };
-	static const struct provision_ap_defaults mms_defaults =
-		{ OFONO_GPRS_CONTEXT_TYPE_MMS, "MMS", "mms" };
-
-	GSList *internet_apns = NULL;
-	GSList *mms_apns = NULL;
-
-	/* Split internet and mms apns, delete all others */
-	while (apns) {
-		GSList *link = apns;
-		struct ofono_gprs_provision_data *ap = link->data;
-
-		apns = g_slist_remove_link(apns, link);
-		if (ap->type == OFONO_GPRS_CONTEXT_TYPE_INTERNET) {
-			internet_apns = g_slist_concat(internet_apns, link);
-		} else if (ap->type == OFONO_GPRS_CONTEXT_TYPE_MMS) {
-			mms_apns = g_slist_concat(mms_apns, link);
-		} else {
-			g_slist_free_full(link, provision_free_ap);
-		}
-	}
-
-	/* Pick the best ap of each type and concatenate them */
-	return g_slist_concat(
-		provision_pick_best_ap(internet_apns, spn,
-			mbpi_default_internet_proto, &internet_defaults),
-		provision_pick_best_ap(mms_apns, spn,
-			mbpi_default_mms_proto, &mms_defaults));
-}
-
-int provision_get_settings(const char *mcc, const char *mnc,
-				const char *spn,
-				struct ofono_gprs_provision_data **settings,
-				int *count)
-{
-	GSList *l;
-	GSList *apns;
-	GError *error = NULL;
-	int ap_count;
-	int i;
-
-	ofono_info("Provisioning for MCC %s, MNC %s, SPN '%s'", mcc, mnc, spn);
-
-	/*
-	 * Passing FALSE to mbpi_lookup_apn() would return
-	 * an empty list if duplicates are found.
-	 */
-	apns = mbpi_lookup_apn(mcc, mnc, TRUE, &error);
-	if (error != NULL) {
-		ofono_error("%s", error->message);
-		g_error_free(error);
-	}
-
-	DBG("Found %d APs in MBPI", g_slist_length(apns));
-	apns = provision_normalize_apn_list(apns, spn);
-	ap_count = g_slist_length(apns);
-
-	DBG("Provisioning %d APs", ap_count);
-	*settings = g_new0(struct ofono_gprs_provision_data, ap_count);
-	*count = ap_count;
-
-	for (l = apns, i = 0; l; l = l->next, i++) {
-		struct ofono_gprs_provision_data *ap = l->data;
-
-		ofono_info("Name: '%s'", ap->name);
-		ofono_info("  APN: '%s'", ap->apn);
-		ofono_info("  Type: %s", mbpi_ap_type(ap->type));
-		ofono_info("  Username: '%s'", ap->username);
-		ofono_info("  Password: '%s'", ap->password);
-
-		memcpy(*settings + i, ap,
-			sizeof(struct ofono_gprs_provision_data));
-
-		g_free(ap);
-	}
-
-	g_slist_free(apns);
-
-	return 0;
-}
-
-static struct ofono_gprs_provision_driver provision_driver = {
-	.name		= "Provisioning",
-	.get_settings	= provision_get_settings
-};
-
-static int provision_init(void)
-{
-	DBG("");
-	return ofono_gprs_provision_driver_register(&provision_driver);
-}
-
-static void provision_exit(void)
-{
-	DBG("");
-	ofono_gprs_provision_driver_unregister(&provision_driver);
-}
-
-OFONO_PLUGIN_DEFINE(provision, "Provisioning Plugin", VERSION,
-			OFONO_PLUGIN_PRIORITY_DEFAULT,
-			provision_init, provision_exit)
-
-/*
- * Local Variables:
- * mode: C
- * c-basic-offset: 8
- * indent-tabs-mode: t
- * End:
- */

ofono/src/gprs.c

@@ -34,7 +34,6 @@
 #include <net/route.h>
 #include <netinet/in.h>
 #include <arpa/inet.h>
-#include <ctype.h>
 
 #include <glib.h>
 #include <gdbus.h>
@@ -259,10 +258,6 @@ static gboolean gprs_proto_from_string(const char *str,
 static const char *gprs_auth_method_to_string(enum ofono_gprs_auth_method auth)
 {
 	switch (auth) {
-	case OFONO_GPRS_AUTH_METHOD_ANY:
-		return "any";
-	case OFONO_GPRS_AUTH_METHOD_NONE:
-		return "none";
 	case OFONO_GPRS_AUTH_METHOD_CHAP:
 		return "chap";
 	case OFONO_GPRS_AUTH_METHOD_PAP:
@@ -281,12 +276,6 @@ static gboolean gprs_auth_method_from_string(const char *str,
 	} else if (g_str_equal(str, "pap")) {
 		*auth = OFONO_GPRS_AUTH_METHOD_PAP;
 		return TRUE;
-	} else if (g_str_equal(str, "any")) {
-		*auth = OFONO_GPRS_AUTH_METHOD_ANY;
-		return TRUE;
-	} else if (g_str_equal(str, "none")) {
-		*auth = OFONO_GPRS_AUTH_METHOD_NONE;
-		return TRUE;
 	}
 
 	return FALSE;
@@ -652,48 +641,7 @@ static gboolean pri_parse_proxy(struct pri_context *ctx, const char *proxy)
 	}
 
 	g_free(ctx->proxy_host);
-	ctx->proxy_host = NULL;
-
-	if (host[0] == '0' || strstr(host, ".0")) {
-		/*
-		 * Some operators provide IP address of the MMS proxy
-		 * prepending zeros to each number shorter then 3 digits,
-		 * e.g. "192.168.094.023" instead of "192.168.94.23".
-		 * That may look nicer but it's actually wrong because
-		 * the numbers starting with zeros are interpreted as
-		 * octal numbers. In the example above 023 actually means
-		 * 16 and 094 is not a valid number at all.
-		 *
-		 * In addition to publishing these broken settings on their
-		 * web sites, some of the operators send them over the air,
-		 * in which case we can't even blame the user for entering
-		 * an invalid IP address. We better be prepared to deal with
-		 * those.
-		 *
-		 * Since nobody in the world seems to be actually using the
-		 * octal notation to write an IP address, let's remove the
-		 * leading zeros if we find them in the host part of the MMS
-		 * proxy URL.
-		 */
-		char** parts = g_strsplit(host, ".", -1);
-		guint count = g_strv_length(parts);
-		if (count == 4) {
-			char** ptr = parts;
-			while (*ptr) {
-				char* part = *ptr;
-				while (part[0] == '0' && isdigit(part[1])) {
-					memmove(part, part+1, strlen(part));
-				}
-				*ptr++ = part;
-			}
-			ctx->proxy_host = g_strjoinv(".", parts);
-			DBG("%s => %s", host, ctx->proxy_host);
-		}
-		g_strfreev(parts);
-	}
-
-	if (!ctx->proxy_host)
-		ctx->proxy_host = g_strdup(host);
+	ctx->proxy_host = g_strdup(host);
 
 	g_free(scheme);
 	return TRUE;
@@ -944,13 +892,6 @@ static void pri_reset_context_properties(struct pri_context *ctx,
 				gprs_proto_to_string(ctx->context.proto));
 	}
 
-	if (ctx->context.auth_method != ap->auth_method) {
-		ctx->context.auth_method = ap->auth_method;
-		changed = TRUE;
-		pri_str_signal_change(ctx, "AuthenticationMethod",
-			gprs_auth_method_to_string(ctx->context.auth_method));
-	}
-
 	if (ap->type == OFONO_GPRS_CONTEXT_TYPE_MMS) {
 		if (pri_str_update(ctx->message_proxy, ap->message_proxy,
 				sizeof(ctx->message_proxy))) {

ofono/src/simfs.c

@@ -226,9 +226,6 @@ void sim_fs_notify_file_watches(struct sim_fs *fs, int id)
 		struct ofono_sim_context *context = l->data;
 		GSList *k;
 
-		if (context->file_watches == NULL)
-			continue;
-
 		for (k = context->file_watches->items; k; k = k->next) {
 			struct file_watch *w = k->data;
 			ofono_sim_file_changed_cb_t notify = w->item.notify;

ofono/src/ussd.c

@@ -514,20 +514,6 @@ void ofono_ussd_notify(struct ofono_ussd *ussd, int status, int dcs,
 				DBUS_TYPE_STRING, &str, DBUS_TYPE_INVALID);
 
 		ussd_change_state(ussd, new_state);
-		goto free;
-	} else if (ussd->state == USSD_STATE_USER_ACTION &&
-				status != OFONO_USSD_STATUS_ACTION_REQUIRED) {
-		ussd_change_state(ussd, USSD_STATE_IDLE);
-
-		if (status == OFONO_USSD_STATUS_NOTIFY && str && str[0]) {
-			const char *path = __ofono_atom_get_path(ussd->atom);
-
-			g_dbus_emit_signal(conn, path,
-				OFONO_SUPPLEMENTARY_SERVICES_INTERFACE,
-				"NotificationReceived", DBUS_TYPE_STRING,
-				&str, DBUS_TYPE_INVALID);
-		}
-
 		goto free;
 	} else {
 		ofono_error("Received an unsolicited USSD but can't handle.");

rpm/ofono.spec

@@ -11,7 +11,7 @@ Requires:   dbus
 Requires:   systemd
 Requires:   ofono-configs
 Requires:   libgrilio >= 1.0.10
-Requires:   libglibutil >= 1.0.22
+Requires:   libglibutil >= 1.0.19
 Requires(preun): systemd
 Requires(post): systemd
 Requires(postun): systemd
@@ -21,7 +21,7 @@ BuildRequires:  pkgconfig(libudev) >= 145
 BuildRequires:  pkgconfig(mobile-broadband-provider-info)
 BuildRequires:  pkgconfig(libwspcodec) >= 2.0
 BuildRequires:  pkgconfig(libgrilio) >= 1.0.10
-BuildRequires:  pkgconfig(libglibutil) >= 1.0.22
+BuildRequires:  pkgconfig(libglibutil) >= 1.0.19
 BuildRequires:  pkgconfig(libdbuslogserver-dbus)
 BuildRequires:  pkgconfig(libmce-glib)
 BuildRequires:  libtool
@@ -72,7 +72,6 @@ autoreconf --force --install
     --enable-debuglog \
     --enable-jolla-rilmodem \
     --enable-sailfishos \
-    --enable-sailfish-provision \
     --disable-add-remove-context \
     --disable-isimodem \
     --disable-qmimodem \