diff options
author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2020-09-28 17:34:31 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2020-09-29 08:41:53 +0200 |
commit | 9699096f1eccd742d898f471b046181ea90dd582 (patch) | |
tree | 1b62f4c159a6ae9b90eac7d750a4833b0e30a7dc /lib | |
parent | 12d74001d52911dfb96d81b774183aeaba21370c (diff) | |
download | barebox-9699096f1eccd742d898f471b046181ea90dd582.tar.gz barebox-9699096f1eccd742d898f471b046181ea90dd582.tar.xz |
treewide: replace strerror(-PTR_ERR(errno)) with %pe format specifier
Using %pe instead of PTR_ERR has the benefit of being less verbose and
less error-prone (no negation necessary) while potentially reducing
code size. Make use of it.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/cmdlinepart.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/lib/cmdlinepart.c b/lib/cmdlinepart.c index 5a16462874..5e95760bae 100644 --- a/lib/cmdlinepart.c +++ b/lib/cmdlinepart.c @@ -30,7 +30,6 @@ int cmdlinepart_do_parse_one(const char *devname, const char *partstr, char *end; char buf[PATH_MAX] = {}; unsigned long flags = 0; - int ret = 0; struct cdev *cdev; memset(buf, 0, PATH_MAX); @@ -85,11 +84,11 @@ int cmdlinepart_do_parse_one(const char *devname, const char *partstr, cdev = devfs_add_partition(devname, *offset, size, flags, buf); if (IS_ERR(cdev)) { - ret = PTR_ERR(cdev); - printf("cannot create %s: %s\n", buf, strerror(-ret)); + printf("cannot create %s: %pe\n", buf, cdev); + return PTR_ERR(cdev); } - return ret; + return 0; } int cmdlinepart_do_parse(const char *devname, const char *parts, loff_t devsize, |