I wasn't sure how get_options() worked, so I looked at examples. And by sheer chance the first example I picked the only example which uses it incorrectly... I've added some comments that hopefully help. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> diff --git a/lib/cmdline.c b/lib/cmdline.c index 4c0888c4a68d..0eb8d0ab60db 100644 --- a/lib/cmdline.c +++ b/lib/cmdline.c @@ -73,14 +73,15 @@ EXPORT_SYMBOL(get_option); /** * get_options - Parse a string into a list of integers * @str: String to be parsed - * @nints: size of integer array + * @nints: size of integer array (including the extra int at the start) * @ints: integer array * * This function parses a string containing a comma-separated * list of integers, a hyphen-separated range of _positive_ integers, * or a combination of both. The parse halts when the array is * full, or when no more numbers can be retrieved from the - * string. + * string. It stores the number of numbers as the first element in the + * array. * * Return value is the character in the string which caused * the parse to end (typically a null terminator, if @str is -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
