From: Guillaume Zajac <guillaume.zajac@linux.intel.com>
To: ofono@ofono.org
Subject: Re: [PATCH 2/5] speedup: Check for supported modem capabilities first
Date: Wed, 11 Jan 2012 11:48:49 +0100 [thread overview]
Message-ID: <4F0D6911.5050007@linux.intel.com> (raw)
In-Reply-To: <4F09A7D2.5090500@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1123 bytes --]
Hi Denis,
>> We can have a private enum modem type into plugin, but we won't be able
>> to use it into any atoms.
>> Maybe by deduction we can add an enum sim_type into sim.h store the type
>> into the plugin data and pass it at SIM atom creation.
>> What do you think?
> Inside the plugin you can do whatever you want for determining the cdma
> vs gsm modem type. I'm not quite sure I follow your comment about sim.h...
To use only PIN management and retrieving IMSI from current sim atom,
sim type (R-UIM or SIM) has to be specified at atom creation.
The sim type can is determined into Huawei plugin at the same time as
modem type:
- If modem type if GSM, sim type SIM.
- If modem type is CDMA, sim is R-UIM (sim atom is not created in
case of UIM/ROMSIM)
I was proposing to have those 2 types into include/sim.h.
Thus, sim atom can be flagged as SIM or R-UIM to access or not to the
file system and use PIN management only e.g. in:
sim_initialize() and sim_initialize_after_pin() we just get the imsi we
don't try to read the sim file system.
Kind regards,
Guillaume
next prev parent reply other threads:[~2012-01-11 10:48 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-09 13:10 [PATCH 0/5] Unify SpeedUp plugin Guillaume Zajac
2012-01-09 13:10 ` [PATCH 1/5] udevng: Remove huaweicdma label from driver_list Guillaume Zajac
2012-01-07 18:26 ` Denis Kenzior
2012-01-09 13:10 ` [PATCH 2/5] speedup: Check for supported modem capabilities first Guillaume Zajac
2012-01-07 18:26 ` Denis Kenzior
2012-01-09 19:36 ` Marcel Holtmann
2012-01-07 19:26 ` Denis Kenzior
2012-01-10 13:43 ` Guillaume Zajac
2012-01-08 10:12 ` Denis Kenzior
2012-01-10 15:37 ` Guillaume Zajac
2012-01-08 14:27 ` Denis Kenzior
2012-01-11 10:48 ` Guillaume Zajac [this message]
2012-01-11 11:06 ` Denis Kenzior
2012-01-11 15:12 ` Guillaume Zajac
2012-01-09 13:10 ` [PATCH 3/5] speedup: Create GSM/UMTS atoms only if support has been detected Guillaume Zajac
2012-01-09 13:10 ` [PATCH 4/5] speedup: Create CDMA atoms " Guillaume Zajac
2012-01-09 13:11 ` [PATCH 5/5] udevng: Simplify vendor and driver list for SpeedUp Guillaume Zajac
2012-01-09 13:14 ` [PATCH] speedupcdma: Delete unused plugin Guillaume Zajac
2012-01-12 10:00 ` [PATCH 0/5] Unify SpeedUp plugin Guillaume Zajac
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=4F0D6911.5050007@linux.intel.com \
--to=guillaume.zajac@linux.intel.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.