From e22a4a0f67030da9e5c7688237ca976e7211abfb Mon Sep 17 00:00:00 2001 From: Benjamin Romer Date: Mon, 18 Aug 2014 09:34:54 -0400 Subject: [PATCH] staging: unisys: remove generally useless elses from visorchipset_main.c There are several else clauses that aren't necessary listed by checkpatch.pl in visorchipset_main.c. Remove these. Signed-off-by: Benjamin Romer Signed-off-by: Greg Kroah-Hartman --- .../unisys/visorchipset/visorchipset_main.c | 26 +++++++------------ 1 file changed, 10 insertions(+), 16 deletions(-) diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c b/drivers/staging/unisys/visorchipset/visorchipset_main.c index 2ca94614f358..1eeeef83442f 100644 --- a/drivers/staging/unisys/visorchipset/visorchipset_main.c +++ b/drivers/staging/unisys/visorchipset/visorchipset_main.c @@ -58,8 +58,7 @@ NONULLSTR(char *s) { if (s) return s; - else - return ""; + return ""; } static int serverregistered; @@ -379,8 +378,7 @@ static ssize_t toolaction_store(struct device *dev, if (ret) return ret; - else - return count; + return count; } static ssize_t boottotool_show(struct device *dev, @@ -416,8 +414,7 @@ static ssize_t boottotool_store(struct device *dev, if (ret) return ret; - else - return count; + return count; } static ssize_t error_show(struct device *dev, struct device_attribute *attr, @@ -446,8 +443,7 @@ static ssize_t error_store(struct device *dev, struct device_attribute *attr, &error, sizeof(u32)); if (ret) return ret; - else - return count; + return count; } static ssize_t textid_show(struct device *dev, struct device_attribute *attr, @@ -476,8 +472,7 @@ static ssize_t textid_store(struct device *dev, struct device_attribute *attr, &textId, sizeof(u32)); if (ret) return ret; - else - return count; + return count; } @@ -509,8 +504,7 @@ static ssize_t remaining_steps_store(struct device *dev, &remainingSteps, sizeof(u16)); if (ret) return ret; - else - return count; + return count; } #if 0 @@ -1544,8 +1538,8 @@ read_controlvm_event(CONTROLVM_MESSAGE *msg) if (msg->hdr.Flags.testMessage == 1) { LOGERR("ignoring bad CONTROLVM_QUEUE_EVENT msg with controlvm_msg_id=0x%x because Flags.testMessage is nonsensical (=1)", msg->hdr.Id); return FALSE; - } else - return TRUE; + } + return TRUE; } return FALSE; } @@ -2305,8 +2299,8 @@ static ssize_t chipsetready_store(struct device *dev, } else if (strcmp(msgtype, "MODULES_LOADED") == 0) { chipset_events[1] = 1; return count; - } else - return -EINVAL; + } + return -EINVAL; } /* The parahotplug/devicedisabled interface gets called by our support script -- 2.30.2