diff options
author | Hans-Frieder Vogt <hfvogt@gmx.net> | 2012-07-15 12:56:47 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-07-30 20:55:14 -0400 |
commit | fc4b3fa714fb2a078531bccb501cf2ef8c7f888a (patch) | |
tree | d502bfe4a77bfca8061cc25fa71afb2cbda1920c /drivers/media | |
parent | 26ff63137c45886169ed102bddd6e90d6c27f00d (diff) |
[media] rtl2832.c: minor cleanup
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>
Diffstat (limited to 'drivers/media')
-rw-r--r-- | drivers/media/dvb/frontends/rtl2832.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/dvb/frontends/rtl2832.c b/drivers/media/dvb/frontends/rtl2832.c index 2da592fb38ad..28269ccaeab7 100644 --- a/drivers/media/dvb/frontends/rtl2832.c +++ b/drivers/media/dvb/frontends/rtl2832.c | |||
@@ -589,7 +589,7 @@ static int rtl2832_set_frontend(struct dvb_frontend *fe) | |||
589 | return -EINVAL; | 589 | return -EINVAL; |
590 | } | 590 | } |
591 | 591 | ||
592 | for (j = 0; j < sizeof(bw_params[j]); j++) { | 592 | for (j = 0; j < sizeof(bw_params[0]); j++) { |
593 | ret = rtl2832_wr_regs(priv, 0x1c+j, 1, &bw_params[i][j], 1); | 593 | ret = rtl2832_wr_regs(priv, 0x1c+j, 1, &bw_params[i][j], 1); |
594 | if (ret) | 594 | if (ret) |
595 | goto err; | 595 | goto err; |