From: Borislav Petkov <[email protected]> This driver is Intel-only so loading on anything which is not Intel is pointless. Prevent it from doing so.
While at it, correct the "not supported" print statement to say CPU "model" which is what that test does. Suggested-by: Erwan Velu <[email protected]> Signed-off-by: Borislav Petkov <[email protected]> Cc: Len Brown <[email protected]> Cc: [email protected] Cc: Rafael J. Wysocki <[email protected]> CC: Srinivas Pandruvada <[email protected]> Cc: Viresh Kumar <[email protected]> --- drivers/cpufreq/intel_pstate.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c index b599c7318aab..2986119dd31f 100644 --- a/drivers/cpufreq/intel_pstate.c +++ b/drivers/cpufreq/intel_pstate.c @@ -2596,6 +2596,9 @@ static int __init intel_pstate_init(void) const struct x86_cpu_id *id; int rc; + if (boot_cpu_data.x86_vendor != X86_VENDOR_INTEL) + return -ENODEV; + if (no_load) return -ENODEV; @@ -2611,7 +2614,7 @@ static int __init intel_pstate_init(void) } else { id = x86_match_cpu(intel_pstate_cpu_ids); if (!id) { - pr_info("CPU ID not supported\n"); + pr_info("CPU model not supported\n"); return -ENODEV; } -- 2.21.0 -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.
