Re: [PATCH] staging: speakup: replace simple_strtoul with kstrtou8

From: Samuel Thibault
Date: Thu Mar 02 2017 - 10:29:46 EST


Marcin Ciupak, on jeu. 02 mars 2017 15:28:23 +0100, wrote:
> - int val;
> + int ret;
>
> - val = simple_strtoul(skip_spaces(start), &start, 10);
> + ret = kstrtou8(skip_spaces(start), 10, dest);

This is not the same, you need to have start properly move, since it's
used below:

> if (*start == ',')
> start++;
> - *dest = (u_char)val;
> return start;

Samuel