staging: tidspbridge: Remove cfg_get_dev_object() and do a trivial cleanup
authorIvan Gomez Castellanos <ivan.gomez@ti.com>
Wed, 25 Aug 2010 22:08:58 +0000 (17:08 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 31 Aug 2010 18:23:15 +0000 (11:23 -0700)
The cfg_get_dev_object function is only used in one place and because of
its simplicity, it can be removed.

The parameter *value can be left uninitialized if the strcmp() returns a
nonzero value, so in the function dev_remove_device(), the hdev_obj could
be used uninitialized and could be dereferenced in dev_destroy_device().
This patch fixes this issue, and also removes the dev_obj pointer which
is not used.

Signed-off-by: Ivan Gomez Castellanos <ivan.gomez@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/tidspbridge/include/dspbridge/cfg.h
drivers/staging/tidspbridge/pmgr/dev.c
drivers/staging/tidspbridge/services/cfg.c

index eea90001bfff79f271baffb73ebc63744190cd3b..9a259a1dddf8bda1dca06ab4e237269ed5480f19 100644 (file)
  */
 extern int cfg_get_cd_version(u32 *version);
 
-/*
- *  ======== cfg_get_dev_object ========
- *  Purpose:
- *      Retrieve the Device Object handle for a given devnode.
- *  Parameters:
- *      dev_node_obj:  Platform's dev_node handle from which to retrieve
- *                     value.
- *      value:          Ptr to location to store the value.
- *  Returns:
- *      0:                Success.
- *      -EFAULT: dev_node_obj is invalid or device_obj is invalid.
- *      -ENODATA: The resource is not available.
- *  Requires:
- *      CFG initialized.
- *  Ensures:
- *      0:    *value is set to the retrieved u32.
- *      else:       *value is set to 0L.
- */
-extern int cfg_get_dev_object(struct cfg_devnode *dev_node_obj,
-                                    u32 *value);
-
 /*
  *  ======== cfg_get_exec_file ========
  *  Purpose:
index 4ddf03d3b1aba25aa3bcd584e4fb82e0ef7ca83b..7893f9a0cf0eb5e78df04c76775089205b55f315 100644 (file)
@@ -85,6 +85,11 @@ struct dev_object {
        struct node_mgr *hnode_mgr;
 };
 
+struct drv_ext {
+       struct list_head link;
+       char sz_string[MAXREGPATHLENGTH];
+};
+
 /*  ----------------------------------- Globals */
 static u32 refs;               /* Module reference count */
 
@@ -812,18 +817,31 @@ int dev_remove_device(struct cfg_devnode *dev_node_obj)
 {
        struct dev_object *hdev_obj;    /* handle to device object */
        int status = 0;
-       struct dev_object *dev_obj;
+       struct drv_data *drv_datap = dev_get_drvdata(bridge);
+
+       if (!drv_datap)
+               status = -ENODATA;
+
+       if (!dev_node_obj)
+               status = -EFAULT;
 
        /* Retrieve the device object handle originaly stored with
         * the dev_node: */
-       status = cfg_get_dev_object(dev_node_obj, (u32 *) &hdev_obj);
        if (!status) {
-               /* Remove the Processor List */
-               dev_obj = (struct dev_object *)hdev_obj;
-               /* Destroy the device object. */
-               status = dev_destroy_device(hdev_obj);
+               /* check the device string and then store dev object */
+               if (!strcmp((char *)((struct drv_ext *)dev_node_obj)->sz_string,
+                                                               "TIOMAP1510")) {
+                       hdev_obj = drv_datap->dev_object;
+                       /* Destroy the device object. */
+                       status = dev_destroy_device(hdev_obj);
+               } else {
+                       status = -EPERM;
+               }
        }
 
+       if (status)
+               pr_err("%s: Failed, status 0x%x\n", __func__, status);
+
        return status;
 }
 
index b9c38829921b28dfa9f05dd6c86b9450ce1d55d4..587c71a5975ffac09cd0a404bcbf3032c5659ed0 100644 (file)
 #include <dspbridge/cfg.h>
 #include <dspbridge/drv.h>
 
-struct drv_ext {
-       struct list_head link;
-       char sz_string[MAXREGPATHLENGTH];
-};
-
-/*
- *  ======== cfg_get_dev_object ========
- *  Purpose:
- *      Retrieve the Device Object handle for a given devnode.
- */
-int cfg_get_dev_object(struct cfg_devnode *dev_node_obj,
-                             u32 *value)
-{
-       int status = 0;
-       u32 dw_buf_size;
-       struct drv_data *drv_datap = dev_get_drvdata(bridge);
-
-       if (!drv_datap)
-               status = -EPERM;
-
-       if (!dev_node_obj)
-               status = -EFAULT;
-
-       if (!value)
-               status = -EFAULT;
-
-       dw_buf_size = sizeof(value);
-       if (!status) {
-
-               /* check the device string and then store dev object */
-               if (!
-                   (strcmp
-                    ((char *)((struct drv_ext *)dev_node_obj)->sz_string,
-                     "TIOMAP1510")))
-                       *value = (u32)drv_datap->dev_object;
-       }
-       if (status)
-               pr_err("%s: Failed, status 0x%x\n", __func__, status);
-       return status;
-}
-
 /*
  *  ======== cfg_get_exec_file ========
  *  Purpose: