Re: [MM] [PATCH] novatel-lte: increase number of retries for connection status checks

2013-05-27 Thread Ben Chan
60s seems a bit excessive as users most likely give up earlier. Revised the patch to have a 30s timeout. Thanks, Ben On Mon, May 27, 2013 at 12:52 AM, Aleksander Morgado wrote: > On 26/05/13 08:45, Ben Chan wrote: > > This patch increases the number of retries, from 4 to 10, for connection > >

[MM] [PATCH v2] novatel-lte: increase number of retries for connection status checks

2013-05-27 Thread Ben Chan
This patch increases the number of retries, from 4 to 30, for connection status check during a connection / disconnection request, which handles some scenario when the connection / disconnection request takes more than 5 seconds to complete. --- plugins/novatel/mm-broadband-bearer-novatel-lte.c |

[MM] [PATCH v2] novatel-lte: normalize QMI status when included in DBus error message

2013-05-27 Thread Ben Chan
This patches normalize a response for the AT$NWQMISTATUS command, by replacing white-space characters with a space, before the response is included in a DBus error message. --- plugins/novatel/mm-broadband-bearer-novatel-lte.c | 33 --- 1 file changed, 29 insertions(+), 4 delet

Re: [MM] [PATCH] Implement modem time interface in Gobi plugin

2013-05-27 Thread Aleksander Morgado
On 05/27/2013 04:16 PM, Heiher wrote: > Hello, > > Implement modem time interface in Gobi plugin for MC7750. (I'm not sure > if this will work on other devices. May be is ok ;)) > So this is just the same implementation as in the Sierra plugin, right? In the best case, we can re-use the same co

Re: [PATCH] MM: Wavecom: Forbidden Sierra Wireless MC7750

2013-05-27 Thread Aleksander Morgado
On 05/27/2013 03:55 PM, Heiher wrote: > It works! > > Thanks! > Pushed to git master then. > > On Mon, May 27, 2013 at 9:48 PM, Aleksander Morgado > mailto:aleksan...@lanedo.com>> wrote: > > On 05/27/2013 03:03 PM, Aleksander Morgado wrote: > > If we need to handle this case, we would

Re: [PATCH] MM: Wavecom: Forbidden Sierra Wireless MC7750

2013-05-27 Thread Aleksander Morgado
On 05/27/2013 03:03 PM, Aleksander Morgado wrote: > If we need to handle this case, we would add a forbidden-drivers filter > in the Wavecom plugin, including both "sierra" and "sierra-net". Are you able to re-test with the attached patch installed? -- Aleksander >From fdd03fd77f303a367137d73a1

Re: [PATCH] MM: Wavecom: Forbidden Sierra Wireless MC7750

2013-05-27 Thread Aleksander Morgado
On 05/27/2013 03:35 PM, Bjørn Mork wrote: > Aleksander Morgado writes: >> On 05/27/2013 04:07 AM, Heiher wrote: >>> Hello, >>> >>> When I disable qmi-wwan kernel module, the default plugin for MC7750 is >>> 'wavecom' in ModemManager. The Sierra Wireless MC7750 (0x114f:0x68a2) is >>> not wavecom co

Re: [PATCH] MM: Wavecom: Forbidden Sierra Wireless MC7750

2013-05-27 Thread Bjørn Mork
Aleksander Morgado writes: > On 05/27/2013 04:07 AM, Heiher wrote: >> Hello, >> >> When I disable qmi-wwan kernel module, the default plugin for MC7750 is >> 'wavecom' in ModemManager. The Sierra Wireless MC7750 (0x114f:0x68a2) is >> not wavecom complatible device. Should be forbidden it? >> > >

Re: [PATCH] MM: Wavecom: Forbidden Sierra Wireless MC7750

2013-05-27 Thread Aleksander Morgado
On 05/27/2013 04:07 AM, Heiher wrote: > Hello, > > When I disable qmi-wwan kernel module, the default plugin for MC7750 is > 'wavecom' in ModemManager. The Sierra Wireless MC7750 (0x114f:0x68a2) is > not wavecom complatible device. Should be forbidden it? > That's completely unexpected. Did the

Re: [MM] [PATCH] novatel-lte: increase number of retries for connection status checks

2013-05-27 Thread Aleksander Morgado
On 26/05/13 08:45, Ben Chan wrote: > This patch increases the number of retries, from 4 to 10, for connection > status check during a connection / disconnection request, which handles > some scenario when the connection / disconnection request takes more > than 5 seconds to complete. We recently u

Re: [MM] [PATCH] novatel-lte: normalize QMI status when included in DBus error message

2013-05-27 Thread Aleksander Morgado
On 26/05/13 08:43, Ben Chan wrote: > This patches normalize a response for the AT$NWQMISTATUS command, by > replacing white-space characters with a space, before the response is > included in a DBus error message. > --- If the only purpose is to normalize the message when it's set in a GError/GAs