Hi Guillaume,
>>>>>>>>>> and what about the case when
the SIM card is present, but PIN locked?
>>>>>>>>>>
>>>>>>>>>>> According to the result, it might be
interesting to send ATI when the
>>>>>>>>>>> constructor plugin is probe by oFono.
>>>>>>>>>>> Thus with +GCAP info we can decide which
driver to use.
>>>>>>>>>> Is sending +GCAP after ATI really a standard?
Have we tried anything
>>>>>>>>>> else besides Huawei or ZTE?
>>>>>>>>> I tried with more dongles from different vendors, as
attached table.
>>>>>>>>> The scenarios include:
>>>>>>>>> With valid sim card, sim card PIN locked, no sim
card, sim card locked.
>>>>>>>>> N(ROM) in table indicates the SIM in ROM already.
>>>>>>>>> ATI command can always return GCAP content in all
tests.
>>>>>>>> and what about other manufactures other than Huawei, ZTE
and SpeedUp?
>>>>>>>> What about Sierra, Ericsson etc.?
>>>>>>> Just checked Dell 5530 with Ericsson module,
>>>>>>> With SIM card or not, at+gcap can return +GCAP:+CGSM, +DS
>>>>>>> But the ATI only returns: D5530
>>>>>> I think it is clear that we need to do our homework here and
properly
>>>>>> document the different manufacturers. Someone sending patches
for our
>>>>>> doc/ directory?
>>>>> There're many vendors of 3G dongle..
>>>>> Huawei, ZTE (they share 70%+ of global market), Longcheer, Haier,
Sentar, Viton, D-link, SCV, BandRich, Strongrising.. (more than 30 vendors in China)
>>>>> Sierra, Sony-Ericsson, Option, Novatel, Alcatel, Samsung, LG,
AnyData, C-motech, Micromax...
>>>>> We can try with them step by step, but can we work out the 2 biggest
firstly?
>>>>> Looks ATI command can work for both Huawei and ZTE dongles.
>>>>>
>>>> I agree here, the work to be done over all manufacturers will be
>>>> fastidious and might require a lot of dongles that we don't have
currently.
>>>> Maybe we could do as Ying An proposed as we are sure ATI works for
>>>> Huawei and ZTE (at least the ones we have).
>>>> However, conerning ZTE I haven't seen any CDMA dongle for the
moment.
>>>>
>>>>>>>>>> Also you do realize that the GAtChat object and
thus the file descriptor
>>>>>>>>>> is owned by the modem plugin. The plugin itself
is the only one that
>>>>>>>>>> should do any kind of IO.
>>>>>>>>>>
>>>>>>>>>> So if we require to run ATI first to identify if
we are GSM or CDMA,
>>>>>>>>>> then this is a per modem manufacture specific
detail. And we rather add
>>>>>>>>>> a helper function like we did for CPIN polling
that makes this easier.
>>>>>>>>>>
>>>>>>>>> In current code the 'driver' is hardcoded by
comparing with vendor_list[].
>>>>>>>>> So if it possible to break the step into several:
>>>>>>>>> vendor_list[] in udevng just cares about vendor - by
comparing vendor ID
>>>>>> only,
>>>>>>>>> and add all possible drivers according to that
vendor - (for example add
>>>>>>>>> WCDMA, CDMA2k, TDSCDMA, LTE ...drivers if Huawei
dongle is plugged
>>>>>> in),
>>>>>>>>> and the probe interface in each driver does real
probe work as to issue
>>>>>>>>> ATI command to ensure only correct driver will be
loaded?
>>>>>>>> As I said before, the only time IO can be started is
when the ->enable()
>>>>>>>> callback of the modem plugin is called. Not a second
earlier.
>>>>>>> But if done after enable() called, from semantic aspect the
correct driver has
>>>>>> been
>>>>>>> chosen. Indeed the probe() interface in each driver is not
doing something to
>>>>>> probe,
>>>>>>> then can the work be done in probe()? As set CFUN=1 then
doing some dongle
>>>>>> vendor
>>>>>>> specific work as query model or network mode by ATI, AT+GCAP
command,
>>>>>> etc..? After
>>>>>>> that disable dongle when quit probe()?
>>>>>> The probe() callback is for accepting the driver and allocating
required
>>>>>> local data structures. It is not for IO. And as you can see it
has no
>>>>>> callback handling like enable() with set_powered().
>>>>>>
>>>>>> As I said before, no AT commands before enable() has been
called. That
>>>>>> is how it is suppose to be. We are not changing this.
>>>> First, ATI command is working without sending AT+CFUN=1, we could keep
>>>> CFUN=1 into enable() as we do some vendor/modem type specific job
there.
>>>>
>>>> Then vendor plugin can be chosen using udevng using Vendor ID, however
>>>> driver type (CDMA/GSM) can't lie on the Product ID. So it will be
hard
>>>> to chose the right vendor plugin with right type.
>>>> And if we can't send AT command before enable() time we will face to
bag
>>>> end e.g. :
>>>> For huawei plugin we send GSM specific AT command (AT^RFSWITCH) during
>>>> the enable() time.
>>>> We are also querying the sim state using polling mechanism that might
>>>> fail for CDMA modems that is not using SIM.
>>>> What would you suggest here?
>>> as I said before, no AT commands before ->enable() callback from the
>>> core.
>>>
>>> The callback ->probe() is for accepting the modem driver binding and
>>> allocating modem specific data memory. The callback ->remove() is for
>>> cleanup.
>>>
>>> The callbacks ->enable(), ->disable() and ->set_online() are the
only IO
>>> entry points for every modem driver. And we need to keep it like this.
>> Ok, so I suggest to do the ATI at the very beginning of ->enable() callback.
> the first command has to be always ATE0 +CMEE=1 since otherwise you a)
> can not use the permissive syntax parser and b) your error values will
> be useless.
>
> But yes, after that it is fine to send ATI.
>
Ok
>> Then depending on the ATI answer:
>> - tag the huawei modem data with GSM / CDMA type.
>> - send the GSM / CDMA specific AT commands followed by AT+CFUN=1.
> What different commands depending on GSM or CDMA do you actually have?
>
> The AT^RFSWITCH=? is exactly designed to handle if that command is
> supported or not. There are plenty of GSM versions of the Huawei that do
> not support AT^RFSWITCH. You do need to know if this is supported or
> not.
I see, so we can send AT^RFSWITCH for both type. If it is not supported,
it will be ignored using terminator and then use
default AT+CFUN=5.
> Also we do not send AT+CFUN=1 in ->enable() callback. We bring the modem
> into offline mode. The only time you send AT+CFUN=1 is if you have
> hardware that does not support online/offline distinction. So if this is
> true for Huawei CDMA modems, then the obvious questions is why that is
> the case? Or is this a bug with our CDMA support not supporting offline
> mode.
>
For the moment, CDMA modems are not using ->set_online() callback (it is
automatically set online into modem.c).
We will have to make some test to check AT+CFUN=5 is working on CDMA modems.
The Huawei GSM support is using ->set_online() callback. And so this
means that you need it also for CDMA support. Otherwise you are back at
square one.
Actually you need to test the full CDMA support and if it can properly
handle ->set_online() support. If not, then this needs fixing.
Also I am still not seeing proper support for CDMA SIM atom. We need to
stop hacking around here. The core functionality needs to be implemented
first. Without it, the modem plugins can not function properly.
So why are we wasting time with modem details here. If I remember
correctly, Denis and I made it pretty clear that SIM atom and network
registration atom is a fundamental requirement for CDMA support.
Regards
Marcel