hfp_hf_bluez5: Remove duplicated include

handsfree-audio: Reject SCO if agent is unavailable

This patch rejects the incoming SCO connection if there isn't a
Handsfree Audio Agent registered.

handsfree-audio: Reject SCO if Card is not ready

The Audio Card is being created when the NewConnection from BlueZ
Profile is received, and registered when the service level connection
negotiation finishes. This patch rejects SCO connection if the SCO
incoming connection arrives when the service level negotiation is
ongoing.

handsfree-audio: Check local SCO address

This patch verifies if the local Bluetooth address of the incoming
connection also matches with one of available audio cards.

handsfree-audio: Add NewConnection

This patch adds Agent NewConnection call. The card object path, the SCO
file descriptor, and the codec are being passed to the agent. This
initial version supports CVSD codec only.

handsfree-audio: Add SCO handling

Unlike the previous implementation in the plugin, the SCO/SLC matching
is done based on the Audio Card objects.

Audio Cards are created when the RFCOMM fd descriptor is received, and
registered when the service level connetion is established.

hfp_hf_bluez5: Remove SCO handling

This patch moves the SCO socket handling from hfp_hf_bluez5 plugin.  It
will be instead handled by the handsfree-audio functionality. This is
the initial step to be able to support sending the file descriptor
through the Agent NewConnection method.

hfp_hf_bluez5: Use new bluetooth.h header

bluez5: Remove Bluetooth related utilities

core: Add new Bluetooth header

This patch adds the Bluetooth utility funtions and socket type
declarations to a new header src/bluetooth.h, allowing to share it
between core, and plugins.

bluez5: Add bt_register_profile_with_role

This function allows to specify which role is registering when the
profile does not have specific UUIDs for each role.

handsfree-audio: Check CVSD when registering agent

This patch makes CVSD codec mandatory when registering a Handsfree
Audio Agent.

bluez5: Move ref / unref handsfree audio

This commit moves the ref / unref to hfp_hf_bluez5 to make
bluez5 more generic and reusable.

hfp_hf_bluez5: Remove Card on RequestDisconnection

For local initiated disconnection the GAtChat disconnect callback is not
called. For this case, Audio Card remove function needs to be called.

hfp_hf_bluez5: Remove Card when modem is disabled

hfp_hf_bluez5: Remove Card if SLC is disconnected

This patch removes previous registered Audio Card when the remote
Bluetooth device triggers the service level disconnection.

hfp_hf_bluez5: Remove Card when SLC fails

This patch removes the previously created Audio Card if the service
level negotiation fails.

hfp_hf_bluez5: Add local address to Audio Card

hfp_hf_bluez5: Add registering Audio Card

This patch registers the Handsfree Audio Card when the service level
connection is established.

handsfree-audio: properly increment next_card_id

doc: Add possible handsfree-audio API errors

This patch adds possible errors that can be returned by Register and
Unregister methods of the Manager interface.

bluez4: Increase Bluez 4 D-Bus timeouts to 60 seconds

RequestAuthorization method call may trigger pairing and/or pop up a
dialog in the remote device to which the user needs to react. Thus
increase D-Bus timeouts to 60 seconds for the Bluez 4 plugin.

handsfree-audio: Impelement CardRemoved signal

handsfree-audio: Implement CardAdded signal

handsfree-audio: Implement the GetCards method

handsfree-audio: Implement card GetProperties

handsfree-audio: Unregister from DBus on removal

handsfree-audio: Add skeleton D-Bus registration

For HandsfreeAudioCard objects

handsfree-audio: Don't hardcode '/' path

Use OFONO_MANAGER_PATH instead

include: Add handsfree card register method

handsfree-audio: Implement card create / remove

include: Add handsfree card create / remove API

audio-settings: Fix unneeded async for GetProperties

GetProperties method call for Audio settings returns the reply in the
same iteraction, it doesn't need to be asynchronous.

gdbus: Always unreference the message in g_dbus_send_message()

A quick fix on g_dbus_send_message(), if check_signal() fails it returns
FALSE without unreferencing the message as it should. This patch fixes
it.

AUTHORS: Mention Paulo's contributions

dundee: Fix typo in dundee-disconnect test

dundee: Fix typo in Manager's signal DeviceAdded

This typo was preventing dundee from sending the signal.

hfpmodem: Fix segfault in CIEV GAtChat callback

This patch fixes segmentation fault when the network registration
watch is called without being initialized. CIEV GAtChat callback can
be called before ofono_netreg_register().

==15101== Invalid read of size 8
==15101==    at 0x492B56: ofono_netreg_register (network.c:2073)
==15101==    by 0x47245E: hfp_netreg_probe (network-registration.c:311)
==15101==    by 0x492A8D: ofono_netreg_create (network.c:1881)
==15101==    by 0x4849D5: hfp_pre_sim (hfp_hf_bluez5.c:288)
==15101==    by 0x48C486: ofono_modem_set_powered (modem.c:1194)
==15101==    by 0x484E9D: slc_established (hfp_hf_bluez5.c:85)
==15101==    by 0x4702AD: chld_cb (slc.c:147)
==15101==    by 0x440457: at_chat_finish_command (gatchat.c:461)
==15101==    by 0x44109F: new_bytes (gatchat.c:532)
==15101==    by 0x4433B7: received_data (gatio.c:122)
==15101==    by 0x3CBAA47824: g_main_context_dispatch (gmain.c:2539)
==15101==    by 0x3CBAA47B57: g_main_context_iterate.isra.23
(gmain.c:3146)
==15101==  Address 0x18 is not stack'd, malloc'd or (recently) free'd

bluez5: ref / unref handsfree audio

handsfree-audio: Add ref / unref support

include: Add skeleton for handsfree-audio

hfp_ag_bluez5: Send HFP version when registering

hfp_hf_bluez5: Send HFP version when registering

bluez5: Add profile version argument

This patch adds sending of the Version argument to RegisterProfile
method.  The version is required to create the service record
containing the correct profile version.

hfp_hf_bluez5: Add local HFP version tracking

Fallback to HFP 1.5 if defer setup is not supported since it is not
possible to proceed with codec negotiation.

handsfree-audio: Don't go over 80 characters

handsfree-audio: Add Agent tracking

This patch tracks when the Handsfree Audio Agent leaves the system BUS,
and free agent allocated resources.

handsfree-audio: Call Agent "Release" when exiting

This patch notifies the Agent implementation that it is no longer
registered to the service daemon.

handsfree-audio: Add Agent "Unregister"

This patch implements the "Unregister" method of the Handsfree Audio
Manager. The agent is unregistered if sender and path match.

handsfree-audio: Free agent when exiting

Free agent allocated resources when oFono exits.

handsfree-audio: Add codec array validation

This patch checks if the codec array included in the "Register" method
contains valid codec ID's.

handsfree-audio: Add Agent "Register" method

This patch adds the initial Handsfree Audio Manager "Register"
method implementation. It adds the parsing of the arguments included
in the message and checks if there is an agent registered already.

main: Call handsfree audio manager init and cleanup

handsfree-audio: Add Manager registration

Adds the initial implementation of new experimental Handsfree Audio
Manager interface. This patch adds the interface registration and
the declaration of it's methods.

doc: Add experimental handsfree-audio API

hfp_hf_bluez5: Ensure active SLC for SCO links

Whenever a SCO connection is attempted, ensure that the SLC link is
already active between us and the same remote peer.

hfp_hf_bluez5: Remove use of hash table

The number of paired devices is quite small, so the use of the hash
table for lookup seems unnecessary.  The typical user will have no more
than 1-2 devices paired, maybe half a dozen in a less common case.

hfp_hf_bluez5: Use faster method of disable()

modem: Implement ofono_modem_find

include: Add ofono_modem_find

hfp_hf_bluez5: Remove unneeded modem hash lookup

Modem object is created when the Device Proxy is added, and it is
paired. Another path is when the Device "Paired" property changes
to True.

hfp_hf_bluez5: Use some tricks to optimize code

gdbus: Add g_dbus_proxy_set_removed_watch

hfp_hf_bluez5: Add support for Enabling/Disabling the modem

Now that we have the support for sending the correct messages
to BlueZ, we are able to dynamically power up/down the HFP modem.

We add another property to the modem to be able to get the D-Bus
object path that represents the remote device.

hfp_hf_bluez5: Add Profile RequestDisconnection

This patch implements RequestDisconnection method of the Profile1
interface. This method gets called when the profile gets disconnected.
The profile implementation needs to cleanup the resources related to
the informed device.

bluez5: Add support for Connecting/Disconnecting per UUID

This adds supports for sending the ConnectProfile() and
DisconnectProfile() messages for the org.bluez.Device1 interface.

bluez5: Rename BlueZ Profile registration functions

This patch renames external profile registration functions replacing the
prefix from 'bluetooth_' to 'bt_' in order to follow the name convention
adopted for bluez5 functions.

gdbus: Fix missing PropertiesChanged signal

If D-Bus ObjectManager is not supported, InterfacesAdded signal
checking needs to be ignored otherwise PropertiesChanged signal
will never be sent.

gdbus: Don't call property changed callback during client init

When the client uses ObjectManager to init properties, do not call
property changed callbacks. They should only be called once the proxy
added has been successfully signaled since the proxy itself provides
a full copy of available properties.

gdbus: Fix unpublished interface signal emission

If we haven't published an interface yet (i.e. it's in the data->added
list), we should just ignore any property changed indications as the
values for the properties will anyway be part of the InterfacesAdded
signal.

hfp_ag: Add BlueZ 5 version

This patch adds initial implementation of the D-Bus Profile1
interface and methods responsible for handling Bluetooth connections.

The HFP AG profile interface is registered as soon as a voice capable
modem is registered in oFono.

hfp_hf_bluez5: Remove BlueZ devices proxies hash

The hash table to track the devices is not necessary anymore since
dynamic modem registration on NewConnection was removed.

hfp_hf_bluez5: Fix registering modem on NewConnection

HFP modem will be registered when Proxy Added callback gets called
or when Pair is True. This patch removes the support for dynamic modem
registration when a new connection is notified and there isn't a modem
associated with the Bluetooth remote device.

BlueZ behaviour has been changed and a NewConnection is not notified
before the service discovery finishes.

hfp_hf_bluez5: Remove unneeded error handling

Apart from g_io_channel_unix_new() not setting errno, it never
returns an NULL pointer on error, because in practice it only
calls g_new().

bluetooth: Rename HFP AG plugin

As BlueZ 5 introduced backwards incompatible API changes, and we want to
keep support for BlueZ 4 based HFP AG plugin for some time, we need to
separate HFP AG plugin which is based on BlueZ 4 from the one based on
BlueZ 5.

The hfp_ag.c plugin is renamed to hfp_ag_bluez4. This will make it easy
to add an HFP AG plugin for BlueZ 5.

hfp_hf_bluez5: Only register modems for Paired devices

When there are many devices around that support the HFP AG profile,
we may have a lot of modems that the user will never use.

hfp_hf_bluez5: Improve Proxy property changed handling

This patch removes unnecessary GDBusProxy object access when property
changed is triggered. The property name and the argument iterator is
informed in the function callback.

TODO: Add task for supporting BlueZ 5 in dundee

hfp_hf_bluez5: Fix missing fd close

This patch fix an unusual scenario, service_level_connection() fails if
GIOChannel or GAtChat memory allocation fails.

hfp_hf_bluez5: Rename 'Address' to 'Remote'

Rename this property to better indicate that it is the remote Address
being referred to.

bluez5: Add RFCOMM socket address declaration

This patch adds a copy of Bluetooth RFCOMM socket declaration.

bluez5: Add bt_bacmp()

Adds a copy of BlueZ bacmp() function.

bluez5: Add bt_ba2str()

Adds a copy of BlueZ ba2str() function.

hfp_hf_bluez5: Add SCO listen socket

This patch adds the initial SCO server socket handling. BtIO and BlueZ
functions should not be used, with the new Profile API the objetive is
to get rid of these dependencies.

bluez5: Add bt_bacpy()

Adds a copy of BlueZ bacpy() function.

bluez5: Add SCO socket declarations

Adds local copy of SCO Bluetooth sockets declarations, since the
objective to avoid including BlueZ library headers.

hfp_hf_bluez5: Add SLC establishment procedure

When receiving a NewConnection call from BlueZ, initiates the Service
Level Connection using the received file descriptor. The HFP modem
sub-components (devinfo, voicecall, netreg, handsfree and callvolume)
are created at this point.

hfp_hf_bluez5: Handle NewConnection from BlueZ

Parse the essential arguments in the message, in this case only the
file descriptor, and register the modem if it is not already
registered. This is necessary because in some cases, we may receive a
NewConnection call, and the SDP process is still taking place.

hfp_hf_bluez5: Follow HFP AG device Alias property

If the device Alias property changes we should also change the name of
the modem.

hfp_hf_bluez5: Register modem for HFP AG devices

Now that we are able to keep track of devices appearing and
disappearing from BlueZ, we are able to register the modem when a
device that supports the HFP AG UUID appears.

bluez5: Add HFP_AG_UUID

hfp_hf_bluez5: Add tracking of bluetooth devices

This patch tracks the GDBusProxy for Bluetooth devices in order to be
able to get their properties.

hfp_hf_bluez5: Initial GDBusClient for BlueZ

This patch adds the initial callbacks to track when BlueZ connects so
we can register our HFP external profile handler. And tracks the
interfaces added or removed.

bluez5: Add additional defines

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.