diff options
author | Hans de Goede <hdegoede@redhat.com> | 2021-02-04 17:01:58 +0300 |
---|---|---|
committer | Hans de Goede <hdegoede@redhat.com> | 2021-02-04 17:15:19 +0300 |
commit | 9aa422f676c9bbd6621080924c135707510096bc (patch) | |
tree | 7e2a34562dbd3ce915ebeb386ad2e2a6819d7ad7 /crypto/adiantum.c | |
parent | d2386d791cb72e0dcaa5f43d509a4f71c44f47d8 (diff) | |
download | linux-9aa422f676c9bbd6621080924c135707510096bc.tar.xz |
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 'crypto/adiantum.c')
0 files changed, 0 insertions, 0 deletions