Commit fb088674 authored by Roel Kluin's avatar Roel Kluin Committed by David S. Miller

IRDA: cnt is off by 1

If no prior break occurs, cnt reaches 101 after the loop, so we are still able
to change speed when cnt has become 100.
Signed-off-by: default avatarRoel Kluin <roel.kluin@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent acdb602f
...@@ -585,7 +585,7 @@ static int mcs_speed_change(struct mcs_cb *mcs) ...@@ -585,7 +585,7 @@ static int mcs_speed_change(struct mcs_cb *mcs)
mcs_get_reg(mcs, MCS_RESV_REG, &rval); mcs_get_reg(mcs, MCS_RESV_REG, &rval);
} while(cnt++ < 100 && (rval & MCS_IRINTX)); } while(cnt++ < 100 && (rval & MCS_IRINTX));
if(cnt >= 100) { if (cnt > 100) {
IRDA_ERROR("unable to change speed\n"); IRDA_ERROR("unable to change speed\n");
ret = -EIO; ret = -EIO;
goto error; goto error;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment