Hi Arjan, > > static ssize_t show_in(struct device *dev, char *buf, int nr) > > { > > struct adm1026_data *data = adm1026_update_device(dev); > > return sprintf(buf,"%d\n", INS_FROM_REG(nr, data->in[nr])); > > } > > any chance you could make this use snprintf instead ? None of the other hardware monitoring clients driver do, nor do I think they should. What are we trying to improve? Thanks. -- Jean Delvare http://khali.linux-fr.org/