* [PATCHv2] bluetooth: hci_bcsp: Clean up code Fix
@ 2015-06-09 11:42 Mutharaju, Prasanna (P.)
2015-06-09 11:45 ` Marcel Holtmann
0 siblings, 1 reply; 4+ messages in thread
From: Mutharaju, Prasanna (P.) @ 2015-06-09 11:42 UTC (permalink / raw)
To: marcel@holtmann.org, gustavo@padovan.org, johan.hedberg@gmail.com
Cc: linux-bluetooth@vger.kernel.org
From: Prasanna Karthik <mkarthi3@visteon.com>
Fix for spaces prohibited around that '->' reported by checkpatch and space=
prohibited
between function name and open parenthesis '('
Signed-off-by: Prasanna Karthik <mkarthi3@visteon.com>
---
V2: Reverted back Indentation,Now patch only has above fix
---
---
drivers/bluetooth/hci_bcsp.c | 16 ++++++++--------
1 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/bluetooth/hci_bcsp.c b/drivers/bluetooth/hci_bcsp.c
index dc8e3d4..764939d 100644
--- a/drivers/bluetooth/hci_bcsp.c
+++ b/drivers/bluetooth/hci_bcsp.c
@@ -436,7 +436,7 @@ static inline void bcsp_unslip_one_byte(struct bcsp_str=
uct *bcsp, unsigned char
break;
default:
memcpy(skb_put(bcsp->rx_skb, 1), &byte, 1);
- if ((bcsp->rx_skb-> data[0] & 0x40) !=3D 0 &&=20
+ if ((bcsp->rx_skb->data[0] & 0x40) !=3D 0 &&=20
bcsp->rx_state !=3D BCSP_W4_CRC)
bcsp_crc_update(&bcsp->message_crc, byte);
bcsp->rx_count--;
@@ -447,24 +447,24 @@ static inline void bcsp_unslip_one_byte(struct bcsp_s=
truct *bcsp, unsigned char
switch (byte) {
case 0xdc:
memcpy(skb_put(bcsp->rx_skb, 1), &c0, 1);
- if ((bcsp->rx_skb-> data[0] & 0x40) !=3D 0 &&=20
+ if ((bcsp->rx_skb->data[0] & 0x40) !=3D 0 &&=20
bcsp->rx_state !=3D BCSP_W4_CRC)
- bcsp_crc_update(&bcsp-> message_crc, 0xc0);
+ bcsp_crc_update(&bcsp->message_crc, 0xc0);
bcsp->rx_esc_state =3D BCSP_ESCSTATE_NOESC;
bcsp->rx_count--;
break;
=20
case 0xdd:
memcpy(skb_put(bcsp->rx_skb, 1), &db, 1);
- if ((bcsp->rx_skb-> data[0] & 0x40) !=3D 0 &&=20
+ if ((bcsp->rx_skb->data[0] & 0x40) !=3D 0 &&=20
bcsp->rx_state !=3D BCSP_W4_CRC)=20
- bcsp_crc_update(&bcsp-> message_crc, 0xdb);
+ bcsp_crc_update(&bcsp->message_crc, 0xdb);
bcsp->rx_esc_state =3D BCSP_ESCSTATE_NOESC;
bcsp->rx_count--;
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);
--=20
1.7.0.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCHv2] bluetooth: hci_bcsp: Clean up code Fix
2015-06-09 11:42 Mutharaju, Prasanna (P.)
@ 2015-06-09 11:45 ` Marcel Holtmann
0 siblings, 0 replies; 4+ messages in thread
From: Marcel Holtmann @ 2015-06-09 11:45 UTC (permalink / raw)
To: Mutharaju, Prasanna (P.)
Cc: Gustavo F. Padovan, Johan Hedberg,
linux-bluetooth@vger.kernel.org
Hi Prasanna,
> From: Prasanna Karthik <mkarthi3@visteon.com>
>
> Fix for spaces prohibited around that '->' reported by checkpatch and space prohibited
> between function name and open parenthesis '('
>
> Signed-off-by: Prasanna Karthik <mkarthi3@visteon.com>
>
> ---
> V2: Reverted back Indentation,Now patch only has above fix
> ---
> ---
> drivers/bluetooth/hci_bcsp.c | 16 ++++++++--------
> 1 files changed, 8 insertions(+), 8 deletions(-)
the patch does not apply cleanly with git am.
Applying: bluetooth: hci_bcsp: Clean up code Fix
/data/kernel/maintainer-bluetooth-next/.git/rebase-apply/patch:17: trailing whitespace.
if ((bcsp->rx_skb->data[0] & 0x40) != 0 &&
/data/kernel/maintainer-bluetooth-next/.git/rebase-apply/patch:26: trailing whitespace.
if ((bcsp->rx_skb->data[0] & 0x40) != 0 &&
/data/kernel/maintainer-bluetooth-next/.git/rebase-apply/patch:37: trailing whitespace.
if ((bcsp->rx_skb->data[0] & 0x40) != 0 &&
fatal: 3 lines add whitespace errors.
Patch failed at 0001 bluetooth: hci_bcsp: Clean up code Fix
Regards
Marcel
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCHv2] bluetooth: hci_bcsp: Clean up code Fix
@ 2015-06-09 11:51 Mutharaju, Prasanna (P.)
2015-06-09 12:00 ` Marcel Holtmann
0 siblings, 1 reply; 4+ messages in thread
From: Mutharaju, Prasanna (P.) @ 2015-06-09 11:51 UTC (permalink / raw)
To: marcel@holtmann.org, gustavo@padovan.org, johan.hedberg@gmail.com
Cc: linux-bluetooth@vger.kernel.org
From: Prasanna Karthik <mkarthi3@visteon.com>
Fix for spaces prohibited around that '->' reported by checkpatch and space=
prohibited
between function name and open parenthesis '('
Signed-off-by: Prasanna Karthik <mkarthi3@visteon.com>
---
V3: Reverted back Indentation,Now patch only has above fix, cleanedup
---
---
drivers/bluetooth/hci_bcsp.c | 16 ++++++++--------
1 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/bluetooth/hci_bcsp.c b/drivers/bluetooth/hci_bcsp.c
index dc8e3d4..764939d 100644
--- a/drivers/bluetooth/hci_bcsp.c
+++ b/drivers/bluetooth/hci_bcsp.c
@@ -436,7 +436,7 @@ static inline void bcsp_unslip_one_byte(struct bcsp_str=
uct *bcsp, unsigned char
break;
default:
memcpy(skb_put(bcsp->rx_skb, 1), &byte, 1);
- if ((bcsp->rx_skb-> data[0] & 0x40) !=3D 0 &&=20
+ if ((bcsp->rx_skb->data[0] & 0x40) !=3D 0 &&
bcsp->rx_state !=3D BCSP_W4_CRC)
bcsp_crc_update(&bcsp->message_crc, byte);
bcsp->rx_count--;
@@ -447,24 +447,24 @@ static inline void bcsp_unslip_one_byte(struct bcsp_s=
truct *bcsp, unsigned char
switch (byte) {
case 0xdc:
memcpy(skb_put(bcsp->rx_skb, 1), &c0, 1);
- if ((bcsp->rx_skb-> data[0] & 0x40) !=3D 0 &&=20
+ if ((bcsp->rx_skb->data[0] & 0x40) !=3D 0 &&
bcsp->rx_state !=3D BCSP_W4_CRC)
- bcsp_crc_update(&bcsp-> message_crc, 0xc0);
+ bcsp_crc_update(&bcsp->message_crc, 0xc0);
bcsp->rx_esc_state =3D BCSP_ESCSTATE_NOESC;
bcsp->rx_count--;
break;
=20
case 0xdd:
memcpy(skb_put(bcsp->rx_skb, 1), &db, 1);
- if ((bcsp->rx_skb-> data[0] & 0x40) !=3D 0 &&=20
+ if ((bcsp->rx_skb->data[0] & 0x40) !=3D 0 &&
bcsp->rx_state !=3D BCSP_W4_CRC)=20
- bcsp_crc_update(&bcsp-> message_crc, 0xdb);
+ bcsp_crc_update(&bcsp->message_crc, 0xdb);
bcsp->rx_esc_state =3D BCSP_ESCSTATE_NOESC;
bcsp->rx_count--;
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);
--=20
1.7.0.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCHv2] bluetooth: hci_bcsp: Clean up code Fix
2015-06-09 11:51 [PATCHv2] bluetooth: hci_bcsp: Clean up code Fix Mutharaju, Prasanna (P.)
@ 2015-06-09 12:00 ` Marcel Holtmann
0 siblings, 0 replies; 4+ messages in thread
From: Marcel Holtmann @ 2015-06-09 12:00 UTC (permalink / raw)
To: Mutharaju, Prasanna (P.)
Cc: Gustavo F. Padovan, Johan Hedberg,
linux-bluetooth@vger.kernel.org
Hi Prasanna,
> From: Prasanna Karthik <mkarthi3@visteon.com>
>
> Fix for spaces prohibited around that '->' reported by checkpatch and space prohibited
> between function name and open parenthesis '('
>
> Signed-off-by: Prasanna Karthik <mkarthi3@visteon.com>
>
> ---
> V3: Reverted back Indentation,Now patch only has above fix, cleanedup
> ---
> ---
> drivers/bluetooth/hci_bcsp.c | 16 ++++++++--------
> 1 files changed, 8 insertions(+), 8 deletions(-)
patch has been applied to bluetooth-next tree.
Regards
Marcel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-06-09 12:00 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-09 11:51 [PATCHv2] bluetooth: hci_bcsp: Clean up code Fix Mutharaju, Prasanna (P.)
2015-06-09 12:00 ` Marcel Holtmann
-- strict thread matches above, loose matches on Subject: below --
2015-06-09 11:42 Mutharaju, Prasanna (P.)
2015-06-09 11:45 ` Marcel Holtmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).