diff options
author | Dan Streetman <ddstreet@ieee.org> | 2015-06-16 22:47:52 +0200 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2015-06-23 07:57:37 +0200 |
commit | 5104b7d7678b0029417f6ac08243773a77259ac6 (patch) | |
tree | bf1a7a2703674d474db5618302cf8b5acf3f8adc /kernel/params.c | |
parent | params: suppress unused variable error, warn once just in case code changes. (diff) | |
download | linux-5104b7d7678b0029417f6ac08243773a77259ac6.tar.xz linux-5104b7d7678b0029417f6ac08243773a77259ac6.zip |
module: make perm const
Change the struct kernel_param.perm field to a const, as it should never
be changed.
Signed-off-by: Dan Streetman <ddstreet@ieee.org>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au> (cut from larger patch)
Diffstat (limited to 'kernel/params.c')
-rw-r--r-- | kernel/params.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/kernel/params.c b/kernel/params.c index e906874da5fc..a8b09f6c87dc 100644 --- a/kernel/params.c +++ b/kernel/params.c @@ -395,12 +395,11 @@ EXPORT_SYMBOL(param_ops_invbool); int param_set_bint(const char *val, const struct kernel_param *kp) { - struct kernel_param boolkp; + /* Match bool exactly, by re-using it. */ + struct kernel_param boolkp = *kp; bool v; int ret; - /* Match bool exactly, by re-using it. */ - boolkp = *kp; boolkp.arg = &v; ret = param_set_bool(val, &boolkp); @@ -480,9 +479,8 @@ static int param_array_get(char *buffer, const struct kernel_param *kp) { int i, off, ret; const struct kparam_array *arr = kp->arr; - struct kernel_param p; + struct kernel_param p = *kp; - p = *kp; for (i = off = 0; i < (arr->num ? *arr->num : arr->max); i++) { if (i) buffer[off++] = ','; |