[PATCH] staging: speakup: replace simple_strtoul with kstrtou8

Samuel Thibault samuel.thibault at ens-lyon.org
Thu Mar 2 10:17:12 EST 2017


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


More information about the Speakup mailing list