From: Inaky Perez-Gonzalez <inaky@linux.intel.com>
To: ofono@ofono.org
Subject: [patch 00/20] SMS D-Bus support and misc small patches
Date: Fri, 23 Jul 2010 13:59:48 -0700 [thread overview]
Message-ID: <cover.1279918330.git.inaky.perez-gonzalez@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 3223 bytes --]
From: Inaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com>
This (3rd? 4th?) version of the patchset builds the D-Bus support on
top of the new _txq_submit() callback mechanism.
Note there are still a couple of opens that need discussion:
- the message ID is generated based on the contents of the message --
thus, the current way doesn't work. We need the caller to
_txq_submit() to generate it. It's been left out of the STK
stc.c:handle_command_send_sms() because I am not sure what is the
right way to do it -- need feedback on that.
- The generation of the SMS message ID based on contents still has
shortcomings: if we submit two messages with the same content and
destination number, the ID is the same [sms.c:sms_msg_send()]. What
other factor would make sense to add? time?
The following changes since commit 94344e967b4cd3edd65aa5254ef4b4f5dd037e69:
Denis Kenzior (1):
TODO: Major updates to STK related tasks
are available in the git repository at:
git://gitorious.org/~inakypg/ofono/ofono-inakypg.git master
Patches follow for reviewing convenience.
Inaky Perez-Gonzalez (20):
bug.h: Add BUILD_BUG_ON() and friends for compile-time assert checking
write_file: make transaction-safe
manpage: explain debugging options to -d
SMS: introduce message ID API
introduce DECLARE_SMS_ADDR_STR()
_assembly_encode_address: export and rename
SMS: implement SHA256-based message IDs [incomplete]
sms: document the org.ofono.SMSMessage D-Bus interface
SMS: document handle_sms_status_report()
sms_text_prepare: document @use_delivery_reports
SMS: rename create_tx_queue_entry() to tx_queue_entry_new()
struct tx_queue_entry: add a destructor
SMS: encapsulate D-Bus specific data in 'struct sms_msg_dbus_data'
SMS: introduce bare state machine and transitions
SMS: introduce Wait-for-Status-Report state and infrastructure
SMS: introduce a state change callback for TX messages
SMS: export outgoing messages over D-Bus
SMS: send D-Bus SMS-MSG::PropertyChanged signals when message changes status
SMS: introduce sms_msg_cancel and its D-Bus wrapper
SMS: Implement D-Bus SMS-MSG::GetProperties
HACKING | 10 +
Makefile.am | 5 +-
doc/ofonod.8 | 5 +-
doc/sms-api.txt | 49 ++++-
src/bug.h | 50 ++++
src/ofono.h | 42 +++-
src/sms.c | 598 ++++++++++++++++++++++++++++++++++------
src/smsutil.c | 206 ++++++++++++++-
src/smsutil.h | 122 ++++++++
src/stk.c | 24 ++-
src/storage.c | 42 ++-
test/test-sms-msg-cancel | 173 ++++++++++++
test/test-sms-msg-state-change | 24 ++
unit/test-sms-msg-id.c | 212 ++++++++++++++
14 files changed, 1449 insertions(+), 113 deletions(-)
create mode 100644 src/bug.h
create mode 100755 test/test-sms-msg-cancel
create mode 100755 test/test-sms-msg-state-change
create mode 100644 unit/test-sms-msg-id.c
next reply other threads:[~2010-07-23 20:59 UTC|newest]
Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-23 20:59 Inaky Perez-Gonzalez [this message]
2010-07-23 20:59 ` [patch 01/20] bug.h: Add BUILD_BUG_ON() and friends for compile-time assert checking Inaky Perez-Gonzalez
2010-07-23 21:41 ` Denis Kenzior
2010-07-23 21:57 ` Inaky Perez-Gonzalez
2010-07-23 21:59 ` Denis Kenzior
2010-07-23 20:59 ` [patch 02/20] write_file: make transaction-safe Inaky Perez-Gonzalez
2010-07-23 21:57 ` Denis Kenzior
2010-07-23 22:31 ` Inaky Perez-Gonzalez
2010-07-23 20:59 ` [patch 03/20] manpage: explain debugging options to -d Inaky Perez-Gonzalez
2010-07-23 22:05 ` Denis Kenzior
2010-07-23 20:59 ` [patch 04/20] SMS: introduce message ID API Inaky Perez-Gonzalez
2010-07-27 0:10 ` Denis Kenzior
2010-07-23 20:59 ` [patch 05/20] introduce DECLARE_SMS_ADDR_STR() Inaky Perez-Gonzalez
2010-07-23 22:30 ` Denis Kenzior
2010-07-23 20:59 ` [patch 06/20] _assembly_encode_address: export and rename Inaky Perez-Gonzalez
2010-07-23 22:31 ` Denis Kenzior
2010-07-23 20:59 ` [patch 07/20] SMS: implement SHA256-based message IDs [incomplete] Inaky Perez-Gonzalez
2010-07-27 17:03 ` Denis Kenzior
2010-07-29 21:26 ` Inaky Perez-Gonzalez
2010-07-29 21:37 ` Denis Kenzior
2010-07-31 0:22 ` Inaky Perez-Gonzalez
2010-07-23 20:59 ` [patch 08/20] sms: document the org.ofono.SMSMessage D-Bus interface Inaky Perez-Gonzalez
2010-07-23 23:11 ` Denis Kenzior
2010-07-26 17:19 ` Inaky Perez-Gonzalez
2010-07-26 18:05 ` Denis Kenzior
2010-07-26 20:41 ` Inaky Perez-Gonzalez
2010-07-23 20:59 ` [patch 09/20] SMS: document handle_sms_status_report() Inaky Perez-Gonzalez
2010-07-23 20:59 ` [patch 10/20] sms_text_prepare: document @use_delivery_reports Inaky Perez-Gonzalez
2010-07-23 23:01 ` Denis Kenzior
2010-07-23 20:59 ` [patch 11/20] SMS: rename create_tx_queue_entry() to tx_queue_entry_new() Inaky Perez-Gonzalez
2010-07-23 23:02 ` Denis Kenzior
2010-07-26 20:49 ` Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 12/20] struct tx_queue_entry: add a destructor Inaky Perez-Gonzalez
2010-07-23 23:06 ` Denis Kenzior
2010-07-23 23:11 ` Inaky Perez-Gonzalez
2010-07-23 23:14 ` Denis Kenzior
2010-07-26 18:48 ` Inaky Perez-Gonzalez
2010-07-26 20:49 ` Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 13/20] SMS: encapsulate D-Bus specific data in 'struct sms_msg_dbus_data' Inaky Perez-Gonzalez
2010-07-27 17:08 ` Denis Kenzior
2010-07-29 21:47 ` Inaky Perez-Gonzalez
2010-07-29 22:17 ` Denis Kenzior
2010-07-29 23:23 ` Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 14/20] SMS: introduce bare state machine and transitions Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 15/20] SMS: introduce Wait-for-Status-Report state and infrastructure Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 16/20] SMS: introduce a state change callback for TX messages Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 17/20] SMS: export outgoing messages over D-Bus Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 18/20] SMS: send D-Bus SMS-MSG::PropertyChanged signals when message changes status Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 19/20] SMS: introduce sms_msg_cancel and its D-Bus wrapper Inaky Perez-Gonzalez
2010-07-27 17:16 ` Denis Kenzior
2010-07-30 23:12 ` Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 20/20] SMS: Implement D-Bus SMS-MSG::GetProperties Inaky Perez-Gonzalez
2010-07-27 17:18 ` Denis Kenzior
2010-08-02 19:14 ` Inaky Perez-Gonzalez
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=cover.1279918330.git.inaky.perez-gonzalez@intel.com \
--to=inaky@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox