From be5adb4323a4a2a0461de24904a1a65909011252 Mon Sep 17 00:00:00 2001 From: Emil Gydesen Date: Tue, 26 Mar 2024 11:06:43 +0100 Subject: [PATCH] Bluetooth: BAP: Unicast Client: Replace some LOG_ERR/WRN with DBG When returning an error code directly to an application in the API, it should not log as a warning or error. The lack of PAC records is also reported directly to the application, and is not an error (in fact it is valid not to have e.g. sink or source pacs). Signed-off-by: Emil Gydesen --- subsys/bluetooth/audio/bap_unicast_client.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/subsys/bluetooth/audio/bap_unicast_client.c b/subsys/bluetooth/audio/bap_unicast_client.c index 9fcf8203d6f..b6c579d423b 100644 --- a/subsys/bluetooth/audio/bap_unicast_client.c +++ b/subsys/bluetooth/audio/bap_unicast_client.c @@ -2529,24 +2529,24 @@ static int stream_param_check(const struct bt_bap_unicast_group_stream_param *pa { CHECKIF(param->stream == NULL) { - LOG_ERR("param->stream is NULL"); + LOG_DBG("param->stream is NULL"); return -EINVAL; } CHECKIF(param->qos == NULL) { - LOG_ERR("param->qos is NULL"); + LOG_DBG("param->qos is NULL"); return -EINVAL; } if (param->stream != NULL && param->stream->group != NULL) { - LOG_WRN("stream %p already part of group %p", param->stream, param->stream->group); + LOG_DBG("stream %p already part of group %p", param->stream, param->stream->group); return -EALREADY; } CHECKIF(bt_audio_verify_qos(param->qos) != BT_BAP_ASCS_REASON_NONE) { - LOG_ERR("Invalid QoS"); + LOG_DBG("Invalid QoS"); return -EINVAL; } @@ -3998,7 +3998,7 @@ static uint8_t unicast_client_pac_discover_cb(struct bt_conn *conn, int err; if (attr == NULL) { - LOG_ERR("Unable to find %s PAC", bt_audio_dir_str(client->dir)); + LOG_DBG("Unable to find %s PAC", bt_audio_dir_str(client->dir)); discover_cb(conn, BT_ATT_ERR_ATTRIBUTE_NOT_FOUND);