All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: Cinterion EHS6 support
Date: Thu, 30 Apr 2015 12:19:21 -0500	[thread overview]
Message-ID: <55426419.9010801@gmail.com> (raw)
In-Reply-To: <5541DBC3.4030003@dynamicdevices.co.uk>

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

Hi Alex,

 >> The error message flags up a problem for the integrator. In the case of
>> OFONO_VENDOR_NOKIA, OFONO_VENDOR_SAMSUNG in at_creg_set_cb() we can
>> assume this has been investigated, and no good alternative event
>> reporting solution is available (or a choice was made not to implement)
>> so this is disabled.
>>

Essentially.  We either don't have the vendor specific commands for 
those modems or they simply don't exist.

>> However there remains an exercise to go through for Cinterion to check
>> whether non-CIND reporting mechanism is available and enable it? If so
>> I'll have a look through the documentation.
>>
>> On a separate but related note, can you tell me if ofono supports
>> monitoring of signal strength whilst a data connection is up, e.g. via
>> use of GSM 07.10  or with modems such as Cinterion that expose multiple
>> virtual ports over USB?

Yes, of course.  Both ways are supported.

> The EHS6 requires use of the AT^SIND command to enable signal strength
> reporting, which are reported with textual indication descriptors (as
> opposed to numerics) in the +CIEV indication, similar to the Telit.
>
> The TC65 documentation doesn't show the "rssi" indicator as being
> supported by AT^SIND (pp87), although it is documented as supported by
> AT+CIND (pp81).
>
> Additionally the TC65 documentation states that "All indicators provided
> by AT+CIND can be handled with AT^SIND as well" so there's some
> confusion there, and unfortunately without the TC65 hardware I can't test.
>
> I have some code here which is correctly handling the "rssi" +CIEVs on
> the EHS6 as a "cinterion" plugin as far as I can see.
>
> If you are happy with the above I could provide a patch-set to replace
> tc65 with a "cinterion" plugin using AT^SIND to setup signal strength
> reporting for the Cinterion vendor?
>

That sounds like a good start.  We can always split the driver into two. 
  That is easy.  Merging two drivers into one is much harder.

Regards,
-Denis

  reply	other threads:[~2015-04-30 17:19 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-29 16:45 Cinterion EHS6 support Alex J Lennon
2015-04-29 18:17 ` Denis Kenzior
2015-04-29 18:24   ` Alex J Lennon
2015-04-29 18:50     ` Denis Kenzior
2015-04-29 19:11       ` Alex J Lennon
2015-04-29 19:17         ` Denis Kenzior
2015-04-29 20:03           ` Alex J Lennon
2015-04-29 20:11             ` Alex J Lennon
2015-04-29 22:57               ` Denis Kenzior
2015-04-30  4:42                 ` Alex J Lennon
2015-04-30  7:37                   ` Alex J Lennon
2015-04-30 17:19                     ` Denis Kenzior [this message]
2015-04-30  9:20                   ` Alex J Lennon

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=55426419.9010801@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.