All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH] atmodem: CEREG support for LTE network status reporting in AT modem
Date: Wed, 23 Feb 2011 10:49:37 -0600	[thread overview]
Message-ID: <4D653AA1.8000203@gmail.com> (raw)
In-Reply-To: <8281606748F03E4390BB98A3F47F2E948A9F98CDC7@irsmsx501.ger.corp.intel.com>

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

Hi Redouane,

On 02/23/2011 10:36 AM, Soum, RedouaneX wrote:
> Hi Denis,
> 
> 
>>> In my opinion, combined gprs atom would be easier to do and probably 
>>> enough, separate atoms would be more "looking into the future" like 
>>> but I am not sure if this division is necessary.
>>
>> Looking at the current ConnectionManager API, none of the properties 
>> (Powered, Attached, Suspended, RoamingAllowed) are applicable to LTE.
>> If we use a separate LTE atom then the Bearer property's 'lte' value is 
>> redundant as well.  So why would you want to carry this baggage around 
>> for the user of LTE-only systems?
> 
> After discussing internally with Fred Joly I would like to come back on the 4 properties :
> - Suspended is applicable in LTE in case of CS Voice Call using CSFB

My understanding was that CS Fallback physically switches technologies.
 It is not 'suspending' LTE like it does with GPRS.  Am I wrong on this one?

> - Powered is applicable the only point is that you'll not be registered to LTE network if PS is disabled.

It sounds like you're talking about RadioSettings properties here.

Powered=False is implemented by simply not attaching today.  The
implementation logic for LTE would be completely different.  So most
likely this will not work out nicely anyway.

> - Attached is also applicable and here also if you are not attached then you'll not be registered to LTE network.

Sounds pointless to expose for LTE to me.

> - RoamingAllowed it's a settings from the APE so we can imagine that we would like not to use LTE if we are in roaming.

Perhaps, but again we have no control of the attach procedure with LTE.
 So the implementation has to rely on vendor specific radio settings.
Same arguments about shoehorning logic apply here.

Regards,
-Denis

  reply	other threads:[~2011-02-23 16:49 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-22 13:46 [PATCH] atmodem: CEREG support for LTE network status reporting in AT modem Vijay.Nayani
2011-02-22 14:09 ` Tomasz Gregorek
2011-02-22 14:59   ` Vijay.Nayani
2011-02-22 15:20     ` Tomasz Gregorek
2011-02-22 16:08     ` Soum, RedouaneX
2011-02-22 16:35       ` Denis Kenzior
2011-02-23 10:13         ` Tomasz Gregorek
2011-02-23 15:58           ` Denis Kenzior
2011-02-23 16:36             ` Soum, RedouaneX
2011-02-23 16:49               ` Denis Kenzior [this message]
2011-02-23 18:24                 ` Joly, Frederic
2011-02-23 18:52                   ` Denis Kenzior
2011-02-24 10:34                     ` Arun Ravindran
2011-02-24 16:12                       ` Denis Kenzior
  -- strict thread matches above, loose matches on Subject: below --
2011-02-18  3:52 Tomasz Gregorek

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=4D653AA1.8000203@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.