Bluetooth: Conn: Add const for conn in bt_conn_get_remote_info

The connection object is, and should not, be modified by the function. Making it const makes it possible to do this simple operation in functions that also won't change the conn object (i.e. that uses `const`). Signed-off-by: Emil Gydesen <[email protected]>
zephyrproject-rtos · Feb 20, 2025 · 0e063a1 · 0e063a1
1 parent e8e1555
commit 0e063a1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/include/zephyr/bluetooth/conn.h b/include/zephyr/bluetooth/conn.h
@@ -1069,8 +1069,7 @@ bool bt_conn_is_type(const struct bt_conn *conn, enum bt_conn_type type);
  *  @return -EBUSY The remote information is not yet available.
  *  @return -EINVAL @p conn is not a valid @ref BT_CONN_TYPE_LE or @ref BT_CONN_TYPE_BR connection.
  */
-int bt_conn_get_remote_info(struct bt_conn *conn,
-			    struct bt_conn_remote_info *remote_info);
+int bt_conn_get_remote_info(const struct bt_conn *conn, struct bt_conn_remote_info *remote_info);
 
 /** @brief Get connection transmit power level.
  *

diff --git a/subsys/bluetooth/host/conn.c b/subsys/bluetooth/host/conn.c
@@ -3003,8 +3003,7 @@ bool bt_conn_is_type(const struct bt_conn *conn, enum bt_conn_type type)
 	return (conn->type & type) != 0;
 }
 
-int bt_conn_get_remote_info(struct bt_conn *conn,
-			    struct bt_conn_remote_info *remote_info)
+int bt_conn_get_remote_info(const struct bt_conn *conn, struct bt_conn_remote_info *remote_info)
 {
 	if (!bt_conn_is_type(conn, BT_CONN_TYPE_LE | BT_CONN_TYPE_BR)) {
 		LOG_DBG("Invalid connection type: %u for %p", conn->type, conn);