[media] rtl2832.c: minor cleanup
authorHans-Frieder Vogt <hfvogt@gmx.net>
Sun, 15 Jul 2012 16:56:47 +0000 (13:56 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 31 Jul 2012 00:55:14 +0000 (21:55 -0300)
The current formulation of the bw_params loop uses the counter j as an
index for the first dimension of the bw_params array which is later
incremented by the variable i. It is evaluated correctly only, because j
is initialized to 0 at the beginning of the loop. I think that
explicitly using the index 0 better reflects the intent of the
expression.

Signed-off-by: Hans-Frieder Vogt <hfvogt@gmx.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/frontends/rtl2832.c

index 2da592fb38adc0c03244fb75d4dd51d9ca40a6ff..28269ccaeab709a95ce5a02ab345f1a86f6ec305 100644 (file)
@@ -589,7 +589,7 @@ static int rtl2832_set_frontend(struct dvb_frontend *fe)
                return -EINVAL;
        }
 
-       for (j = 0; j < sizeof(bw_params[j]); j++) {
+       for (j = 0; j < sizeof(bw_params[0]); j++) {
                ret = rtl2832_wr_regs(priv, 0x1c+j, 1, &bw_params[i][j], 1);
                if (ret)
                        goto err;