diff options
| author | Thomas Gleixner | 2016-11-17 19:35:26 +0100 |
|---|---|---|
| committer | Guenter Roeck | 2016-12-02 13:28:13 -0800 |
| commit | 3d7e0a24947c076a6cf6080c5f076c60ae8d9543 (patch) | |
| tree | a30450f607f34f29e220e9f2d1c4d347ed166bc6 /drivers/hwmon | |
| parent | f4dc811c16b91ed0a8c9dd5a863409279d14a581 (diff) | |
hwmon: (via-cputemp) Remove pointless CPU check on each CPU
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>
Diffstat (limited to 'drivers/hwmon')
| -rw-r--r-- | drivers/hwmon/via-cputemp.c | 16 |
1 files changed, 1 insertions, 15 deletions
diff --git a/drivers/hwmon/via-cputemp.c b/drivers/hwmon/via-cputemp.c index ac91c07e3f90..5b9866b1b437 100644 --- a/drivers/hwmon/via-cputemp.c +++ b/drivers/hwmon/via-cputemp.c @@ -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)) { |