hwmon: (via-cputemp) Remove pointless CPU check on each CPU
authorThomas Gleixner <tglx@linutronix.de>
Thu, 17 Nov 2016 18:35:26 +0000 (19:35 +0100)
committerGuenter Roeck <linux@roeck-us.net>
Fri, 2 Dec 2016 21:28:13 +0000 (13:28 -0800)
The check loop for the cpu type is pointless as we already have a cpu model
match before that. The only thing which is not covered by that check would
be a smp system with two different cores. Not likely to happen.

Cc: Jean Delvare <jdelvare@suse.com>
Cc: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/via-cputemp.c

index ac91c07e3f906869858a4a359d157ebf89aa7b36..5b9866b1b437273c7a1da800f83f8210e7d959ce 100644 (file)
@@ -319,22 +319,8 @@ static int __init via_cputemp_init(void)
                goto exit;
 
        cpu_notifier_register_begin();
-       for_each_online_cpu(i) {
-               struct cpuinfo_x86 *c = &cpu_data(i);
-
-               if (c->x86 != 6)
-                       continue;
-
-               if (c->x86_model < 0x0a)
-                       continue;
-
-               if (c->x86_model > 0x0f) {
-                       pr_warn("Unknown CPU model 0x%x\n", c->x86_model);
-                       continue;
-               }
-
+       for_each_online_cpu(i)
                via_cputemp_device_add(i);
-       }
 
 #ifndef CONFIG_HOTPLUG_CPU
        if (list_empty(&pdev_list)) {