From 2f15aa97470d5d424db88ff351de44f1ba079b26 Mon Sep 17 00:00:00 2001
From: Martine Lenders <m.lenders@fu-berlin.de>
Date: Tue, 14 Aug 2018 14:39:11 +0200
Subject: [PATCH] nimble: deactivate -Wunused-function with LLVM/clang

When building `nimble` with LLVM/clang the function
`ble_ll_adv_active_chanset_is_sec()` in `ble_ll_adv.c` is reported to
be unused, it is used within an assertions and `ifdef`s within that
file though. So this just disables that warning.
---
 pkg/nimble/Makefile | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/pkg/nimble/Makefile b/pkg/nimble/Makefile
index b612d4326b..50f37b26b6 100644
--- a/pkg/nimble/Makefile
+++ b/pkg/nimble/Makefile
@@ -9,6 +9,12 @@ 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.
 CFLAGS += -Wno-extra
+# 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
+endif
 
 .PHONY: all
 
-- 
GitLab