platform/x86: ideapad-laptop: Remove unnecessary else
authorJiaxun Yang <jiaxun.yang@flygoat.com>
Sat, 2 Dec 2017 13:45:31 +0000 (21:45 +0800)
committerDarren Hart (VMware) <dvhart@infradead.org>
Tue, 12 Dec 2017 00:31:04 +0000 (16:31 -0800)
Address the following checkpatch warning by removing unnecessary else
blocks:
WARNING: else is not generally useful after a break or return

Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Signed-off-by: Darren Hart (VMware) <dvhart@infradead.org>
drivers/platform/x86/ideapad-laptop.c

index 53ab4e0..9f2a4bc 100644 (file)
@@ -124,10 +124,10 @@ static int read_method_int(acpi_handle handle, const char *method, int *val)
        if (ACPI_FAILURE(status)) {
                *val = -1;
                return -1;
-       } else {
-               *val = result;
-               return 0;
        }
+       *val = result;
+       return 0;
+
 }
 
 static int method_gbmd(acpi_handle handle, unsigned long *ret)
@@ -164,10 +164,10 @@ static int method_vpcr(acpi_handle handle, int cmd, int *ret)
        if (ACPI_FAILURE(status)) {
                *ret = -1;
                return -1;
-       } else {
-               *ret = result;
-               return 0;
        }
+       *ret = result;
+       return 0;
+
 }
 
 static int method_vpcw(acpi_handle handle, int cmd, int data)