aboutsummaryrefslogtreecommitdiff
path: root/sys
diff options
context:
space:
mode:
authorPoul-Henning Kamp <phk@FreeBSD.org>2003-05-31 16:36:42 +0000
committerPoul-Henning Kamp <phk@FreeBSD.org>2003-05-31 16:36:42 +0000
commit7021f3f539f6d715d44fecb739bef366bebde31c (patch)
treea832cd95eee0962ff4e48feab26926fb0084f921 /sys
parent239e385f11bbefc165e467356b8aeecf57358407 (diff)
downloadsrc-7021f3f539f6d715d44fecb739bef366bebde31c.tar.gz
src-7021f3f539f6d715d44fecb739bef366bebde31c.zip
Don't use return(foo(...)); in a function returning void.
Found by: FlexeLint
Notes
Notes: svn path=/head/; revision=115454
Diffstat (limited to 'sys')
-rw-r--r--sys/cam/scsi/scsi_all.c14
1 files changed, 8 insertions, 6 deletions
diff --git a/sys/cam/scsi/scsi_all.c b/sys/cam/scsi/scsi_all.c
index fe581af5cda3..72601dae35be 100644
--- a/sys/cam/scsi/scsi_all.c
+++ b/sys/cam/scsi/scsi_all.c
@@ -2453,10 +2453,12 @@ scsi_mode_sense(struct ccb_scsiio *csio, u_int32_t retries,
u_int8_t page, u_int8_t *param_buf, u_int32_t param_len,
u_int8_t sense_len, u_int32_t timeout)
{
- return(scsi_mode_sense_len(csio, retries, cbfcnp, tag_action, dbd,
- page_code, page, param_buf, param_len, 0,
- sense_len, timeout));
+
+ scsi_mode_sense_len(csio, retries, cbfcnp, tag_action, dbd,
+ page_code, page, param_buf, param_len, 0,
+ sense_len, timeout);
}
+
void
scsi_mode_sense_len(struct ccb_scsiio *csio, u_int32_t retries,
void (*cbfcnp)(struct cam_periph *, union ccb *),
@@ -2518,9 +2520,9 @@ scsi_mode_select(struct ccb_scsiio *csio, u_int32_t retries,
u_int8_t *param_buf, u_int32_t param_len, u_int8_t sense_len,
u_int32_t timeout)
{
- return(scsi_mode_select_len(csio, retries, cbfcnp, tag_action,
- scsi_page_fmt, save_pages, param_buf,
- param_len, 0, sense_len, timeout));
+ scsi_mode_select_len(csio, retries, cbfcnp, tag_action,
+ scsi_page_fmt, save_pages, param_buf,
+ param_len, 0, sense_len, timeout);
}
void