Re: [MM] Power down during init

2012-09-20 Thread Marius Kotsbak
Den 19. sep. 2012 07:47, skrev Aleksander Morgado: On 09/18/2012 08:03 PM, Dan Williams wrote: Btw, is this expected?: model: '+CGMM: MF820D' A bug, the handler code here probably needs to strip out the CGMM if found. Most devices don't actually prefix the CGMM response with +CGMM so the

Re: [MM] Power down during init

2012-09-19 Thread Ben Chan
How long? I think the timeout is 3 seconds now so maybe it should be increased. could be up to 30s, which is likely a firmware issue. - Ben ___ networkmanager-list mailing list networkmanager-list@gnome.org

Re: [MM] Power down during init

2012-09-19 Thread Aleksander Morgado
I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. Planning to do some more stress tests later this week. I'm assuming the tests went well? :-) Except for the issue that CFUN+4 can occasionally take a long time to finish :( How long? I think the timeout is 3

Re: [MM] Power down during init

2012-09-19 Thread Aleksander Morgado
We need to run the power-down during init *only* after having checked current/modem capabilities. Will update the patch. I pushed a new 'power-down-during-init' branch to the upstream git tree, including the change of when the power down command is run, so that we only run it after having

Re: [MM] Power down during init (was: Re: [MM master] Re: [MM06] [PATCH] sierra: use +CFUN=4 for powering down)

2012-09-18 Thread Dan Williams
On Tue, 2012-09-18 at 19:45 +0200, Marius Kotsbak wrote: 2012/9/18 Aleksander Morgado aleksan...@lanedo.com Hey Dan and Marius, Power Up = enabled Power Down = disabled So we already have those states in ModemManager via the

Re: [MM] Power down during init (was: Re: [MM master] Re: [MM06] [PATCH] sierra: use +CFUN=4 for powering down)

2012-09-18 Thread Marius Kotsbak
2012/9/18 Marius Kotsbak marius.kots...@gmail.com 2012/9/18 Aleksander Morgado aleksan...@lanedo.com Hey Dan and Marius, Power Up = enabled Power Down = disabled So we already have those states in ModemManager via the Enable() method and the modem states. It's just that not all

Re: [MM] Power down during init (was: Re: [MM master] Re: [MM06] [PATCH] sierra: use +CFUN=4 for powering down)

2012-09-18 Thread Ben Chan
I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. Planning to do some more stress tests later this week. It may not be a bad idea to allow a plugin to override the power-down-on-initialization behavior. Thanks, Ben On Tue, Sep 18, 2012 at 7:24 AM, Aleksander Morgado

Re: [MM] Power down during init

2012-09-18 Thread Aleksander Morgado
I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. Planning to do some more stress tests later this week. I'm assuming the tests went well? :-) It may not be a bad idea to allow a plugin to override the power-down-on-initialization behavior. Not a big deal I guess; given

Re: [MM] Power down during init

2012-09-18 Thread Aleksander Morgado
Hey Dan and Marius, Power Up = enabled Power Down = disabled So we already have those states in ModemManager via the Enable() method and the modem states. It's just that not all devices actually implement

Re: [MM] Power down during init

2012-09-18 Thread Ben Chan
On Tue, Sep 18, 2012 at 10:05 PM, Aleksander Morgado aleksan...@lanedo.com wrote: I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. Planning to do some more stress tests later this week. I'm assuming the tests went well? :-) Except for the issue that CFUN+4 can occasionally

Re: [MM] Power down during init

2012-09-18 Thread Aleksander Morgado
I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. Planning to do some more stress tests later this week. I'm assuming the tests went well? :-) Except for the issue that CFUN+4 can occasionally take a long time to finish :( Still haven't found the root cause or a

Re: [MM] Power down during init

2012-09-18 Thread Aleksander Morgado
On 09/18/2012 08:03 PM, Dan Williams wrote: Btw, is this expected?: model: '+CGMM: MF820D' A bug, the handler code here probably needs to strip out the CGMM if found. Most devices don't actually prefix the CGMM response with +CGMM so the code wasn't built to handle that, either in 05/06