Skip to contentSkip to conversation

Bluetooth:hci_bcsp:Fix coding style -- Clean up

PATCH In review Mutharaju, Prasanna (P.) 2 participants 2 messages Jun 22, 2015 Bluetooth drivers

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 Bluetooth:hci_bcsp:Fix coding style -- Clean up

From: Prasanna Karthik <[email protected]>

Space prohibited between function name and open parenthesis '(' reported by Checkpatch

----
V2 Cleaned up the patch to remove trailing whitespace
---

---
 drivers/bluetooth/hci_bcsp.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)
drivers/bluetooth/hci_bcsp.c +0 -0
index dc8e3d4..9b8233b 100644
@@ -464,7 +464,7 @@ static inline void bcsp_unslip_one_byte(struct bcsp_str=

uct *bcsp, unsigned char
break;
=20
default:
- BT_ERR ("Invalid byte %02x after esc byte", byte);
+ BT_ERR("Invalid byte %02x after esc byte", byte);
kfree_skb(bcsp->rx_skb);
bcsp->rx_skb =3D NULL;
bcsp->rx_state =3D BCSP_W4_PKT_DELIMITER;
@@ -527,7 +527,7 @@ static void bcsp_complete_rx_pkt(struct hci_uart *hu)
=20
hci_recv_frame(hu->hdev, bcsp->rx_skb);
} else {
- BT_ERR ("Packet for unknown channel (%u %s)",
+ BT_ERR("Packet for unknown channel (%u %s)",
bcsp->rx_skb->data[1] & 0x0f, bcsp->rx_skb->data[0] & 0x80 ?=20 "reliable" : "unreliable");
@@ -587,7 +587,7 @@ static int bcsp_recv(struct hci_uart *hu, const void *d=
ata, int count)
}
if (bcsp->rx_skb->data[0] & 0x80 /* reliable pkt */
&& (bcsp->rx_skb->data[0] & 0x07) !=3D bcsp->rxseq_txack) {
- BT_ERR ("Out-of-order packet arrived, got %u expected %u",
+ BT_ERR("Out-of-order packet arrived, got %u expected %u",
bcsp->rx_skb->data[0] & 0x07, bcsp->rxseq_txack);
=20
kfree_skb(bcsp->rx_skb);
@@ -610,7 +610,7 @@ static int bcsp_recv(struct hci_uart *hu, const void *d=
ata, int count)
=20
case BCSP_W4_CRC:
if (bitrev16(bcsp->message_crc) !=3D bscp_get_crc(bcsp)) {
- BT_ERR ("Checksum failed: computed %04x received %04x",
+ BT_ERR("Checksum failed: computed %04x received %04x",
bitrev16(bcsp->message_crc), bscp_get_crc(bcsp));
=20
--=20
1.7.0.4

maintainer
#1 lore raw

Hi Prasanna,

Quoted 5 lines

Space prohibited between function name and open parenthesis '(' reported by Checkpatch

Quoted 8 lines

----
V2 Cleaned up the patch to remove trailing whitespace
---

---
drivers/bluetooth/hci_bcsp.c | 8 ++++----

1 files changed, 4 insertions(+), 4 deletions(-)

the patch is still malformed when it arrives on the mailing list. Please use git send-email to actually send the patch. Maybe your email client is mangling it somehow.

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in