Hi Jingoo, On Fri, Jul 19 2013, Jingoo Han wrote: > The usage of strict_strtol() is not preferred, because > strict_strtol() is obsolete. Thus, kstrtol() should be > used. > > Also, both kstrtol() and copy_from_user() can be replaced > with kstrtol_from_user() to make the code simpler. > > Signed-off-by: Jingoo Han <jg1.han@xxxxxxxxxxx> > --- > drivers/mmc/card/mmc_test.c | 14 ++++---------- > 1 file changed, 4 insertions(+), 10 deletions(-) > > diff --git a/drivers/mmc/card/mmc_test.c b/drivers/mmc/card/mmc_test.c > index a69df52..0c0fc52 100644 > --- a/drivers/mmc/card/mmc_test.c > +++ b/drivers/mmc/card/mmc_test.c > @@ -2849,18 +2849,12 @@ static ssize_t mtf_test_write(struct file *file, const char __user *buf, > struct seq_file *sf = (struct seq_file *)file->private_data; > struct mmc_card *card = (struct mmc_card *)sf->private; > struct mmc_test_card *test; > - char lbuf[12]; > long testcase; > + int ret; > > - if (count >= sizeof(lbuf)) > - return -EINVAL; > - > - if (copy_from_user(lbuf, buf, count)) > - return -EFAULT; > - lbuf[count] = '\0'; > - > - if (strict_strtol(lbuf, 10, &testcase)) > - return -EINVAL; > + ret = kstrtol_from_user(buf, count, 10, &testcase); > + if (ret) > + return ret; > > test = kzalloc(sizeof(struct mmc_test_card), GFP_KERNEL); > if (!test) Thanks, pushed to mmc-next for 3.12. - Chris. -- Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/> -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
