Bluetooth: sco: Use the correct print format
authorKai Ye <yekai13@huawei.com>
Thu, 3 Jun 2021 07:41:04 +0000 (15:41 +0800)
committerMarcel Holtmann <marcel@holtmann.org>
Sat, 26 Jun 2021 05:12:42 +0000 (07:12 +0200)
According to Documentation/core-api/printk-formats.rst,
Use the correct print format. Printing an unsigned int value should use %u
instead of %d. Otherwise printk() might end up displaying negative numbers.

Signed-off-by: Kai Ye <yekai13@huawei.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/bluetooth/sco.c

index 3bd4156..d9a4e88 100644 (file)
@@ -310,7 +310,7 @@ static void sco_recv_frame(struct sco_conn *conn, struct sk_buff *skb)
        if (!sk)
                goto drop;
 
-       BT_DBG("sk %p len %d", sk, skb->len);
+       BT_DBG("sk %p len %u", sk, skb->len);
 
        if (sk->sk_state != BT_CONNECTED)
                goto drop;
@@ -905,7 +905,7 @@ static int sco_sock_getsockopt_old(struct socket *sock, int optname,
 
                opts.mtu = sco_pi(sk)->conn->mtu;
 
-               BT_DBG("mtu %d", opts.mtu);
+               BT_DBG("mtu %u", opts.mtu);
 
                len = min_t(unsigned int, len, sizeof(opts));
                if (copy_to_user(optval, (char *)&opts, len))
@@ -1167,7 +1167,7 @@ static void sco_connect_cfm(struct hci_conn *hcon, __u8 status)
        if (hcon->type != SCO_LINK && hcon->type != ESCO_LINK)
                return;
 
-       BT_DBG("hcon %p bdaddr %pMR status %d", hcon, &hcon->dst, status);
+       BT_DBG("hcon %p bdaddr %pMR status %u", hcon, &hcon->dst, status);
 
        if (!status) {
                struct sco_conn *conn;
@@ -1196,7 +1196,7 @@ void sco_recv_scodata(struct hci_conn *hcon, struct sk_buff *skb)
        if (!conn)
                goto drop;
 
-       BT_DBG("conn %p len %d", conn, skb->len);
+       BT_DBG("conn %p len %u", conn, skb->len);
 
        if (skb->len) {
                sco_recv_frame(conn, skb);