UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

ACPI: sysfs: Refactor param_get_trace_state() to drop dead code 

The param_get_trace_state() has a few dead code issues:
 - 'return 0;' is never reachable
 - a few 'else' keywords are redundant

Refactor param_get_trace_state() to drop dead code.

Note, leave one 'else' in order to have the best readability.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
Andy Shevchenko 2021-06-16 20:03:35 +03:00 committed by Rafael J. Wysocki
parent b272c05984
commit 0023b28bdb
1 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
drivers/acpi/sysfs.c
View File

@ -255,16 +255,12 @@ static int param_get_trace_state(char *buffer, const struct kernel_param *kp)
{ {
if (!(acpi_gbl_trace_flags & ACPI_TRACE_ENABLED)) if (!(acpi_gbl_trace_flags & ACPI_TRACE_ENABLED))
return sprintf(buffer, "disable\n"); return sprintf(buffer, "disable\n");
else { if (!acpi_gbl_trace_method_name)
if (acpi_gbl_trace_method_name) { return sprintf(buffer, "enable\n");
if (acpi_gbl_trace_flags & ACPI_TRACE_ONESHOT) if (acpi_gbl_trace_flags & ACPI_TRACE_ONESHOT)
return sprintf(buffer, "method-once\n"); return sprintf(buffer, "method-once\n");
else else
return sprintf(buffer, "method\n"); return sprintf(buffer, "method\n");
} else
return sprintf(buffer, "enable\n");
}
return 0;
} }
module_param_call(trace_state, param_set_trace_state, param_get_trace_state, module_param_call(trace_state, param_set_trace_state, param_get_trace_state,