All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH v2] Add CDMA/EVDO tasks in TODO
Date: Wed, 27 Oct 2010 23:03:44 -0500	[thread overview]
Message-ID: <4CC8F620.3080306@gmail.com> (raw)
In-Reply-To: <1288228358-17225-1-git-send-email-caiwen.zhang@windriver.com>

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

Hi Caiwen,

On 10/27/2010 08:12 PM, Zhang Caiwen wrote:
> ---
>  TODO |   52 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 52 insertions(+), 0 deletions(-)
> 
> diff --git a/TODO b/TODO
> index 239bab0..e368070 100644
> --- a/TODO
> +++ b/TODO
> @@ -543,3 +543,55 @@ Miscellaneous
>  
>    Priority: Low
>    Complexity: C4
> +
> +
> +CDMA/EVDO
> +=========
> +
> +- Extend modem interface with a 'Mode' property to indicate the modem type.
> +
> +  Priority: High
> +  Complexity: C1
> +
> +- Extend radio setting,add CDMA radio access technology support. The CDMA modem

Can we watch that punctuation is done properly? Space after the comma,
preferably two spaces after each period.

> +  preferred technology can be CDMA, HDR or CDMA/HDR hybrid.
> +
> +  Priority: Medium
> +  Complexity: C1
> +  Owner: Zhang Caiwen <caiwen.zhang@windriver.com>
> +
> +- Add CDMA Network registration support. CDMA device will register to network
> +  automaticlly. Unlike GSM,generally,it don't support manually register to

Can you check the spelling as well?  And again, the punctuation.

> +  network.So its mainly function is to get CDMA network information. The
> +  information include: registration status, service domain, CDMA 1x signal
> +  strength, HDR signal strength, MCC, MNC, SID, NID etc.
> +
> +  Priority: Medium
> +  Complexity: C1
> +  Owner: Zhang Caiwen <caiwen.zhang@windriver.com>
> +
> +- Add UIM support. UIM support change pin, enter pin, reset pin, set pin lock,
> +  get pin lock setting, get pin remain retry time, and get information that
> +  stored in UIM, include MCC/MNC, ruimid, IMSI,MDN, PRL version etc.
> +
> +  Priority: High
> +  Complexity: C4
> +  Owner: Zhang Caiwen <caiwen.zhang@windriver.com>
> +
> +- Add CDMA voice call support. CDMA voice call will support originate/end/hold
> +  a call,transfer a call, 3WC(three-way calling), conference call and DTMF. For
> +  a voice call, can get its status, CLI and ending result(if it is end).
> +
> +  Priority: High
> +  Complexity: C8
> +  Owner: Zhang Zhengguang <zhengguang.Zhang@windriver.com>
> +
> +- Add CDMA Data connection support. CDMA data connection manager support
> +  start/end a data connect according to the settings from other module(upper
> +  layer or ConnMan).
> +
> +  Priority: High
> +  Complexity: C4
> +  Owner: Zhang Caiwen <caiwen.zhang@windriver.com>
> +
> +

As a general comment, please try to stick to the wording that the rest
of the TODO uses.  E.g.: Foo Support.  'Add foo property...' or 'Extend
foo to support foo...'

The TODO should be accessible to non-telephony geeks. Think of it as a
high level design and roadmap document, with the same level of importance.

Regards,
-Denis

      reply	other threads:[~2010-10-28  4:03 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-28  1:12 [PATCH v2] Add CDMA/EVDO tasks in TODO Zhang Caiwen
2010-10-28  4:03 ` Denis Kenzior [this message]

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=4CC8F620.3080306@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.