Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=c4d5e375470862fd741f93bf0686d7ac2f7fdce4
Commit:     c4d5e375470862fd741f93bf0686d7ac2f7fdce4
Parent:     bd0076cc330f303905018a17d9dcfbabde497572
Author:     David Gibson <[EMAIL PROTECTED]>
AuthorDate: Thu Sep 20 11:22:25 2007 +1000
Committer:  Josh Boyer <[EMAIL PROTECTED]>
CommitDate: Thu Sep 20 07:37:16 2007 -0500

    [POWERPC] Cleanups for physmap_of.c (v2)
    
    This patch includes a whole batch of smallish cleanups for
    drivers/mtd/physmap_of.c.
    
        - A bunch of uneeded #includes are removed
        - We switch to the modern linux/of.h etc. in place of
    asm/prom.h
        - Use some helper macros to avoid some ugly inline #ifdefs
        - A few lines of unreachable code are removed
        - A number of indentation / line-wrapping fixes
        - More consistent use of kernel idioms such as if (!p) instead
    of if (p == NULL)
        - Clarify some printk()s and other informative strings.
        - parse_obsolete_partitions() now returns 0 if no partition
    information is found, instead of returning -ENOENT which the caller
    had to handle specially.
        - (the big one) Despite the name, this driver really has
    nothing to do with drivers/mtd/physmap.c.  The fact that the flash
    chips must be physically direct mapped is a constrant, but doesn't
    really say anything about the actual purpose of this driver, which is
    to instantiate MTD devices based on information from the device tree.
    Therefore the physmap name is replaced everywhere within the file with
    "of_flash".  The file itself and the Kconfig option is not renamed for
    now (so that the diff is actually a diff).  That can come later.
    
    Signed-off-by: David Gibson <[EMAIL PROTECTED]>
    Signed-off-by: Josh Boyer <[EMAIL PROTECTED]>
---
 drivers/mtd/maps/physmap_of.c |  224 ++++++++++++++++++-----------------------
 1 files changed, 99 insertions(+), 125 deletions(-)

diff --git a/drivers/mtd/maps/physmap_of.c b/drivers/mtd/maps/physmap_of.c
index 096dd47..cf75a56 100644
--- a/drivers/mtd/maps/physmap_of.c
+++ b/drivers/mtd/maps/physmap_of.c
@@ -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 <[EMAIL PROTECTED]>
@@ -15,20 +15,15 @@
 
 #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 <[EMAIL PROTECTED]>");
-MODULE_DESCRIPTION("Configurable MTD map driver for OF");
+MODULE_DESCRIPTION("Device tree based MTD map driver");
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to