Skip to content

Commit

Permalink
Bluetooth: hci.h: Do not include conn.h
Browse files Browse the repository at this point in the history
Commit 0f06c7d introduced the inclusion
of conn.h in hci.h. This is in fact not required because bluetooth.h has
forward declarations of many of the structs present in conn.h,
including struct bt_conn.

As part of this change, include the necessary Bluetooth headers
required by hci_core.h.

Fixes zephyrproject-rtos#58214.

Signed-off-by: Carles Cufi <[email protected]>
  • Loading branch information
carlescufi committed Jun 8, 2023
1 parent 54aa6f1 commit a07a00d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion include/zephyr/bluetooth/hci.h
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@
#include <zephyr/sys/util.h>
#include <zephyr/net/buf.h>
#include <zephyr/bluetooth/addr.h>
#include <zephyr/bluetooth/bluetooth.h>
#include <zephyr/bluetooth/hci_err.h>
#include <zephyr/bluetooth/conn.h>

#ifdef __cplusplus
extern "C" {
Expand Down
4 changes: 4 additions & 0 deletions subsys/bluetooth/host/hci_core.h
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,10 @@
* SPDX-License-Identifier: Apache-2.0
*/

#include <zephyr/bluetooth/addr.h>
#include <zephyr/bluetooth/bluetooth.h>
#include <zephyr/bluetooth/conn.h>

/* LL connection parameters */
#define LE_CONN_LATENCY 0x0000
#define LE_CONN_TIMEOUT 0x002a
Expand Down

0 comments on commit a07a00d

Please sign in to comment.