diff options
author | Víctor Manuel Jáquez Leal | 2012-03-09 01:03:48 +0100 |
---|---|---|
committer | Greg Kroah-Hartman | 2012-03-09 22:21:08 +0100 |
commit | a05c5dc3fc73c11fbb06c493b1ae6d77c4ac172e (patch) | |
tree | 9b9676e417cf8b76c154169b6b1007464964aec5 /drivers/staging/tidspbridge | |
parent | staging: tidspbridge: remove dmm_init() and dmm_exit() (diff) | |
download | kernel-qcow2-linux-a05c5dc3fc73c11fbb06c493b1ae6d77c4ac172e.tar.gz kernel-qcow2-linux-a05c5dc3fc73c11fbb06c493b1ae6d77c4ac172e.tar.xz kernel-qcow2-linux-a05c5dc3fc73c11fbb06c493b1ae6d77c4ac172e.zip |
staging: tidspbridge: remove dev_init() and dev_exit()
The dev module has a dev_init() and a dev_exit() whose only purpose is
to keep a reference counting which is not used at all.
This patch removes these functions and the reference count variable.
There is no functional changes.
Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/tidspbridge')
-rw-r--r-- | drivers/staging/tidspbridge/include/dspbridge/dev.h | 27 | ||||
-rw-r--r-- | drivers/staging/tidspbridge/pmgr/dev.c | 29 | ||||
-rw-r--r-- | drivers/staging/tidspbridge/pmgr/dspapi.c | 22 |
3 files changed, 3 insertions, 75 deletions
diff --git a/drivers/staging/tidspbridge/include/dspbridge/dev.h b/drivers/staging/tidspbridge/include/dspbridge/dev.h index f92b4be0b413..fa2d79ef6cc8 100644 --- a/drivers/staging/tidspbridge/include/dspbridge/dev.h +++ b/drivers/staging/tidspbridge/include/dspbridge/dev.h @@ -478,33 +478,6 @@ extern int dev_get_bridge_context(struct dev_object *hdev_obj, **phbridge_context); /* - * ======== dev_exit ======== - * Purpose: - * Decrement reference count, and free resources when reference count is - * 0. - * Parameters: - * Returns: - * Requires: - * DEV is initialized. - * Ensures: - * When reference count == 0, DEV's private resources are freed. - */ -extern void dev_exit(void); - -/* - * ======== dev_init ======== - * Purpose: - * Initialize DEV's private state, keeping a reference count on each call. - * Parameters: - * Returns: - * TRUE if initialized; FALSE if error occurred. - * Requires: - * Ensures: - * TRUE: A requirement for the other public DEV functions. - */ -extern bool dev_init(void); - -/* * ======== dev_insert_proc_object ======== * Purpose: * Inserts the Processor Object into the List of PROC Objects diff --git a/drivers/staging/tidspbridge/pmgr/dev.c b/drivers/staging/tidspbridge/pmgr/dev.c index c5e627ce8b8f..6234ffb5e8a3 100644 --- a/drivers/staging/tidspbridge/pmgr/dev.c +++ b/drivers/staging/tidspbridge/pmgr/dev.c @@ -81,9 +81,6 @@ struct drv_ext { char sz_string[MAXREGPATHLENGTH]; }; -/* ----------------------------------- Globals */ -static u32 refs; /* Module reference count */ - /* ----------------------------------- Function Prototypes */ static int fxn_not_implemented(int arg, ...); static int init_cod_mgr(struct dev_object *dev_obj); @@ -649,32 +646,6 @@ int dev_get_bridge_context(struct dev_object *hdev_obj, } /* - * ======== dev_exit ======== - * Purpose: - * Decrement reference count, and free resources when reference count is - * 0. - */ -void dev_exit(void) -{ - refs--; -} - -/* - * ======== dev_init ======== - * Purpose: - * Initialize DEV's private state, keeping a reference count on each call. - */ -bool dev_init(void) -{ - bool ret = true; - - if (ret) - refs++; - - return ret; -} - -/* * ======== dev_notify_clients ======== * Purpose: * Notify all clients of this device of a change in device status. diff --git a/drivers/staging/tidspbridge/pmgr/dspapi.c b/drivers/staging/tidspbridge/pmgr/dspapi.c index b9ca24c18e13..9ef1ad9527af 100644 --- a/drivers/staging/tidspbridge/pmgr/dspapi.c +++ b/drivers/staging/tidspbridge/pmgr/dspapi.c @@ -265,11 +265,8 @@ void api_exit(void) { api_c_refs--; - if (api_c_refs == 0) { - /* Release all modules initialized in api_init(). */ - dev_exit(); + if (api_c_refs == 0) mgr_exit(); - } } /* @@ -280,23 +277,10 @@ void api_exit(void) bool api_init(void) { bool ret = true; - bool fdev; - bool fmgr; - - if (api_c_refs == 0) { - /* initialize driver and other modules */ - fmgr = mgr_init(); - fdev = dev_init(); - ret = fdev && fmgr; - if (!ret) { - if (fmgr) - mgr_exit(); + if (api_c_refs == 0) + ret = mgr_init(); - if (fdev) - dev_exit(); - } - } if (ret) api_c_refs++; |