linux-toradex.git/lib/cmdline.c, branch v5.0-rc3

lib/cmdline.c: remove meaningless comment

2017-09-09T01:26:49+00:00

One line of code was commented out by c++ style comment for debugging, but
forgot removing it.

Clean it up.

Link: http://lkml.kernel.org/r/1503312113-11843-1-git-send-email-bhe@redhat.com
Signed-off-by: Baoquan He 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

lib/cmdline.c: fix get_options() overflow while parsing ranges

2017-06-23T23:15:55+00:00

When using get_options() it's possible to specify a range of numbers,
like 1-100500.  The problem is that it doesn't track array size while
calling internally to get_range() which iterates over the range and
fills the memory with numbers.

Link: http://lkml.kernel.org/r/2613C75C-B04D-4BFF-82A6-12F97BA0F620@gmail.com
Signed-off-by: Ilya V. Matveychikov 
Cc: Jonathan Corbet 
Cc: 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

boot/param: Move next_arg() function to lib/cmdline.c for later reuse

2017-04-18T08:37:13+00:00

next_arg() will be used to parse boot parameters in the x86/boot/compressed code,
so move it to lib/cmdline.c for better code reuse.

No change in functionality.

Signed-off-by: Baoquan He 
Cc: Andrew Morton 
Cc: Gustavo Padovan 
Cc: Jens Axboe 
Cc: Jessica Yu 
Cc: Johannes Berg 
Cc: Josh Triplett 
Cc: Larry Finger 
Cc: Linus Torvalds 
Cc: Niklas Söderlund 
Cc: Peter Zijlstra 
Cc: Petr Mladek 
Cc: Rasmus Villemoes 
Cc: Thomas Gleixner 
Cc: dan.j.williams@intel.com
Cc: dave.jiang@intel.com
Cc: dyoung@redhat.com
Cc: keescook@chromium.org
Cc: zijun_hu 
Link: http://lkml.kernel.org/r/1492436099-4017-2-git-send-email-bhe@redhat.com
Signed-off-by: Ingo Molnar

lib: Add a generic cmdline parse function parse_option_str

2014-10-03T17:40:58+00:00

There should be a generic function to parse params like a=b,c
Adding parse_option_str in lib/cmdline.c which will return true
if there's specified option set in the params.

Also updated efi=old_map parsing code to use the new function

Signed-off-by: Dave Young 
Signed-off-by: Matt Fleming

lib/cmdline.c: add size unit t/p/e to memparse

2014-08-07T01:01:25+00:00

For modern filesystems such as btrfs, t/p/e size level operations are
common.  add size unit t/p/e parsing to memparse

Signed-off-by: Gui Hecheng 
Acked-by: David Rientjes 
Reviewed-by: Satoru Takeuchi 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

lib/cmdline.c: declare exported symbols immediately

2014-01-24T00:36:57+00:00

WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
+EXPORT_SYMBOL(memparse);

WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
+EXPORT_SYMBOL(get_option);

WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
+EXPORT_SYMBOL(get_options);

Signed-off-by: Felipe Contreras 
Cc: Levente Kurusa 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

lib/cmdline.c: fix style issues

2014-01-24T00:36:57+00:00

WARNING: space prohibited between function name and open parenthesis '('
+int get_option (char **str, int *pint)

WARNING: space prohibited between function name and open parenthesis '('
+	*pint = simple_strtol (cur, str, 0);

ERROR: trailing whitespace
+ $

WARNING: please, no spaces at the start of a line
+ $

WARNING: space prohibited between function name and open parenthesis '('
+		res = get_option ((char **)&str, ints + i);

Signed-off-by: Felipe Contreras 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

lib: reduce the use of module.h wherever possible

2012-03-07T20:04:04+00:00

For files only using THIS_MODULE and/or EXPORT_SYMBOL, map
them onto including export.h -- or if the file isn't even
using those, then just delete the include.  Fix up any implicit
include dependencies that were being masked by module.h along
the way.

Signed-off-by: Paul Gortmaker

generic, memparse(): constify argument

2008-07-28T13:05:23+00:00

memparse()'s first argument can be const, so it should be.

Signed-off-by: Jeremy Fitzhardinge 
Cc: Andrew Morton 
Signed-off-by: Ingo Molnar

lib: allow memparse() to accept a NULL and ignorable second parm

2008-07-25T17:53:27+00:00

Extend memparse() to allow the caller to use a NULL second parameter, which
would represent no interest in returning the address of the end of the parsed
string.

In numerous cases, callers invoke memparse() to parse a possibly-suffixed
string (such as "64K" or "2G" or whatever) and define a character pointer to
accept the end pointer being returned by memparse() even though they have no
interest in it and promptly throw it away.

This (backward-compatible) enhancement allows callers to use NULL in the cases
where they just don't care about getting back that end pointer.

[akpm@linux-foundation.org: coding-style fixes]
Signed-off-by: Robert P. J. Day 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds