diff options
| author | Andrey Smirnov <andrew.smirnov@gmail.com> | 2018-12-06 23:32:18 -0800 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2018-12-10 09:57:00 +0100 |
| commit | 32c01aa6c8a6310d09b706fd634f439cc3d115e3 (patch) | |
| tree | edb7bdab3db4d4521b03a25379a423bb574ec992 /lib | |
| parent | 1d48fdfe8be18e92ac8acc7890a6c238182ac331 (diff) | |
| download | barebox-32c01aa6c8a6310d09b706fd634f439cc3d115e3.tar.gz barebox-32c01aa6c8a6310d09b706fd634f439cc3d115e3.tar.xz | |
lib/parameter: Fix typecases to match corresponding PARAM_TYPE_*
This change should be a no-op in terms of behavior, but it makes code
less confusing when PARAM_TYPE_* matches the type used in the type
cast.
Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/parameter.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/parameter.c b/lib/parameter.c index a21b8fa4a4..00e9a9ff4e 100644 --- a/lib/parameter.c +++ b/lib/parameter.c @@ -343,10 +343,10 @@ static int param_int_set(struct device_d *dev, struct param_d *p, const char *va ret = strtobool(val, pi->value); break; case PARAM_TYPE_INT32: - *(uint32_t *)pi->value = simple_strtol(val, NULL, 0); + *(int32_t *)pi->value = simple_strtol(val, NULL, 0); break; case PARAM_TYPE_UINT32: - *(int32_t *)pi->value = simple_strtoul(val, NULL, 0); + *(uint32_t *)pi->value = simple_strtoul(val, NULL, 0); break; case PARAM_TYPE_INT64: *(int64_t *)pi->value = simple_strtoll(val, NULL, 0); |