[PATCH v3, Part2, 3/3] udev: Add nokiacdma device

[PATCH v3, Part2, 3/3] udev: Add nokiacdma device

[Dara Spieker-Doyle]

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

---
 plugins/udev.c |   21 +++++++++++++++++++++
 1 files changed, 21 insertions(+), 0 deletions(-)

diff --git a/plugins/udev.c b/plugins/udev.c
index 255755e..9447e5c 100644
--- a/plugins/udev.c
+++ b/plugins/udev.c
@@ -449,6 +449,25 @@ static void add_isi(struct ofono_modem *modem,
 	ofono_modem_register(modem);
 }
 
+static void add_nokiacdma(struct ofono_modem *modem,
+					struct udev_device *udev_device)
+{
+	const char *devnode;
+	int registered;
+
+	DBG("modem %p", modem);
+
+	devnode = udev_device_get_devnode(udev_device);
+
+	registered = ofono_modem_get_integer(modem, "Registered");
+	if (registered != 0)
+		return;
+
+	ofono_modem_set_string(modem, "Device", devnode);
+	ofono_modem_set_integer(modem, "Registered", 1);
+	ofono_modem_register(modem);
+}
+
 static void add_modem(struct udev_device *udev_device)
 {
 	struct ofono_modem *modem;
@@ -533,6 +552,8 @@ done:
 		add_isi(modem, udev_device);
 	else if (g_strcmp0(driver, "n900") == 0)
 		add_isi(modem, udev_device);
+	else if (g_strcmp0(driver, "nokiacdma") == 0)
+		add_nokiacdma(modem, udev_device);
 }
 
 static gboolean devpath_remove(gpointer key, gpointer value, gpointer user_data)
-- 
1.7.0.4

Re: [PATCH v3, Part2, 3/3] udev: Add nokiacdma device

[Denis Kenzior]

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

Hi Dara,

On 12/21/2010 01:23 PM, Dara Spieker-Doyle wrote:
> ---
>  plugins/udev.c |   21 +++++++++++++++++++++
>  1 files changed, 21 insertions(+), 0 deletions(-)
> 
> diff --git a/plugins/udev.c b/plugins/udev.c
> index 255755e..9447e5c 100644
> --- a/plugins/udev.c
> +++ b/plugins/udev.c
> @@ -449,6 +449,25 @@ static void add_isi(struct ofono_modem *modem,
>  	ofono_modem_register(modem);
>  }
>  
> +static void add_nokiacdma(struct ofono_modem *modem,
> +					struct udev_device *udev_device)
> +{
> +	const char *devnode;
> +	int registered;
> +
> +	DBG("modem %p", modem);
> +
> +	devnode = udev_device_get_devnode(udev_device);
> +
> +	registered = ofono_modem_get_integer(modem, "Registered");
> +	if (registered != 0)
> +		return;
> +
> +	ofono_modem_set_string(modem, "Device", devnode);
> +	ofono_modem_set_integer(modem, "Registered", 1);
> +	ofono_modem_register(modem);

This looks a little fishy to me.  How many ttys are created when this
device is plugged in?  Is it really just one or are you relying on luck
to select the right tty?

> +}
> +
>  static void add_modem(struct udev_device *udev_device)
>  {
>  	struct ofono_modem *modem;
> @@ -533,6 +552,8 @@ done:
>  		add_isi(modem, udev_device);
>  	else if (g_strcmp0(driver, "n900") == 0)
>  		add_isi(modem, udev_device);
> +	else if (g_strcmp0(driver, "nokiacdma") == 0)
> +		add_nokiacdma(modem, udev_device);
>  }
>  
>  static gboolean devpath_remove(gpointer key, gpointer value, gpointer user_data)

Regards,
-Denis

Re: [PATCH v3, Part2, 3/3] udev: Add nokiacdma device

[Dara Spieker-Doyle]

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

Hi Denis

On 12/22/2010 06:23 PM, ext Denis Kenzior wrote:
> Hi Dara,
>
> On 12/21/2010 01:23 PM, Dara Spieker-Doyle wrote:
>> ---
>>   plugins/udev.c |   21 +++++++++++++++++++++
>>   1 files changed, 21 insertions(+), 0 deletions(-)
>>
>> diff --git a/plugins/udev.c b/plugins/udev.c
>> index 255755e..9447e5c 100644
>> --- a/plugins/udev.c
>> +++ b/plugins/udev.c
>> @@ -449,6 +449,25 @@ static void add_isi(struct ofono_modem *modem,
>>   	ofono_modem_register(modem);
>>   }
>>
>> +static void add_nokiacdma(struct ofono_modem *modem,
>> +					struct udev_device *udev_device)
>> +{
>> +	const char *devnode;
>> +	int registered;
>> +
>> +	DBG("modem %p", modem);
>> +
>> +	devnode = udev_device_get_devnode(udev_device);
>> +
>> +	registered = ofono_modem_get_integer(modem, "Registered");
>> +	if (registered != 0)
>> +		return;
>> +
>> +	ofono_modem_set_string(modem, "Device", devnode);
>> +	ofono_modem_set_integer(modem, "Registered", 1);
>> +	ofono_modem_register(modem);
>
> This looks a little fishy to me.  How many ttys are created when this
> device is plugged in?  Is it really just one or are you relying on luck
> to select the right tty?
>
When the current nokiacdma (Nokia 7205) device is plugged in there is 
only one TTY created.

Thanks
Dara