diff options
author | Jayamohan Kallickal <jayamohank@serverengines.com> | 2010-07-21 18:46:00 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-07-28 10:05:28 -0400 |
commit | 457ff3b7dc3796d8778286217ad304ff122e948f (patch) | |
tree | 32a0a7e6c67d2bb564efc9238abefa9239d38c72 /drivers/scsi/be2iscsi/be_cmds.c | |
parent | bbe56c734cc1ecccd7b2b143e1767bf2b1eafc76 (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.c | 24 |
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); |