OSDN Git Service

platform/x86: thinkpad_acpi: Accept ibm_init_struct.init() returning -ENODEV
authorHans de Goede <hdegoede@redhat.com>
Sun, 21 Nov 2021 19:11:23 +0000 (20:11 +0100)
committerHans de Goede <hdegoede@redhat.com>
Mon, 22 Nov 2021 12:53:17 +0000 (13:53 +0100)
commitc15f86856bec5bbf9a5ea909ce5ccc5b05744eb1
tree269d89fc495d74122c86616b67e201186eaeb45a
parentff448bbaacfb6f216ae101c1f16d8c5142c16fdf
platform/x86: thinkpad_acpi: Accept ibm_init_struct.init() returning -ENODEV

Commit 79f960e29cfc ("platform/x86: thinkpad_acpi: Convert platform driver
to use dev_groups") accidentally modified tpacpi_kbdlang_init() causing it
to return -ENODEV instead of 0 on machines without kbdlang support
(which are most of them).

ibm_init() sees this -ENODEV as an error causing the entire module to not
load, not good.

Note that technically tpacpi_kbdlang_init() was already buggy before, it
should have returned 1 instead of 0 if the feature is not present.

Rather then fixing tpacpi_kbdlang_init() though, IMHO it is bettter to
just make ibm_init() treat -ENODEV as 1 to fix the issue; and then in
a followup commit also change all the existing "return 1"s from
ibm_init_struct.init() callbacks to "return -ENODEV" as -ENODEV clearly
states what it going on where as a magic return of "1" requires a deep
dive into the code to figure out what is going on.

This will also allow removing some extra ifs to translate -ENODEV to
return 1 in a couple of init() callbacks.

Fixes: 79f960e29cfc ("platform/x86: thinkpad_acpi: Convert platform driver to use dev_groups")
Cc: Len Baker <len.baker@gmx.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20211121191129.256713-2-hdegoede@redhat.com
drivers/platform/x86/thinkpad_acpi.c