Age | Commit message (Collapse) | Author | Files | Lines | |
---|---|---|---|---|---|
2009-09-10 | Fix cut-paste error | Denis Kenzior | 1 | -1/+1 | |
2009-09-10 | Add initial Cell Broadcast atom implementation | Denis Kenzior | 1 | -0/+65 | |
2009-09-08 | Add modem properties for booleans | Marcel Holtmann | 1 | -0/+6 | |
2009-09-02 | Get rid of node argument in ofono_modem_create | Denis Kenzior | 1 | -1/+1 | |
2009-09-02 | Add modem property capability | Denis Kenzior | 1 | -0/+8 | |
Add ofono_modem_set_string & ofono_modem_get_string Add ofono_modem_set_integer & ofono_modem_get_integer | |||||
2009-09-01 | populate should return void | Denis Kenzior | 1 | -1/+1 | |
2009-09-01 | Make remove functions not return anything | Denis Kenzior | 13 | -13/+13 | |
2009-09-01 | change int vendor to unsigned int vendor | Denis Kenzior | 12 | -25/+39 | |
2009-09-01 | Add vendor flag capability to voicecalls | Denis Kenzior | 1 | -1/+2 | |
2009-09-01 | Add vendor flag capability to ussd | Denis Kenzior | 1 | -2/+2 | |
2009-09-01 | Add vendor flag capability to ssn | Denis Kenzior | 1 | -2/+2 | |
2009-09-01 | Add vendor flag capability for sms | Denis Kenzior | 1 | -2/+2 | |
2009-09-01 | Add vendor flag capability to sim | Denis Kenzior | 1 | -2/+2 | |
2009-09-01 | Add vendor flag capability to phonebook | Denis Kenzior | 1 | -1/+2 | |
2009-09-01 | Add vendor flag capability to netreg | Denis Kenzior | 1 | -2/+2 | |
2009-09-01 | Add vendor flag capability to devinfo | Denis Kenzior | 1 | -1/+2 | |
2009-09-01 | Add vendor flag to call-settings | Denis Kenzior | 1 | -1/+2 | |
2009-09-01 | Add vendor parameter to call-meter | Denis Kenzior | 1 | -1/+2 | |
2009-09-01 | Add vendor flag to call-forwarding | Denis Kenzior | 1 | -1/+2 | |
2009-09-01 | Introduce vendor parameter to call-barring | Denis Kenzior | 1 | -1/+2 | |
2009-08-31 | Expose service dialling numbers stored on SIM through SimManager. | Andrzej Zaborowski | 1 | -0/+9 | |
2009-08-30 | Reformat comments a little | Denis Kenzior | 1 | -6/+8 | |
2009-08-30 | driver callback naming | Andres Salomon | 1 | -0/+9 | |
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. | |||||
2009-08-24 | Convert to fully non-recursive build system | Marcel Holtmann | 1 | -26/+0 | |
2009-08-24 | Fix broken dependency on include/ofono/*.h files | Rémi Denis-Courmont | 1 | -7/+9 | |
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. | |||||
2009-08-21 | Introduce the Modem driver API | Denis Kenzior | 2 | -2/+30 | |
2009-08-20 | Rename ofono_modem_*_userdata | Denis Kenzior | 1 | -2/+2 | |
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 | |||||
2009-08-20 | Break out device info into a separate driver | Denis Kenzior | 3 | -1/+70 | |
The modem info ops (Serial, Manufacturer, etc) are broken into a separate oFono atom. This is mainly to support a proper modem device driver infrastructure. As a result the data structures have been tweaked a little. driver.h is now empty and has been removed. | |||||
2009-08-19 | Evolve the voicecall driver | Denis Kenzior | 3 | -1/+125 | |
2009-08-19 | Evolve the netreg driver | Denis Kenzior | 2 | -1/+114 | |
2009-08-19 | Move mcc & mnc defines to types.h | Denis Kenzior | 1 | -0/+4 | |
2009-08-19 | Evolve message waiting low level API | Denis Kenzior | 2 | -1/+42 | |
2009-08-19 | Evolve SIM driver | Denis Kenzior | 2 | -1/+137 | |
2009-08-19 | Move ofono_destroy_fun to types.h | Denis Kenzior | 1 | -0/+2 | |
2009-08-19 | Evolve SMS driver API | Denis Kenzior | 2 | -1/+77 | |
2009-08-17 | Evolve the USSD API | Denis Kenzior | 2 | -1/+64 | |
2009-08-14 | Refactor cssn into ofono_ssn | Denis Kenzior | 2 | -1/+62 | |
Make this into a fully fledged entity, with a driver instead of the current kludge. This means modem drivers can actually choose whether to instantiate a CSSN atom or not Move the notification functions from voicecall.c into ssn.c. Move the cssn.h header into include/ssn.h and refactor Update call barring to utilize ofono_ssn and use the new atom_watch functionality to detect when ssn has been added or removed | |||||
2009-08-14 | Move CALL_SETTINGS_INTERFACE define to dbus.h | Denis Kenzior | 1 | -0/+1 | |
2009-08-14 | Move PHONEBOOK_INTERFACE define to dbus.h | Denis Kenzior | 1 | -0/+1 | |
2009-08-14 | Move CALL_METER_INTERFACE define to dbus.h | Denis Kenzior | 1 | -1/+1 | |
2009-08-14 | Move CALL_FORWARDING_INTERFACE define to dbus.h | Denis Kenzior | 1 | -0/+2 | |
2009-08-14 | Move CALL_BARRING_INTERFACE definition to dbus.h | Denis Kenzior | 1 | -0/+1 | |
2009-08-14 | Evolve the phonebook driver | Denis Kenzior | 2 | -1/+73 | |
2009-08-14 | Evolve call settings driver | Denis Kenzior | 2 | -1/+80 | |
2009-08-14 | Move clir and cug option enums to types.h | Denis Kenzior | 1 | -0/+13 | |
2009-08-14 | Evolve the call meter driver API | Denis Kenzior | 2 | -1/+88 | |
2009-08-14 | Evolve the call-forwarding driver | Denis Kenzior | 2 | -1/+89 | |
2009-08-14 | Evolve the call-barring driver | Denis Kenzior | 2 | -1/+70 | |
- Move call barring related functionality out of driver.h into a dedicated file, call-barring.h - Update to use the new atom framework - Remove over-reliance on the modem structure, call-barring structure should now be used instead whenever possible | |||||
2009-08-13 | Move certain parts of driver.h to types.h | Denis Kenzior | 2 | -1/+55 | |
2009-08-12 | Add function for printing warnings | Marcel Holtmann | 1 | -0/+2 | |