bluez5: Fix copy paste error

netreg: Fix broken CMER behavior on MBM

mbm: Delay init until EMRDY

If we send these commands before the EMRDY notification has been seen,
they will simply return an error.

hfpmodem: Add comment for previous commit

hfpmodem: Fix release-and-swap without +CIEV

Some phones do not send the corresponding call state update (+CIEV)
after a successful release-and-swap operation (AT+CHLD=1).

This has been observed with a Nokia 500, while testing ReleaseAndSwap()
while an active and a held call exist:

ofonod[20414]: > AT+CLCC\r
ofonod[20414]: < \r\n+CLCC: 1,0,1,0,0,"<number1>",145\r\n
ofonod[20414]: < \r\n+CLCC: 2,0,0,0,0,"<number2>",145\r\n
ofonod[20414]: < \r\nOK\r\n
ofonod[20414]: > AT+CHLD=1\r
ofonod[20414]: < \r\nOK\r\n

After this, no +CIEV is received, but the call has been hung up.

The proposed approach to solve this consists of using AT+CLCC, unless
a call release has been received within a specific time period.

The result fixes the problem as can be seen below:

ofonod[20847]: < \r\n+CLCC: 1,0,1,0,0,"<number1>",145\r\n
ofonod[20847]: < \r\n+CLCC: 2,0,0,0,0,"<number2>",145\r\n
ofonod[20847]: < \r\nOK\r\n
ofonod[20847]: > AT+CHLD=1\r
ofonod[20847]: < \r\nOK\r\n
ofonod[20847]: > AT+CLCC\r
ofonod[20847]: < \r\n+CLCC: 1,0,0,0,0,"<number1>",145\r\n
ofonod[20847]: < \r\nOK\r\n
ofonod[20847]: < \r\n+CIEV: 5,2\r\n
ofonod[20847]: < \r\n+CIEV: 5,0\r\n

hfpmodem: Avoid transitional voicecall states

While processing the result of AT+CLCC, process the differences in a way
that disconnections are reported first, then call state changes and
finally new calls.

The goal is to avoid unnecessary transitional states such as two active
calls existing at the same time.

hfpmodem: Refactor voicecall notify with foreach

Define a helper function in order to use foreach statements when
ofono_voicecall_notify() needs to be called.

gdbus: Check for valid path before handling properties

hfpmodem: Add support for sending the supported codecs

Right now, only the mandatory CVSD codec is supported. The mSBC
mandatory codec is "temporarily" not supported.

The spec alows this, HFP 1.6 Spec Section 4.34.1 page 92: "If wide band
speech is supported then the mandatory codec (mSBC) shall be included
unless it is temporarily unavailable."

hfpmodem: Add version defines for HFP 1.6

AUTHORS: Mention Claudio's contributions

build: Use AM_CPPFLAGS instead of INCLUDES

Makefile.am:518: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')

build: Do not use deprecated AM_CONFIG_HEADER

The long-obsoleted AM_CONFIG_HEADER macro was removed in automake 1.13.
Use AC_CONFIG_HEADERS instead.

storage: Include sys/types.h for ssize_t

src/storage.h:32:1: error: unknown type name 'ssize_t'
src/storage.h:36:1: error: unknown type name 'ssize_t'

hfp_hf_bluez5: Add BlueZ Profile handler

This patch declares the external HFP Profile handler. It contains the
initial implementation of the D-Bus Profile1 interface and methods
responsible for handling Bluetooth connections.

bluez5: Add a couple of defines

hfp_hf_bluez5: Add initial skeleton implementation

bluetooth: Add profile register / unregister

Adds utility API to handle BlueZ 5 RegisterProfile and UnregisterProfile
methods.

hfp_hf: Add initial file for external HFP

This patch adds the initial file to support external HFP profile and
BlueZ 5. "hfp_hf_bluez5" plugin will implement an external Bluetooth
profile compatible with BlueZ 5, and "hfp_hf_bluez4" plugin will keep
the compatibility with BlueZ 4.

bluetooth: Initial files for BlueZ 5

This patch adds the file for Bluetooth(BlueZ 5) oFono plugin.

bluetooth: Rename bluetooth plugins

As BlueZ 5 introduced backwards incompatible API changes, and we want to
keep support for BlueZ 4 based plugins for some time, we need to separate
the plugins that are based on BlueZ 4 from the ones based on BlueZ 5.

The bluetooth.c plugin has now been renamed to bluez4.c and the hfp_hf.c
plugin is renamed to hfp_hf_bluez4.  This will make it easy to add a
HFP HF plugin for BlueZ 5.

build: Add configure option for BlueZ 5 support

Going forward, BlueZ 5 will be supported by default.  BlueZ 4 support
will have to be explicitly enabled by using '--enable-bluez4'.

As before, the '--disable-bluetooth' configure option will disable all
bluetooth support (BlueZ 4 and BlueZ 5)

sim: Fix whitespace violation

phonesim: Add support for SIM hotswap

stk: Fix crash with Modem-Handled SIM Refresh

Revert "stk: Fix crash in SIM refresh handling"

This reverts commit dfd1fae50b8197b0b4ed81fed3139726e4ae1bf5.

AUTHORS: Mention Forest's contributions

sierra: Initialize GSM error reporting separately

sierra: Create GPRS context in post_sim function

This matches the huawei plugin.

gdbus: Fix memory leak on properties_set()

The pointer returned by dbus_message_iter_get_signature() must be freed
with dbus_free().

Fixes this memory leak:

==1857== 16 bytes in 1 blocks are definitely lost in loss record 104 of
251
==1857==    at 0x402BF52: realloc (in
/usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==1857==    by 0x415E286: dbus_realloc (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x415E70B: ??? (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x415F17B: ??? (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x414CB33: dbus_message_iter_get_signature (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==1857==    by 0x8053239: properties_set (object.c:899)
==1857==    by 0x5FFFFF: ???
==1857==

gdbus: Check signature of property value before calling setter

gdbus: Don't include just added interfaces in GetManagedObjects

If we received a call to ObjectManager.GetManagedObject we should not
include in the response the interfaces in data->added. This is because
it's not guaranteed that those interfaces will trigger an
InterfacesAdded signal, which is the case if the interface is removed in
the same mainloop iteration.

gdbus: Simplify generated introspection

The generated introspection is not supposed to be read as is by human,
so there's no point in printing the indentation or writing more code to
use auto-close tags.

If it's desired to read the raw xml file, user can always use other
tools to transform the output such as "xmllint --format".

This also fixes a missing </property> when property is deprecated.

gdbus: Hold client reference during get name owner reply

gdbus: Call check_signals when sending signals with g_dbus_send_message

If message passed to g_dbus_send_message is a signal verify if it is a
valid and there really exists an interface with respective signal name.

gdbus: Check if the interface being registered is valid

This prevent registering interfaces that are empty or have all members
marked as experiemental.

gdbus: Introduce G_DBUS_PROPERTY_FLAG_EXPERIMENTAL

This flag can be used to mark properties as experimental, marked
properties are disabled by default and can be enabled by setting
G_DBUS_FLAG_ENABLE_EXPERIMENTAL using g_dbus_set_flags.

gdbus: Introduce G_DBUS_SIGNAL_FLAG_EXPERIMENTAL

This flag can be used to mark signals as experimental, marked
signals are disabled by default and can be enabled by setting
G_DBUS_FLAG_ENABLE_EXPERIMENTAL using g_dbus_set_flags.

gdbus: Introduce G_DBUS_METHOD_FLAG_EXPERIMENTAL

This flag can be used to mark methods as experimental, marked
methods are disable by default and can be enabled by setting
G_DBUS_FLAG_ENABLE_EXPERIMENTAL using g_dbus_set_flags.

gdbus: Include changed properties only once per signal

gdbus: Update properties on D-Bus client re-connections

gdbus: Add function to manually refresh properties

gdbus: Add support for proxy property change notifications

gdbus: Fix handling of client connect/disconnect signals

gdbus: Protect standalone proxy creation with client reference

gdbus: Use object manager only if callback functions are set

gdbus: Use client service name and not hardcoded org.bluez

gdbus: Add support for creating D-Bus proxies without object manager

gdbus: Avoid using g_ptr_array_new_full convenience function

The g_ptr_array_new_full function only got introduced with GLib 2.30
and to avoid requiring a newer GLib use g_ptr_array_set_free_func instead.

Release 1.12

doc: Add USAT certification status document

build: Compile support for new D-Bus client helpers

gdbus: Increase the method call timeout to 5 minutes

gdbus: Add helper for sending async error replies

gdbus: Fix race condition with creating duplicate client proxies

gdbus: Fix issue with unref of client while parsing interfaces

gdbus: Fix issue with unref of client in connect handler

gdbus: Add support for D-Bus client method calls

gdbus: Add callback support for handling property changes

gdbus: Handle property changed signals and update property

gdbus: Set property changed filter match rule for each proxy

gdbus: Use a GPtrArray for the match rules

gdbus: Add support for setting D-Bus client basic properties

gdbus: Deep copy of variants also requires a signature

gdbus: Add support for complex D-Bus client properties

gdbus: Add support for D-Bus client properties

gdbus: Add support for D-Bus client proxies

gdbus: Add helper functions for simple D-Bus clients

telit: set baud rate to 115200

Telit HE910 needs the baud rate to be set.

sim: return from cpin when sim ready for telit

Especially for Telit HE910 it is not enough to wait for
entering a PIN code.
If we do not wait for #QSS: 3, subsequent commands,
like +CMER will report SIM BUSY and the network registration
atom will be removed as a consequence.

atutil: Remove unused macro

With commit 6ee21a3fc18460cc12dbddb3b7ec70e941fa4bbc
the usage DECLARE_FAILURE was removed.

test: Tweak list-modems to display battchg

hfp: use ofono_handsfree_battchg_notify

handsfree: Implement BatteryChargeLevel

include: Add battery charge level notify API

doc: Add BatteryChargeLevel to Handsfree

tools: Make two functions of stktest static

build: Use AC_PROG_MKDIR_P instead of AM_PROG_MKDIR_P

build: Reorder default developer build options

build: Reorder configure options

AUTHORS: Mention Mingli's contributions

atmodem: remove pending idle callbacks at removal

If device probe and removal happen in short succession, it's possible
that the idle handler registered in the probe function doesn't run before
the device is removed.  In this case, the idle handler needs to be
unregistered so that it does not run and try to access the data that's
destroyed during the removal.

atmodem: Re-work the CBS driver

We might have mis-interpreted how 27.007 intends for CBS to work.  After
studying the implementation notes of the IMC 6260 modem, the spec intent
made a little bit more sense.

qmi: Include persistent device configuration service

qmi: Print service number if name is not known

gdbus: Add g_dbus_add_properties_watch function

Convenient function to create watches for D-Bus properties.

gdbus: Don't automatically attach ObjectManager

Let each project attach the object manager interface instead of
registering it automatically.

gdbus: Don't register DBus.Properties with no properties

Delay registering DBus.Properties interface until the moment there are
properties on that path. This is needed for objects that currently don't
expose any property to not export the interface.

sms: Don't try to start the queue if already active

If the queue is currently inside a submit operation and the network
registration state changes, do not try to schedule the next operation.

ifx: Bump the setup timeout

The newer IFX firmware was changed, it no longer accepts AT commands
until the UA frame has been sent back to the AP.  The commands are
simply dropped on the floor.  This requires a proper fix in GAtMux, but
until then simply bump up the timeout to get around this issue.

ifx: Fix the case of modem-originated call

During STK Set Up Call, we have modem-originated calls that do not go
through the core 'Dial' method.  Make sure the calls are still detected
in this case.

stk: Fix Set Up Call with pause characters

Set Up Call with extra DTMF characters after the phone number should be
set up with only the dialed number.  Otherwise we get a sequence like
this:
{VoiceCallManager} [CallAdded] /ifx_0/voicecall01 { LineIdentification =
+012340123456c1c2, Name = , Emergency = False, Multiparty = False,
RemoteHeld = False, State = alerting, RemoteMultiparty = False }

stk: Print what kind of info the SIM wants

In Provide Local Information implementation

voicecall: Add extra debug info

voicecall: Fix crash

sim: Fix crash

ofonod[32055]: ++++++++ backtrace ++++++++
ofonod[32055]: #0  0x7f6af0ee3b30 in /lib64/libc.so.6
ofonod[32055]: #1  0x4c2466 in __ofono_watchlist_remove_item() at
src/watch.c:57
ofonod[32055]: #2  0x4b5b73 in ofono_sim_remove_spn_watch() at
src/sim.c:2715
ofonod[32055]: #3  0x497c30 in netreg_unregister() at src/network.c:1817
ofonod[32055]: #4  0x4912e1 in __ofono_atom_unregister() at
src/modem.c:277
ofonod[32055]: #5  0x491387 in flush_atoms() at src/modem.c:425
ofonod[32055]: #6  0x4b6cb8 in __ofono_sim_refresh() at src/sim.c:3154
ofonod[32055]: #7  0x4b8c41 in handle_command_refresh() at
src/stk.c:2302
ofonod[32055]: #8  0x4baf0d in
ofono_stk_proactive_command_handled_notify() at src/stk.c:3048
ofonod[32055]: #9  0x46c60f in satn_notify() at
drivers/ifxmodem/stk.c:229
ofonod[32055]: #10 0x7f6af1711455 in /usr/lib64/libglib-2.0.so.0
ofonod[32055]: #11 0x43e729 in at_chat_match_notify() at
gatchat/gatchat.c:421
ofonod[32055]: #12 0x440da8 in received_data() at gatchat/gatio.c:125
ofonod[32055]: #13 0x441834 in dispatch_sources() at
gatchat/gatmux.c:157
ofonod[32055]: #14 0x441bbd in received_data() at gatchat/gatmux.c:215
ofonod[32055]: #15 0x7f6af173dfc3 in /usr/lib64/libglib-2.0.so.0
ofonod[32055]: #16 0x7f6af16ef065 in /usr/lib64/libglib-2.0.so.0
ofonod[32055]: #17 0x7f6af16efd0f in /usr/lib64/libglib-2.0.so.0
ofonod[32055]: #18 0x7f6af16efef9 in /usr/lib64/libglib-2.0.so.0
ofonod[32055]: #19 0x7f6af16f032f in /usr/lib64/libglib-2.0.so.0
ofonod[32055]: #20 0x48f5f8 in main() at src/main.c:249
ofonod[32055]: #21 0x7f6af0ed04bd in /lib64/libc.so.6
ofonod[32055]: +++++++++++++++++++++++++++

ifx: Rework SIM state handling

When performing a SIM REFRESH, particularly NAA Initialization, we need
to wait until the modem sends XSIMSTATE: 7, not XSIMSTATE:2.  Even on
normal initialization it seems safer to wait until XSIMSTATE:2.

atmodem: For IFX XSIMSTATE 7 should never occur

in the case of a PIN entry, XSIMSTATE of 7 (Ready to Attach) should
never occur

stk: Handle self explanatory icons in idle mode text

test: Use pretty-printing from monitor-ofono