"break" after "return" is at best bogus (good compilers even warn about the "break" being unreachable). Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx> Cc: Rudolf Marek <r.marek@xxxxxxxxxxxx> --- drivers/hwmon/coretemp.c | 2 -- 1 file changed, 2 deletions(-) --- linux-2.6.36-rc4/drivers/hwmon/coretemp.c 2010-09-13 08:45:02.000000000 +0200 +++ 2.6.36-rc4-x86-coretemp-cosmetic/drivers/hwmon/coretemp.c 2010-09-03 17:54:14.000000000 +0200 @@ -279,11 +279,9 @@ static int __devinit get_tjmax(struct cp case 0x1a: dev_warn(dev, "TjMax is assumed as 100 C!\n"); return 100000; - break; case 0x17: case 0x1c: /* Atom CPUs */ return adjust_tjmax(c, id, dev); - break; default: dev_warn(dev, "CPU (model=0x%x) is not supported yet," " using default TjMax of 100C.\n", c->x86_model); _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors