aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/scsi/be2iscsi/be_cmds.c
diff options
context:
space:
mode:
authorJayamohan Kallickal <jayamohank@serverengines.com>2010-07-21 18:46:00 -0400
committerJames Bottomley <James.Bottomley@suse.de>2010-07-28 10:05:28 -0400
commit457ff3b7dc3796d8778286217ad304ff122e948f (patch)
tree32a0a7e6c67d2bb564efc9238abefa9239d38c72 /drivers/scsi/be2iscsi/be_cmds.c
parentbbe56c734cc1ecccd7b2b143e1767bf2b1eafc76 (diff)
[SCSI] be2iscsi: Fix warnings from new checkpatch.pl
The latest checkpatch.pl throws some new warnings. Fixing it to get rid of a bunch of warnings Signed-off-by: Jayamohan Kallickal <jayamohank@serverengines.com> Reviewed-by: Mike Christie <michaelc@cs.wisc.edu> Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/be2iscsi/be_cmds.c')
-rw-r--r--drivers/scsi/be2iscsi/be_cmds.c24
1 files changed, 12 insertions, 12 deletions
diff --git a/drivers/scsi/be2iscsi/be_cmds.c b/drivers/scsi/be2iscsi/be_cmds.c
index cda6642c7368..4f19030c1e3e 100644
--- a/drivers/scsi/be2iscsi/be_cmds.c
+++ b/drivers/scsi/be2iscsi/be_cmds.c
@@ -151,20 +151,20 @@ void beiscsi_async_link_state_process(struct beiscsi_hba *phba,
151{ 151{
152 switch (evt->port_link_status) { 152 switch (evt->port_link_status) {
153 case ASYNC_EVENT_LINK_DOWN: 153 case ASYNC_EVENT_LINK_DOWN:
154 SE_DEBUG(DBG_LVL_1, "Link Down on Physical Port %d \n", 154 SE_DEBUG(DBG_LVL_1, "Link Down on Physical Port %d\n",
155 evt->physical_port); 155 evt->physical_port);
156 phba->state |= BE_ADAPTER_LINK_DOWN; 156 phba->state |= BE_ADAPTER_LINK_DOWN;
157 iscsi_host_for_each_session(phba->shost, 157 iscsi_host_for_each_session(phba->shost,
158 be2iscsi_fail_session); 158 be2iscsi_fail_session);
159 break; 159 break;
160 case ASYNC_EVENT_LINK_UP: 160 case ASYNC_EVENT_LINK_UP:
161 phba->state = BE_ADAPTER_UP; 161 phba->state = BE_ADAPTER_UP;
162 SE_DEBUG(DBG_LVL_1, "Link UP on Physical Port %d \n", 162 SE_DEBUG(DBG_LVL_1, "Link UP on Physical Port %d\n",
163 evt->physical_port); 163 evt->physical_port);
164 break; 164 break;
165 default: 165 default:
166 SE_DEBUG(DBG_LVL_1, "Unexpected Async Notification %d on" 166 SE_DEBUG(DBG_LVL_1, "Unexpected Async Notification %d on"
167 "Physical Port %d \n", 167 "Physical Port %d\n",
168 evt->port_link_status, 168 evt->port_link_status,
169 evt->physical_port); 169 evt->physical_port);
170 } 170 }
@@ -199,7 +199,7 @@ int beiscsi_process_mcc(struct beiscsi_hba *phba)
199 else 199 else
200 SE_DEBUG(DBG_LVL_1, 200 SE_DEBUG(DBG_LVL_1,
201 " Unsupported Async Event, flags" 201 " Unsupported Async Event, flags"
202 " = 0x%08x \n", compl->flags); 202 " = 0x%08x\n", compl->flags);
203 203
204 } else if (compl->flags & CQE_FLAGS_COMPLETED_MASK) { 204 } else if (compl->flags & CQE_FLAGS_COMPLETED_MASK) {
205 status = be_mcc_compl_process(ctrl, compl); 205 status = be_mcc_compl_process(ctrl, compl);
@@ -286,7 +286,7 @@ int be_mbox_notify(struct be_ctrl_info *ctrl)
286 286
287 status = be_mbox_db_ready_wait(ctrl); 287 status = be_mbox_db_ready_wait(ctrl);
288 if (status != 0) { 288 if (status != 0) {
289 SE_DEBUG(DBG_LVL_1, " be_mbox_db_ready_wait failed 1\n"); 289 SE_DEBUG(DBG_LVL_1, " be_mbox_db_ready_wait failed\n");
290 return status; 290 return status;
291 } 291 }
292 val = 0; 292 val = 0;
@@ -297,14 +297,14 @@ int be_mbox_notify(struct be_ctrl_info *ctrl)
297 297
298 status = be_mbox_db_ready_wait(ctrl); 298 status = be_mbox_db_ready_wait(ctrl);
299 if (status != 0) { 299 if (status != 0) {
300 SE_DEBUG(DBG_LVL_1, " be_mbox_db_ready_wait failed 2\n"); 300 SE_DEBUG(DBG_LVL_1, " be_mbox_db_ready_wait failed\n");
301 return status; 301 return status;
302 } 302 }
303 if (be_mcc_compl_is_new(compl)) { 303 if (be_mcc_compl_is_new(compl)) {
304 status = be_mcc_compl_process(ctrl, &mbox->compl); 304 status = be_mcc_compl_process(ctrl, &mbox->compl);
305 be_mcc_compl_use(compl); 305 be_mcc_compl_use(compl);
306 if (status) { 306 if (status) {
307 SE_DEBUG(DBG_LVL_1, "After be_mcc_compl_process \n"); 307 SE_DEBUG(DBG_LVL_1, "After be_mcc_compl_process\n");
308 return status; 308 return status;
309 } 309 }
310 } else { 310 } else {
@@ -500,7 +500,7 @@ int be_cmd_fw_initialize(struct be_ctrl_info *ctrl)
500 500
501 status = be_mbox_notify(ctrl); 501 status = be_mbox_notify(ctrl);
502 if (status) 502 if (status)
503 SE_DEBUG(DBG_LVL_1, "be_cmd_fw_initialize Failed \n"); 503 SE_DEBUG(DBG_LVL_1, "be_cmd_fw_initialize Failed\n");
504 504
505 spin_unlock(&ctrl->mbox_lock); 505 spin_unlock(&ctrl->mbox_lock);
506 return status; 506 return status;
@@ -517,7 +517,7 @@ int beiscsi_cmd_cq_create(struct be_ctrl_info *ctrl,
517 void *ctxt = &req->context; 517 void *ctxt = &req->context;
518 int status; 518 int status;
519 519
520 SE_DEBUG(DBG_LVL_8, "In beiscsi_cmd_cq_create \n"); 520 SE_DEBUG(DBG_LVL_8, "In beiscsi_cmd_cq_create\n");
521 spin_lock(&ctrl->mbox_lock); 521 spin_lock(&ctrl->mbox_lock);
522 memset(wrb, 0, sizeof(*wrb)); 522 memset(wrb, 0, sizeof(*wrb));
523 523
@@ -550,7 +550,7 @@ int beiscsi_cmd_cq_create(struct be_ctrl_info *ctrl,
550 cq->id = le16_to_cpu(resp->cq_id); 550 cq->id = le16_to_cpu(resp->cq_id);
551 cq->created = true; 551 cq->created = true;
552 } else 552 } else
553 SE_DEBUG(DBG_LVL_1, "In be_cmd_cq_create, status=ox%08x \n", 553 SE_DEBUG(DBG_LVL_1, "In be_cmd_cq_create, status=ox%08x\n",
554 status); 554 status);
555 spin_unlock(&ctrl->mbox_lock); 555 spin_unlock(&ctrl->mbox_lock);
556 556
@@ -619,7 +619,7 @@ int beiscsi_cmd_q_destroy(struct be_ctrl_info *ctrl, struct be_queue_info *q,
619 u8 subsys = 0, opcode = 0; 619 u8 subsys = 0, opcode = 0;
620 int status; 620 int status;
621 621
622 SE_DEBUG(DBG_LVL_8, "In beiscsi_cmd_q_destroy \n"); 622 SE_DEBUG(DBG_LVL_8, "In beiscsi_cmd_q_destroy\n");
623 spin_lock(&ctrl->mbox_lock); 623 spin_lock(&ctrl->mbox_lock);
624 memset(wrb, 0, sizeof(*wrb)); 624 memset(wrb, 0, sizeof(*wrb));
625 be_wrb_hdr_prepare(wrb, sizeof(*req), true, 0); 625 be_wrb_hdr_prepare(wrb, sizeof(*req), true, 0);