summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWolfram Sang <w.sang@pengutronix.de>2012-10-20 17:12:56 +0200
committerArtem Bityutskiy <artem.bityutskiy@linux.intel.com>2012-11-12 17:20:30 +0200
commitd81ac614683e3de9b21b82c4dcdb1eb81a4f849a (patch)
treee3ee5ff3f133d8041aed86085e9b52863f7094a7
parent334221cf5a94ec3a75c7620661eb5b7b5332256c (diff)
ubiformat: really skip some messages when quiet
Both logic (only print when not quiet) and the indentation suggest that the braces around the block have been forgotten. Signed-off-by: Wolfram Sang <w.sang@pengutronix.de> Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
-rw-r--r--ubi-utils/ubiformat.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/ubi-utils/ubiformat.c b/ubi-utils/ubiformat.c
index a265a9a..8f9bd3a 100644
--- a/ubi-utils/ubiformat.c
+++ b/ubi-utils/ubiformat.c
@@ -875,11 +875,12 @@ int main(int argc, char * const argv[])
* erase counters.
*/
if (percent < 50) {
- if (!args.yes || !args.quiet)
+ if (!args.yes || !args.quiet) {
warnmsg("only %d of %d eraseblocks have valid erase counter",
si->ok_cnt, si->good_cnt);
normsg("erase counter 0 will be used for all eraseblocks");
normsg("note, arbitrary erase counter value may be specified using -e option");
+ }
if (!args.yes && want_exit()) {
if (args.yes && !args.quiet)
printf("yes\n");
@@ -888,11 +889,12 @@ int main(int argc, char * const argv[])
args.ec = 0;
args.override_ec = 1;
} else if (percent < 95) {
- if (!args.yes || !args.quiet)
+ if (!args.yes || !args.quiet) {
warnmsg("only %d of %d eraseblocks have valid erase counter",
si->ok_cnt, si->good_cnt);
normsg("mean erase counter %lld will be used for the rest of eraseblock",
si->mean_ec);
+ }
if (!args.yes && want_exit()) {
if (args.yes && !args.quiet)
printf("yes\n");