Skip to content
Snippets Groups Projects

change(ble): Add Time Update service

Closed change(ble): Add Time Update service
hauke/firmware:ble-time-update into master
1 unresolved thread
Closed Hauke Mehrtens requested to merge hauke/firmware:ble-time-update into master
1 unresolved thread

This adds BLE a characteristic which allows to set the current time. The time is set in milliseconds since 1.1.1970 as 64 bit value.

Signed-off-by: Hauke Mehrtens hauke@hauke-m.de

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Hauke Mehrtens @hauke started a thread on an old version of the diff
    • epicardium/ble/card10.c
    224 uint64_t timeNet;
    225 uint64_t time;
    226 uint32_t sec;
    227 uint32_t ssec;
    228 int res;
    229
    230 if (len < sizeof(uint64_t)) {
    231 return ATT_ERR_LENGTH;
    232 }
    233 memcpy(&timeNet, pValue, sizeof(timeNet));
    234
    235 time = __ntohl(timeNet);
    236 sec = time / 1000;
    237 ssec = (time % 1000);
    238 ssec *= 256;
    239 ssec /= 1000;
  • rahix added 4-BLE label

    added 4-BLE label

  • Hauke Mehrtens added 3 commits

    added 3 commits

    • 69775a15 - 1 commit from branch card10:master
    • 8370f330 - RTC: Add function to set time
    • 25b17c23 - change(ble): Add Time Update service

    Compare with previous version

  • Hauke Mehrtens added 2 commits

    added 2 commits

    • 95af6056 - RTC: Add function to set time
    • 966497ac - change(ble): Add Time Update service

    Compare with previous version

  • Annoyatron
    Annoyatron @annoyatron ·
    Guest

    Looks good to me! Thanks for keeping our codebase clean.

    However, I'm only a bot - so a human will still have to approve this request.

  • schneider
    schneider @schneider ·
    Owner

    Also needs a rebase

  • Hauke Mehrtens added 6 commits

    added 6 commits

    Compare with previous version

  • H
    Hauke Mehrtens @hauke ·
    Author Maintainer
    • rebase done
    • code adapted to recent changes
    • documentation added
  • Hauke Mehrtens added 3 commits

    added 3 commits

    • 14930b38 - 1 commit from branch card10:master
    • 109eb75a - RTC: Add function to set time
    • be113b4a - change(ble): Add Time Update service

    Compare with previous version

  • schneider mentioned in merge request !108 (merged)

    mentioned in merge request !108 (merged)

  • schneider
    schneider @schneider ·
    Owner

    Merged in https://git.card10.badge.events.ccc.de/card10/firmware/merge_requests/108

  • schneider closed

    closed

    • Please register or sign in to reply