mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2024-11-28 05:00:20 +00:00
Code Issues Projects Releases Packages Wiki Activity

Ensure all found BLE services are tracked

Browse source 
If a device has more than one service the order of service discovery
determined the selection of the service that we intend to interact
with. This assumption is not accurate and is even platform dependent.

Thinking ahead, it is likely that some devices may require us to keep
track and interact with multiple services at the time.

The new logic still suffers from the fact that there is no way
to select the correct service for interaction. This will require
higher level stack changes.

Signed-off-by: Alex Blasche <alexander.blasche@qt.io>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
Alex Blasche 2017-06-27 14:56:30 +02:00 committed by Linus Torvalds
parent 81dabe5ace
commit 57753321b0
2 changed files with 26 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
core/qt-ble.cpp
View file

@ -22,7 +22,9 @@ void BLEObject::serviceStateChanged(QLowEnergyService::ServiceState s)
{
QList<QLowEnergyCharacteristic> list;
list = service->characteristics();
auto service = qobject_cast<QLowEnergyService*>(sender());
if (service)
list = service->characteristics();
Q_FOREACH(QLowEnergyCharacteristic c, list) {
qDebug() << " " << c.uuid().toString();
@ -50,9 +52,10 @@ void BLEObject::addService(const QBluetoothUuid &newService)
return;
}
service = controller->createServiceObject(newService, this);
auto service = controller->createServiceObject(newService, this);
qDebug() << " .. created service object" << service;
if (service) {
services.append(service);
connect(service, &QLowEnergyService::stateChanged, this, &BLEObject::serviceStateChanged);
connect(service, &QLowEnergyService::characteristicChanged, this, &BLEObject::characteristcStateChanged);
connect(service, &QLowEnergyService::descriptorWritten, this, &BLEObject::writeCompleted);
@ -72,7 +75,7 @@ BLEObject::~BLEObject()
dc_status_t BLEObject::write(const void* data, size_t size, size_t *actual)
{
QList<QLowEnergyCharacteristic> list = service->characteristics();
QList<QLowEnergyCharacteristic> list = preferredService()->characteristics();
QByteArray bytes((const char *)data, (int) size);
if (!list.isEmpty()) {
@ -83,7 +86,7 @@ dc_status_t BLEObject::write(const void* data, size_t size, size_t *actual)
QLowEnergyService::WriteWithoutResponse :
QLowEnergyService::WriteWithResponse;
service->writeCharacteristic(c, bytes, mode);
preferredService()->writeCharacteristic(c, bytes, mode);
return DC_STATUS_SUCCESS;
}
@ -93,7 +96,7 @@ dc_status_t BLEObject::write(const void* data, size_t size, size_t *actual)
dc_status_t BLEObject::read(void* data, size_t size, size_t *actual)
{
if (receivedPackets.isEmpty()) {
QList<QLowEnergyCharacteristic> list = service->characteristics();
QList<QLowEnergyCharacteristic> list = preferredService()->characteristics();
if (list.isEmpty())
return DC_STATUS_IO;
@ -188,11 +191,19 @@ dc_status_t qt_ble_open(dc_custom_io_t *io, dc_context_t *context, const char *d
loop.exec();
qDebug() << " .. done waiting";
if (ble->preferredService() == nullptr) {
qDebug() << "failed to find suitable service on" << devaddr;
report_error("Failed to find suitable service on '%s'", devaddr);
controller->disconnectFromDevice();
delete controller;
return DC_STATUS_IO;
}
qDebug() << " .. enabling notifications";
/* Enable notifications */
QList<QLowEnergyCharacteristic> list = ble->service->characteristics();
QList<QLowEnergyCharacteristic> list = ble->preferredService()->characteristics();
if (!list.isEmpty()) {
const QLowEnergyCharacteristic &c = list.constLast();
@ -209,7 +220,7 @@ dc_status_t qt_ble_open(dc_custom_io_t *io, dc_context_t *context, const char *d
d = l.first();
qDebug() << "now writing \"0x0100\" to the first descriptor";
ble->service->writeDescriptor(d, QByteArray::fromHex("0100"));
ble->preferredService()->writeDescriptor(d, QByteArray::fromHex("0100"));
}
}

10
core/qt-ble.h
View file

@ -2,6 +2,7 @@
#ifndef QT_BLE_H
#define QT_BLE_H
#include <QVector>
#include <QLowEnergyController>
#include <QEventLoop>
@ -14,7 +15,10 @@ public:
~BLEObject();
dc_status_t write(const void* data, size_t size, size_t *actual);
dc_status_t read(void* data, size_t size, size_t *actual);
QLowEnergyService *service;
//TODO: need better mode of selecting the desired service than below
inline QLowEnergyService *preferredService()
{ return services.isEmpty() ? nullptr : services[0]; }
public slots:
void addService(const QBluetoothUuid &newService);
@ -23,7 +27,9 @@ public slots:
void writeCompleted(const QLowEnergyDescriptor &d, const QByteArray &value);
private:
QLowEnergyController *controller;
QVector<QLowEnergyService *> services;
QLowEnergyController *controller = nullptr;
QList<QByteArray> receivedPackets;
QEventLoop waitForPacket;
};