1
0
mirror of https://github.com/RIOT-OS/RIOT.git synced 2025-01-18 12:52:44 +01:00

Merge pull request #10196 from haukepetersen/fix_nimble_nodevelhelp

pkg/nimble: prepare for building without DEVELHELP
This commit is contained in:
Hauke Petersen 2018-11-08 14:40:07 +01:00 committed by GitHub
commit fa520a09cc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 3 deletions

View File

@ -90,6 +90,7 @@ static void start_advertise(void)
rc = ble_gap_adv_start(own_addr_type, NULL, BLE_HS_FOREVER,
&advp, gap_event_cb, NULL);
assert(rc == 0);
(void)rc;
}
static void app_ble_sync_cb(void)
@ -101,6 +102,7 @@ static void app_ble_sync_cb(void)
rc = ble_hs_id_infer_auto(0, &own_addr_type);
assert(rc == 0);
(void)rc;
/* generate the advertising data */
update_ad();

View File

@ -6,14 +6,17 @@ PKG_LICENSE = Apache-2.0
TDIR = $(RIOTPKG)/$(PKG_NAME)
PDIR = $(PKG_BUILDDIR)
# As of now, NimBLE does not build without warnings for -Wextra, so we disable
# that flag for this package. Will hopefully be fixed some time in the future.
# NimBLE is not optimized for building with all (extra) warnings enabled. So for
# now, we disable a number of selected compiler warnings when building NimBLE.
CFLAGS += -Wno-extra
ifeq (llvm,$(TOOLCHAIN))
# the static function `ble_ll_adv_active_chanset_is_sec()` in
# `nimble/controller/src/ble_ll_adv.c` isn't used in our compilation path, so
# tell LLVM/clang not to be so pedantic with this.
ifeq (llvm,$(TOOLCHAIN))
CFLAGS += -Wno-unused-function
CFLAGS += -Wno-sometimes-uninitialized
else
CFLAGS += -Wno-unused-but-set-variable
endif
.PHONY: all