aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/media/tuners/tda9887.c
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <m.chehab@samsung.com>2013-11-01 11:40:40 -0400
committerMauro Carvalho Chehab <m.chehab@samsung.com>2013-11-07 12:03:10 -0500
commit408679cd3ccfd91a5b63337ec4dd0b6b3d6a6504 (patch)
treea457a7c6b0ac39681510ddc386c88c38ed6d7174 /drivers/media/tuners/tda9887.c
parent80f93c7b0f4599ffbdac8d964ecd1162b8b618b9 (diff)
[media] tda9887: remove an warning when compiling for alpha
There's no need to zero the buffer, as if the routine gets an error, rc will be different than one. That fixes the following warning: drivers/media/tuners/tda9887.c: In function 'tda9887_status': drivers/media/tuners/tda9887.c:539:2: warning: value computed is not used [-Wunused-value] While here, make fix the CodingStyle on this function. Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com> Reviewed-by: Hans Verkuil <hans.verkuil@cisco.com> Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Diffstat (limited to 'drivers/media/tuners/tda9887.c')
-rw-r--r--drivers/media/tuners/tda9887.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/media/tuners/tda9887.c b/drivers/media/tuners/tda9887.c
index 300005c535ba..9823248d743f 100644
--- a/drivers/media/tuners/tda9887.c
+++ b/drivers/media/tuners/tda9887.c
@@ -536,8 +536,8 @@ static int tda9887_status(struct dvb_frontend *fe)
536 unsigned char buf[1]; 536 unsigned char buf[1];
537 int rc; 537 int rc;
538 538
539 memset(buf,0,sizeof(buf)); 539 rc = tuner_i2c_xfer_recv(&priv->i2c_props, buf, 1);
540 if (1 != (rc = tuner_i2c_xfer_recv(&priv->i2c_props,buf,1))) 540 if (rc != 1)
541 tuner_info("i2c i/o error: rc == %d (should be 1)\n", rc); 541 tuner_info("i2c i/o error: rc == %d (should be 1)\n", rc);
542 dump_read_message(fe, buf); 542 dump_read_message(fe, buf);
543 return 0; 543 return 0;