diff options
| author |   Hans de Goede <hdegoede@redhat.com> | 2021-02-04 15:01:58 +0100 |
|---|---|---|
| committer | Hans de Goede <hdegoede@redhat.com> | 2021-02-04 15:15:19 +0100 |
| commit | 9aa422f676c9bbd6621080924c135707510096bc (patch) | |
| tree | 7e2a34562dbd3ce915ebeb386ad2e2a6819d7ad7 /lib/test_bitmap.c | |
| parent | platform/x86: thinkpad_acpi: Fix 'warning: no previous prototype for' warnings (diff) | |
| download | linux-dev-9aa422f676c9bbd6621080924c135707510096bc.tar.xz linux-dev-9aa422f676c9bbd6621080924c135707510096bc.zip | |
platform/x86: thinkpad_acpi: Replace ifdef CONFIG_ACPI_PLATFORM_PROFILE with depends on
With the #if IS_ENABLED(CONFIG_ACPI_PLATFORM_PROFILE), we get the following
errors when thinkpad_acpi is builtin while CONFIG_ACPI_PLATFORM_PROFILE=m :
drivers/platform/x86/thinkpad_acpi.c:10186: undefined reference to `platform_profile_notify'
drivers/platform/x86/thinkpad_acpi.c:10226: undefined reference to `platform_profile_register'
drivers/platform/x86/thinkpad_acpi.c:10246: undefined reference to `platform_profile_remove'
This could be fixed by changing the IS_ENABLED to IS_REACHABLE, but
I believe that it is better to just switch to using depends on.
Using depends on ensures that platform-profile support is always
available when thinkpad_acpi is build, hopefully leading to less
confusing bug-reports about it sometimes not working.
Cc: Mark Pearson <markpearson@lenovo.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20210204140158.268289-2-hdegoede@redhat.com
Diffstat (limited to 'lib/test_bitmap.c')
0 files changed, 0 insertions, 0 deletions