[PATCH v2 3/3] mtd:nor:ppb_unlock: remove repeated chip unlock

Honza Petrouš jpetrous at gmail.com
Tue May 30 02:51:27 PDT 2017


The Persistent Protection Bits (PPB) locking of cfi_cmdset_0002.c
doesn't support per-sector-unlocking, so any unlock request
unlocks the whole chip. Because of that limitation the driver
does the unlock in three steps:
 1) remember all locked sector
 2) do the whole chip unlock
 3) lock back only the necessary sectors

Unfortunately in step 2 (unlocking the chip) there is used
cfi_varsize_frob() for per-sector unlock, what ends up
in multiple chip unlocking calls (exactly the chip unlock
is called for every sector in the unlock area) even the only one
unlock per chip is enough.

Also optimized processing only inside necessary chip array in case
of multi-chip setting.

Signed-off-by: Honza Petrous <jpetrous at gmail.com>
---
 drivers/mtd/chips/cfi_cmdset_0002.c | 30 +++++++++++++++++++-----------
 1 file changed, 19 insertions(+), 11 deletions(-)

diff --git a/drivers/mtd/chips/cfi_cmdset_0002.c
b/drivers/mtd/chips/cfi_cmdset_0002.c
index 29a0b7f..22a27c8 100644
--- a/drivers/mtd/chips/cfi_cmdset_0002.c
+++ b/drivers/mtd/chips/cfi_cmdset_0002.c
@@ -2531,7 +2531,7 @@ static int cfi_atmel_unlock(struct mtd_info
*mtd, loff_t ofs, uint64_t len)
  */

 struct ppb_lock {
-    struct flchip *chip;
+    int chipid;
     loff_t offset;
     int locked;
 };
@@ -2635,6 +2635,8 @@ static int __maybe_unused cfi_ppb_unlock(struct
mtd_info *mtd, loff_t ofs,
     int i;
     int sectors;
     int ret;
+    int fromchip;
+    int tochip;

     /*
      * PPB unlocking always unlocks all sectors of the flash chip.
@@ -2656,6 +2658,8 @@ static int __maybe_unused cfi_ppb_unlock(struct
mtd_info *mtd, loff_t ofs,
     sectors = 0;
     offset = 0;
     length = mtd->size;
+    fromchip = ofs >> cfi->chipshift;
+    tochip = (ofs + len - 1) >> cfi->chipshift;

     while (length) {
         int size = regions[i].erasesize;
@@ -2676,8 +2680,8 @@ static int __maybe_unused cfi_ppb_unlock(struct
mtd_info *mtd, loff_t ofs,
             if (sectors >= MAX_SECTORS) {
                 printk(KERN_ERR "Only %d sectors for PPB locking supported!\n",
                        MAX_SECTORS);
-                kfree(sect);
-                return -EINVAL;
+                ret = -EINVAL;
+                goto out;
             }
         }

@@ -2697,12 +2701,13 @@ static int __maybe_unused
cfi_ppb_unlock(struct mtd_info *mtd, loff_t ofs,
         }
     }

-    /* Now unlock the whole chip */
-    ret = cfi_varsize_frob(mtd, do_ppb_xxlock, ofs, len,
-                   DO_XXLOCK_ONEBLOCK_UNLOCK);
-    if (ret) {
-        kfree(sect);
-        return ret;
+    /* Now unlock the all necessary chips (full unlock) */
+    for (chipnum = fromchip; chipnum <= tochip; chipnum++) {
+        ret = do_ppb_xxlock(map, &cfi->chips[chipnum],
+                    0, 1 << cfi->chipshift,
+                    DO_XXLOCK_ONEBLOCK_UNLOCK);
+        if (ret)
+            goto out;
     }

     /*
@@ -2710,11 +2715,14 @@ static int __maybe_unused
cfi_ppb_unlock(struct mtd_info *mtd, loff_t ofs,
      * We need to re-lock all previously locked sectors.
      */
     for (i = 0; i < sectors; i++) {
-        if (sect[i].locked)
-            do_ppb_xxlock(map, sect[i].chip, sect[i].offset, 0,
+        if (sect[i].locked &&
+            (sect[i].chipid >= fromchip && sect[i].chipid <= tochip))
+            do_ppb_xxlock(map, &cfi->chips[sect[i].chipid],
+                      sect[i].offset, 0,
                       DO_XXLOCK_ONEBLOCK_LOCK);
     }

+out:
     kfree(sect);
     return ret;
 }
-- 
2.9.3



More information about the linux-mtd mailing list