diff options
author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-12-27 09:44:50 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-12-29 05:17:13 -0500 |
commit | fce50ac80e38b1d205939ff7d4777536701263a6 (patch) | |
tree | 65baca04f33fddfa698435e5acb09b655e6151a1 /drivers/media/radio | |
parent | 9ae2ae35f8a8a13997b9ec719ae7c151a2182e80 (diff) |
[media] radio-wl1273: Fix two warnings
drivers/media/radio/radio-wl1273.c: In function ‘wl1273_fm_upload_firmware_patch’:
drivers/media/radio/radio-wl1273.c:675:2: warning: ‘n’ may be used uninitialized in this function
drivers/media/radio/radio-wl1273.c:675:2: warning: ‘i’ may be used uninitialized in this function
Those vars are never initialized, and the debug message makes no sense, as it
will show just two random values.
Cc: Matti J. Aaltonen <matti.j.aaltonen@nokia.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/radio')
-rw-r--r-- | drivers/media/radio/radio-wl1273.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/radio/radio-wl1273.c b/drivers/media/radio/radio-wl1273.c index 1813790b8e9b..dd6bd364efa0 100644 --- a/drivers/media/radio/radio-wl1273.c +++ b/drivers/media/radio/radio-wl1273.c | |||
@@ -645,7 +645,7 @@ static int wl1273_fm_upload_firmware_patch(struct wl1273_device *radio) | |||
645 | const char *fw_name = "radio-wl1273-fw.bin"; | 645 | const char *fw_name = "radio-wl1273-fw.bin"; |
646 | struct device *dev = radio->dev; | 646 | struct device *dev = radio->dev; |
647 | __u8 *ptr; | 647 | __u8 *ptr; |
648 | int i, n, r; | 648 | int r; |
649 | 649 | ||
650 | dev_dbg(dev, "%s:\n", __func__); | 650 | dev_dbg(dev, "%s:\n", __func__); |
651 | 651 | ||
@@ -672,7 +672,6 @@ static int wl1273_fm_upload_firmware_patch(struct wl1273_device *radio) | |||
672 | /* ignore possible error here */ | 672 | /* ignore possible error here */ |
673 | wl1273_fm_write_cmd(core, WL1273_RESET, 0); | 673 | wl1273_fm_write_cmd(core, WL1273_RESET, 0); |
674 | 674 | ||
675 | dev_dbg(dev, "n: %d, i: %d\n", n, i); | ||
676 | dev_dbg(dev, "%s - download OK, r: %d\n", __func__, r); | 675 | dev_dbg(dev, "%s - download OK, r: %d\n", __func__, r); |
677 | out: | 676 | out: |
678 | release_firmware(fw_p); | 677 | release_firmware(fw_p); |