[PATCH 4/4] staging: unisys: remove generally useless elses from visorchipset_main.c

[Benjamin Romer]

From: Benjamin Romer <bro...@comcast.net>

There are several else clauses that aren't necessary listed by checkpatch.pl in
visorchipset_main.c. Remove these.

Signed-off-by: Benjamin Romer <benjamin.ro...@unisys.com>
---
 .../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 2ca9461..1eeeef8 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
-- 
1.9.1

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel