All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH_v0 1/6] doc: Add DormantInterval property
Date: Wed, 07 Dec 2011 06:34:41 -0600	[thread overview]
Message-ID: <4EDF5D61.8080609@gmail.com> (raw)
In-Reply-To: <1323273951-24162-2-git-send-email-guillaume.zajac@linux.intel.com>

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

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.

>  		string Username [readwrite]
>  
>  			Holds the username to be used for authentication

Regards,
-Denis

  reply	other threads:[~2011-12-07 12:34 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 [this message]
2011-12-08  9:25     ` Guillaume Zajac
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=4EDF5D61.8080609@gmail.com \
    --to=denkenz@gmail.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.