PM / Sleep: Remove unnecessary label from suspend_freeze_processes()
authorRafael J. Wysocki <rjw@sisk.pl>
Sat, 11 Feb 2012 21:40:23 +0000 (22:40 +0100)
committerRafael J. Wysocki <rjw@sisk.pl>
Mon, 13 Feb 2012 15:26:13 +0000 (16:26 +0100)
The Finish label in suspend_freeze_processes() is in fact unnecessary
and makes the function look more complicated than it really is, so
remove that label (along with a few empty lines).

Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Acked-by: Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com>
kernel/power/power.h

index 21724ee..398d42b 100644 (file)
@@ -234,16 +234,14 @@ static inline int suspend_freeze_processes(void)
        int error;
 
        error = freeze_processes();
-
        /*
         * freeze_processes() automatically thaws every task if freezing
         * fails. So we need not do anything extra upon error.
         */
        if (error)
-               goto Finish;
+               return error;
 
        error = freeze_kernel_threads();
-
        /*
         * freeze_kernel_threads() thaws only kernel threads upon freezing
         * failure. So we have to thaw the userspace tasks ourselves.
@@ -251,7 +249,6 @@ static inline int suspend_freeze_processes(void)
        if (error)
                thaw_processes();
 
- Finish:
        return error;
 }