OMAP5 VirtQueue: Removed VirtQueue_postInitDone().

Looking at the Linux side trace, we see this message:

  omap-rproc omap-rproc.0: Dropping unknown message ffffff07

proving the Linux driver is not even processing this message.

Per Suman:
"This call was added as part of optimizing the boot-time for the Ducati image,
where in we starting loading at OPP100, and this message is used to trigger
releasing the OPP100 constraint.

You can remove it or keep it, the rproc message is discarding it anyway."

So, it was removed.

Signed-off-by: G Anthony <a0783926@ti.com>
Signed-off-by: Chris Ring <cring@ti.com>

diff --git a/packages/ti/ipc/family/omap54xx/VirtQueue.c b/packages/ti/ipc/family/omap54xx/VirtQueue.c
index 778d368cdc3287d59b38abb3bd9d16b633568c58..8c8df1c1f97aa5946d049317c44e51d264ef28f7 100644 (file)
--- a/packages/ti/ipc/family/omap54xx/VirtQueue.c
+++ b/packages/ti/ipc/family/omap54xx/VirtQueue.c
@@ -568,14 +568,6 @@ Void VirtQueue_postCrashToMailbox(Void)
     InterruptProxy_intSend(0, (UInt)RP_MSG_MBOX_CRASH);
 }
 
-/*!
- * ======== VirtQueue_postInitDone ========
- */
-Void VirtQueue_postInitDone(Void)
-{
-    InterruptProxy_intSend(0, (UInt)RP_MSG_BOOTINIT_DONE);
-}
-
 #define CACHE_WB_TICK_PERIOD    5
 
 /*!

diff --git a/packages/ti/ipc/family/omap54xx/VirtQueue.h b/packages/ti/ipc/family/omap54xx/VirtQueue.h
index c14f0fe7417baf5db37f84c74ace43ca3da49306..5ee14394dc247bbe245abecbc15718921ae28924 100644 (file)
--- a/packages/ti/ipc/family/omap54xx/VirtQueue.h
+++ b/packages/ti/ipc/family/omap54xx/VirtQueue.h
@@ -229,12 +229,6 @@ Int VirtQueue_addUsedBuf(VirtQueue_Handle vq, Int16 token, Int len);
  */
 Void VirtQueue_postCrashToMailbox(Void);
 
-/*!
- *  @brief      Post a init done message to host
- */
-Void VirtQueue_postInitDone(Void);
-
-
 #if defined (__cplusplus)
 }
 #endif /* defined (__cplusplus) */

diff --git a/packages/ti/srvmgr/omx/OmxSrvMgr.c b/packages/ti/srvmgr/omx/OmxSrvMgr.c
index 7202a701f3b5741c494043fe87b0c91b716123d2..3b115a1792bc079d16a511dc5266e87f8842aaf4 100644 (file)
--- a/packages/ti/srvmgr/omx/OmxSrvMgr.c
+++ b/packages/ti/srvmgr/omx/OmxSrvMgr.c
@@ -47,8 +47,6 @@
 #include <ti/ipc/MultiProc.h>
 
 #include <ti/ipc/rpmsg/MessageQCopy.h>
-/* TBD: ONLY WORKS FORM OMAP5 FOR NOW! */
-#include <ti/ipc/family/omap54xx/VirtQueue.h>
 #include <ti/ipc/rpmsg/NameMap.h>
 #include <ti/srvmgr/rpmsg_omx.h>
 #include <ti/srvmgr/ServiceMgr.h>
@@ -85,9 +83,6 @@ Void OmxSrvMgr_taskFxn(UArg arg0, UArg arg1)
 
 #ifdef SMP
     NameMap_register("rpmsg-omx1", OMX_MGR_PORT);
-    System_printf("OmxSrvMgr: Proc#%d sending BOOTINIT_DONE\n",
-                        MultiProc_self());
-    VirtQueue_postInitDone();
 #else
     if (MultiProc_self() == MultiProc_getId("CORE0")) {
         NameMap_register("rpmsg-omx0", OMX_MGR_PORT);
@@ -98,13 +93,6 @@ Void OmxSrvMgr_taskFxn(UArg arg0, UArg arg1)
     if (MultiProc_self() == MultiProc_getId("DSP")) {
         NameMap_register("rpmsg-omx2", OMX_MGR_PORT);
     }
-
-    if ((MultiProc_self() == MultiProc_getId("CORE1")) ||
-        (MultiProc_self() == MultiProc_getId("DSP"))) {
-        System_printf("OmxSrvMgr: Proc#%d sending BOOTINIT_DONE\n",
-                        MultiProc_self());
-        VirtQueue_postInitDone();
-    }
 #endif
 
     while (1) {

diff --git a/packages/ti/srvmgr/omx/package.xs b/packages/ti/srvmgr/omx/package.xs
index e65a7d90b6fb02d87bba4766489dc20075c820e3..dfb9b38b44ae8dabad7540c98b0604c3d482a6a2 100644 (file)
--- a/packages/ti/srvmgr/omx/package.xs
+++ b/packages/ti/srvmgr/omx/package.xs
@@ -78,9 +78,3 @@ function getLibs(prog)
 
     return libAry.join(";");
 }
-
-function close()
-{
-    /* TBD: just like the .c file, this only works for OMAP5 right now */
-    xdc.loadPackage('ti.ipc.family.omap54xx');
-}
\ No newline at end of file