Thread (68 messages) flat view 68 messages, 6 authors, 2020-07-15

Re: [PATCH 09/13] cpufreq: acpi-cpufreq: Remove unused ID structs

From: Lee Jones <hidden>
Date: 2020-07-14 21:33:40
Also in: linux-pm, lkml

On Tue, 14 Jul 2020, Rafael J. Wysocki wrote:
On Tue, Jul 14, 2020 at 4:51 PM Lee Jones [off-list ref] wrote:
quoted
Can't see them being used anywhere and the compiler doesn't complain
that they're missing, so ...
Aren't they needed for automatic module loading in certain configurations?
Any idea how that works, or where the code is for that?
quoted
Fixes the following W=1 kernel build warning(s):

 drivers/cpufreq/acpi-cpufreq.c:1004:36: warning: ‘processor_device_ids’ defined but not used [-Wunused-const-variable=]
 997 | static const struct x86_cpu_id acpi_cpufreq_ids[] = {
 | ^~~~~~~~~~~~~~~~
 drivers/cpufreq/acpi-cpufreq.c:997:32: warning: ‘acpi_cpufreq_ids’ defined but not used [-Wunused-const-variable=]
 619 | static const struct acpi_device_id processor_device_ids[] = {
 | ^~~~~~~~~~~~~~~~~~~~

Cc: Andy Grover <redacted>
Cc: Paul Diefenbaugh <redacted>
Cc: Dominik Brodowski <redacted>
Cc: Denis Sadykov <redacted>
Signed-off-by: Lee Jones <redacted>
---
 drivers/cpufreq/acpi-cpufreq.c | 14 --------------
 1 file changed, 14 deletions(-)
diff --git a/drivers/cpufreq/acpi-cpufreq.c b/drivers/cpufreq/acpi-cpufreq.c
index d38a693b48e03..fc68f13352695 100644
--- a/drivers/cpufreq/acpi-cpufreq.c
+++ b/drivers/cpufreq/acpi-cpufreq.c
@@ -995,18 +995,4 @@ MODULE_PARM_DESC(acpi_pstate_strict,
 late_initcall(acpi_cpufreq_init);
 module_exit(acpi_cpufreq_exit);

-static const struct x86_cpu_id acpi_cpufreq_ids[] = {
-       X86_MATCH_FEATURE(X86_FEATURE_ACPI, NULL),
-       X86_MATCH_FEATURE(X86_FEATURE_HW_PSTATE, NULL),
-       {}
-};
-MODULE_DEVICE_TABLE(x86cpu, acpi_cpufreq_ids);
-
-static const struct acpi_device_id processor_device_ids[] = {
-       {ACPI_PROCESSOR_OBJECT_HID, },
-       {ACPI_PROCESSOR_DEVICE_HID, },
-       {},
-};
-MODULE_DEVICE_TABLE(acpi, processor_device_ids);
-
 MODULE_ALIAS("acpi");
-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help