On Fri, 2011-08-05 at 09:00 -0400, Dmitry Artamonow wrote: > Export caseopen alarm status into userspace for > Winbond w83627* chips and also implement alarm clear knob. > > Signed-off-by: Dmitry Artamonow <mad_soft@xxxxxxxx> [ ... ] > +static ssize_t > +store_caseopen_clear(struct device *dev, struct device_attribute *attr, > + const char *buf, size_t count) Another note: Please drop the "_clear" from the function name. It is redundant with "store_". Another possible name would be "clear_caseopen". Thanks, Guenter _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors