Skip to content

Commit 18aeefd

Browse files
committed
Bluetooth: ISO: rm extra log statements in chan_send
bt_iso_chan_send and bt_iso_chan_send_ts would log twice for each TX, which is unncessary. LOG_DBG statements include the function name by default, so the 2nd log statement was unncessary, and were removed. Added logging of the ts in bt_iso_chan_send_ts which would still make it possible to tell the 2 apart, even if CONFIG_LOG_FUNC_NAME_PREFIX_DBG=n. Signed-off-by: Emil Gydesen <[email protected]>
1 parent 9ae10ea commit 18aeefd

File tree

1 file changed

+1
-3
lines changed
  • subsys/bluetooth/host

1 file changed

+1
-3
lines changed

subsys/bluetooth/host/iso.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -967,7 +967,6 @@ int bt_iso_chan_send(struct bt_iso_chan *chan, struct net_buf *buf, uint16_t seq
967967

968968
iso_conn = chan->iso;
969969

970-
BT_ISO_DATA_DBG("send-iso (no ts)");
971970
return conn_iso_send(iso_conn, buf, BT_ISO_TS_ABSENT);
972971
}
973972

@@ -983,7 +982,7 @@ int bt_iso_chan_send_ts(struct bt_iso_chan *chan, struct net_buf *buf, uint16_t
983982
return err;
984983
}
985984

986-
BT_ISO_DATA_DBG("chan %p len %zu", chan, net_buf_frags_len(buf));
985+
BT_ISO_DATA_DBG("chan %p len %zu ts %u", chan, net_buf_frags_len(buf), ts);
987986

988987
hdr = net_buf_push(buf, sizeof(*hdr));
989988
hdr->ts = sys_cpu_to_le32(ts);
@@ -993,7 +992,6 @@ int bt_iso_chan_send_ts(struct bt_iso_chan *chan, struct net_buf *buf, uint16_t
993992

994993
iso_conn = chan->iso;
995994

996-
LOG_DBG("send-iso (ts)");
997995
return conn_iso_send(iso_conn, buf, BT_ISO_TS_PRESENT);
998996
}
999997

0 commit comments

Comments
 (0)