diff options
| author | Sascha Hauer <s.hauer@pengutronix.de> | 2014-07-24 12:21:03 +0200 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2014-10-08 16:33:34 +0200 |
| commit | ade058360233b8a701c66db282a21502e3a27f79 (patch) | |
| tree | 46a9b7773fbf8eed6268ee8594a6a80ed1ca461d /common/globalvar.c | |
| parent | d096cafff7a83c6b946f350ef441918a65f9933d (diff) | |
| download | barebox-ade058360233b8a701c66db282a21502e3a27f79.tar.gz barebox-ade058360233b8a701c66db282a21502e3a27f79.tar.xz | |
param: let dev_add_param return the newly created param
dev_add_param creates a new parameter so it makes more sense to
return it than to return an error code. Since the return value
is hardly ever checked this is only a small patch.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'common/globalvar.c')
| -rw-r--r-- | common/globalvar.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/common/globalvar.c b/common/globalvar.c index 1f9b0cc1ee..c72f147d5f 100644 --- a/common/globalvar.c +++ b/common/globalvar.c @@ -17,7 +17,12 @@ int globalvar_add(const char *name, const char *(*get)(struct device_d *, struct param_d *p), unsigned long flags) { - return dev_add_param(&global_device, name, set, get, flags); + struct param_d *param; + + param = dev_add_param(&global_device, name, set, get, flags); + if (IS_ERR(param)) + return PTR_ERR(param); + return 0; } /* |