the interfaceTag is actually initialised in for loop of this function and its not needed to initialise it before for loop. and also remove the comment that is obvious about this variable. Cc: Mikko Virkkilä <mikko.virkkila@xxxxxxxxxxxx> Cc: Lauri Hintsala <Lauri.Hintsala@xxxxxxxxxxxx> Cc: Riku Mettälä <riku.mettala@xxxxxxxxxxxx> Cc: Veli-Pekka Peltola <veli-pekka.peltola@xxxxxxxxxxxx> Signed-off-by: Devendra Naga <develkernel412222@xxxxxxxxx> --- drivers/staging/csr/bh.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/csr/bh.c b/drivers/staging/csr/bh.c index 2597e22..3193df2 100644 --- a/drivers/staging/csr/bh.c +++ b/drivers/staging/csr/bh.c @@ -164,7 +164,7 @@ uf_wait_for_thread_to_stop(unifi_priv_t *priv, struct uf_thread *thread) handle_bh_error(unifi_priv_t *priv) { u8 conf_param = CONFIG_IND_ERROR; - u8 interfaceTag = 0; /* used as a loop counter */ + u8 interfaceTag; /* Block unifi_run_bh() until the error has been handled. */ -- 1.7.9.5 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel