* Re: [PATCH] staging: speakup: Replaced obsolete simple_strtoul
[not found] <1475699578-31474-1-git-send-email-jkhasdev@gmail.com>
@ ` Samuel Thibault
0 siblings, 0 replies; only message in thread
From: Samuel Thibault @ UTC (permalink / raw)
To: Jitendra Khasdev
Cc: w.d.hubbs, chris, kirk, gregkh, bankarsandhya512, saurabh.truth,
amitoj1606, speakup, devel, linux-kernel
Hello,
Jitendra Khasdev, on Thu 06 Oct 2016 02:02:58 +0530, wrote:
> From: Jitendra Kumar Khasdev <jkhasdev@gmail.com>
>
> This patch is for replacing obsolete simple_strtoul to kstrtoul which remove warning produce by checkpatch.
> + unsigned long val;
> +
> + if (kstrtoul(start, 10, &val))
> + return NULL;
>
> - val = simple_strtoul(skip_spaces(start), &start, 10);
> if (*start == ',')
simple_strtoul modifies start, so you can't just replace it with
kstrtoul.
Samuel
^ permalink raw reply [flat|nested] only message in thread