summaryrefslogtreecommitdiffstats
path: root/sys/cam
diff options
context:
space:
mode:
authorsmh <smh@FreeBSD.org>2013-04-26 15:31:52 +0000
committersmh <smh@FreeBSD.org>2013-04-26 15:31:52 +0000
commitbb465a0ea2adda799d70e444a0c74c76d2aa2aa7 (patch)
tree4331afc3efbdc0ba50afea380f2c1b3e54ab48c4 /sys/cam
parent92fe819b2bf3d39de203f8c4081832b443628a90 (diff)
downloadFreeBSD-src-bb465a0ea2adda799d70e444a0c74c76d2aa2aa7.zip
FreeBSD-src-bb465a0ea2adda799d70e444a0c74c76d2aa2aa7.tar.gz
Removed unneeded tests in dadeletemethodset changing it to return void
Reviewed by: mav Approved by: pjd (mentor) MFC after: 2 weeks
Diffstat (limited to 'sys/cam')
-rw-r--r--sys/cam/scsi/scsi_da.c11
1 files changed, 4 insertions, 7 deletions
diff --git a/sys/cam/scsi/scsi_da.c b/sys/cam/scsi/scsi_da.c
index a7a5db5..d2318bf 100644
--- a/sys/cam/scsi/scsi_da.c
+++ b/sys/cam/scsi/scsi_da.c
@@ -868,7 +868,7 @@ static void daasync(void *callback_arg, u_int32_t code,
static void dasysctlinit(void *context, int pending);
static int dacmdsizesysctl(SYSCTL_HANDLER_ARGS);
static int dadeletemethodsysctl(SYSCTL_HANDLER_ARGS);
-static int dadeletemethodset(struct da_softc *softc,
+static void dadeletemethodset(struct da_softc *softc,
da_delete_methods delete_method);
static periph_ctor_t daregister;
static periph_dtor_t dacleanup;
@@ -1568,12 +1568,10 @@ dacmdsizesysctl(SYSCTL_HANDLER_ARGS)
return (0);
}
-static int
+static void
dadeletemethodset(struct da_softc *softc, da_delete_methods delete_method)
{
- if (delete_method < 0 || delete_method > DA_DELETE_MAX)
- return (EINVAL);
softc->delete_method = delete_method;
@@ -1581,8 +1579,6 @@ dadeletemethodset(struct da_softc *softc, da_delete_methods delete_method)
softc->disk->d_flags |= DISKFLAG_CANDELETE;
else
softc->disk->d_flags &= ~DISKFLAG_CANDELETE;
-
- return (0);
}
static int
@@ -1607,7 +1603,8 @@ dadeletemethodsysctl(SYSCTL_HANDLER_ARGS)
for (i = 0; i <= DA_DELETE_MAX; i++) {
if (strcmp(buf, da_delete_method_names[i]) != 0)
continue;
- return dadeletemethodset(softc, i);
+ dadeletemethodset(softc, i);
+ return (0);
}
return (EINVAL);
}
OpenPOWER on IntegriCloud