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 <m...@tobin.cc>
---
 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
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to