Add a separate plugin for connection to phone simulator

Update modem.conf example

Fix emitting Modems Property in the wrong place

Mention Andres' contributions

G1: Add a G1 syntax for parsing

This is based on the generic_at parser, with unnecessary stuff removed.

The G1 routinely screws up CRLFs, so the parser needs to account for
that.  This parser ignores leading CRLFs (which is what reference-ril
does as well), as well as trailing LFs (which are sometimes left out).
CRs are used as end-of-message indicators.  Since we're not bothering
tracking CRLFs, there's also no need for a GARBAGE state, or MULTILINE
stuff.

G1: Add initial HTC G1 modem support

G1 plugin is based on generic_at, with a bunch of stuff dropped
and simplified.  We use AT+CFUN=1 for powering on rather than having
a configurable init string.  We also manually set the default state
during init (the G1 appears to start in mode V0 by default).  The
device (/dev/smd0) is hardcoded.

Squash compiler warning

Update modemconf driver to the new ofono_modem_create

Change plugins to the new ofono_modem_create api

Get rid of node argument in ofono_modem_create

Add simple utility to validate paths

Add modem property capability

Add ofono_modem_set_string & ofono_modem_get_string
Add ofono_modem_set_integer & ofono_modem_get_integer

Use OFONO_AT_DEBUG even for MBM plugin

Unset the driver data when MBM driver is removed

Add example for modem.conf and install it if requested

Add really simple modem configuration parsing

Add skeleton for plugin to handle static modem configurations

Update makefile.am with new doc files

Add call-settings api doc

Add call-meter api doc

Add call-forwarding api doc

Add address & netmask properties to GPRS api

Update GPRS api with context use type

Add test script for listing all operator details

Add test script to monitor all PropertyChanged signals

Add two extra test programs for modem interface

Add initial plugin for Ericsson MBM devices

Rename isi drivers to isimodem

Be more consistent with directory naming

Rename generic_at drivers to atmodem

Be more consistent with directory naming

populate should return void

Make remove functions not return anything

change int vendor to unsigned int vendor

Add first steps for a udev based modem detection plugin

Update D-Bus configuration options and check for udev

Add ofono_modem_get_powered

Add proposed data connection manager API

Add stub vendor.h file

Add vendor flag capability to voicecalls

Add vendor flag capability to ussd

Add vendor flag capability to ssn

Add vendor flag capability for sms

Add vendor flag capability to sim

Add vendor flag capability to phonebook

Add vendor flag capability to netreg

Add vendor flag capability to devinfo

Add vendor flag to call-settings

Add vendor parameter to call-meter

Add vendor flag to call-forwarding

Introduce vendor parameter to call-barring

gisi: Getter function for pipe handle

Fix being able to set powered after failure

If enable failed, subsequent attempts will also fail.

Add PDU listing support

Don't return sdn numbers until ready

Simplify EFsdn reading code

Rename MNCLength to MobileNetworkCodeLength

Refactor sim mnc length

Style Fix

Simplify nested ifs

Simplify overly complicated code

Expose service dialling numbers stored on SIM through SimManager.

Decode and encode alpha-identifier fields

Add identifier argument to sim_adn_parse and sim_adn_build.

Also fix the number length passed to extract_bcd_number in sim_adn_parse.

Read EFad and expose the IMSI MNC length (2-3)

Reformat comments a little

driver callback naming

On Sun, 30 Aug 2009 13:45:45 -0500
Denis Kenzior <denkenz@gmail.com> wrote:

> Hi Andres,
>
> > static struct ofono_modem_driver g1_driver = {
> >         .name = "HTC G1",
> >         .probe = g1_probe,
> >         .enable = g1_enable,
> >         .disable = g1_disable,
> >         .remove = g1_remove,
> >         .populate = g1_populate,
> > };
> >
>
> So the current intention:
> .probe - Detect whether device is really supported by the plugin,
> initialize any data structures specific to the device
> .remove - Destroy data structures
> .enable - Perform power up
> .disable - Perform power down
> .populate - Populate the atoms supported by this device (e.g. netreg,
> voicecall, etc)  This is called by the core after every power cycle,
> when the device is brought up.
>

Thanks!  See patch below.

> >
> > Of course, I'm also wondering why there needs to be two separate
> > layers of calls in the first place.  Why not have drivers register
> > everything from within probe, call ofono_set_powered(modem, TRUE)
> > once the device is ready, and be done with it?
>
> The reason for this is e.g. airplane mode, where you physically want
> to turn off the device.  Another case is for battery / power reasons,
> e.g. a netbook with a USB modem that is not being used.
>

Fair enough.  In the kernel, we have callbacks named suspend/resume
to handle that.

> > The only reason why this doesn't blow up in the generic_at plugin is
> > because the driver_data is leaked.  If one were to free it from
> > generic_at_exit in the wrong place (since it's allocated from
> > generic_at_init, it would make sense to free it in generic_at_exit),
> > one would see the same SEGV/SIGBUS/SIGILL errors upon ctrl-c.
>
> So the leak has now been fixed.
>
> I think you're being unnecessarily harsh here.  To be fair, the
> generic_at driver does something like this at init:

My criticism is simply w/ the naming.  'enable'/'disable' doesn't imply
anything about power.  powerup/powerdown, poweron/poweroff,
suspend/resume would all imply power state changes (at least the latter
would be familiar to those who do kernel stuff).  Having comments that
describe what the callbacks do would also work, though.

>From 80a7b54d52201dfd7d8b590457450ae0a4f72888 Mon Sep 17 00:00:00 2001
From: Andres Salomon <dilinger@collabora.co.uk>
Date: Sun, 30 Aug 2009 15:56:16 -0400
Subject: [PATCH] Add comments to ofono_modem_driver struct

Document what all the callbacks do.

Fix memory leak in generic_at driver

Add debug to modem_create

Make history drivers work again

Fix a problem with CtrlZ echo

When a PDU is submitted and echoed by the modem, the CtrlZ character
is also echoed back.  Fix the parser to handle this case appropriately.

gisi: return GPRS interface index/name

gisi: Fix indication debugging

Rename all files under drivers/isimodem/

Fix typo in technology name

Convert to fully non-recursive build system

Use script to generate list of builtin plugin symbols

Use proper variables for various commands

Add straw-man for rest of ISI modem driver

Fix broken dependency on include/ofono/*.h files

The fact that the directory exists does not prove that (all of) the
header file symbolic links have been created. This is particularly
annoying when a new header file gets added.

Fix build warning

Fix typos

gisi: Fix warning in g_isi_subscribe()

Save the GIsiModem instance in GIsiClient, so that calling
g_isi_subscribe() doesn't throw a warning.

Fix memory leak in ISI phonebook driver

gisi: Add debugging hooks

Fix memory leak in isimodem.c

Move dump_msg() to isi.h as static inline

Fix getting the offset into sms fragment bitmap.

Otherwise only 64 fragments fit.

Add generic_at modem driver

Useful for testing or when the device is extremely 27.007 compliant,
e.g. phonesim

Update the isi driver to the new modem driver api

Introduce the Modem driver API

Remove org.ofono.at.Manager

Add CONFIGDIR automake magic

Remove unused function

Enable multi-modem support

This patch enables concurrent use of multiple ISI modems, e.g., all
connected via USB using cdc_phonet.

Enable multiple instances of phonebook driver

Enable multiple instances of devinfo driver

Move struct isi_data to private

Refactor isidevinfo

Remove storing of pending request objects to linked list;
g_isi_client_destroy() will clear those properly. Fix naming of enums,
and use DECLARE_SUCCESS() macro.

Pass modem instance to phonebook driver

Fix unused variable l

Rename ofono_modem_*_userdata

Rename ofono_modem_get_userdata to ofono_modem_get_data
Rename ofono_modem_set_userdata to ofono_modem_set_data

For consistency with other drivers

Hide the ofono_modem datastructure