Commit 21167399 authored by Jarod Wilson's avatar Jarod Wilson Committed by Greg Kroah-Hartman

Staging: lirc: fix compiler warning

On Fri, Aug 13, 2010 at 03:38:40PM +0200, Dan Carpenter wrote:
> Speak of left over stuff, it's weird that I didn't notice this before
> but gcc complains about an unitialized variable in
> imon_incoming_packet().
>
> drivers/staging/lirc/lirc_imon.c: In function ‘imon_incoming_packet’:
> drivers/staging/lirc/lirc_imon.c:661: warning: ‘chunk_num’ may be used
> 	uninitialized in this function
>
> I don't know how to fix that, but it looks important.

Ew. Yeah, that doesn't look so hot like it is right now. The old lirc_imon
driver had chunk_num = buf[7], and made much more extensive use of
chunk_num. Simply removing chunk_num and using buf[7] should be fine.
Signed-off-by: default avatarJarod Wilson <jarod@redhat.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent c37c6d21
...@@ -596,7 +596,7 @@ static void imon_incoming_packet(struct imon_context *context, ...@@ -596,7 +596,7 @@ static void imon_incoming_packet(struct imon_context *context,
struct device *dev = context->driver->dev; struct device *dev = context->driver->dev;
int octet, bit; int octet, bit;
unsigned char mask; unsigned char mask;
int i, chunk_num; int i;
/* /*
* just bail out if no listening IR client * just bail out if no listening IR client
...@@ -655,7 +655,7 @@ static void imon_incoming_packet(struct imon_context *context, ...@@ -655,7 +655,7 @@ static void imon_incoming_packet(struct imon_context *context,
} }
} }
if (chunk_num == 10) { if (buf[7] == 10) {
if (context->rx.count) { if (context->rx.count) {
submit_data(context); submit_data(context);
context->rx.count = 0; context->rx.count = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment