Open Source Telephony
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: ofono@ofono.org
Subject: Re: [PATCH 10/16] gprs: Add ofono_gprs_create_context method
Date: Thu, 12 Aug 2010 07:57:38 -0400	[thread overview]
Message-ID: <1281614258.12579.272.camel@localhost.localdomain> (raw)
In-Reply-To: <1281493601-14351-11-git-send-email-zhenhua.zhang@intel.com>

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

Hi Zhenhua,

> DUN server may create one primary context if none of contexts existing
> on the GPRS atom.

so Denis and I had a chat about this. And we agreed that oFono core
should just create one Internet context if none exists. If the plugin
driver registers a GPRS atom, we should just create one context anyway.
ConnMan requires this anyway and it makes sense. We can also go one step
ahead and fail to remove this default Internet context. It should be
present all the time.

Regards

Marcel



  reply	other threads:[~2010-08-12 11:57 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-11  2:26 [PATCH 00/16] Implement DUN server for oFono Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 01/16] gprs: Rename status_watch to netreg_status_watch Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 02/16] emulator: Add status watches for ofono emulator Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 03/16] emulator: Add emulator status watches in gprs atom Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 04/16] emulator: Add APIs to send GAtServer result Zhenhua Zhang
2010-08-12 11:52   ` Marcel Holtmann
2010-08-13  1:08     ` Zhang, Zhenhua
2010-08-13 13:06       ` Marcel Holtmann
2010-08-11  2:26 ` [PATCH 05/16] emulator: Implement dialing up for DUN Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 06/16] gprs: Make gprs_proto_to/from_string non-static Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 07/16] emulator: Register mandatory AT command handlers Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 08/16] gprs: Refactor to share remove_context method Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 09/16] gprs: Add DUN +CGATT support in gprs atom Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 10/16] gprs: Add ofono_gprs_create_context method Zhenhua Zhang
2010-08-12 11:57   ` Marcel Holtmann [this message]
2010-08-13  1:42     ` Zhang, Zhenhua
2010-08-13 13:04       ` Marcel Holtmann
2010-08-11  2:26 ` [PATCH 11/16] gprs: Add DUN +CGDCONT support in gprs atom Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 12/16] gprs: Add DUN server GPRS connect support Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 13/16] gprs: Add status watch functions Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 14/16] gprs: Add gprs_get_status Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 15/16] emulator: Watch GPRS status changes Zhenhua Zhang
2010-08-11  2:26 ` [PATCH 16/16] emulator: Watch netreg " Zhenhua Zhang

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=1281614258.12579.272.camel@localhost.localdomain \
    --to=marcel@holtmann.org \
    --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