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 /include | |
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 'include')
-rw-r--r-- | include/param.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/param.h b/include/param.h index 9856a2e552..b8dd9b1869 100644 --- a/include/param.h +++ b/include/param.h @@ -26,7 +26,7 @@ const char *dev_get_param(struct device_d *dev, const char *name); int dev_set_param(struct device_d *dev, const char *name, const char *val); struct param_d *get_param_by_name(struct device_d *dev, const char *name); -int dev_add_param(struct device_d *dev, const char *name, +struct param_d *dev_add_param(struct device_d *dev, const char *name, int (*set)(struct device_d *dev, struct param_d *p, const char *val), const char *(*get)(struct device_d *, struct param_d *p), unsigned long flags); @@ -92,7 +92,7 @@ static inline struct param_d *get_param_by_name(struct device_d *dev, return NULL; } -static inline int dev_add_param(struct device_d *dev, char *name, +static inline struct param_d *dev_add_param(struct device_d *dev, char *name, int (*set)(struct device_d *dev, struct param_d *p, const char *val), const char *(*get)(struct device_d *, struct param_d *p), unsigned long flags) |