Since a negative value may be assigned to 'rc', use a signed type for that variable. Cc: Anil Gurumurthy <anil.gurumurthy@xxxxxxxxxx> Cc: Sudarsana Kalluru <sudarsana.kalluru@xxxxxxxxxx> Fixes: b85daafe46ee ("[SCSI] bfa: Add BSG interface to support ELS, CT and vendor commands.") Signed-off-by: Bart Van Assche <bvanassche@xxxxxxx> --- drivers/scsi/bfa/bfad_bsg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/bfa/bfad_bsg.c b/drivers/scsi/bfa/bfad_bsg.c index be8dfbe13e90..371856bc67a8 100644 --- a/drivers/scsi/bfa/bfad_bsg.c +++ b/drivers/scsi/bfa/bfad_bsg.c @@ -3552,7 +3552,7 @@ bfad_im_bsg_request(struct bsg_job *job) { struct fc_bsg_request *bsg_request = job->request; struct fc_bsg_reply *bsg_reply = job->reply; - uint32_t rc = BFA_STATUS_OK; + int32_t rc = BFA_STATUS_OK; switch (bsg_request->msgcode) { case FC_BSG_HST_VENDOR: