All of lore.kernel.org
 help / color / mirror / Atom feed
From: Antoine Aubert <a.aubert@overkiz.com>
To: ofono@ofono.org
Subject: Re: [PATCH 2/4] plugins: support for Telit LE910 V2 modem
Date: Thu, 26 Jan 2017 10:40:37 +0100	[thread overview]
Message-ID: <710ce16f-892b-3114-6ca2-039a630fb6f6@overkiz.com> (raw)
In-Reply-To: <3026e500-7392-f3e8-3ee1-0e0817e5b392@gmail.com>

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

Hi,

I'v recently add support for ge910 quad,
(https://github.com/p1ng0o/ofono/commits/ovk) and I had some ugly issues.

1/ doesn't support proper atmux
(https://github.com/p1ng0o/ofono/commit/1b7f64b2faaca91ce711774a12ba79ad35cbb8ed)

Quote from support(a)telit:

"In Telit implementation module is NEVER an initiator since it is up to
controller to send the
SABM command to DLCI 0
* Module is ALWAYS a responder
* Module send the V24 frame as F9 01 EF 09 E3 05 0B 8D 9A F9 where C/R
bit is set to 1
* According to Table 1 this is a response frame
* According to above DTE don’t have to copy the status."

thread with denis,
https://lists.ofono.org/pipermail/ofono/2016-October/016590.html

2/ Ugly init process, forced me to create plugin from scratch
(https://github.com/p1ng0o/ofono/commit/6534001cd5801a30b0c237b697226ed2d02fe774)

- I have to wait QSS=2 for initialize all devices, sometimes it take
more than modem powered timeout !

- Sometime I didn't receive QSS notification at all

Quote from support(a)telit:

"just try to increase the waiting time for QSS answer (also 1 minute at
the beginning: it depends also on the number of records in the SIM
phonebook, ecc.)"

Did you face same issues ?

Antoine Aubert
a.aubert(a)overkiz.com

Le 26/01/2017 à 10:09, gluedig(a)gmail.com a écrit :
> On 01/25/2017 05:37 PM, Denis Kenzior wrote:
>> On 01/25/2017 04:41 AM, Piotr Haber wrote:
>>> LE910 V2 is next generation Telit LTE modem.
>>> It supports 3GPP Rel. 9 LTE Cat. 4 over multiple bands.
>>> Default USB composition uses PID 0x36 and
>>> consists of 6 CDC-ACM serial ports and 1 CDC-NCM network adapter.
>>> ---
>>>  Makefile.am       |   3 +
>>>  plugins/le910v2.c | 400
>>> ++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>>  2 files changed, 403 insertions(+)
>>>  create mode 100644 plugins/le910v2.c
>>>
>> Can you add the LE910 changes into the HE910 driver?  The two are 99%
>> the same except for HE910 supporting voice calls and LE910 using a
>> different gprs_context driver.
>>
>> There's no sense in maintaining a 400 line driver that is virtually
>> identical to what we already have.
>>
>> Regards,
>> -Denis
>>
> Hi,
> of course you are right.
>
> I would prepare a series of patches that renames he910 plugin to xe910 -
> this is how Telit calls the whole family that consists of lots of models
> form 2g (GE910) through 3/3.5g (HE910, DE910, UE910, HE910V2) to
> 4g(LE910, LE910V2)
>
> Then I'll add support for LE910V2 to xe910 plugin.
> How does it sound?
>
> Piotr
> _______________________________________________
> ofono mailing list
> ofono(a)ofono.org
> https://lists.ofono.org/mailman/listinfo/ofono


  reply	other threads:[~2017-01-26  9:40 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-24 14:21 [PATCH v2] drivers: add support for Telit LE910 V2 modem Piotr Haber
2017-01-24 18:55 ` Denis Kenzior
2017-01-25 10:41   ` [PATCH 0/4] add support for Telit LE910V2 Piotr Haber
2017-01-25 10:41     ` [PATCH 1/4] telitmodem: support for CDC-NCM network adapter Piotr Haber
2017-01-25 16:33       ` Denis Kenzior
2017-01-25 10:41     ` [PATCH 2/4] plugins: support for Telit LE910 V2 modem Piotr Haber
2017-01-25 16:37       ` Denis Kenzior
2017-01-26  9:09         ` gluedig
2017-01-26  9:40           ` Antoine Aubert [this message]
2017-01-26 11:13             ` gluedig
2017-01-26 13:46           ` Denis Kenzior
2017-01-25 10:41     ` [PATCH 3/4] doc: description of Telit LE910V2 network setup Piotr Haber
2017-01-25 16:38       ` Denis Kenzior
2017-01-25 10:41     ` [PATCH 4/4] udevng: setup of Telit LE910V2 Piotr Haber

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=710ce16f-892b-3114-6ca2-039a630fb6f6@overkiz.com \
    --to=a.aubert@overkiz.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.