Re: [PATCH 1/1] Extended modem.conf

[Ryan Raasch <ryan.raasch@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 6751 bytes --]

An updated patch for the addition of serial options in the modem.conf 
file. I used the kernel coding style.

Andres Salomon wrote:
> On Mon, 26 Oct 2009 12:46:26 -0500
> Denis Kenzior <denkenz@gmail.com> wrote:
> 
>> Hi Ryan,
>>
>>> Hello,
>>>
>>> This a patch for allowing the addition of more options to the
>>> generic atgen driver. Comments :) ?
>> Before I look further into this, please re-submit the patch and
>> follow the oFono code style conventions.  oFono uses kernel coding
>> style and no maintainer will even look at patches that do not follow
>> this convention.
> 
> Also, it would be helpful if the patch was inline so that people could
> easily comment on various pieces of it in their replies.
> 

Using git imap-send, does anyone know how to use this with git-diff 
instead of making a commit, using
format-patch, then reversing the commit? It is a patch for approval.

---
  gatchat/gattty.c    |   16 ++++++------
  plugins/atgen.c     |   49 +++++++++++++++++++++++++++++++++++--
  plugins/modemconf.c |   67 
++++++++++++++++++++++++++++++++++++++++++++++++++-
  3 files changed, 120 insertions(+), 12 deletions(-)

diff --git a/gatchat/gattty.c b/gatchat/gattty.c
index f18eca4..64dd8ab 100644
--- a/gatchat/gattty.c
+++ b/gatchat/gattty.c
@@ -208,21 +208,21 @@ static int open_device(const char *tty, GHashTable 
*options)
  							(void *) &value)) {
  			gboolean ok = FALSE;

-			if (g_str_equal(key, "baud"))
+			if (g_str_equal(key, "Baud"))
  				ok = set_baud(value, &ti);
-			else if (g_str_equal(key, "stopbits"))
+			else if (g_str_equal(key, "StopBits"))
  				ok = set_stop_bits(value, &ti);
-			else if (g_str_equal(key, "databits"))
+			else if (g_str_equal(key, "DataBits"))
  				ok = set_data_bits(value, &ti);
-			else if (g_str_equal(key, "parity"))
+			else if (g_str_equal(key, "Parity"))
  				ok = set_parity(value, &ti);
-			else if (g_str_equal(key, "xonxoff"))
+			else if (g_str_equal(key, "XonXoff"))
  				ok = set_xonxoff(value, &ti);
-			else if (g_str_equal(key, "rtscts"))
+			else if (g_str_equal(key, "Rtscts"))
  				ok = set_rtscts(value, &ti);
-			else if (g_str_equal(key, "local"))
+			else if (g_str_equal(key, "Local"))
  				ok = set_local(value, &ti);
-			else if (g_str_equal(key, "read"))
+			else if (g_str_equal(key, "Read"))
  				ok = set_read(value, &ti);

  			if (ok == FALSE)
diff --git a/plugins/atgen.c b/plugins/atgen.c
index c0fbbb2..b72426e 100644
--- a/plugins/atgen.c
+++ b/plugins/atgen.c
@@ -48,6 +48,24 @@
  #include <ofono/ussd.h>
  #include <ofono/voicecall.h>

+static const char* tty_opts[] =
+  {
+	  "Baud",
+	  "Read",
+	  "Local",
+	  "StopBits",
+	  "DataBits",
+	  "Parity",
+	  "XonXoff",
+	  "Rtscts",
+	  NULL,
+  };
+
+static void unregister_tty_option(gpointer data)
+{
+	g_free(data);
+}
+
  static int atgen_probe(struct ofono_modem *modem)
  {
  	return 0;
@@ -68,6 +86,9 @@ static int atgen_enable(struct ofono_modem *modem)
  	GIOChannel *channel;
  	GAtSyntax *syntax;
  	const char *device;
+	const char *value;
+	GHashTable *options;
+	int i = 0;

  	DBG("%p", modem);

@@ -75,23 +96,45 @@ static int atgen_enable(struct ofono_modem *modem)
  	if (!device)
  		return -EINVAL;

-	channel = g_at_tty_open(device, NULL);
-	if (!channel)
+	options = g_hash_table_new_full(g_str_hash, g_str_equal,
+					g_free, unregister_tty_option);
+	if (!options) {
+		return -ENOMEM;
+	}
+
+	while(tty_opts[i]) {
+
+		value = ofono_modem_get_string(modem, tty_opts[i]);
+		if(value) {
+			g_hash_table_replace(options, g_strdup(tty_opts[i]),
+					     g_strdup(value));
+		}
+		i++;
+	};
+
+	channel = g_at_tty_open(device, options);
+	if (!channel) {
+		g_hash_table_destroy(options);
  		return -EIO;
+	}

  	syntax = g_at_syntax_new_gsmv1();
  	chat = g_at_chat_new(channel, syntax);
  	g_at_syntax_unref(syntax);
  	g_io_channel_unref(channel);

-	if (!chat)
+	if (!chat) {
+		g_hash_table_destroy(options);
  		return -ENOMEM;
+	}

  	if (getenv("OFONO_AT_DEBUG"))
  		g_at_chat_set_debug(chat, atgen_debug, NULL);

  	ofono_modem_set_data(modem, chat);

+	g_hash_table_destroy(options);
+
  	return 0;
  }

diff --git a/plugins/modemconf.c b/plugins/modemconf.c
index 192faa6..8973774 100644
--- a/plugins/modemconf.c
+++ b/plugins/modemconf.c
@@ -72,7 +72,7 @@ static int set_address(struct ofono_modem *modem,
  static int set_device(struct ofono_modem *modem,
  					GKeyFile *keyfile, const char *group)
  {
-	char *device;
+	char *device, *value;

  	device = g_key_file_get_string(keyfile, group, "Device", NULL);
  	if (!device)
@@ -82,6 +82,71 @@ 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");
+
+	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");
+
+	value = g_key_file_get_string(keyfile, group, "Rtscts", NULL);
+	if (value) {
+		ofono_modem_set_string(modem, "Rtscts", value);
+		g_free(value);
+	}
+	else
+		ofono_modem_set_string(modem, "Rtscts", "on");
+
+
  	return 0;
  }

-- 
1.6.4.GIT


> _______________________________________________
> ofono mailing list
> ofono(a)ofono.org
> http://lists.ofono.org/listinfo/ofono