diff options
| author | Wolfram Sang <w.sang@pengutronix.de> | 2012-10-20 17:12:56 +0200 |
|---|---|---|
| committer | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2012-11-12 17:20:30 +0200 |
| commit | d81ac614683e3de9b21b82c4dcdb1eb81a4f849a (patch) | |
| tree | e3ee5ff3f133d8041aed86085e9b52863f7094a7 | |
| parent | 334221cf5a94ec3a75c7620661eb5b7b5332256c (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.c | 6 |
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"); |