]> git.ipfire.org Git - people/ms/u-boot.git/blobdiff - drivers/mtd/ubi/build.c
ubi,ubifs: sync with linux v4.2
[people/ms/u-boot.git] / drivers / mtd / ubi / build.c
index 290d524b1be2577d8a9da710a8c3518cc2d1d632..f0a3b6794222206e7cff03f6b2f2bdc024364c9b 100644 (file)
@@ -80,6 +80,7 @@ static struct mtd_dev_param __initdata mtd_dev_param[UBI_MAX_DEVICES];
 #ifdef CONFIG_MTD_UBI_FASTMAP
 /* UBI module parameter to enable fastmap automatically on non-fastmap images */
 static bool fm_autoconvert;
+static bool fm_debug;
 #endif
 #else
 #ifdef CONFIG_MTD_UBI_FASTMAP
@@ -87,10 +88,12 @@ static bool fm_autoconvert;
 #define CONFIG_MTD_UBI_FASTMAP_AUTOCONVERT 0
 #endif
 static bool fm_autoconvert = CONFIG_MTD_UBI_FASTMAP_AUTOCONVERT;
+#if !defined(CONFIG_MTD_UBI_FM_DEBUG)
+#define CONFIG_MTD_UBI_FM_DEBUG 0
+#endif
+static bool fm_debug = CONFIG_MTD_UBI_FM_DEBUG;
 #endif
 #endif
-/* Root UBI "class" object (corresponds to '/<sysfs>/class/ubi/') */
-struct class *ubi_class;
 
 /* Slab cache for wear-leveling entries */
 struct kmem_cache *ubi_wl_entry_slab;
@@ -110,7 +113,7 @@ static struct ubi_device *ubi_devices[UBI_MAX_DEVICES];
 #else
 struct ubi_device *ubi_devices[UBI_MAX_DEVICES];
 #endif
-
 #ifndef __UBOOT__
 /* Serializes UBI devices creations and removals */
 DEFINE_MUTEX(ubi_devices_mutex);
@@ -126,8 +129,17 @@ static ssize_t ubi_version_show(struct class *class,
 }
 
 /* UBI version attribute ('/<sysfs>/class/ubi/version') */
-static struct class_attribute ubi_version =
-       __ATTR(version, S_IRUGO, ubi_version_show, NULL);
+static struct class_attribute ubi_class_attrs[] = {
+       __ATTR(version, S_IRUGO, ubi_version_show, NULL),
+       __ATTR_NULL
+};
+
+/* Root UBI "class" object (corresponds to '/<sysfs>/class/ubi/') */
+struct class ubi_class = {
+       .name           = UBI_NAME_STR,
+       .owner          = THIS_MODULE,
+       .class_attrs    = ubi_class_attrs,
+};
 
 static ssize_t dev_attribute_show(struct device *dev,
                                  struct device_attribute *attr, char *buf);
@@ -169,23 +181,22 @@ static struct device_attribute dev_mtd_num =
  */
 int ubi_volume_notify(struct ubi_device *ubi, struct ubi_volume *vol, int ntype)
 {
+       int ret;
        struct ubi_notification nt;
 
        ubi_do_get_device_info(ubi, &nt.di);
        ubi_do_get_volume_info(ubi, vol, &nt.vi);
 
-#ifdef CONFIG_MTD_UBI_FASTMAP
        switch (ntype) {
        case UBI_VOLUME_ADDED:
        case UBI_VOLUME_REMOVED:
        case UBI_VOLUME_RESIZED:
        case UBI_VOLUME_RENAMED:
-               if (ubi_update_fastmap(ubi)) {
-                       ubi_err("Unable to update fastmap!");
-                       ubi_ro_mode(ubi);
-               }
+               ret = ubi_update_fastmap(ubi);
+               if (ret)
+                       ubi_msg(ubi, "Unable to write a new fastmap: %i", ret);
        }
-#endif
+
        return blocking_notifier_call_chain(&ubi_notifiers, ntype, &nt);
 }
 
@@ -406,6 +417,22 @@ static ssize_t dev_attribute_show(struct device *dev,
        return ret;
 }
 
+static struct attribute *ubi_dev_attrs[] = {
+       &dev_eraseblock_size.attr,
+       &dev_avail_eraseblocks.attr,
+       &dev_total_eraseblocks.attr,
+       &dev_volumes_count.attr,
+       &dev_max_ec.attr,
+       &dev_reserved_for_bad.attr,
+       &dev_bad_peb_count.attr,
+       &dev_max_vol_count.attr,
+       &dev_min_io_size.attr,
+       &dev_bgt_enabled.attr,
+       &dev_mtd_num.attr,
+       NULL
+};
+ATTRIBUTE_GROUPS(ubi_dev);
+
 static void dev_release(struct device *dev)
 {
        struct ubi_device *ubi = container_of(dev, struct ubi_device, dev);
@@ -428,45 +455,15 @@ static int ubi_sysfs_init(struct ubi_device *ubi, int *ref)
 
        ubi->dev.release = dev_release;
        ubi->dev.devt = ubi->cdev.dev;
-       ubi->dev.class = ubi_class;
+       ubi->dev.class = &ubi_class;
+       ubi->dev.groups = ubi_dev_groups;
        dev_set_name(&ubi->dev, UBI_NAME_STR"%d", ubi->ubi_num);
        err = device_register(&ubi->dev);
        if (err)
                return err;
 
        *ref = 1;
-       err = device_create_file(&ubi->dev, &dev_eraseblock_size);
-       if (err)
-               return err;
-       err = device_create_file(&ubi->dev, &dev_avail_eraseblocks);
-       if (err)
-               return err;
-       err = device_create_file(&ubi->dev, &dev_total_eraseblocks);
-       if (err)
-               return err;
-       err = device_create_file(&ubi->dev, &dev_volumes_count);
-       if (err)
-               return err;
-       err = device_create_file(&ubi->dev, &dev_max_ec);
-       if (err)
-               return err;
-       err = device_create_file(&ubi->dev, &dev_reserved_for_bad);
-       if (err)
-               return err;
-       err = device_create_file(&ubi->dev, &dev_bad_peb_count);
-       if (err)
-               return err;
-       err = device_create_file(&ubi->dev, &dev_max_vol_count);
-       if (err)
-               return err;
-       err = device_create_file(&ubi->dev, &dev_min_io_size);
-       if (err)
-               return err;
-       err = device_create_file(&ubi->dev, &dev_bgt_enabled);
-       if (err)
-               return err;
-       err = device_create_file(&ubi->dev, &dev_mtd_num);
-       return err;
+       return 0;
 }
 
 /**
@@ -475,17 +472,6 @@ static int ubi_sysfs_init(struct ubi_device *ubi, int *ref)
  */
 static void ubi_sysfs_close(struct ubi_device *ubi)
 {
-       device_remove_file(&ubi->dev, &dev_mtd_num);
-       device_remove_file(&ubi->dev, &dev_bgt_enabled);
-       device_remove_file(&ubi->dev, &dev_min_io_size);
-       device_remove_file(&ubi->dev, &dev_max_vol_count);
-       device_remove_file(&ubi->dev, &dev_bad_peb_count);
-       device_remove_file(&ubi->dev, &dev_reserved_for_bad);
-       device_remove_file(&ubi->dev, &dev_max_ec);
-       device_remove_file(&ubi->dev, &dev_volumes_count);
-       device_remove_file(&ubi->dev, &dev_total_eraseblocks);
-       device_remove_file(&ubi->dev, &dev_avail_eraseblocks);
-       device_remove_file(&ubi->dev, &dev_eraseblock_size);
        device_unregister(&ubi->dev);
 }
 #endif
@@ -541,7 +527,7 @@ static int uif_init(struct ubi_device *ubi, int *ref)
         */
        err = alloc_chrdev_region(&dev, 0, ubi->vtbl_slots + 1, ubi->ubi_name);
        if (err) {
-               ubi_err("cannot register UBI character devices");
+               ubi_err(ubi, "cannot register UBI character devices");
                return err;
        }
 
@@ -552,7 +538,7 @@ static int uif_init(struct ubi_device *ubi, int *ref)
 
        err = cdev_add(&ubi->cdev, dev, 1);
        if (err) {
-               ubi_err("cannot add character device");
+               ubi_err(ubi, "cannot add character device");
                goto out_unreg;
        }
 
@@ -564,7 +550,7 @@ static int uif_init(struct ubi_device *ubi, int *ref)
                if (ubi->volumes[i]) {
                        err = ubi_add_volume(ubi, ubi->volumes[i]);
                        if (err) {
-                               ubi_err("cannot add volume %d", i);
+                               ubi_err(ubi, "cannot add volume %d", i);
                                goto out_volumes;
                        }
                }
@@ -580,7 +566,8 @@ out_sysfs:
        cdev_del(&ubi->cdev);
 out_unreg:
        unregister_chrdev_region(ubi->cdev.dev, ubi->vtbl_slots + 1);
-       ubi_err("cannot initialize UBI %s, error %d", ubi->ubi_name, err);
+       ubi_err(ubi, "cannot initialize UBI %s, error %d",
+               ubi->ubi_name, err);
        return err;
 }
 
@@ -674,7 +661,7 @@ static int io_init(struct ubi_device *ubi, int max_beb_per1024)
                 * guess we should just pick the largest region. But this is
                 * not implemented.
                 */
-               ubi_err("multiple regions, not implemented");
+               ubi_err(ubi, "multiple regions, not implemented");
                return -EINVAL;
        }
 
@@ -709,7 +696,7 @@ static int io_init(struct ubi_device *ubi, int max_beb_per1024)
         * which allows us to avoid costly division operations.
         */
        if (!is_power_of_2(ubi->min_io_size)) {
-               ubi_err("min. I/O unit (%d) is not power of 2",
+               ubi_err(ubi, "min. I/O unit (%d) is not power of 2",
                        ubi->min_io_size);
                return -EINVAL;
        }
@@ -726,7 +713,7 @@ static int io_init(struct ubi_device *ubi, int max_beb_per1024)
        if (ubi->max_write_size < ubi->min_io_size ||
            ubi->max_write_size % ubi->min_io_size ||
            !is_power_of_2(ubi->max_write_size)) {
-               ubi_err("bad write buffer size %d for %d min. I/O unit",
+               ubi_err(ubi, "bad write buffer size %d for %d min. I/O unit",
                        ubi->max_write_size, ubi->min_io_size);
                return -EINVAL;
        }
@@ -763,7 +750,7 @@ static int io_init(struct ubi_device *ubi, int max_beb_per1024)
 
        /* The shift must be aligned to 32-bit boundary */
        if (ubi->vid_hdr_shift % 4) {
-               ubi_err("unaligned VID header shift %d",
+               ubi_err(ubi, "unaligned VID header shift %d",
                        ubi->vid_hdr_shift);
                return -EINVAL;
        }
@@ -773,7 +760,7 @@ static int io_init(struct ubi_device *ubi, int max_beb_per1024)
            ubi->leb_start < ubi->vid_hdr_offset + UBI_VID_HDR_SIZE ||
            ubi->leb_start > ubi->peb_size - UBI_VID_HDR_SIZE ||
            ubi->leb_start & (ubi->min_io_size - 1)) {
-               ubi_err("bad VID header (%d) or data offsets (%d)",
+               ubi_err(ubi, "bad VID header (%d) or data offsets (%d)",
                        ubi->vid_hdr_offset, ubi->leb_start);
                return -EINVAL;
        }
@@ -793,14 +780,14 @@ static int io_init(struct ubi_device *ubi, int max_beb_per1024)
         * read-only mode.
         */
        if (ubi->vid_hdr_offset + UBI_VID_HDR_SIZE <= ubi->hdrs_min_io_size) {
-               ubi_warn("EC and VID headers are in the same minimal I/O unit, switch to read-only mode");
+               ubi_warn(ubi, "EC and VID headers are in the same minimal I/O unit, switch to read-only mode");
                ubi->ro_mode = 1;
        }
 
        ubi->leb_size = ubi->peb_size - ubi->leb_start;
 
        if (!(ubi->mtd->flags & MTD_WRITEABLE)) {
-               ubi_msg("MTD device %d is write-protected, attach in read-only mode",
+               ubi_msg(ubi, "MTD device %d is write-protected, attach in read-only mode",
                        ubi->mtd->index);
                ubi->ro_mode = 1;
        }
@@ -833,7 +820,7 @@ static int autoresize(struct ubi_device *ubi, int vol_id)
        int err, old_reserved_pebs = vol->reserved_pebs;
 
        if (ubi->ro_mode) {
-               ubi_warn("skip auto-resize because of R/O mode");
+               ubi_warn(ubi, "skip auto-resize because of R/O mode");
                return 0;
        }
 
@@ -854,21 +841,22 @@ static int autoresize(struct ubi_device *ubi, int vol_id)
                vtbl_rec = ubi->vtbl[vol_id];
                err = ubi_change_vtbl_record(ubi, vol_id, &vtbl_rec);
                if (err)
-                       ubi_err("cannot clean auto-resize flag for volume %d",
+                       ubi_err(ubi, "cannot clean auto-resize flag for volume %d",
                                vol_id);
        } else {
                desc.vol = vol;
                err = ubi_resize_volume(&desc,
                                        old_reserved_pebs + ubi->avail_pebs);
                if (err)
-                       ubi_err("cannot auto-resize volume %d", vol_id);
+                       ubi_err(ubi, "cannot auto-resize volume %d",
+                               vol_id);
        }
 
        if (err)
                return err;
 
-       ubi_msg("volume %d (\"%s\") re-sized from %d to %d LEBs", vol_id,
-               vol->name, old_reserved_pebs, vol->reserved_pebs);
+       ubi_msg(ubi, "volume %d (\"%s\") re-sized from %d to %d LEBs",
+               vol_id, vol->name, old_reserved_pebs, vol->reserved_pebs);
        return 0;
 }
 
@@ -909,7 +897,7 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num,
        for (i = 0; i < UBI_MAX_DEVICES; i++) {
                ubi = ubi_devices[i];
                if (ubi && mtd->index == ubi->mtd->index) {
-                       ubi_err("mtd%d is already attached to ubi%d",
+                       ubi_err(ubi, "mtd%d is already attached to ubi%d",
                                mtd->index, i);
                        return -EEXIST;
                }
@@ -924,7 +912,7 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num,
         * no sense to attach emulated MTD devices, so we prohibit this.
         */
        if (mtd->type == MTD_UBIVOLUME) {
-               ubi_err("refuse attaching mtd%d - it is already emulated on top of UBI",
+               ubi_err(ubi, "refuse attaching mtd%d - it is already emulated on top of UBI",
                        mtd->index);
                return -EINVAL;
        }
@@ -935,7 +923,7 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num,
                        if (!ubi_devices[ubi_num])
                                break;
                if (ubi_num == UBI_MAX_DEVICES) {
-                       ubi_err("only %d UBI devices may be created",
+                       ubi_err(ubi, "only %d UBI devices may be created",
                                UBI_MAX_DEVICES);
                        return -ENFILE;
                }
@@ -945,7 +933,7 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num,
 
                /* Make sure ubi_num is not busy */
                if (ubi_devices[ubi_num]) {
-                       ubi_err("ubi%d already exists", ubi_num);
+                       ubi_err(ubi, "already exists");
                        return -EEXIST;
                }
        }
@@ -969,21 +957,24 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num,
         */
        ubi->fm_pool.max_size = min(((int)mtd_div_by_eb(ubi->mtd->size,
                ubi->mtd) / 100) * 5, UBI_FM_MAX_POOL_SIZE);
-       if (ubi->fm_pool.max_size < UBI_FM_MIN_POOL_SIZE)
-               ubi->fm_pool.max_size = UBI_FM_MIN_POOL_SIZE;
+       ubi->fm_pool.max_size = max(ubi->fm_pool.max_size,
+               UBI_FM_MIN_POOL_SIZE);
 
-       ubi->fm_wl_pool.max_size = UBI_FM_WL_POOL_SIZE;
+       ubi->fm_wl_pool.max_size = ubi->fm_pool.max_size / 2;
        ubi->fm_disabled = !fm_autoconvert;
+       if (fm_debug)
+               ubi_enable_dbg_chk_fastmap(ubi);
 
        if (!ubi->fm_disabled && (int)mtd_div_by_eb(ubi->mtd->size, ubi->mtd)
            <= UBI_FM_MAX_START) {
-               ubi_err("More than %i PEBs are needed for fastmap, sorry.",
+               ubi_err(ubi, "More than %i PEBs are needed for fastmap, sorry.",
                        UBI_FM_MAX_START);
                ubi->fm_disabled = 1;
        }
 
-       ubi_msg("default fastmap pool size: %d", ubi->fm_pool.max_size);
-       ubi_msg("default fastmap WL pool size: %d", ubi->fm_wl_pool.max_size);
+       ubi_msg(ubi, "default fastmap pool size: %d", ubi->fm_pool.max_size);
+       ubi_msg(ubi, "default fastmap WL pool size: %d",
+               ubi->fm_wl_pool.max_size);
 #else
        ubi->fm_disabled = 1;
 #endif
@@ -991,10 +982,10 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num,
        mutex_init(&ubi->ckvol_mutex);
        mutex_init(&ubi->device_mutex);
        spin_lock_init(&ubi->volumes_lock);
-       mutex_init(&ubi->fm_mutex);
-       init_rwsem(&ubi->fm_sem);
+       init_rwsem(&ubi->fm_protect);
+       init_rwsem(&ubi->fm_eba_sem);
 
-       ubi_msg("attaching mtd%d to ubi%d", mtd->index, ubi_num);
+       ubi_msg(ubi, "attaching mtd%d", mtd->index);
 
        err = io_init(ubi, max_beb_per1024);
        if (err)
@@ -1013,7 +1004,8 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num,
 #endif
        err = ubi_attach(ubi, 0);
        if (err) {
-               ubi_err("failed to attach mtd%d, error %d", mtd->index, err);
+               ubi_err(ubi, "failed to attach mtd%d, error %d",
+                       mtd->index, err);
                goto out_free;
        }
 
@@ -1034,28 +1026,28 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num,
        ubi->bgt_thread = kthread_create(ubi_thread, ubi, "%s", ubi->bgt_name);
        if (IS_ERR(ubi->bgt_thread)) {
                err = PTR_ERR(ubi->bgt_thread);
-               ubi_err("cannot spawn \"%s\", error %d", ubi->bgt_name,
-                       err);
+               ubi_err(ubi, "cannot spawn \"%s\", error %d",
+                       ubi->bgt_name, err);
                goto out_debugfs;
        }
 
-       ubi_msg("attached mtd%d (name \"%s\", size %llu MiB) to ubi%d",
-               mtd->index, mtd->name, ubi->flash_size >> 20, ubi_num);
-       ubi_msg("PEB size: %d bytes (%d KiB), LEB size: %d bytes",
+       ubi_msg(ubi, "attached mtd%d (name \"%s\", size %llu MiB)",
+               mtd->index, mtd->name, ubi->flash_size >> 20);
+       ubi_msg(ubi, "PEB size: %d bytes (%d KiB), LEB size: %d bytes",
                ubi->peb_size, ubi->peb_size >> 10, ubi->leb_size);
-       ubi_msg("min./max. I/O unit sizes: %d/%d, sub-page size %d",
+       ubi_msg(ubi, "min./max. I/O unit sizes: %d/%d, sub-page size %d",
                ubi->min_io_size, ubi->max_write_size, ubi->hdrs_min_io_size);
-       ubi_msg("VID header offset: %d (aligned %d), data offset: %d",
+       ubi_msg(ubi, "VID header offset: %d (aligned %d), data offset: %d",
                ubi->vid_hdr_offset, ubi->vid_hdr_aloffset, ubi->leb_start);
-       ubi_msg("good PEBs: %d, bad PEBs: %d, corrupted PEBs: %d",
+       ubi_msg(ubi, "good PEBs: %d, bad PEBs: %d, corrupted PEBs: %d",
                ubi->good_peb_count, ubi->bad_peb_count, ubi->corr_peb_count);
-       ubi_msg("user volume: %d, internal volumes: %d, max. volumes count: %d",
+       ubi_msg(ubi, "user volume: %d, internal volumes: %d, max. volumes count: %d",
                ubi->vol_count - UBI_INT_VOL_COUNT, UBI_INT_VOL_COUNT,
                ubi->vtbl_slots);
-       ubi_msg("max/mean erase counter: %d/%d, WL threshold: %d, image sequence number: %u",
+       ubi_msg(ubi, "max/mean erase counter: %d/%d, WL threshold: %d, image sequence number: %u",
                ubi->max_ec, ubi->mean_ec, CONFIG_MTD_UBI_WL_THRESHOLD,
                ubi->image_seq);
-       ubi_msg("available PEBs: %d, total reserved PEBs: %d, PEBs reserved for bad PEB handling: %d",
+       ubi_msg(ubi, "available PEBs: %d, total reserved PEBs: %d, PEBs reserved for bad PEB handling: %d",
                ubi->avail_pebs, ubi->rsvd_pebs, ubi->beb_rsvd_pebs);
 
        /*
@@ -1064,7 +1056,20 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num,
         */
        spin_lock(&ubi->wl_lock);
        ubi->thread_enabled = 1;
+#ifndef __UBOOT__
        wake_up_process(ubi->bgt_thread);
+#else
+       /*
+        * U-Boot special: We have no bgt_thread in U-Boot!
+        * So just call do_work() here directly.
+        */
+       err = do_work(ubi);
+       if (err) {
+               ubi_err(ubi, "%s: work failed with error code %d",
+                       ubi->bgt_name, err);
+       }
+#endif
+
        spin_unlock(&ubi->wl_lock);
 
        ubi_devices[ubi_num] = ubi;
@@ -1124,7 +1129,7 @@ int ubi_detach_mtd_dev(int ubi_num, int anyway)
                        return -EBUSY;
                }
                /* This may only happen if there is a bug */
-               ubi_err("%s reference count %d, destroy anyway",
+               ubi_err(ubi, "%s reference count %d, destroy anyway",
                        ubi->ubi_name, ubi->ref_count);
        }
        ubi_devices[ubi_num] = NULL;
@@ -1132,11 +1137,14 @@ int ubi_detach_mtd_dev(int ubi_num, int anyway)
 
        ubi_assert(ubi_num == ubi->ubi_num);
        ubi_notify_all(ubi, UBI_VOLUME_REMOVED, NULL);
-       ubi_msg("detaching mtd%d from ubi%d", ubi->mtd->index, ubi_num);
+       ubi_msg(ubi, "detaching mtd%d", ubi->mtd->index);
 #ifdef CONFIG_MTD_UBI_FASTMAP
        /* If we don't write a new fastmap at detach time we lose all
-        * EC updates that have been made since the last written fastmap. */
-       ubi_update_fastmap(ubi);
+        * EC updates that have been made since the last written fastmap.
+        * In case of fastmap debugging we omit the update to simulate an
+        * unclean shutdown. */
+       if (!ubi_dbg_chk_fastmap(ubi))
+               ubi_update_fastmap(ubi);
 #endif
        /*
         * Before freeing anything, we have to stop the background thread to
@@ -1160,7 +1168,7 @@ int ubi_detach_mtd_dev(int ubi_num, int anyway)
        put_mtd_device(ubi->mtd);
        vfree(ubi->peb_buf);
        vfree(ubi->fm_buf);
-       ubi_msg("mtd%d is detached from ubi%d", ubi->mtd->index, ubi->ubi_num);
+       ubi_msg(ubi, "mtd%d is detached", ubi->mtd->index);
        put_device(&ubi->dev);
        return 0;
 }
@@ -1185,9 +1193,9 @@ static struct mtd_info * __init open_mtd_by_chdev(const char *mtd_dev)
                return ERR_PTR(err);
 
        /* MTD device number is defined by the major / minor numbers */
-       major = imajor(path.dentry->d_inode);
-       minor = iminor(path.dentry->d_inode);
-       mode = path.dentry->d_inode->i_mode;
+       major = imajor(d_backing_inode(path.dentry));
+       minor = iminor(d_backing_inode(path.dentry));
+       mode = d_backing_inode(path.dentry)->i_mode;
        path_put(&path);
        if (major != MTD_CHAR_MAJOR || !S_ISCHR(mode))
                return ERR_PTR(-EINVAL);
@@ -1250,28 +1258,20 @@ int ubi_init(void)
        BUILD_BUG_ON(sizeof(struct ubi_vid_hdr) != 64);
 
        if (mtd_devs > UBI_MAX_DEVICES) {
-               ubi_err("too many MTD devices, maximum is %d", UBI_MAX_DEVICES);
+               pr_err("UBI error: too many MTD devices, maximum is %d",
+                      UBI_MAX_DEVICES);
                return -EINVAL;
        }
 
        /* Create base sysfs directory and sysfs files */
-       ubi_class = class_create(THIS_MODULE, UBI_NAME_STR);
-       if (IS_ERR(ubi_class)) {
-               err = PTR_ERR(ubi_class);
-               ubi_err("cannot create UBI class");
-               goto out;
-       }
-
-       err = class_create_file(ubi_class, &ubi_version);
-       if (err) {
-               ubi_err("cannot create sysfs file");
-               goto out_class;
-       }
+       err = class_register(&ubi_class);
+       if (err < 0)
+               return err;
 
        err = misc_register(&ubi_ctrl_cdev);
        if (err) {
-               ubi_err("cannot register device");
-               goto out_version;
+               pr_err("UBI error: cannot register device");
+               goto out;
        }
 
        ubi_wl_entry_slab = kmem_cache_create("ubi_wl_entry_slab",
@@ -1297,7 +1297,8 @@ int ubi_init(void)
                mtd = open_mtd_device(p->name);
                if (IS_ERR(mtd)) {
                        err = PTR_ERR(mtd);
-                       ubi_err("cannot open mtd %s, error %d", p->name, err);
+                       pr_err("UBI error: cannot open mtd %s, error %d",
+                              p->name, err);
                        /* See comment below re-ubi_is_module(). */
                        if (ubi_is_module())
                                goto out_detach;
@@ -1309,7 +1310,8 @@ int ubi_init(void)
                                         p->vid_hdr_offs, p->max_beb_per1024);
                mutex_unlock(&ubi_devices_mutex);
                if (err < 0) {
-                       ubi_err("cannot attach mtd%d", mtd->index);
+                       pr_err("UBI error: cannot attach mtd%d",
+                              mtd->index);
                        put_mtd_device(mtd);
 
                        /*
@@ -1332,7 +1334,7 @@ int ubi_init(void)
 
        err = ubiblock_init();
        if (err) {
-               ubi_err("block: cannot initialize, error %d", err);
+               pr_err("UBI error: block: cannot initialize, error %d", err);
 
                /* See comment above re-ubi_is_module(). */
                if (ubi_is_module())
@@ -1353,16 +1355,13 @@ out_slab:
        kmem_cache_destroy(ubi_wl_entry_slab);
 out_dev_unreg:
        misc_deregister(&ubi_ctrl_cdev);
-out_version:
-       class_remove_file(ubi_class, &ubi_version);
-out_class:
-       class_destroy(ubi_class);
 out:
 #ifdef __UBOOT__
        /* Reset any globals that the driver depends on being zeroed */
        mtd_devs = 0;
 #endif
-       ubi_err("cannot initialize UBI, error %d", err);
+       class_unregister(&ubi_class);
+       pr_err("UBI error: cannot initialize UBI, error %d", err);
        return err;
 }
 late_initcall(ubi_init);
@@ -1386,8 +1385,7 @@ void ubi_exit(void)
        ubi_debugfs_exit();
        kmem_cache_destroy(ubi_wl_entry_slab);
        misc_deregister(&ubi_ctrl_cdev);
-       class_remove_file(ubi_class, &ubi_version);
-       class_destroy(ubi_class);
+       class_unregister(&ubi_class);
 #ifdef __UBOOT__
        /* Reset any globals that the driver depends on being zeroed */
        mtd_devs = 0;
@@ -1409,7 +1407,7 @@ static int __init bytes_str_to_int(const char *str)
 
        result = simple_strtoul(str, &endp, 0);
        if (str == endp || result >= INT_MAX) {
-               ubi_err("incorrect bytes count: \"%s\"\n", str);
+               pr_err("UBI error: incorrect bytes count: \"%s\"\n", str);
                return -EINVAL;
        }
 
@@ -1425,7 +1423,7 @@ static int __init bytes_str_to_int(const char *str)
        case '\0':
                break;
        default:
-               ubi_err("incorrect bytes count: \"%s\"\n", str);
+               pr_err("UBI error: incorrect bytes count: \"%s\"\n", str);
                return -EINVAL;
        }
 
@@ -1467,15 +1465,15 @@ int ubi_mtd_param_parse(const char *val, struct kernel_param *kp)
                return -EINVAL;
 
        if (mtd_devs == UBI_MAX_DEVICES) {
-               ubi_err("too many parameters, max. is %d\n",
-                       UBI_MAX_DEVICES);
+               pr_err("UBI error: too many parameters, max. is %d\n",
+                      UBI_MAX_DEVICES);
                return -EINVAL;
        }
 
        len = strnlen(val, MTD_PARAM_LEN_MAX);
        if (len == MTD_PARAM_LEN_MAX) {
-               ubi_err("parameter \"%s\" is too long, max. is %d\n",
-                       val, MTD_PARAM_LEN_MAX);
+               pr_err("UBI error: parameter \"%s\" is too long, max. is %d\n",
+                      val, MTD_PARAM_LEN_MAX);
                return -EINVAL;
        }
 
@@ -1494,7 +1492,7 @@ int ubi_mtd_param_parse(const char *val, struct kernel_param *kp)
                tokens[i] = strsep(&pbuf, ",");
 
        if (pbuf) {
-               ubi_err("too many arguments at \"%s\"\n", val);
+               pr_err("UBI error: too many arguments at \"%s\"\n", val);
                return -EINVAL;
        }
 
@@ -1514,8 +1512,8 @@ int ubi_mtd_param_parse(const char *val, struct kernel_param *kp)
                int err = kstrtoint(token, 10, &p->max_beb_per1024);
 
                if (err) {
-                       ubi_err("bad value for max_beb_per1024 parameter: %s",
-                               token);
+                       pr_err("UBI error: bad value for max_beb_per1024 parameter: %s",
+                              token);
                        return -EINVAL;
                }
        }
@@ -1525,7 +1523,8 @@ int ubi_mtd_param_parse(const char *val, struct kernel_param *kp)
                int err = kstrtoint(token, 10, &p->ubi_num);
 
                if (err) {
-                       ubi_err("bad value for ubi_num parameter: %s", token);
+                       pr_err("UBI error: bad value for ubi_num parameter: %s",
+                              token);
                        return -EINVAL;
                }
        } else
@@ -1552,6 +1551,8 @@ MODULE_PARM_DESC(mtd, "MTD devices to attach. Parameter format: mtd=<name|num|pa
 #ifdef CONFIG_MTD_UBI_FASTMAP
 module_param(fm_autoconvert, bool, 0644);
 MODULE_PARM_DESC(fm_autoconvert, "Set this parameter to enable fastmap automatically on images without a fastmap.");
+module_param(fm_debug, bool, 0);
+MODULE_PARM_DESC(fm_debug, "Set this parameter to enable fastmap debugging by default. Warning, this will make fastmap slow!");
 #endif
 MODULE_VERSION(__stringify(UBI_VERSION));
 MODULE_DESCRIPTION("UBI - Unsorted Block Images");