* Guenter Roeck <guenter.roeck@xxxxxxxxxxxx> wrote: > Commit e40cc4bdfd4b89813f072f72bd9c7055814d3f0f introduced > a build breakage if CONFIG_SMP is undefined. This patch > fixes the problem. > > Reported-by: Ingo Molnar <mingo@xxxxxxx> > Cc: Fenghua Yu <fenghua.yu@xxxxxxxxx> > Cc: Jean Delvare <khali@xxxxxxxxxxxx> > Signed-off-by: Guenter Roeck <guenter.roeck@xxxxxxxxxxxx> > > --- > drivers/hwmon/coretemp.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/hwmon/coretemp.c b/drivers/hwmon/coretemp.c > index baa842a..58a3c2d 100644 > --- a/drivers/hwmon/coretemp.c > +++ b/drivers/hwmon/coretemp.c > @@ -503,9 +503,11 @@ static void coretemp_device_remove(unsigned int cpu) > list_del(&p->list); > mutex_unlock(&pdev_list_mutex); > kfree(p); > +#ifdef CONFIG_SMP > for_each_cpu(i, cpu_sibling_mask(cpu)) > if (i != cpu && !coretemp_device_add(i)) > break; > +#endif Wont this result in an 'unused variable i' compiler warning? Thanks, Ingo _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
