diff options
author | Jacob Pan <jacob.jun.pan@linux.intel.com> | 2016-11-14 14:08:45 -0500 |
---|---|---|
committer | Zhang Rui <rui.zhang@intel.com> | 2016-11-21 07:54:40 -0500 |
commit | ec638db8cb9ddd5ca08b23f2835b6c9c15eb616d (patch) | |
tree | b1863f1f3e95f247fb83f2a7bb198952eeae1398 | |
parent | b58ec8b5824717875b844a9e8cc62fc6eb5fd289 (diff) |
thermal/powerclamp: add back module device table
Commit 3105f234e0aba43e44e277c20f9b32ee8add43d4 replaced module
cpu id table with a cpu feature check, which is logically correct.
But we need the module device table to allow module auto loading.
Cc: stable@vger.kernel.org # 4.8
Fixes:3105f234 thermal/powerclamp: correct cpu support check
Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
-rw-r--r-- | drivers/thermal/intel_powerclamp.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/drivers/thermal/intel_powerclamp.c b/drivers/thermal/intel_powerclamp.c index 7a223074df3d..afada655f861 100644 --- a/drivers/thermal/intel_powerclamp.c +++ b/drivers/thermal/intel_powerclamp.c | |||
@@ -669,9 +669,16 @@ static struct thermal_cooling_device_ops powerclamp_cooling_ops = { | |||
669 | .set_cur_state = powerclamp_set_cur_state, | 669 | .set_cur_state = powerclamp_set_cur_state, |
670 | }; | 670 | }; |
671 | 671 | ||
672 | static const struct x86_cpu_id __initconst intel_powerclamp_ids[] = { | ||
673 | { X86_VENDOR_INTEL, X86_FAMILY_ANY, X86_MODEL_ANY, X86_FEATURE_MWAIT }, | ||
674 | {} | ||
675 | }; | ||
676 | MODULE_DEVICE_TABLE(x86cpu, intel_powerclamp_ids); | ||
677 | |||
672 | static int __init powerclamp_probe(void) | 678 | static int __init powerclamp_probe(void) |
673 | { | 679 | { |
674 | if (!boot_cpu_has(X86_FEATURE_MWAIT)) { | 680 | |
681 | if (!x86_match_cpu(intel_powerclamp_ids)) { | ||
675 | pr_err("CPU does not support MWAIT"); | 682 | pr_err("CPU does not support MWAIT"); |
676 | return -ENODEV; | 683 | return -ENODEV; |
677 | } | 684 | } |