1 files changed, 22 insertions, 54 deletions

diff --git a/plugins/modemconf.c b/plugins/modemconf.c
index 6b317e60..440b205e 100644
--- a/plugins/modemconf.c
+++ b/plugins/modemconf.c
@@ -35,6 +35,18 @@
 
 static GSList *modem_list = NULL;
 
+static const char *tty_opts[] = {
+	"Baud",
+	"Read",
+	"Local",
+	"StopBits",
+	"DataBits",
+	"Parity",
+	"XonXoff",
+	"RtsCts",
+	NULL,
+};
+
 static int set_address(struct ofono_modem *modem,
 					GKeyFile *keyfile, const char *group)
 {
@@ -72,7 +84,9 @@ static int set_address(struct ofono_modem *modem,
 static int set_device(struct ofono_modem *modem,
 					GKeyFile *keyfile, const char *group)
 {
-	char *device, *value;
+	char *device;
+	char *value;
+	int i;
 
 	device = g_key_file_get_string(keyfile, group, "Device", NULL);
 	if (!device)
@@ -82,62 +96,16 @@ static int set_device(struct ofono_modem *modem,
 
 	g_free(device);
 
-	value = g_key_file_get_string(keyfile, group, "Baud", NULL);
-	if (value) {
-		ofono_modem_set_string(modem, "Baud", value);
-		g_free(value);
-	} else
-		ofono_modem_set_string(modem, "Baud", "115200");
-
-	value = g_key_file_get_string(keyfile, group, "Read", NULL);
-	if (value) {
-		ofono_modem_set_string(modem, "Read", value);
-		g_free(value);
-	} else
-		ofono_modem_set_string(modem, "Read", "on");
-
-	value = g_key_file_get_string(keyfile, group, "Local", NULL);
-	if (value) {
-		ofono_modem_set_string(modem, "Local", value);
-		g_free(value);
-	} else
-		ofono_modem_set_string(modem, "Local", "none");
-
-	value = g_key_file_get_string(keyfile, group, "StopBits", NULL);
-	if (value) {
-		ofono_modem_set_string(modem, "StopBits", value);
-		g_free(value);
-	} else
-		ofono_modem_set_string(modem, "StopBits", "1");
-
-	value = g_key_file_get_string(keyfile, group, "DataBits", NULL);
-	if (value) {
-		ofono_modem_set_string(modem, "DataBits", value);
-		g_free(value);
-	} else
-		ofono_modem_set_string(modem, "Databits", "8");
-
-	value = g_key_file_get_string(keyfile, group, "Parity", NULL);
-	if (value) {
-		ofono_modem_set_string(modem, "Parity", value);
-		g_free(value);
-	} else
-		ofono_modem_set_string(modem, "Parity", "none");
+	for (i = 0; tty_opts[i]; i++) {
+		value = g_key_file_get_string(keyfile, group,
+						tty_opts[i], NULL);
 
-	value = g_key_file_get_string(keyfile, group, "XonXoff", NULL);
-	if (value) {
-		ofono_modem_set_string(modem, "XonXoff", value);
-		g_free(value);
-	} else
-		ofono_modem_set_string(modem, "XonXoff", "off");
+		if (value == NULL)
+			continue;
 
-	value = g_key_file_get_string(keyfile, group, "Rtscts", NULL);
-	if (value) {
-		ofono_modem_set_string(modem, "Rtscts", value);
+		ofono_modem_set_string(modem, tty_opts[i], value);
 		g_free(value);
-	} else
-		ofono_modem_set_string(modem, "Rtscts", "on");
-
+	}
 
 	return 0;
 }