Open Source Telephony
 help / color / mirror / Atom feed
From: Guillaume Zajac <guillaume.zajac@linux.intel.com>
To: ofono@ofono.org
Subject: Re: [PATCH_v0 1/6] doc: Add DormantInterval property
Date: Thu, 08 Dec 2011 10:25:03 +0100	[thread overview]
Message-ID: <4EE0826F.50707@linux.intel.com> (raw)
In-Reply-To: <4EDF5D61.8080609@gmail.com>

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

Hi Denis,

On 07/12/2011 13:34, Denis Kenzior wrote:
> Hi Guillaume,
>
> On 12/07/2011 10:05 AM, Guillaume Zajac wrote:
>> ---
>>   doc/cdma-connman-api.txt |    5 +++++
>>   1 files changed, 5 insertions(+), 0 deletions(-)
>>
>> diff --git a/doc/cdma-connman-api.txt b/doc/cdma-connman-api.txt
>> index 48699a3..a6d1e56 100644
>> --- a/doc/cdma-connman-api.txt
>> +++ b/doc/cdma-connman-api.txt
>> @@ -35,6 +35,11 @@ Properties	boolean Powered [readwrite]
>>   			Contains whether the connection is dormant.  Will
>>   			always be false if the connection is not powered.
>>
>> +		int DormantInterval [readwrite]
>> +
>> +			Contains the interval in seconds to enter in dormant
>> +			Mode.
>> +
> Why would the user want to mess with this setting? What would be the
> right interval?  Remember, oFono only exports settings that are useful
> to show to the user, and that the user can understand what to do with.
> This this one certainly doesn't fit the above description.
>
> Besides, we should simply disable dormancy mode in the first place.  The
> base station can more intelligently manage that anyway.
>

Right, the user can't manage this value, however Dormant property is 
ReadOnly.
So should we have a specific DBus API to disable/enable the dormancy mode?

Kind regards,
Guillaume

  reply	other threads:[~2011-12-08  9:25 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-07 16:05 [PATCH_v0 0/6] Manage Dormant property for CDMA modems Guillaume Zajac
2011-12-07 16:05 ` [PATCH_v0 1/6] doc: Add DormantInterval property Guillaume Zajac
2011-12-07 12:34   ` Denis Kenzior
2011-12-08  9:25     ` Guillaume Zajac [this message]
2011-12-08 11:00       ` Denis Kenzior
2011-12-07 16:05 ` [PATCH_v0 2/6] cdma-connman: Add dormant option configuration and notification Guillaume Zajac
2011-12-07 16:05 ` [PATCH_v0 3/6] cdma-connman: Add DormantInterval porperty and dormant notification Guillaume Zajac
2011-12-07 16:05 ` [PATCH_v0 4/6] huaweicdma: Specify vendor in creating cdma-connman atom Guillaume Zajac
2011-12-07 16:05 ` [PATCH_v0 5/6] cdmamodem: Add dormant notification and dormant interval setting Guillaume Zajac
2011-12-07 16:05 ` [PATCH_v0 6/6] test: Add script to set dormant interval Guillaume Zajac

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EE0826F.50707@linux.intel.com \
    --to=guillaume.zajac@linux.intel.com \
    --cc=ofono@ofono.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox