diff options
| author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-12-27 11:44:50 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-12-29 08:17:13 -0200 |
| commit | fce50ac80e38b1d205939ff7d4777536701263a6 (patch) | |
| tree | 65baca04f33fddfa698435e5acb09b655e6151a1 | |
| 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>
| -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 1813790b8e9..dd6bd364efa 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) const char *fw_name = "radio-wl1273-fw.bin"; struct device *dev = radio->dev; __u8 *ptr; - int i, n, r; + int r; dev_dbg(dev, "%s:\n", __func__); @@ -672,7 +672,6 @@ static int wl1273_fm_upload_firmware_patch(struct wl1273_device *radio) /* ignore possible error here */ wl1273_fm_write_cmd(core, WL1273_RESET, 0); - dev_dbg(dev, "n: %d, i: %d\n", n, i); dev_dbg(dev, "%s - download OK, r: %d\n", __func__, r); out: release_firmware(fw_p); |