[source] mtd: fix up error messages

LEDE Commits lede-commits at lists.infradead.org
Sat Oct 15 02:37:12 PDT 2016


blogic pushed a commit to source.git, branch master:
https://git.lede-project.org/c66658441b78736e5537c10ed2b380b8aa568d24

commit c66658441b78736e5537c10ed2b380b8aa568d24
Author: John Crispin <john at phrozen.org>
AuthorDate: Thu Oct 6 23:59:34 2016 +0200

    mtd: fix up error messages
    
    remove the "Error fixing up TRX header" message which is misleading.
    
    Signed-off-by: John Crispin <john at phrozen.org>
---
 package/system/mtd/src/mtd.c | 2 +-
 package/system/mtd/src/trx.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/system/mtd/src/mtd.c b/package/system/mtd/src/mtd.c
index 8fbddf5..0d5ad22 100644
--- a/package/system/mtd/src/mtd.c
+++ b/package/system/mtd/src/mtd.c
@@ -586,7 +586,7 @@ resume:
 				if (!quiet)
 					fprintf(stderr, "\b\b\b   ");
 				if (quiet < 2)
-					fprintf(stderr, "\nAppending jffs2 data from %s to %s...", jffs2file, mtd);
+					fprintf(stderr, "\nAppending jffs2 data from %s to %s..\n.", jffs2file, mtd);
 				/* got an EOF marker - this is the place to add some jffs2 data */
 				skip = mtd_replace_jffs2(mtd, fd, e, jffs2file);
 				jffs2_replaced = 1;
diff --git a/package/system/mtd/src/trx.c b/package/system/mtd/src/trx.c
index 360a127..1f5c529 100644
--- a/package/system/mtd/src/trx.c
+++ b/package/system/mtd/src/trx.c
@@ -81,6 +81,7 @@ trx_fixup(int fd, const char *name)
 	ptr = mmap(NULL, len, PROT_READ|PROT_WRITE, MAP_SHARED, bfd, 0);
 	if (!ptr || (ptr == (void *) -1)) {
 		perror("mmap");
+		fprintf(stderr, "Mapping the TRX header failed\n");
 		goto err1;
 	}
 
@@ -100,7 +101,6 @@ trx_fixup(int fd, const char *name)
 err1:
 	close(bfd);
 err:
-	fprintf(stderr, "Error fixing up TRX header\n");
 	return -1;
 }
 



More information about the lede-commits mailing list