On Sat, 8 Jun 2013 11:38:00 +0400 Maxim Uvarov <muvarov at gmail.com> wrote: [snip] > > +static int copy_to(void *target, void *src, size_t size, int > > userbuf) +{ > > + if (userbuf) { > > + if (copy_to_user(target, src, size)) > > + return -EFAULT; > > + return 0; > > + } else { > > + memcpy(target, src, size); > > + return 0; > > + } > > +} > > > > It is better to do return 0 in the end of function. Sure, I will change that. Thanks! Michael