There are several missing curly braces so this function doesn't work as
intended.

Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>

diff --git a/drivers/scsi/bfa/bfa_ioc.c b/drivers/scsi/bfa/bfa_ioc.c
index 315d6d6..59c3492 100644
--- a/drivers/scsi/bfa/bfa_ioc.c
+++ b/drivers/scsi/bfa/bfa_ioc.c
@@ -3662,7 +3662,7 @@ bfa_cb_sfp_state_query(struct bfa_sfp_s *sfp)
        bfa_trc(sfp, sfp->portspeed);
        if (sfp->media) {
                bfa_sfp_media_get(sfp);
-               if (sfp->state_query_cbfn)
+               if (sfp->state_query_cbfn) {
                        sfp->state_query_cbfn(sfp->state_query_cbarg,
                                        sfp->status);
                        sfp->media = NULL;
@@ -3670,14 +3670,16 @@ bfa_cb_sfp_state_query(struct bfa_sfp_s *sfp)
 
                if (sfp->portspeed) {
                        sfp->status = bfa_sfp_speed_valid(sfp, sfp->portspeed);
-                       if (sfp->state_query_cbfn)
+                       if (sfp->state_query_cbfn) {
                                sfp->state_query_cbfn(sfp->state_query_cbarg,
                                                sfp->status);
                                sfp->portspeed = BFA_PORT_SPEED_UNKNOWN;
+                       }
                }
 
                sfp->state_query_lock = 0;
                sfp->state_query_cbfn = NULL;
+       }
 }
 
 /*
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to