diff options
author | Jayamohan Kallickal <jayamohank@serverengines.com> | 2010-07-21 18:47:16 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-07-28 10:05:29 -0400 |
commit | 03a1231009927b7168d6d86a7a7f6c7f9b4be85a (patch) | |
tree | d8dd64421f69e6c9a535bf33bbaadc87f7230daa /drivers | |
parent | d3ad2bb31c26d7314fad98da8abb04f4fa24ed16 (diff) |
[SCSI] be2iscsi: Fixing the return type of functions
Fixing some functions return values that did not match with
the possible return values
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')
-rw-r--r-- | drivers/scsi/be2iscsi/be_cmds.h | 2 | ||||
-rw-r--r-- | drivers/scsi/be2iscsi/be_main.c | 2 | ||||
-rw-r--r-- | drivers/scsi/be2iscsi/be_mgmt.c | 13 | ||||
-rw-r--r-- | drivers/scsi/be2iscsi/be_mgmt.h | 10 |
4 files changed, 13 insertions, 14 deletions
diff --git a/drivers/scsi/be2iscsi/be_cmds.h b/drivers/scsi/be2iscsi/be_cmds.h index f94df6cdebc8..30a293f52ef7 100644 --- a/drivers/scsi/be2iscsi/be_cmds.h +++ b/drivers/scsi/be2iscsi/be_cmds.h | |||
@@ -423,7 +423,7 @@ int beiscsi_cmd_mccq_create(struct beiscsi_hba *phba, | |||
423 | struct be_queue_info *cq); | 423 | struct be_queue_info *cq); |
424 | 424 | ||
425 | int be_poll_mcc(struct be_ctrl_info *ctrl); | 425 | int be_poll_mcc(struct be_ctrl_info *ctrl); |
426 | unsigned char mgmt_check_supported_fw(struct be_ctrl_info *ctrl, | 426 | int mgmt_check_supported_fw(struct be_ctrl_info *ctrl, |
427 | struct beiscsi_hba *phba); | 427 | struct beiscsi_hba *phba); |
428 | unsigned int be_cmd_get_mac_addr(struct beiscsi_hba *phba); | 428 | unsigned int be_cmd_get_mac_addr(struct beiscsi_hba *phba); |
429 | void free_mcc_tag(struct be_ctrl_info *ctrl, unsigned int tag); | 429 | void free_mcc_tag(struct be_ctrl_info *ctrl, unsigned int tag); |
diff --git a/drivers/scsi/be2iscsi/be_main.c b/drivers/scsi/be2iscsi/be_main.c index e6259d7498f3..3b54dc72371e 100644 --- a/drivers/scsi/be2iscsi/be_main.c +++ b/drivers/scsi/be2iscsi/be_main.c | |||
@@ -3295,7 +3295,7 @@ static void hwi_purge_eq(struct beiscsi_hba *phba) | |||
3295 | 3295 | ||
3296 | static void beiscsi_clean_port(struct beiscsi_hba *phba) | 3296 | static void beiscsi_clean_port(struct beiscsi_hba *phba) |
3297 | { | 3297 | { |
3298 | unsigned char mgmt_status; | 3298 | int mgmt_status; |
3299 | 3299 | ||
3300 | mgmt_status = mgmt_epfw_cleanup(phba, CMD_CONNECTION_CHUTE_0); | 3300 | mgmt_status = mgmt_epfw_cleanup(phba, CMD_CONNECTION_CHUTE_0); |
3301 | if (mgmt_status) | 3301 | if (mgmt_status) |
diff --git a/drivers/scsi/be2iscsi/be_mgmt.c b/drivers/scsi/be2iscsi/be_mgmt.c index ff8b1cd6fec4..dcb70fac2fce 100644 --- a/drivers/scsi/be2iscsi/be_mgmt.c +++ b/drivers/scsi/be2iscsi/be_mgmt.c | |||
@@ -21,7 +21,7 @@ | |||
21 | #include "be_mgmt.h" | 21 | #include "be_mgmt.h" |
22 | #include "be_iscsi.h" | 22 | #include "be_iscsi.h" |
23 | 23 | ||
24 | unsigned char mgmt_get_fw_config(struct be_ctrl_info *ctrl, | 24 | int mgmt_get_fw_config(struct be_ctrl_info *ctrl, |
25 | struct beiscsi_hba *phba) | 25 | struct beiscsi_hba *phba) |
26 | { | 26 | { |
27 | struct be_mcc_wrb *wrb = wrb_from_mbox(&ctrl->mbox_mem); | 27 | struct be_mcc_wrb *wrb = wrb_from_mbox(&ctrl->mbox_mem); |
@@ -65,7 +65,7 @@ unsigned char mgmt_get_fw_config(struct be_ctrl_info *ctrl, | |||
65 | return status; | 65 | return status; |
66 | } | 66 | } |
67 | 67 | ||
68 | unsigned char mgmt_check_supported_fw(struct be_ctrl_info *ctrl, | 68 | int mgmt_check_supported_fw(struct be_ctrl_info *ctrl, |
69 | struct beiscsi_hba *phba) | 69 | struct beiscsi_hba *phba) |
70 | { | 70 | { |
71 | struct be_dma_mem nonemb_cmd; | 71 | struct be_dma_mem nonemb_cmd; |
@@ -117,8 +117,7 @@ unsigned char mgmt_check_supported_fw(struct be_ctrl_info *ctrl, | |||
117 | return status; | 117 | return status; |
118 | } | 118 | } |
119 | 119 | ||
120 | 120 | int mgmt_epfw_cleanup(struct beiscsi_hba *phba, unsigned short chute) | |
121 | unsigned char mgmt_epfw_cleanup(struct beiscsi_hba *phba, unsigned short chute) | ||
122 | { | 121 | { |
123 | struct be_ctrl_info *ctrl = &phba->ctrl; | 122 | struct be_ctrl_info *ctrl = &phba->ctrl; |
124 | struct be_mcc_wrb *wrb = wrb_from_mccq(phba); | 123 | struct be_mcc_wrb *wrb = wrb_from_mccq(phba); |
@@ -144,7 +143,7 @@ unsigned char mgmt_epfw_cleanup(struct beiscsi_hba *phba, unsigned short chute) | |||
144 | return status; | 143 | return status; |
145 | } | 144 | } |
146 | 145 | ||
147 | unsigned char mgmt_invalidate_icds(struct beiscsi_hba *phba, | 146 | unsigned int mgmt_invalidate_icds(struct beiscsi_hba *phba, |
148 | struct invalidate_command_table *inv_tbl, | 147 | struct invalidate_command_table *inv_tbl, |
149 | unsigned int num_invalidate, unsigned int cid) | 148 | unsigned int num_invalidate, unsigned int cid) |
150 | { | 149 | { |
@@ -202,7 +201,7 @@ unsigned char mgmt_invalidate_icds(struct beiscsi_hba *phba, | |||
202 | return tag; | 201 | return tag; |
203 | } | 202 | } |
204 | 203 | ||
205 | unsigned char mgmt_invalidate_connection(struct beiscsi_hba *phba, | 204 | unsigned int mgmt_invalidate_connection(struct beiscsi_hba *phba, |
206 | struct beiscsi_endpoint *beiscsi_ep, | 205 | struct beiscsi_endpoint *beiscsi_ep, |
207 | unsigned short cid, | 206 | unsigned short cid, |
208 | unsigned short issue_reset, | 207 | unsigned short issue_reset, |
@@ -239,7 +238,7 @@ unsigned char mgmt_invalidate_connection(struct beiscsi_hba *phba, | |||
239 | return tag; | 238 | return tag; |
240 | } | 239 | } |
241 | 240 | ||
242 | unsigned char mgmt_upload_connection(struct beiscsi_hba *phba, | 241 | unsigned int mgmt_upload_connection(struct beiscsi_hba *phba, |
243 | unsigned short cid, unsigned int upload_flag) | 242 | unsigned short cid, unsigned int upload_flag) |
244 | { | 243 | { |
245 | struct be_ctrl_info *ctrl = &phba->ctrl; | 244 | struct be_ctrl_info *ctrl = &phba->ctrl; |
diff --git a/drivers/scsi/be2iscsi/be_mgmt.h b/drivers/scsi/be2iscsi/be_mgmt.h index 3d316b82feb1..74b885a4b836 100644 --- a/drivers/scsi/be2iscsi/be_mgmt.h +++ b/drivers/scsi/be2iscsi/be_mgmt.h | |||
@@ -86,14 +86,14 @@ struct mcc_wrb { | |||
86 | struct mcc_wrb_payload payload; | 86 | struct mcc_wrb_payload payload; |
87 | }; | 87 | }; |
88 | 88 | ||
89 | unsigned char mgmt_epfw_cleanup(struct beiscsi_hba *phba, unsigned short chute); | 89 | int mgmt_epfw_cleanup(struct beiscsi_hba *phba, unsigned short chute); |
90 | int mgmt_open_connection(struct beiscsi_hba *phba, struct sockaddr *dst_addr, | 90 | int mgmt_open_connection(struct beiscsi_hba *phba, struct sockaddr *dst_addr, |
91 | struct beiscsi_endpoint *beiscsi_ep); | 91 | struct beiscsi_endpoint *beiscsi_ep); |
92 | 92 | ||
93 | unsigned char mgmt_upload_connection(struct beiscsi_hba *phba, | 93 | unsigned int mgmt_upload_connection(struct beiscsi_hba *phba, |
94 | unsigned short cid, | 94 | unsigned short cid, |
95 | unsigned int upload_flag); | 95 | unsigned int upload_flag); |
96 | unsigned char mgmt_invalidate_icds(struct beiscsi_hba *phba, | 96 | unsigned int mgmt_invalidate_icds(struct beiscsi_hba *phba, |
97 | struct invalidate_command_table *inv_tbl, | 97 | struct invalidate_command_table *inv_tbl, |
98 | unsigned int num_invalidate, unsigned int cid); | 98 | unsigned int num_invalidate, unsigned int cid); |
99 | 99 | ||
@@ -237,10 +237,10 @@ struct beiscsi_endpoint { | |||
237 | u16 cid_vld; | 237 | u16 cid_vld; |
238 | }; | 238 | }; |
239 | 239 | ||
240 | unsigned char mgmt_get_fw_config(struct be_ctrl_info *ctrl, | 240 | int mgmt_get_fw_config(struct be_ctrl_info *ctrl, |
241 | struct beiscsi_hba *phba); | 241 | struct beiscsi_hba *phba); |
242 | 242 | ||
243 | unsigned char mgmt_invalidate_connection(struct beiscsi_hba *phba, | 243 | unsigned int mgmt_invalidate_connection(struct beiscsi_hba *phba, |
244 | struct beiscsi_endpoint *beiscsi_ep, | 244 | struct beiscsi_endpoint *beiscsi_ep, |
245 | unsigned short cid, | 245 | unsigned short cid, |
246 | unsigned short issue_reset, | 246 | unsigned short issue_reset, |