]> err.no Git - linux-2.6/blobdiff - drivers/mtd/ubi/build.c
olpc: sdhci: add quirk for the Marvell CaFe's vdd/powerup issue
[linux-2.6] / drivers / mtd / ubi / build.c
index 0714543766433b18210f39720d0320ac3bd2d8d6..961416ac06167350bf689c631a85b0ece7f067ec 100644 (file)
  * struct mtd_dev_param - MTD device parameter description data structure.
  * @name: MTD device name or number string
  * @vid_hdr_offs: VID header offset
- * @data_offs: data offset
  */
 struct mtd_dev_param
 {
        char name[MTD_PARAM_LEN_MAX];
        int vid_hdr_offs;
-       int data_offs;
 };
 
 /* Numbers of elements set in the @mtd_dev_param array */
@@ -68,9 +66,6 @@ static struct mtd_dev_param mtd_dev_param[UBI_MAX_DEVICES];
 /* Root UBI "class" object (corresponds to '/<sysfs>/class/ubi/') */
 struct class *ubi_class;
 
-/* Slab cache for lock-tree entries */
-struct kmem_cache *ubi_ltree_slab;
-
 /* Slab cache for wear-leveling entries */
 struct kmem_cache *ubi_wl_entry_slab;
 
@@ -124,6 +119,8 @@ static struct device_attribute dev_min_io_size =
        __ATTR(min_io_size, S_IRUGO, dev_attribute_show, NULL);
 static struct device_attribute dev_bgt_enabled =
        __ATTR(bgt_enabled, S_IRUGO, dev_attribute_show, NULL);
+static struct device_attribute dev_mtd_num =
+       __ATTR(mtd_num, S_IRUGO, dev_attribute_show, NULL);
 
 /**
  * ubi_get_device - get UBI device.
@@ -246,7 +243,7 @@ static ssize_t dev_attribute_show(struct device *dev,
        else if (attr == &dev_total_eraseblocks)
                ret = sprintf(buf, "%d\n", ubi->good_peb_count);
        else if (attr == &dev_volumes_count)
-               ret = sprintf(buf, "%d\n", ubi->vol_count);
+               ret = sprintf(buf, "%d\n", ubi->vol_count - UBI_INT_VOL_COUNT);
        else if (attr == &dev_max_ec)
                ret = sprintf(buf, "%d\n", ubi->max_ec);
        else if (attr == &dev_reserved_for_bad)
@@ -259,8 +256,10 @@ static ssize_t dev_attribute_show(struct device *dev,
                ret = sprintf(buf, "%d\n", ubi->min_io_size);
        else if (attr == &dev_bgt_enabled)
                ret = sprintf(buf, "%d\n", ubi->thread_enabled);
+       else if (attr == &dev_mtd_num)
+               ret = sprintf(buf, "%d\n", ubi->mtd->index);
        else
-               BUG();
+               ret = -EINVAL;
 
        ubi_put_device(ubi);
        return ret;
@@ -316,6 +315,9 @@ static int ubi_sysfs_init(struct ubi_device *ubi)
        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;
 }
 
@@ -325,6 +327,7 @@ static int ubi_sysfs_init(struct ubi_device *ubi)
  */
 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);
@@ -363,9 +366,6 @@ static int uif_init(struct ubi_device *ubi)
        int i, err;
        dev_t dev;
 
-       mutex_init(&ubi->volumes_mutex);
-       spin_lock_init(&ubi->volumes_lock);
-
        sprintf(ubi->ubi_name, UBI_NAME_STR "%d", ubi->ubi_num);
 
        /*
@@ -512,7 +512,7 @@ static int io_init(struct ubi_device *ubi)
                return -EINVAL;
        }
 
-       if (ubi->vid_hdr_offset < 0 || ubi->leb_start < ubi->vid_hdr_offset)
+       if (ubi->vid_hdr_offset < 0)
                return -EINVAL;
 
        /*
@@ -562,10 +562,8 @@ static int io_init(struct ubi_device *ubi)
        }
 
        /* Similar for the data offset */
-       if (ubi->leb_start == 0) {
-               ubi->leb_start = ubi->vid_hdr_offset + ubi->vid_hdr_alsize;
-               ubi->leb_start = ALIGN(ubi->leb_start, ubi->min_io_size);
-       }
+       ubi->leb_start = ubi->vid_hdr_offset + UBI_EC_HDR_SIZE;
+       ubi->leb_start = ALIGN(ubi->leb_start, ubi->min_io_size);
 
        dbg_msg("vid_hdr_offset   %d", ubi->vid_hdr_offset);
        dbg_msg("vid_hdr_aloffset %d", ubi->vid_hdr_aloffset);
@@ -608,8 +606,16 @@ static int io_init(struct ubi_device *ubi)
                ubi->ro_mode = 1;
        }
 
-       dbg_msg("leb_size         %d", ubi->leb_size);
-       dbg_msg("ro_mode          %d", ubi->ro_mode);
+       ubi_msg("physical eraseblock size:   %d bytes (%d KiB)",
+               ubi->peb_size, ubi->peb_size >> 10);
+       ubi_msg("logical eraseblock size:    %d bytes", ubi->leb_size);
+       ubi_msg("smallest flash I/O unit:    %d", ubi->min_io_size);
+       if (ubi->hdrs_min_io_size != ubi->min_io_size)
+               ubi_msg("sub-page size:              %d",
+                       ubi->hdrs_min_io_size);
+       ubi_msg("VID header offset:          %d (aligned %d)",
+               ubi->vid_hdr_offset, ubi->vid_hdr_aloffset);
+       ubi_msg("data offset:                %d", ubi->leb_start);
 
        /*
         * Note, ideally, we have to initialize ubi->bad_peb_count here. But
@@ -622,23 +628,74 @@ static int io_init(struct ubi_device *ubi)
        return 0;
 }
 
+/**
+ * autoresize - re-size the volume which has the "auto-resize" flag set.
+ * @ubi: UBI device description object
+ * @vol_id: ID of the volume to re-size
+ *
+ * This function re-sizes the volume marked by the @UBI_VTBL_AUTORESIZE_FLG in
+ * the volume table to the largest possible size. See comments in ubi-header.h
+ * for more description of the flag. Returns zero in case of success and a
+ * negative error code in case of failure.
+ */
+static int autoresize(struct ubi_device *ubi, int vol_id)
+{
+       struct ubi_volume_desc desc;
+       struct ubi_volume *vol = ubi->volumes[vol_id];
+       int err, old_reserved_pebs = vol->reserved_pebs;
+
+       /*
+        * Clear the auto-resize flag in the volume in-memory copy of the
+        * volume table, and 'ubi_resize_volume()' will propogate this change
+        * to the flash.
+        */
+       ubi->vtbl[vol_id].flags &= ~UBI_VTBL_AUTORESIZE_FLG;
+
+       if (ubi->avail_pebs == 0) {
+               struct ubi_vtbl_record vtbl_rec;
+
+               /*
+                * No avalilable PEBs to re-size the volume, clear the flag on
+                * flash and exit.
+                */
+               memcpy(&vtbl_rec, &ubi->vtbl[vol_id],
+                      sizeof(struct ubi_vtbl_record));
+               err = ubi_change_vtbl_record(ubi, vol_id, &vtbl_rec);
+               if (err)
+                       ubi_err("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);
+       }
+
+       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);
+       return 0;
+}
+
 /**
  * ubi_attach_mtd_dev - attach an MTD device.
  * @mtd_dev: MTD device description object
+ * @ubi_num: number to assign to the new UBI device
  * @vid_hdr_offset: VID header offset
- * @data_offset: data offset
  *
- * This function attaches an MTD device to UBI. It first treats @mtd_dev as the
- * MTD device name, and tries to open it by this name. If it is unable to open,
- * it tries to convert @mtd_dev to an integer and open the MTD device by its
- * number. Returns new UBI device's number in case of success and a negative
- * error code in case of failure.
+ * This function attaches MTD device @mtd_dev to UBI and assign @ubi_num number
+ * to the newly created UBI device, unless @ubi_num is %UBI_DEV_NUM_AUTO, in
+ * which case this function finds a vacant device nubert and assings it
+ * automatically. Returns the new UBI device number in case of success and a
+ * negative error code in case of failure.
  *
  * Note, the invocations of this function has to be serialized by the
  * @ubi_devices_mutex.
  */
-int ubi_attach_mtd_dev(struct mtd_info *mtd, int vid_hdr_offset,
-                      int data_offset)
+int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num, int vid_hdr_offset)
 {
        struct ubi_device *ubi;
        int i, err;
@@ -649,22 +706,47 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int vid_hdr_offset,
         * Note, this function assumes that UBI devices creations and deletions
         * are serialized, so it does not take the &ubi_devices_lock.
         */
-       for (i = 0; i < UBI_MAX_DEVICES; i++)
+       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",
+                       dbg_err("mtd%d is already attached to ubi%d",
                                mtd->index, i);
-                       return -EINVAL;
+                       return -EEXIST;
                }
+       }
 
-       /* Search for an empty slot in the @ubi_devices array */
-       for (i = 0; i < UBI_MAX_DEVICES; i++)
-               if (!ubi_devices[i])
-                       break;
+       /*
+        * Make sure this MTD device is not emulated on top of an UBI volume
+        * already. Well, generally this recursion works fine, but there are
+        * different problems like the UBI module takes a reference to itself
+        * by attaching (and thus, opening) the emulated MTD device. This
+        * results in inability to unload the module. And in general it makes
+        * 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", mtd->index);
+               return -EINVAL;
+       }
 
-       if (i == UBI_MAX_DEVICES) {
-               ubi_err("only %d UBI devices may be created", UBI_MAX_DEVICES);
-               return -ENFILE;
+       if (ubi_num == UBI_DEV_NUM_AUTO) {
+               /* Search for an empty slot in the @ubi_devices array */
+               for (ubi_num = 0; ubi_num < UBI_MAX_DEVICES; ubi_num++)
+                       if (!ubi_devices[ubi_num])
+                               break;
+               if (ubi_num == UBI_MAX_DEVICES) {
+                       dbg_err("only %d UBI devices may be created", UBI_MAX_DEVICES);
+                       return -ENFILE;
+               }
+       } else {
+               if (ubi_num >= UBI_MAX_DEVICES)
+                       return -EINVAL;
+
+               /* Make sure ubi_num is not busy */
+               if (ubi_devices[ubi_num]) {
+                       dbg_err("ubi%d already exists", ubi_num);
+                       return -EEXIST;
+               }
        }
 
        ubi = kzalloc(sizeof(struct ubi_device), GFP_KERNEL);
@@ -672,18 +754,21 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int vid_hdr_offset,
                return -ENOMEM;
 
        ubi->mtd = mtd;
-       ubi->ubi_num = i;
+       ubi->ubi_num = ubi_num;
        ubi->vid_hdr_offset = vid_hdr_offset;
-       ubi->leb_start = data_offset;
+       ubi->autoresize_vol_id = -1;
 
-       dbg_msg("attaching mtd%d to ubi%d: VID header offset %d data offset %d",
-               mtd->index, ubi->ubi_num, vid_hdr_offset, data_offset);
+       mutex_init(&ubi->buf_mutex);
+       mutex_init(&ubi->ckvol_mutex);
+       mutex_init(&ubi->volumes_mutex);
+       spin_lock_init(&ubi->volumes_lock);
+
+       ubi_msg("attaching mtd%d to ubi%d", mtd->index, ubi_num);
 
        err = io_init(ubi);
        if (err)
                goto out_free;
 
-       mutex_init(&ubi->buf_mutex);
        ubi->peb_buf1 = vmalloc(ubi->peb_size);
        if (!ubi->peb_buf1)
                goto out_free;
@@ -705,6 +790,12 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int vid_hdr_offset,
                goto out_free;
        }
 
+       if (ubi->autoresize_vol_id != -1) {
+               err = autoresize(ubi, ubi->autoresize_vol_id);
+               if (err)
+                       goto out_detach;
+       }
+
        err = uif_init(ubi);
        if (err)
                goto out_detach;
@@ -717,18 +808,11 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int vid_hdr_offset,
                goto out_uif;
        }
 
-       ubi_msg("attached mtd%d to ubi%d", mtd->index, ubi->ubi_num);
+       ubi_msg("attached mtd%d to ubi%d", mtd->index, ubi_num);
        ubi_msg("MTD device name:            \"%s\"", mtd->name);
        ubi_msg("MTD device size:            %llu MiB", ubi->flash_size >> 20);
-       ubi_msg("physical eraseblock size:   %d bytes (%d KiB)",
-               ubi->peb_size, ubi->peb_size >> 10);
-       ubi_msg("logical eraseblock size:    %d bytes", ubi->leb_size);
        ubi_msg("number of good PEBs:        %d", ubi->good_peb_count);
        ubi_msg("number of bad PEBs:         %d", ubi->bad_peb_count);
-       ubi_msg("smallest flash I/O unit:    %d", ubi->min_io_size);
-       ubi_msg("VID header offset:          %d (aligned %d)",
-               ubi->vid_hdr_offset, ubi->vid_hdr_aloffset);
-       ubi_msg("data offset:                %d", ubi->leb_start);
        ubi_msg("max. allowed volumes:       %d", ubi->vtbl_slots);
        ubi_msg("wear-leveling threshold:    %d", CONFIG_MTD_UBI_WL_THRESHOLD);
        ubi_msg("number of internal volumes: %d", UBI_INT_VOL_COUNT);
@@ -746,8 +830,8 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int vid_hdr_offset,
                wake_up_process(ubi->bgt_thread);
        }
 
-       ubi_devices[ubi->ubi_num] = ubi;
-       return ubi->ubi_num;
+       ubi_devices[ubi_num] = ubi;
+       return ubi_num;
 
 out_uif:
        uif_close(ubi);
@@ -788,23 +872,24 @@ int ubi_detach_mtd_dev(int ubi_num, int anyway)
        spin_lock(&ubi_devices_lock);
        ubi = ubi_devices[ubi_num];
        if (!ubi) {
-               spin_lock(&ubi_devices_lock);
+               spin_unlock(&ubi_devices_lock);
                return -EINVAL;
        }
 
        if (ubi->ref_count) {
                if (!anyway) {
-                       spin_lock(&ubi_devices_lock);
+                       spin_unlock(&ubi_devices_lock);
                        return -EBUSY;
                }
                /* This may only happen if there is a bug */
                ubi_err("%s reference count %d, destroy anyway",
                        ubi->ubi_name, ubi->ref_count);
        }
-       ubi_devices[ubi->ubi_num] = NULL;
+       ubi_devices[ubi_num] = NULL;
        spin_unlock(&ubi_devices_lock);
 
-       dbg_msg("detaching mtd%d from ubi%d", ubi->mtd->index, ubi->ubi_num);
+       ubi_assert(ubi_num == ubi->ubi_num);
+       dbg_msg("detaching mtd%d from ubi%d", ubi->mtd->index, ubi_num);
 
        /*
         * Before freeing anything, we have to stop the background thread to
@@ -828,56 +913,30 @@ int ubi_detach_mtd_dev(int ubi_num, int anyway)
        return 0;
 }
 
-/**
- * ltree_entry_ctor - lock tree entries slab cache constructor.
- * @obj: the lock-tree entry to construct
- * @cache: the lock tree entry slab cache
- * @flags: constructor flags
- */
-static void ltree_entry_ctor(struct kmem_cache *cache, void *obj)
-{
-       struct ubi_ltree_entry *le = obj;
-
-       le->users = 0;
-       init_rwsem(&le->mutex);
-}
-
 /**
  * find_mtd_device - open an MTD device by its name or number.
  * @mtd_dev: name or number of the device
  *
- * This function tries to open and MTD device with name @mtd_dev, and if it
- * fails, then it tries to interpret the @mtd_dev string as an ASCII-coded
- * integer and open an MTD device with this number. Returns MTD device
- * description object in case of success and a negative error code in case of
- * failure.
+ * This function tries to open and MTD device described by @mtd_dev string,
+ * which is first treated as an ASCII number, and if it is not true, it is
+ * treated as MTD device name. Returns MTD device description object in case of
+ * success and a negative error code in case of failure.
  */
 static struct mtd_info * __init open_mtd_device(const char *mtd_dev)
 {
        struct mtd_info *mtd;
+       int mtd_num;
+       char *endp;
 
-       mtd = get_mtd_device_nm(mtd_dev);
-       if (IS_ERR(mtd)) {
-               int mtd_num;
-               char *endp;
-
-               if (PTR_ERR(mtd) != -ENODEV)
-                       return mtd;
-
+       mtd_num = simple_strtoul(mtd_dev, &endp, 0);
+       if (*endp != '\0' || mtd_dev == endp) {
                /*
-                * Probably this is not MTD device name but MTD device number -
-                * check this out.
+                * This does not look like an ASCII integer, probably this is
+                * MTD device name.
                 */
-               mtd_num = simple_strtoul(mtd_dev, &endp, 0);
-               if (*endp != '\0' || mtd_dev == endp) {
-                       ubi_err("incorrect MTD device: \"%s\"", mtd_dev);
-                       return ERR_PTR(-ENODEV);
-               }
-
+               mtd = get_mtd_device_nm(mtd_dev);
+       } else
                mtd = get_mtd_device(NULL, mtd_num);
-               if (IS_ERR(mtd))
-                       return mtd;
-       }
 
        return mtd;
 }
@@ -891,8 +950,7 @@ static int __init ubi_init(void)
        BUILD_BUG_ON(sizeof(struct ubi_vid_hdr) != 64);
 
        if (mtd_devs > UBI_MAX_DEVICES) {
-               printk(KERN_ERR "UBI error: too many MTD devices, "
-                      "maximum is %d\n", UBI_MAX_DEVICES);
+               ubi_err("too many MTD devices, maximum is %d", UBI_MAX_DEVICES);
                return -EINVAL;
        }
 
@@ -900,33 +958,27 @@ static int __init ubi_init(void)
        ubi_class = class_create(THIS_MODULE, UBI_NAME_STR);
        if (IS_ERR(ubi_class)) {
                err = PTR_ERR(ubi_class);
-               printk(KERN_ERR "UBI error: cannot create UBI class\n");
+               ubi_err("cannot create UBI class");
                goto out;
        }
 
        err = class_create_file(ubi_class, &ubi_version);
        if (err) {
-               printk(KERN_ERR "UBI error: cannot create sysfs file\n");
+               ubi_err("cannot create sysfs file");
                goto out_class;
        }
 
        err = misc_register(&ubi_ctrl_cdev);
        if (err) {
-               printk(KERN_ERR "UBI error: cannot register device\n");
+               ubi_err("cannot register device");
                goto out_version;
        }
 
-       ubi_ltree_slab = kmem_cache_create("ubi_ltree_slab",
-                                          sizeof(struct ubi_ltree_entry), 0,
-                                          0, &ltree_entry_ctor);
-       if (!ubi_ltree_slab)
-               goto out_dev_unreg;
-
        ubi_wl_entry_slab = kmem_cache_create("ubi_wl_entry_slab",
-                                               sizeof(struct ubi_wl_entry),
-                                               0, 0, NULL);
+                                             sizeof(struct ubi_wl_entry),
+                                             0, 0, NULL);
        if (!ubi_wl_entry_slab)
-               goto out_ltree;
+               goto out_dev_unreg;
 
        /* Attach MTD devices */
        for (i = 0; i < mtd_devs; i++) {
@@ -942,12 +994,12 @@ static int __init ubi_init(void)
                }
 
                mutex_lock(&ubi_devices_mutex);
-               err = ubi_attach_mtd_dev(mtd, p->vid_hdr_offs, p->data_offs);
+               err = ubi_attach_mtd_dev(mtd, UBI_DEV_NUM_AUTO,
+                                        p->vid_hdr_offs);
                mutex_unlock(&ubi_devices_mutex);
                if (err < 0) {
                        put_mtd_device(mtd);
-                       printk(KERN_ERR "UBI error: cannot attach %s\n",
-                              p->name);
+                       ubi_err("cannot attach mtd%d", mtd->index);
                        goto out_detach;
                }
        }
@@ -962,8 +1014,6 @@ out_detach:
                        mutex_unlock(&ubi_devices_mutex);
                }
        kmem_cache_destroy(ubi_wl_entry_slab);
-out_ltree:
-       kmem_cache_destroy(ubi_ltree_slab);
 out_dev_unreg:
        misc_deregister(&ubi_ctrl_cdev);
 out_version:
@@ -971,7 +1021,7 @@ out_version:
 out_class:
        class_destroy(ubi_class);
 out:
-       printk(KERN_ERR "UBI error: cannot initialize UBI, error %d\n", err);
+       ubi_err("UBI error: cannot initialize UBI, error %d", err);
        return err;
 }
 module_init(ubi_init);
@@ -987,7 +1037,6 @@ static void __exit ubi_exit(void)
                        mutex_unlock(&ubi_devices_mutex);
                }
        kmem_cache_destroy(ubi_wl_entry_slab);
-       kmem_cache_destroy(ubi_ltree_slab);
        misc_deregister(&ubi_ctrl_cdev);
        class_remove_file(ubi_class, &ubi_version);
        class_destroy(ubi_class);
@@ -1020,10 +1069,8 @@ static int __init bytes_str_to_int(const char *str)
        case 'M':
                result *= 1024;
        case 'K':
-       case 'k':
                result *= 1024;
-               if (endp[1] == 'i' && (endp[2] == '\0' ||
-                         endp[2] == 'B'  || endp[2] == 'b'))
+               if (endp[1] == 'i' && endp[2] == 'B')
                        endp += 2;
        case '\0':
                break;
@@ -1050,7 +1097,7 @@ static int __init ubi_mtd_param_parse(const char *val, struct kernel_param *kp)
        struct mtd_dev_param *p;
        char buf[MTD_PARAM_LEN_MAX];
        char *pbuf = &buf[0];
-       char *tokens[3] = {NULL, NULL, NULL};
+       char *tokens[2] = {NULL, NULL};
 
        if (!val)
                return -EINVAL;
@@ -1080,7 +1127,7 @@ static int __init ubi_mtd_param_parse(const char *val, struct kernel_param *kp)
        if (buf[len - 1] == '\n')
                buf[len - 1] = '\0';
 
-       for (i = 0; i < 3; i++)
+       for (i = 0; i < 2; i++)
                tokens[i] = strsep(&pbuf, ",");
 
        if (pbuf) {
@@ -1094,13 +1141,9 @@ static int __init ubi_mtd_param_parse(const char *val, struct kernel_param *kp)
 
        if (tokens[1])
                p->vid_hdr_offs = bytes_str_to_int(tokens[1]);
-       if (tokens[2])
-               p->data_offs = bytes_str_to_int(tokens[2]);
 
        if (p->vid_hdr_offs < 0)
                return p->vid_hdr_offs;
-       if (p->data_offs < 0)
-               return p->data_offs;
 
        mtd_devs += 1;
        return 0;
@@ -1108,16 +1151,15 @@ static int __init ubi_mtd_param_parse(const char *val, struct kernel_param *kp)
 
 module_param_call(mtd, ubi_mtd_param_parse, NULL, NULL, 000);
 MODULE_PARM_DESC(mtd, "MTD devices to attach. Parameter format: "
-                     "mtd=<name|num>[,<vid_hdr_offs>,<data_offs>]. "
+                     "mtd=<name|num>[,<vid_hdr_offs>].\n"
                      "Multiple \"mtd\" parameters may be specified.\n"
-                     "MTD devices may be specified by their number or name. "
-                     "Optional \"vid_hdr_offs\" and \"data_offs\" parameters "
-                     "specify UBI VID header position and data starting "
-                     "position to be used by UBI.\n"
-                     "Example: mtd=content,1984,2048 mtd=4 - attach MTD device"
-                     "with name content using VID header offset 1984 and data "
-                     "start 2048, and MTD device number 4 using default "
-                     "offsets");
+                     "MTD devices may be specified by their number or name.\n"
+                     "Optional \"vid_hdr_offs\" parameter specifies UBI VID "
+                     "header position and data starting position to be used "
+                     "by UBI.\n"
+                     "Example: mtd=content,1984 mtd=4 - attach MTD device"
+                     "with name \"content\" using VID header offset 1984, and "
+                     "MTD device number 4 with default VID header offset.");
 
 MODULE_VERSION(__stringify(UBI_VERSION));
 MODULE_DESCRIPTION("UBI - Unsorted Block Images");