Function checks for condition inside a loop that checks the same condition, this is redundant. Fix also removes checkpatch CHECK. Remove redundant check. Signed-off-by: Tobin C. Harding <me@xxxxxxxx> --- drivers/staging/ks7010/ks_hostif.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c index 460ab13..b1213c8 100644 --- a/drivers/staging/ks7010/ks_hostif.c +++ b/drivers/staging/ks7010/ks_hostif.c @@ -2607,8 +2607,7 @@ void hostif_sme_task(unsigned long dev) DPRINTK(3, "\n"); if (priv->dev_state >= DEVICE_STATE_BOOT) { - if (cnt_smeqbody(priv) > 0 - && priv->dev_state >= DEVICE_STATE_BOOT) { + if (cnt_smeqbody(priv) > 0) { hostif_sme_execute(priv, priv->sme_i.event_buff[priv->sme_i. qhead]); -- 2.7.4 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel