mtd: do not use mtd->sync directly
Artem Bityutskiy [Fri, 30 Dec 2011 14:35:35 +0000 (16:35 +0200)]
This patch teaches 'mtd_sync()' to do nothing when the MTD driver does
not have the '->sync()' method, which allows us to remove all direct
'mtd->sync' accesses.

Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>

drivers/mtd/ftl.c
drivers/mtd/mtdblock.c
drivers/mtd/mtdchar.c
drivers/mtd/mtdswap.c
drivers/mtd/rfd_ftl.c
drivers/mtd/ubi/kapi.c
fs/jffs2/super.c
fs/logfs/dev_mtd.c
include/linux/mtd/mtd.h

index c9c9029..19d6372 100644 (file)
@@ -650,8 +650,7 @@ static int reclaim_block(partition_t *part)
            if (queued) {
                pr_debug("ftl_cs: waiting for transfer "
                      "unit to be prepared...\n");
-               if (part->mbd.mtd->sync)
-                       mtd_sync(part->mbd.mtd);
+               mtd_sync(part->mbd.mtd);
            } else {
                static int ne = 0;
                if (++ne < 5)
index 496e1a6..af65912 100644 (file)
@@ -322,8 +322,7 @@ static int mtdblock_release(struct mtd_blktrans_dev *mbd)
 
        if (!--mtdblk->count) {
                /* It was the last usage. Free the cache */
-               if (mbd->mtd->sync)
-                       mtd_sync(mbd->mtd);
+               mtd_sync(mbd->mtd);
                vfree(mtdblk->cache_data);
        }
 
@@ -341,9 +340,7 @@ static int mtdblock_flush(struct mtd_blktrans_dev *dev)
        mutex_lock(&mtdblk->cache_mutex);
        write_cached_data(mtdblk);
        mutex_unlock(&mtdblk->cache_mutex);
-
-       if (dev->mtd->sync)
-               mtd_sync(dev->mtd);
+       mtd_sync(dev->mtd);
        return 0;
 }
 
index 2020a16..23a5110 100644 (file)
@@ -154,7 +154,7 @@ static int mtdchar_close(struct inode *inode, struct file *file)
        pr_debug("MTD_close\n");
 
        /* Only sync if opened RW */
-       if ((file->f_mode & FMODE_WRITE) && mtd->sync)
+       if ((file->f_mode & FMODE_WRITE))
                mtd_sync(mtd);
 
        iput(mfi->ino);
index 4441c08..fe4426c 100644 (file)
@@ -1047,8 +1047,7 @@ static int mtdswap_flush(struct mtd_blktrans_dev *dev)
 {
        struct mtdswap_dev *d = MTDSWAP_MBD_TO_MTDSWAP(dev);
 
-       if (d->mtd->sync)
-               mtd_sync(d->mtd);
+       mtd_sync(d->mtd);
        return 0;
 }
 
index 5426d42..233b946 100644 (file)
@@ -448,8 +448,7 @@ static int reclaim_block(struct partition *part, u_long *old_sector)
        int rc;
 
        /* we have a race if sync doesn't exist */
-       if (part->mbd.mtd->sync)
-               mtd_sync(part->mbd.mtd);
+       mtd_sync(part->mbd.mtd);
 
        score = 0x7fffffff; /* MAX_INT */
        best_block = -1;
index 9f265cc..9fdb353 100644 (file)
@@ -714,9 +714,7 @@ int ubi_sync(int ubi_num)
        if (!ubi)
                return -ENODEV;
 
-       if (ubi->mtd->sync)
-               mtd_sync(ubi->mtd);
-
+       mtd_sync(ubi->mtd);
        ubi_put_device(ubi);
        return 0;
 }
index e78bf3c..5863a36 100644 (file)
@@ -336,9 +336,7 @@ static void jffs2_put_super (struct super_block *sb)
        jffs2_flash_cleanup(c);
        kfree(c->inocache_list);
        jffs2_clear_xattr_subsystem(c);
-       if (c->mtd->sync)
-               mtd_sync(c->mtd);
-
+       mtd_sync(c->mtd);
        D1(printk(KERN_DEBUG "jffs2_put_super returning\n"));
 }
 
index 136c736..3f46588 100644 (file)
@@ -119,8 +119,7 @@ static void logfs_mtd_sync(struct super_block *sb)
 {
        struct mtd_info *mtd = logfs_super(sb)->s_mtd;
 
-       if (mtd->sync)
-               mtd_sync(mtd);
+       mtd_sync(mtd);
 }
 
 static int logfs_mtd_readpage(void *_sb, struct page *page)
index a58ecf4..305f12b 100644 (file)
@@ -399,7 +399,8 @@ int mtd_writev(struct mtd_info *mtd, const struct kvec *vecs,
 
 static inline void mtd_sync(struct mtd_info *mtd)
 {
-       mtd->sync(mtd);
+       if (mtd->sync)
+               mtd->sync(mtd);
 }
 
 /* Chip-supported device locking */