staging: tidspbridge: remove disp_init() and disp_exit()
authorVíctor Manuel Jáquez Leal <vjaquez@igalia.com>
Fri, 9 Mar 2012 00:03:40 +0000 (01:03 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 9 Mar 2012 21:21:06 +0000 (13:21 -0800)
The disp module has a disp_init() and a disp_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>
drivers/staging/tidspbridge/include/dspbridge/disp.h
drivers/staging/tidspbridge/pmgr/dspapi.c
drivers/staging/tidspbridge/rmgr/disp.c

index 5dfdc8cfb93758ef3022bf7d97f1d4f958df50d8..39d3cea9ca8b9a35430fe323d5920baa0ba32757 100644 (file)
@@ -53,7 +53,6 @@ struct disp_attr {
  *      -ENOMEM:            Insufficient memory for requested resources.
  *      -EPERM:              Unable to create dispatcher.
  *  Requires:
- *      disp_init(void) called.
  *      disp_attrs != NULL.
  *      hdev_obj != NULL.
  *      dispatch_obj != NULL.
@@ -73,38 +72,12 @@ extern int disp_create(struct disp_object **dispatch_obj,
  *      disp_obj:  Node Dispatcher object.
  *  Returns:
  *  Requires:
- *      disp_init(void) called.
  *      Valid disp_obj.
  *  Ensures:
  *      disp_obj is invalid.
  */
 extern void disp_delete(struct disp_object *disp_obj);
 
-/*
- *  ======== disp_exit ========
- *  Discontinue usage of DISP module.
- *
- *  Parameters:
- *  Returns:
- *  Requires:
- *      disp_init(void) previously called.
- *  Ensures:
- *      Any resources acquired in disp_init(void) will be freed when last DISP
- *      client calls disp_exit(void).
- */
-extern void disp_exit(void);
-
-/*
- *  ======== disp_init ========
- *  Initialize the DISP module.
- *
- *  Parameters:
- *  Returns:
- *      TRUE if initialization succeeded, FALSE otherwise.
- *  Ensures:
- */
-extern bool disp_init(void);
-
 /*
  *  ======== disp_node_change_priority ========
  *  Change the priority of a node currently running on the target.
@@ -120,7 +93,6 @@ extern bool disp_init(void);
  *      0:                Success.
  *      -ETIME:           A timeout occurred before the DSP responded.
  *  Requires:
- *      disp_init(void) called.
  *      Valid disp_obj.
  *      hnode != NULL.
  *  Ensures:
@@ -148,7 +120,6 @@ extern int disp_node_change_priority(struct disp_object
  *      -ETIME:   A timeout occurred before the DSP responded.
  *      -EPERM:      A failure occurred, unable to create node.
  *  Requires:
- *      disp_init(void) called.
  *      Valid disp_obj.
  *      pargs != NULL.
  *      hnode != NULL.
@@ -178,7 +149,6 @@ extern int disp_node_create(struct disp_object *disp_obj,
  *      0:        Success.
  *      -ETIME:   A timeout occurred before the DSP responded.
  *  Requires:
- *      disp_init(void) called.
  *      Valid disp_obj.
  *      hnode != NULL.
  *  Ensures:
@@ -204,7 +174,6 @@ extern int disp_node_delete(struct disp_object *disp_obj,
  *      0:        Success.
  *      -ETIME:   A timeout occurred before the DSP responded.
  *  Requires:
- *      disp_init(void) called.
  *      Valid disp_obj.
  *      hnode != NULL.
  *  Ensures:
index a5f1f6514f88da6531abc9bec73f84e7dce8a87e..b1a99f76694cb58c229078a2e25cc0077a0e0b28 100644 (file)
@@ -272,7 +272,6 @@ void api_exit(void)
                msg_exit();
                io_exit();
                strm_exit();
-               disp_exit();
                mgr_exit();
                rmm_exit();
        }
@@ -287,12 +286,11 @@ bool api_init(void)
 {
        bool ret = true;
        bool fdev, fchnl, fmsg, fio;
-       bool fmgr, fdisp, fstrm, frmm;
+       bool fmgr, fstrm, frmm;
 
        if (api_c_refs == 0) {
                /* initialize driver and other modules */
                fmgr = mgr_init();
-               fdisp = disp_init();
                fstrm = strm_init();
                frmm = rmm_init();
                fchnl = chnl_init();
@@ -309,9 +307,6 @@ bool api_init(void)
                        if (fstrm)
                                strm_exit();
 
-                       if (fdisp)
-                               disp_exit();
-
                        if (fchnl)
                                chnl_exit();
 
index e510bb25110be05665fc62980bdfbe229ee32969..4af51b75aeab6f12d057b5b0f63415584122b785 100644 (file)
@@ -69,8 +69,6 @@ struct disp_object {
        u32 data_mau_size;      /* Size of DSP Data MAU */
 };
 
-static u32 refs;
-
 static void delete_disp(struct disp_object *disp_obj);
 static int fill_stream_def(rms_word *pdw_buf, u32 *ptotal, u32 offset,
                                  struct node_strmdef strm_def, u32 max,
@@ -172,29 +170,6 @@ void disp_delete(struct disp_object *disp_obj)
        delete_disp(disp_obj);
 }
 
-/*
- *  ======== disp_exit ========
- *  Discontinue usage of DISP module.
- */
-void disp_exit(void)
-{
-       refs--;
-}
-
-/*
- *  ======== disp_init ========
- *  Initialize the DISP module.
- */
-bool disp_init(void)
-{
-       bool ret = true;
-
-       if (ret)
-               refs++;
-
-       return ret;
-}
-
 /*
  *  ======== disp_node_change_priority ========
  *  Change the priority of a node currently running on the target.