diff options
author | Andrey Smirnov <andrew.smirnov@gmail.com> | 2018-06-07 06:01:04 -0700 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2018-06-11 08:54:10 +0200 |
commit | 247117e7177668ef13515c871a79397d67ddcb2b (patch) | |
tree | f9e2bad57269cb6849848b1ffc83d093751e632f /scripts/imx | |
parent | 319367dd147fff1711c5d996fe6cddbf0d613af0 (diff) | |
download | barebox-247117e7177668ef13515c871a79397d67ddcb2b.tar.gz barebox-247117e7177668ef13515c871a79397d67ddcb2b.tar.xz |
scripts: imx-image: Drop error return from write_dcd()
Write_dcd() exits early in case of failure, so there's no realy reason
to have it return a error code as a result. Drop it and simplify the
caller code.
Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'scripts/imx')
-rw-r--r-- | scripts/imx/imx-image.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/scripts/imx/imx-image.c b/scripts/imx/imx-image.c index b241e8c4b6..a8a26e0eec 100644 --- a/scripts/imx/imx-image.c +++ b/scripts/imx/imx-image.c @@ -427,7 +427,7 @@ static int xwrite(int fd, void *buf, int len) return 0; } -static int write_dcd(const char *outfile) +static void write_dcd(const char *outfile) { int outfd, ret; int dcdsize = curdcd * sizeof(uint32_t); @@ -443,8 +443,6 @@ static int write_dcd(const char *outfile) perror("write"); exit(1); } - - return 0; } static int check(const struct config_data *data, uint32_t cmd, uint32_t addr, @@ -778,10 +776,8 @@ int main(int argc, char *argv[]) check_last_dcd(0); if (dcd_only) { - ret = write_dcd(data.outfile); - if (ret) - exit(1); - exit (0); + write_dcd(data.outfile); + exit(0); } switch (data.header_version) { |