Use pwrite_full() instead of pwrite() to make sure that all of the
data was written to target device. While at it adjust return value of
imx_bbu_write_device() to be zero in case of success.
Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>
---
arch/arm/mach-imx/imx-bbu-internal.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-imx/imx-bbu-internal.c b/arch/arm/mach-imx/imx-bbu-internal.c
index 59fe6218e..504e359bc 100644
--- a/arch/arm/mach-imx/imx-bbu-internal.c
+++ b/arch/arm/mach-imx/imx-bbu-internal.c
@@ -30,6 +30,7 @@
#include <environment.h>
#include <mach/bbu.h>
#include <mach/generic.h>
+#include <libfile.h>
#define IMX_INTERNAL_FLAG_ERASE BIT(30)
@@ -116,9 +117,12 @@ static int imx_bbu_write_device(struct imx_internal_bbu_handler *imx_handler,
}
}
- ret = pwrite(fd, buf, image_len, offset);
- if (ret < 0)
+ ret = pwrite_full(fd, buf, image_len, offset);
+ if (ret < 0) {
+ pr_err("writing to %s failed with %s\n", devicefile,
+ strerror(-ret));
goto err_close;
+ }
imx_bbu_protect(fd, imx_handler, devicefile, offset,
image_len, 1);
@@ -126,7 +130,7 @@ static int imx_bbu_write_device(struct imx_internal_bbu_handler *imx_handler,
err_close:
close(fd);
- return ret;
+ return ret < 0 ? ret : 0;
}
static int __imx_bbu_write_device(struct imx_internal_bbu_handler *imx_handler,
--
2.17.1
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
