]> www.pilppa.org Git - linux-2.6-omap-h63xx.git/blobdiff - drivers/mtd/maps/physmap_of.c
[POWERPC] Cleanups for physmap_of.c (v2)
[linux-2.6-omap-h63xx.git] / drivers / mtd / maps / physmap_of.c
index 096dd47b5d5d7915671ab17b1844787949208c24..cf75a566442ed790e70ef062b755bf3581db7afc 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Normal mappings of chips in physical memory for OF devices
+ * Flash mappings described by the OF (or flattened) device tree
  *
  * Copyright (C) 2006 MontaVista Software Inc.
  * Author: Vitaly Wool <vwool@ru.mvista.com>
 
 #include <linux/module.h>
 #include <linux/types.h>
-#include <linux/kernel.h>
 #include <linux/init.h>
-#include <linux/slab.h>
 #include <linux/device.h>
 #include <linux/mtd/mtd.h>
 #include <linux/mtd/map.h>
 #include <linux/mtd/partitions.h>
-#include <linux/mtd/physmap.h>
-#include <asm/io.h>
-#include <asm/prom.h>
-#include <asm/of_device.h>
-#include <asm/of_platform.h>
+#include <linux/of.h>
+#include <linux/of_platform.h>
 
-struct physmap_flash_info {
+struct of_flash {
        struct mtd_info         *mtd;
        struct map_info         map;
        struct resource         *res;
@@ -38,8 +33,10 @@ struct physmap_flash_info {
 };
 
 #ifdef CONFIG_MTD_PARTITIONS
+#define OF_FLASH_PARTS(info)   ((info)->parts)
+
 static int parse_obsolete_partitions(struct of_device *dev,
-                                    struct physmap_flash_info *info,
+                                    struct of_flash *info,
                                     struct device_node *dp)
 {
        int i, plen, nr_parts;
@@ -50,17 +47,15 @@ static int parse_obsolete_partitions(struct of_device *dev,
 
        part = of_get_property(dp, "partitions", &plen);
        if (!part)
-               return -ENOENT;
+               return 0; /* No partitions found */
 
        dev_warn(&dev->dev, "Device tree uses obsolete partition map binding\n");
 
        nr_parts = plen / sizeof(part[0]);
 
-       info->parts = kzalloc(nr_parts * sizeof(struct mtd_partition), GFP_KERNEL);
-       if (!info->parts) {
-               printk(KERN_ERR "Can't allocate the flash partition data!\n");
+       info->parts = kzalloc(nr_parts * sizeof(*info->parts), GFP_KERNEL);
+       if (!info->parts)
                return -ENOMEM;
-       }
 
        names = of_get_property(dp, "partition-names", &plen);
 
@@ -86,8 +81,8 @@ static int parse_obsolete_partitions(struct of_device *dev,
        return nr_parts;
 }
 
-static int __devinit process_partitions(struct physmap_flash_info *info,
-                                       struct of_device *dev)
+static int __devinit parse_partitions(struct of_flash *info,
+                                     struct of_device *dev)
 {
        const char *partname;
        static const char *part_probe_types[]
@@ -109,89 +104,68 @@ static int __devinit process_partitions(struct physmap_flash_info *info,
        for (pp = dp->child; pp; pp = pp->sibling)
                nr_parts++;
 
-       if (nr_parts) {
-               info->parts = kzalloc(nr_parts * sizeof(struct mtd_partition),
-                                     GFP_KERNEL);
-               if (!info->parts) {
-                       printk(KERN_ERR "Can't allocate the flash partition data!\n");
-                       return -ENOMEM;
-               }
+       if (nr_parts == 0)
+               return parse_obsolete_partitions(dev, info, dp);
+
+       info->parts = kzalloc(nr_parts * sizeof(*info->parts),
+                             GFP_KERNEL);
+       if (!info->parts)
+               return -ENOMEM;
 
-               for (pp = dp->child, i = 0 ; pp; pp = pp->sibling, i++) {
-                       const u32 *reg;
-                       int len;
-
-                       reg = of_get_property(pp, "reg", &len);
-                       if (!reg || (len != 2*sizeof(u32))) {
-                               dev_err(&dev->dev, "Invalid 'reg' on %s\n",
-                                       dp->full_name);
-                               kfree(info->parts);
-                               info->parts = NULL;
-                               return -EINVAL;
-                       }
-                       info->parts[i].offset = reg[0];
-                       info->parts[i].size = reg[1];
-
-                       partname = of_get_property(pp, "label", &len);
-                       if (!partname)
-                               partname = of_get_property(pp, "name", &len);
-                       info->parts[i].name = (char *)partname;
-
-                       if (of_get_property(pp, "read-only", &len))
-                               info->parts[i].mask_flags = MTD_WRITEABLE;
+       for (pp = dp->child, i = 0; pp; pp = pp->sibling, i++) {
+               const u32 *reg;
+               int len;
+
+               reg = of_get_property(pp, "reg", &len);
+               if (!reg || (len != 2*sizeof(u32))) {
+                       dev_err(&dev->dev, "Invalid 'reg' on %s\n",
+                               dp->full_name);
+                       kfree(info->parts);
+                       info->parts = NULL;
+                       return -EINVAL;
                }
-       } else {
-               nr_parts = parse_obsolete_partitions(dev, info, dp);
-               if (nr_parts == -ENOENT)
-                       nr_parts = 0;
-       }
+               info->parts[i].offset = reg[0];
+               info->parts[i].size = reg[1];
 
-       if (nr_parts < 0)
-               return nr_parts;
+               partname = of_get_property(pp, "label", &len);
+               if (!partname)
+                       partname = of_get_property(pp, "name", &len);
+               info->parts[i].name = (char *)partname;
 
-       if (nr_parts > 0)
-               add_mtd_partitions(info->mtd, info->parts, nr_parts);
-       else
-               add_mtd_device(info->mtd);
+               if (of_get_property(pp, "read-only", &len))
+                       info->parts[i].mask_flags = MTD_WRITEABLE;
+       }
 
-       return 0;
+       return nr_parts;
 }
 #else /* MTD_PARTITIONS */
-static int __devinit process_partitions(struct physmap_flash_info *info,
-                                       struct device_node *dev)
-{
-       add_mtd_device(info->mtd);
-       return 0;
-}
+#define        OF_FLASH_PARTS(info)            (0)
+#define parse_partitions(info, dev)    (0)
 #endif /* MTD_PARTITIONS */
 
-static int of_physmap_remove(struct of_device *dev)
+static int of_flash_remove(struct of_device *dev)
 {
-       struct physmap_flash_info *info;
+       struct of_flash *info;
 
        info = dev_get_drvdata(&dev->dev);
-       if (info == NULL)
+       if (!info)
                return 0;
        dev_set_drvdata(&dev->dev, NULL);
 
-       if (info->mtd != NULL) {
-#ifdef CONFIG_MTD_PARTITIONS
-               if (info->parts) {
+       if (info->mtd) {
+               if (OF_FLASH_PARTS(info)) {
                        del_mtd_partitions(info->mtd);
-                       kfree(info->parts);
+                       kfree(OF_FLASH_PARTS(info));
                } else {
                        del_mtd_device(info->mtd);
                }
-#else
-               del_mtd_device(info->mtd);
-#endif
                map_destroy(info->mtd);
        }
 
-       if (info->map.virt != NULL)
+       if (info->map.virt)
                iounmap(info->map.virt);
 
-       if (info->res != NULL) {
+       if (info->res) {
                release_resource(info->res);
                kfree(info->res);
        }
@@ -229,52 +203,49 @@ static struct mtd_info * __devinit obsolete_probe(struct of_device *dev,
                return do_map_probe("jedec_probe", map);
        } else {
                if (strcmp(of_probe, "ROM") != 0)
-                       dev_dbg(&dev->dev, "obsolete_probe: don't know probe type "
-                               "'%s', mapping as rom\n", of_probe);
+                       dev_warn(&dev->dev, "obsolete_probe: don't know probe "
+                                "type '%s', mapping as rom\n", of_probe);
                return do_map_probe("mtd_rom", map);
        }
 }
 
-static int __devinit of_physmap_probe(struct of_device *dev, const struct of_device_id *match)
+static int __devinit of_flash_probe(struct of_device *dev,
+                                   const struct of_device_id *match)
 {
        struct device_node *dp = dev->node;
        struct resource res;
-       struct physmap_flash_info *info;
-       const char *probe_type = (const char *)match->data;
+       struct of_flash *info;
+       const char *probe_type = match->data;
        const u32 *width;
        int err;
 
+       err = -ENXIO;
        if (of_address_to_resource(dp, 0, &res)) {
-               dev_err(&dev->dev, "Can't get the flash mapping!\n");
-               err = -EINVAL;
+               dev_err(&dev->dev, "Can't get IO address from device tree\n");
                goto err_out;
        }
 
-               dev_dbg(&dev->dev, "physmap flash device: %.8llx at %.8llx\n",
-           (unsigned long long)res.end - res.start + 1,
-           (unsigned long long)res.start);
+               dev_dbg(&dev->dev, "of_flash device: %.8llx-%.8llx\n",
+               (unsigned long long)res.start, (unsigned long long)res.end);
 
-       info = kzalloc(sizeof(struct physmap_flash_info), GFP_KERNEL);
-       if (info == NULL) {
-               err = -ENOMEM;
+       err = -ENOMEM;
+       info = kzalloc(sizeof(*info), GFP_KERNEL);
+       if (!info)
                goto err_out;
-       }
        memset(info, 0, sizeof(*info));
 
        dev_set_drvdata(&dev->dev, info);
 
+       err = -EBUSY;
        info->res = request_mem_region(res.start, res.end - res.start + 1,
-                       dev->dev.bus_id);
-       if (info->res == NULL) {
-               dev_err(&dev->dev, "Could not reserve memory region\n");
-               err = -ENOMEM;
+                                      dev->dev.bus_id);
+       if (!info->res)
                goto err_out;
-       }
 
+       err = -ENXIO;
        width = of_get_property(dp, "bank-width", NULL);
-       if (width == NULL) {
-               dev_err(&dev->dev, "Can't get the flash bank width!\n");
-               err = -EINVAL;
+       if (!width) {
+               dev_err(&dev->dev, "Can't get bank width from device tree\n");
                goto err_out;
        }
 
@@ -283,10 +254,10 @@ static int __devinit of_physmap_probe(struct of_device *dev, const struct of_dev
        info->map.size = res.end - res.start + 1;
        info->map.bankwidth = *width;
 
+       err = -ENOMEM;
        info->map.virt = ioremap(info->map.phys, info->map.size);
-       if (info->map.virt == NULL) {
-               dev_err(&dev->dev, "Failed to ioremap flash region\n");
-               err = EIO;
+       if (!info->map.virt) {
+               dev_err(&dev->dev, "Failed to ioremap() flash region\n");
                goto err_out;
        }
 
@@ -297,25 +268,30 @@ static int __devinit of_physmap_probe(struct of_device *dev, const struct of_dev
        else
                info->mtd = obsolete_probe(dev, &info->map);
 
-       if (info->mtd == NULL) {
-               dev_err(&dev->dev, "map_probe failed\n");
-               err = -ENXIO;
+       err = -ENXIO;
+       if (!info->mtd) {
+               dev_err(&dev->dev, "do_map_probe() failed\n");
                goto err_out;
        }
        info->mtd->owner = THIS_MODULE;
 
-       return process_partitions(info, dev);
+       err = parse_partitions(info, dev);
+       if (err < 0)
+               goto err_out;
 
-err_out:
-       of_physmap_remove(dev);
-       return err;
+       if (err > 0)
+               add_mtd_partitions(info->mtd, OF_FLASH_PARTS(info), err);
+       else
+               add_mtd_device(info->mtd);
 
        return 0;
 
-
+err_out:
+       of_flash_remove(dev);
+       return err;
 }
 
-static struct of_device_id of_physmap_match[] = {
+static struct of_device_id of_flash_match[] = {
        {
                .compatible     = "cfi-flash",
                .data           = (void *)"cfi_probe",
@@ -337,30 +313,28 @@ static struct of_device_id of_physmap_match[] = {
        },
        { },
 };
+MODULE_DEVICE_TABLE(of, of_flash_match);
 
-MODULE_DEVICE_TABLE(of, of_physmap_match);
-
-
-static struct of_platform_driver of_physmap_flash_driver = {
-       .name           = "physmap-flash",
-       .match_table    = of_physmap_match,
-       .probe          = of_physmap_probe,
-       .remove         = of_physmap_remove,
+static struct of_platform_driver of_flash_driver = {
+       .name           = "of-flash",
+       .match_table    = of_flash_match,
+       .probe          = of_flash_probe,
+       .remove         = of_flash_remove,
 };
 
-static int __init of_physmap_init(void)
+static int __init of_flash_init(void)
 {
-       return of_register_platform_driver(&of_physmap_flash_driver);
+       return of_register_platform_driver(&of_flash_driver);
 }
 
-static void __exit of_physmap_exit(void)
+static void __exit of_flash_exit(void)
 {
-       of_unregister_platform_driver(&of_physmap_flash_driver);
+       of_unregister_platform_driver(&of_flash_driver);
 }
 
-module_init(of_physmap_init);
-module_exit(of_physmap_exit);
+module_init(of_flash_init);
+module_exit(of_flash_exit);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Vitaly Wool <vwool@ru.mvista.com>");
-MODULE_DESCRIPTION("Configurable MTD map driver for OF");
+MODULE_DESCRIPTION("Device tree based MTD map driver");