diff options
| author | Marcos Paulo de Souza <marcos.mage@gmail.com> | 2012-02-04 22:26:13 +0100 |
|---|---|---|
| committer | Rafael J. Wysocki <rjw@sisk.pl> | 2012-02-09 23:55:43 +0100 |
| commit | 8916e3702ec422b57cc549fbae3986106292100f (patch) | |
| tree | 476b8f1079e607518d8be5333abfd58ef53d792f /kernel/power/suspend.c | |
| parent | 3ed3c7b559f3cae7a5a92860a127ceb0cafd5a9c (diff) | |
PM / Suspend: Avoid code duplication in suspend statistics update
The code
if (error) {
suspend_stats.fail++;
dpm_save_failed_errno(error);
} else
suspend_stats.success++;
Appears in the kernel/power/main.c and kernel/power/suspend.c.
This patch just creates a new function to avoid duplicated code.
Suggested-by: Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com>
Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com>
Acked-by: Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com>
Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Diffstat (limited to 'kernel/power/suspend.c')
| -rw-r--r-- | kernel/power/suspend.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/kernel/power/suspend.c b/kernel/power/suspend.c index 560a639614a..03bc92b4275 100644 --- a/kernel/power/suspend.c +++ b/kernel/power/suspend.c @@ -321,11 +321,7 @@ int pm_suspend(suspend_state_t state) int ret; if (state > PM_SUSPEND_ON && state < PM_SUSPEND_MAX) { ret = enter_state(state); - if (ret) { - suspend_stats.fail++; - dpm_save_failed_errno(ret); - } else - suspend_stats.success++; + suspend_stats_update(ret); return ret; } return -EINVAL; |