diff options
author | Michael Krufky <mkrufky@linuxtv.org> | 2012-05-21 11:28:14 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-06-11 14:59:16 -0400 |
commit | bffaecc75cc6cbbf1e46571256d7b137df61bff6 (patch) | |
tree | ee1562308a4433db5f35fe3bbbfe03a599f11be9 | |
parent | 87736df2401f51e67e55b8d1e6aac98a7990002b (diff) |
[media] lg2160: fix off-by-one error in lg216x_write_regs
Fix an off-by-one error in lg216x_write_regs, causing the last element
of the lg216x init block to be ignored. Spotted by Dan Carpenter.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/media/dvb/frontends/lg2160.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/dvb/frontends/lg2160.c b/drivers/media/dvb/frontends/lg2160.c index a3ab1a5b6597..cc11260e99df 100644 --- a/drivers/media/dvb/frontends/lg2160.c +++ b/drivers/media/dvb/frontends/lg2160.c | |||
@@ -126,7 +126,7 @@ static int lg216x_write_regs(struct lg216x_state *state, | |||
126 | 126 | ||
127 | lg_reg("writing %d registers...\n", len); | 127 | lg_reg("writing %d registers...\n", len); |
128 | 128 | ||
129 | for (i = 0; i < len - 1; i++) { | 129 | for (i = 0; i < len; i++) { |
130 | ret = lg216x_write_reg(state, regs[i].reg, regs[i].val); | 130 | ret = lg216x_write_reg(state, regs[i].reg, regs[i].val); |
131 | if (lg_fail(ret)) | 131 | if (lg_fail(ret)) |
132 | return ret; | 132 | return ret; |