* [PATCH] Bluetooth: Remove dead code
@ 2015-01-05 14:02 Gowtham Anandha Babu
2015-01-13 6:52 ` Gowtham Anandha Babu
2015-01-14 9:21 ` Johan Hedberg
0 siblings, 2 replies; 3+ messages in thread
From: Gowtham Anandha Babu @ 2015-01-05 14:02 UTC (permalink / raw)
To: linux-bluetooth; +Cc: bharat.panda, cpgs, Gowtham Anandha Babu
Variable 'controller' is assigned a value that is never used.
Identified by cppcheck tool.
Signed-off-by: Gowtham Anandha Babu <gowtham.ab@samsung.com>
---
net/bluetooth/cmtp/capi.c | 6 ------
1 file changed, 6 deletions(-)
diff --git a/net/bluetooth/cmtp/capi.c b/net/bluetooth/cmtp/capi.c
index 1ca8a87..75bd2c4 100644
--- a/net/bluetooth/cmtp/capi.c
+++ b/net/bluetooth/cmtp/capi.c
@@ -253,8 +253,6 @@ static void cmtp_recv_interopmsg(struct cmtp_session *session, struct sk_buff *s
if (skb->len < CAPI_MSG_BASELEN + 15)
break;
- controller = CAPIMSG_U32(skb->data, CAPI_MSG_BASELEN + 10);
-
if (!info && ctrl) {
int len = min_t(uint, CAPI_MANUFACTURER_LEN,
skb->data[CAPI_MSG_BASELEN + 14]);
@@ -270,8 +268,6 @@ static void cmtp_recv_interopmsg(struct cmtp_session *session, struct sk_buff *s
if (skb->len < CAPI_MSG_BASELEN + 32)
break;
- controller = CAPIMSG_U32(skb->data, CAPI_MSG_BASELEN + 12);
-
if (!info && ctrl) {
ctrl->version.majorversion = CAPIMSG_U32(skb->data, CAPI_MSG_BASELEN + 16);
ctrl->version.minorversion = CAPIMSG_U32(skb->data, CAPI_MSG_BASELEN + 20);
@@ -285,8 +281,6 @@ static void cmtp_recv_interopmsg(struct cmtp_session *session, struct sk_buff *s
if (skb->len < CAPI_MSG_BASELEN + 17)
break;
- controller = CAPIMSG_U32(skb->data, CAPI_MSG_BASELEN + 12);
-
if (!info && ctrl) {
int len = min_t(uint, CAPI_SERIAL_LEN,
skb->data[CAPI_MSG_BASELEN + 16]);
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* RE: [PATCH] Bluetooth: Remove dead code
2015-01-05 14:02 [PATCH] Bluetooth: Remove dead code Gowtham Anandha Babu
@ 2015-01-13 6:52 ` Gowtham Anandha Babu
2015-01-14 9:21 ` Johan Hedberg
1 sibling, 0 replies; 3+ messages in thread
From: Gowtham Anandha Babu @ 2015-01-13 6:52 UTC (permalink / raw)
To: linux-bluetooth; +Cc: bharat.panda, cpgs
Ping.
> -----Original Message-----
> From: linux-bluetooth-owner@vger.kernel.org [mailto:linux-bluetooth-
> owner@vger.kernel.org] On Behalf Of Gowtham Anandha Babu
> Sent: Monday, January 05, 2015 7:33 PM
> To: linux-bluetooth@vger.kernel.org
> Cc: bharat.panda@samsung.com; cpgs@samsung.com; Gowtham Anandha
> Babu
> Subject: [PATCH] Bluetooth: Remove dead code
>
> Variable 'controller' is assigned a value that is never used.
> Identified by cppcheck tool.
>
> Signed-off-by: Gowtham Anandha Babu <gowtham.ab@samsung.com>
> ---
> net/bluetooth/cmtp/capi.c | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/net/bluetooth/cmtp/capi.c b/net/bluetooth/cmtp/capi.c index
> 1ca8a87..75bd2c4 100644
> --- a/net/bluetooth/cmtp/capi.c
> +++ b/net/bluetooth/cmtp/capi.c
> @@ -253,8 +253,6 @@ static void cmtp_recv_interopmsg(struct
> cmtp_session *session, struct sk_buff *s
> if (skb->len < CAPI_MSG_BASELEN + 15)
> break;
>
> - controller = CAPIMSG_U32(skb->data,
> CAPI_MSG_BASELEN + 10);
> -
> if (!info && ctrl) {
> int len = min_t(uint,
> CAPI_MANUFACTURER_LEN,
> skb-
> >data[CAPI_MSG_BASELEN + 14]); @@ -270,8 +268,6 @@ static void
> cmtp_recv_interopmsg(struct cmtp_session *session, struct sk_buff *s
> if (skb->len < CAPI_MSG_BASELEN + 32)
> break;
>
> - controller = CAPIMSG_U32(skb->data,
> CAPI_MSG_BASELEN + 12);
> -
> if (!info && ctrl) {
> ctrl->version.majorversion =
> CAPIMSG_U32(skb->data, CAPI_MSG_BASELEN + 16);
> ctrl->version.minorversion =
> CAPIMSG_U32(skb->data, CAPI_MSG_BASELEN + 20); @@ -285,8 +281,6
> @@ static void cmtp_recv_interopmsg(struct cmtp_session *session, struct
> sk_buff *s
> if (skb->len < CAPI_MSG_BASELEN + 17)
> break;
>
> - controller = CAPIMSG_U32(skb->data,
> CAPI_MSG_BASELEN + 12);
> -
> if (!info && ctrl) {
> int len = min_t(uint, CAPI_SERIAL_LEN,
> skb-
> >data[CAPI_MSG_BASELEN + 16]);
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth"
in
> the body of a message to majordomo@vger.kernel.org More majordomo
> info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Bluetooth: Remove dead code
2015-01-05 14:02 [PATCH] Bluetooth: Remove dead code Gowtham Anandha Babu
2015-01-13 6:52 ` Gowtham Anandha Babu
@ 2015-01-14 9:21 ` Johan Hedberg
1 sibling, 0 replies; 3+ messages in thread
From: Johan Hedberg @ 2015-01-14 9:21 UTC (permalink / raw)
To: Gowtham Anandha Babu; +Cc: linux-bluetooth, bharat.panda, cpgs
Hi Gowtham,
On Mon, Jan 05, 2015, Gowtham Anandha Babu wrote:
> Variable 'controller' is assigned a value that is never used.
> Identified by cppcheck tool.
>
> Signed-off-by: Gowtham Anandha Babu <gowtham.ab@samsung.com>
> ---
> net/bluetooth/cmtp/capi.c | 6 ------
> 1 file changed, 6 deletions(-)
Applied to bluetooth-next. Thanks.
Johan
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-01-14 9:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-05 14:02 [PATCH] Bluetooth: Remove dead code Gowtham Anandha Babu
2015-01-13 6:52 ` Gowtham Anandha Babu
2015-01-14 9:21 ` Johan Hedberg
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).