[PULL BUG-FIX] git://git.linuxtv.org/mkrufky/mxl111sf lg2160

[PULL BUG-FIX] git://git.linuxtv.org/mkrufky/mxl111sf lg2160

[Michael Krufky]

Mauro,

Please merge:

The following changes since commit abed623ca59a7d1abed6c4e7459be03e25a90a1e:

  [media] radio-sf16fmi: add support for SF16-FMD (2012-05-20 16:10:05 -0300)

are available in the git repository at:

  git://git.linuxtv.org/mkrufky/mxl111sf lg2160

for you to fetch changes up to 80fed89735a02dd03d682261967a246ca9b0129f:

  lg2160: fix off-by-one error in lg216x_write_regs (2012-05-21 12:04:05 -0400)

----------------------------------------------------------------
Michael Krufky (1):
      lg2160: fix off-by-one error in lg216x_write_regs

 drivers/media/dvb/frontends/lg2160.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


On Mon, May 21, 2012 at 12:06 PM, Michael Krufky <mkrufky@kernellabs.com> wrote:
> 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>
> ---
>  drivers/media/dvb/frontends/lg2160.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/dvb/frontends/lg2160.c b/drivers/media/dvb/frontends/lg2160.c
> index a3ab1a5..cc11260 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,
>
>        lg_reg("writing %d registers...\n", len);
>
> -       for (i = 0; i < len - 1; i++) {
> +       for (i = 0; i < len; i++) {
>                ret = lg216x_write_reg(state, regs[i].reg, regs[i].val);
>                if (lg_fail(ret))
>                        return ret;
> --
> 1.7.9.5
>