mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-31 02:21:39 -04:00
platform/x86: thinkpad_acpi: Remove unneeded goto
Remove an unneeded goto statement in hotkey_kthread(). Since the function has a single exit location with no cleanup code, the jump provides no benefit. Per the kernel coding style, returning directly is preferred over goto in such case [1]. [1] https://www.kernel.org/doc/html/latest/process/coding-style.html Signed-off-by: Eduardo Vasconcelos <eduardo@eduardovasconcelos.com> Tested-by: Eduardo Vasconcelos <eduardo@eduardovasconcelos.com> Reviewed-by: Mark Pearson <mpearson-lenovo@squebb.ca> Link: https://patch.msgid.link/20260425063936.9360-1-eduardo@eduardovasconcelos.com Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
This commit is contained in:
committed by
Ilpo Järvinen
parent
0f4a265107
commit
74ab7c4d73
@@ -2469,7 +2469,7 @@ static int hotkey_kthread(void *data)
|
||||
bool was_frozen;
|
||||
|
||||
if (tpacpi_lifecycle == TPACPI_LIFE_EXITING)
|
||||
goto exit;
|
||||
return 0;
|
||||
|
||||
set_freezable();
|
||||
|
||||
@@ -2526,7 +2526,6 @@ static int hotkey_kthread(void *data)
|
||||
si ^= 1;
|
||||
}
|
||||
|
||||
exit:
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user