Update geotag data whilst connected.

Absorb the geotag updates into the disconnect detection event.
Now, as long as we are connected to the camera, we can provide GPS
updates.

src/furble.cpp

@@ -78,8 +78,6 @@ static void remote_control(FurbleCtx *fctx) {
  do {
  ez.yield();
 
- furble_gps_update_geodata(camera);
-
  if (fctx->reconnected) {
  show_shutter_control(shutter_lock, shutter_lock_start_ms);
  fctx->reconnected = false;
@@ -140,12 +138,19 @@ static void remote_control(FurbleCtx *fctx) {
  } while (camera->isConnected());
 }
 
-uint16_t disconnectDetect(void *private_data) {
+/**
+ * Refresh the camera connection status, this includes:
+ * * disconnect detection
+ * * geotag updates
+ */
+static uint16_t statusRefresh(void *private_data) {
  FurbleCtx *fctx = static_cast<FurbleCtx *>(private_data);
  Furble::Camera *camera = fctx->camera;
 
- if (camera->isConnected())
+ if (camera->isConnected()) {
+ furble_gps_update_geodata(camera);
  return 500;
+ }
 
  String buttons = ez.buttons.get();
  String header = ez.header.title();
@@ -177,7 +182,7 @@ static void menu_remote(FurbleCtx *fctx) {
  submenu.addItem("Disconnect");
  submenu.downOnLast("first");
 
- ez.addEvent(disconnectDetect, fctx, 500);
+ ez.addEvent(statusRefresh, fctx, 500);
 
  do {
  submenu.runOnce();
@@ -191,7 +196,7 @@ static void menu_remote(FurbleCtx *fctx) {
  }
  } while (submenu.pickName() != "Disconnect");
 
- ez.removeEvent(disconnectDetect);
+ ez.removeEvent(statusRefresh);
 
  fctx->camera->disconnect();
  ez.backlight.inactivity(USER_SET);
@@ -245,8 +250,6 @@ static void menu_connect(bool scan) {
 
  FurbleCtx fctx = {Furble::CameraList::get(i - 1), false};
 
- furble_gps_update_geodata(fctx.camera);
-
  ezProgressBar progress_bar(FURBLE_STR, "Connecting ...", "");
  if (fctx.camera->connect(settings_load_esp_tx_power(), &update_progress_bar, &progress_bar)) {
  if (scan) {