upstream: [media] tda9887: remove an warning when compiling for alpha
authorMauro Carvalho Chehab <m.chehab@samsung.com>
Fri, 1 Nov 2013 15:40:40 +0000 (12:40 -0300)
committerChanho Park <chanho61.park@samsung.com>
Tue, 18 Nov 2014 02:58:52 +0000 (11:58 +0900)
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>
drivers/media/tuners/tda9887.c

index 300005c..9823248 100644 (file)
@@ -536,8 +536,8 @@ static int tda9887_status(struct dvb_frontend *fe)
        unsigned char buf[1];
        int rc;
 
-       memset(buf,0,sizeof(buf));
-       if (1 != (rc = tuner_i2c_xfer_recv(&priv->i2c_props,buf,1)))
+       rc = tuner_i2c_xfer_recv(&priv->i2c_props, buf, 1);
+       if (rc != 1)
                tuner_info("i2c i/o error: rc == %d (should be 1)\n", rc);
        dump_read_message(fe, buf);
        return 0;