In the patch: http://jdelvare.net1.nerim.net/sensors/lm_sensors-CVS-separate-mallocs.diff to break the kmalloc's out, why did you remove the - default : - printk("lm85: Internal error, invalid kind (%d)!", kind); - err = -EFAULT ; - goto ERROR1; clause from the kind detection switch statement in the ADM1026 and LM85 drivers? Shouldn't we trap any invalid "kind" values? :v)