UBI: rename a local variable
authorArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Fri, 3 Sep 2010 11:53:23 +0000 (14:53 +0300)
committerArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Tue, 19 Oct 2010 14:19:56 +0000 (17:19 +0300)
Rename local variable 'ec_corr' into 'ec_err' to make the code a little bit
more readable. 'ec_err' is more appropriate because it sounds more like 'error
when EC was read' and it looks more logical because we use it together with
'err'. Just a minor nicification which should improve the rather complex
scanning code.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
drivers/mtd/ubi/scan.c

index 6f90807..a15e9bc 100644 (file)
@@ -725,7 +725,7 @@ static int process_eb(struct ubi_device *ubi, struct ubi_scan_info *si,
                      int pnum)
 {
        long long uninitialized_var(ec);
-       int err, bitflips = 0, vol_id, ec_corr = 0;
+       int err, bitflips = 0, vol_id, ec_err = 0;
 
        dbg_bld("scan PEB %d", pnum);
 
@@ -756,12 +756,12 @@ static int process_eb(struct ubi_device *ubi, struct ubi_scan_info *si,
                 * corrupted. Set %bitflips flag in order to make this PEB be
                 * moved and EC be re-created.
                 */
-               ec_corr = err;
+               ec_err = err;
                ec = UBI_SCAN_UNKNOWN_EC;
                bitflips = 1;
        }
 
-       if (!ec_corr) {
+       if (!ec_err) {
                int image_seq;
 
                /* Make sure UBI version is OK */
@@ -817,10 +817,10 @@ static int process_eb(struct ubi_device *ubi, struct ubi_scan_info *si,
        else if (err == UBI_IO_BITFLIPS)
                bitflips = 1;
        else if (err == UBI_IO_BAD_HDR_EBADMSG || err == UBI_IO_BAD_HDR ||
-                (err == UBI_IO_FF && ec_corr) || err == UBI_IO_FF_BITFLIPS) {
+                (err == UBI_IO_FF && ec_err) || err == UBI_IO_FF_BITFLIPS) {
                /* VID header is corrupted */
                if (err == UBI_IO_BAD_HDR_EBADMSG ||
-                   ec_corr == UBI_IO_BAD_HDR_EBADMSG)
+                   ec_err == UBI_IO_BAD_HDR_EBADMSG)
                        si->read_err_count += 1;
                err = add_to_list(si, pnum, ec, &si->corr);
                if (err)
@@ -870,7 +870,7 @@ static int process_eb(struct ubi_device *ubi, struct ubi_scan_info *si,
                }
        }
 
-       if (ec_corr)
+       if (ec_err)
                ubi_warn("valid VID header but corrupted EC header at PEB %d",
                         pnum);
        err = ubi_scan_add_used(ubi, si, pnum, ec, vidh, bitflips);
@@ -878,7 +878,7 @@ static int process_eb(struct ubi_device *ubi, struct ubi_scan_info *si,
                return err;
 
 adjust_mean_ec:
-       if (!ec_corr) {
+       if (!ec_err) {
                si->ec_sum += ec;
                si->ec_count += 1;
                if (ec > si->max_ec)