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 wrote: > Hey D

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 > 2012/9/18 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 >

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 > 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 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 > >> the low-power mode when

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

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 >> the low-power mode when disabling, partially because we're not always >