diff options
author | Matthias Kaehlcke <mka@chromium.org> | 2017-04-17 20:05:03 +0200 |
---|---|---|
committer | Mike Snitzer <snitzer@redhat.com> | 2017-04-24 20:31:53 +0200 |
commit | e36215d87f301f9567c8c99fd34e6c3521a94ddf (patch) | |
tree | c12d12f5bf550823e34b5c26fef5cfd555dc29cd /drivers/md/dm-ioctl.c | |
parent | dm: remove dummy dm_table definition (diff) | |
download | linux-e36215d87f301f9567c8c99fd34e6c3521a94ddf.tar.xz linux-e36215d87f301f9567c8c99fd34e6c3521a94ddf.zip |
dm ioctl: remove double parentheses
The extra pair of parantheses is not needed and causes clang to generate
warnings about the DM_DEV_CREATE_CMD comparison in validate_params().
Also remove another double parentheses that doesn't cause a warning.
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Diffstat (limited to 'drivers/md/dm-ioctl.c')
-rw-r--r-- | drivers/md/dm-ioctl.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/md/dm-ioctl.c b/drivers/md/dm-ioctl.c index 641d5e9cfedd..0956b8659360 100644 --- a/drivers/md/dm-ioctl.c +++ b/drivers/md/dm-ioctl.c @@ -1770,12 +1770,12 @@ static int validate_params(uint cmd, struct dm_ioctl *param) cmd == DM_LIST_VERSIONS_CMD) return 0; - if ((cmd == DM_DEV_CREATE_CMD)) { + if (cmd == DM_DEV_CREATE_CMD) { if (!*param->name) { DMWARN("name not supplied when creating device"); return -EINVAL; } - } else if ((*param->uuid && *param->name)) { + } else if (*param->uuid && *param->name) { DMWARN("only supply one of name or uuid, cmd(%u)", cmd); return -EINVAL; } |