From cc7f5406efa1f24d5ca97267653ab1f1296e617a Mon Sep 17 00:00:00 2001
From: cpfeiffer <cpfeiffer@users.noreply.github.com>
Date: Wed, 30 Mar 2016 21:48:42 +0200
Subject: [PATCH] Use low latency transfer mode for fw update #234

---
 .../gadgetbridge/service/btle/AbstractBTLEOperation.java   | 4 ++++
 .../devices/miband/operations/UpdateFirmwareOperation.java | 7 ++++++-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/btle/AbstractBTLEOperation.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/btle/AbstractBTLEOperation.java
index b7a7e9389..258199ad8 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/btle/AbstractBTLEOperation.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/btle/AbstractBTLEOperation.java
@@ -113,6 +113,10 @@ public abstract class AbstractBTLEOperation<T extends AbstractBTLEDeviceSupport>
         return operationStatus == OperationStatus.RUNNING;
     }
 
+    public boolean isOperationFinished() {
+        return operationStatus == OperationStatus.FINISHED;
+    }
+
     public T getSupport() {
         return mSupport;
     }
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/miband/operations/UpdateFirmwareOperation.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/miband/operations/UpdateFirmwareOperation.java
index 710f4a34e..c953e663c 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/miband/operations/UpdateFirmwareOperation.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/miband/operations/UpdateFirmwareOperation.java
@@ -60,6 +60,7 @@ public class UpdateFirmwareOperation extends AbstractMiBandOperation {
     }
 
     private void done() {
+        LOG.info("Operation done.");
         updateCoordinator = null;
         operationFinished();
         unsetBusy();
@@ -273,6 +274,7 @@ public class UpdateFirmwareOperation extends AbstractMiBandOperation {
 
         try {
             TransactionBuilder builder = performInitialized("send firmware packet");
+            getSupport().setLowLatency(builder);
             for (int i = 0; i < packets; i++) {
                 byte[] fwChunk = Arrays.copyOfRange(fwbytes, i * packetLength, i * packetLength + packetLength);
 
@@ -319,6 +321,7 @@ public class UpdateFirmwareOperation extends AbstractMiBandOperation {
         public boolean sendFwInfo() {
             try {
                 TransactionBuilder builder = performInitialized("send firmware info");
+                getSupport().setLowLatency(builder);
                 builder.add(new SetDeviceBusyAction(getDevice(), getContext().getString(R.string.updating_firmware), getContext()));
                 builder.write(getCharacteristic(MiBandService.UUID_CHARACTERISTIC_CONTROL_POINT), getFirmwareInfo());
                 builder.add(new FirmwareInfoSucceededAction());
@@ -440,7 +443,9 @@ public class UpdateFirmwareOperation extends AbstractMiBandOperation {
     private class FirmwareInfoSucceededAction extends PlainAction {
         @Override
         public boolean run(BluetoothGatt gatt) {
-            firmwareInfoSent = true;
+            if (isOperationRunning()) {
+                firmwareInfoSent = true;
+            }
             return true;
         }
     }