aboutsummaryrefslogtreecommitdiff
path: root/hw
diff options
context:
space:
mode:
authorStewart Smith <stewart@linux.ibm.com>2018-11-29 15:28:33 +1100
committerStewart Smith <stewart@linux.ibm.com>2018-12-10 22:21:12 -0600
commite35f1cb63c1bedd3710a3a0fa7bea9f68910fb86 (patch)
treebe8a59b7a353f42fb4ecf1d26404f2d940e88412 /hw
parentbabd5a27dc06143b3b8860eb121aa382e1315022 (diff)
downloadskiboot-e35f1cb63c1bedd3710a3a0fa7bea9f68910fb86.zip
skiboot-e35f1cb63c1bedd3710a3a0fa7bea9f68910fb86.tar.gz
skiboot-e35f1cb63c1bedd3710a3a0fa7bea9f68910fb86.tar.bz2
hw/bt.c: Move some debug ifdef to make static analysis happy
Okay, so maybe the static analysis warning is all useless, and maybe having the ifdef around a call is actually useful. I'll take the less noise in my CI static analysis thing. Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
Diffstat (limited to 'hw')
-rw-r--r--hw/bt.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/hw/bt.c b/hw/bt.c
index 17c6de4..db214b2 100644
--- a/hw/bt.c
+++ b/hw/bt.c
@@ -409,9 +409,9 @@ static void bt_expire_old_msg(uint64_t tb)
}
}
+#if BT_QUEUE_DEBUG
static void print_debug_queue_info(void)
{
-#if BT_QUEUE_DEBUG
struct bt_msg *msg;
static bool printed;
@@ -426,8 +426,8 @@ static void print_debug_queue_info(void)
printed = true;
prlog(PR_DEBUG, "----- BT Msg Queue Empty -----\n");
}
-#endif
}
+#endif
static void bt_send_and_unlock(void)
{
@@ -473,7 +473,9 @@ static void bt_poll(struct timer *t __unused, void *data __unused,
*/
lock(&bt.lock);
+#if BT_QUEUE_DEBUG
print_debug_queue_info();
+#endif
bt_ctrl = bt_inb(BT_CTRL);