On Tuesday 15 September 2009 04:36:18 yang shaobo wrote:
> Dear caglar :
> Thanks a lot for your patch !
> Now the new dsplink works perfectly with my gstreamer.
>
> According FAQ on DSPLink wiki, the other thing that should be done may
> be adding the dsplink cleanup functions to gstreamer core .
> I will try this .
>

I use something like this.

Regards,
Caglar

--- 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiauddec1.c
    2009-06-15 22:27:27.000000000 +0300
+++ 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiauddec1_mine.c
       2009-07-06 16:13:23.000000000 +0300
@@ -1059,6 +1059,9 @@
             if (!gst_tiauddec1_exit_audio(auddec1)) {
                 return GST_STATE_CHANGE_FAILURE;
             }
+            if (gst_tiauddec1_codec_stop(auddec1) < 0) {
+                GST_ERROR("failed to stop codec\n");
+            }
             break;
 
         case GST_STATE_CHANGE_READY_TO_PAUSED:
@@ -1362,11 +1365,6 @@
         gst_ticircbuffer_data_consumed(auddec1->circBuf, encDataWindow, 0);
     }
 
-    /* Initialize codec engine */
-    if (gst_tiauddec1_codec_stop(auddec1) < 0) {
-        GST_ERROR("failed to stop codec\n");
-    }
-
     /* Notify main thread if it is waiting on decode thread shut-down */
     auddec1->decodeDrained = TRUE;
     Rendezvous_force(auddec1->waitOnDecodeDrain);
--- 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiauddec.c 
    2009-06-15 22:27:27.000000000 +0300
+++ 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiauddec_mine.c
        2009-07-06 16:15:26.000000000 +0300
@@ -1054,6 +1054,10 @@
             if (!gst_tiauddec_exit_audio(auddec)) {
                 return GST_STATE_CHANGE_FAILURE;
             }
+            /* Stop codec engine */
+            if (gst_tiauddec_codec_stop(auddec) < 0) {
+                GST_ERROR("failed to stop codec\n");
+            }
             break;
 
         case GST_STATE_CHANGE_READY_TO_PAUSED:
@@ -1360,12 +1364,6 @@
         gst_ticircbuffer_data_consumed(auddec->circBuf, encDataWindow, 0);
     }
 
-     /* Initialize codec engine */
-    /* Stop codec engine */
-    if (gst_tiauddec_codec_stop(auddec) < 0) {
-        GST_ERROR("failed to stop codec\n");
-    }
-
     /* Notify main thread if it is waiting on decode thread shut-down */
     auddec->decodeDrained = TRUE;
     Rendezvous_force(auddec->waitOnDecodeDrain);
--- 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiimgdec1.c
    2009-06-15 22:27:27.000000000 +0300
+++ 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiimgdec1_mine.c
       2009-07-06 16:17:11.000000000 +0300
@@ -1216,6 +1216,10 @@
             if (!gst_tiimgdec1_exit_image(imgdec1)) {
                 return GST_STATE_CHANGE_FAILURE;
             }
+            /* Stop codec engine */
+            if (gst_tiimgdec1_codec_stop(imgdec1) < 0) {
+                GST_ERROR("failed to stop codec\n");
+            }
             break;
 
         default:
@@ -1539,11 +1543,6 @@
         gst_ticircbuffer_data_consumed(imgdec1->circBuf, encDataWindow, 0);
     }
 
-    /* Stop codec engine */
-    if (gst_tiimgdec1_codec_stop(imgdec1) < 0) {
-        GST_ERROR("failed to stop codec\n");
-    }
-
     imgdec1->decodeDrained = TRUE;
     Rendezvous_force(imgdec1->waitOnDecodeDrain);
 
--- 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiimgdec.c 
    2009-06-15 22:27:27.000000000 +0300
+++ 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiimgdec_mine.c
        2009-07-06 16:20:09.000000000 +0300
@@ -1216,6 +1216,10 @@
             if (!gst_tiimgdec_exit_image(imgdec)) {
                 return GST_STATE_CHANGE_FAILURE;
             }
+            /* Stop codec engine */
+            if (gst_tiimgdec_codec_stop(imgdec) < 0) {
+                GST_ERROR("failed to stop codec\n");
+            }
             break;
 
         default:
@@ -1538,11 +1542,6 @@
         gst_ticircbuffer_data_consumed(imgdec->circBuf, encDataWindow, 0);
     }
 
-    /* Stop codec engine */
-    if (gst_tiimgdec_codec_stop(imgdec) < 0) {
-        GST_ERROR("failed to stop codec\n");
-    }
-
     imgdec->decodeDrained = TRUE;
     Rendezvous_force(imgdec->waitOnDecodeDrain);
 
--- 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiimgenc1.c
    2009-06-15 22:27:26.000000000 +0300
+++ 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiimgenc1_mine.c
       2009-07-06 16:21:36.000000000 +0300
@@ -1519,6 +1519,10 @@
             if (!gst_tiimgenc1_exit_image(imgenc1)) {
                 return GST_STATE_CHANGE_FAILURE;
             }
+            /* Stop codec engine */
+            if (gst_tiimgenc1_codec_stop(imgenc1) < 0) {
+                GST_ERROR("failed to stop codec\n");
+            }
             break;
 
         default:
@@ -1848,11 +1852,6 @@
         gst_ticircbuffer_data_consumed(imgenc1->circBuf, encDataWindow, 0);
     }
 
-    /* Stop codec engine */
-    if (gst_tiimgenc1_codec_stop(imgenc1) < 0) {
-        GST_ERROR("failed to stop codec\n");
-    }
-
     imgenc1->encodeDrained = TRUE;
     Rendezvous_force(imgenc1->waitOnEncodeDrain);
 
--- 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiimgenc.c 
    2009-06-15 22:27:27.000000000 +0300
+++ 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttiimgenc_mine.c
        2009-07-06 16:22:59.000000000 +0300
@@ -1518,6 +1518,10 @@
             if (!gst_tiimgenc_exit_image(imgenc)) {
                 return GST_STATE_CHANGE_FAILURE;
             }
+            /* Stop codec engine */
+            if (gst_tiimgenc_codec_stop(imgenc) < 0) {
+                GST_ERROR("failed to stop codec\n");
+            }
             break;
 
         default:
@@ -1847,11 +1851,6 @@
         gst_ticircbuffer_data_consumed(imgenc->circBuf, encDataWindow, 0);
     }
 
-    /* Stop codec engine */
-    if (gst_tiimgenc_codec_stop(imgenc) < 0) {
-        GST_ERROR("failed to stop codec\n");
-    }
-
     imgenc->encodeDrained = TRUE;
     Rendezvous_force(imgenc->waitOnEncodeDrain);
 
--- 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttividdec2.c
    2009-06-15 22:27:27.000000000 +0300
+++ 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttividdec2_mine.c
       2009-07-06 16:24:54.000000000 +0300
@@ -1112,6 +1112,10 @@
             if (!gst_tividdec2_exit_video(viddec2)) {
                 return GST_STATE_CHANGE_FAILURE;
             }
+            /* stop codec engine */
+            if (gst_tividdec2_codec_stop(viddec2) < 0) {
+                GST_ERROR("failed to stop codec\n");
+            }
             break;
 
         case GST_STATE_CHANGE_READY_TO_PAUSED:
@@ -1496,11 +1500,6 @@
         gst_ticircbuffer_data_consumed(viddec2->circBuf, encDataWindow, 0);
     }
 
-    /* stop codec engine */
-    if (gst_tividdec2_codec_stop(viddec2) < 0) {
-        GST_ERROR("failed to stop codec\n");
-    }
-
     /* Notify main thread if it is waiting on decode thread shut-down */
     viddec2->decodeDrained = TRUE;
     Rendezvous_force(viddec2->waitOnDecodeDrain);
--- 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttividdec.c 
    2009-06-15 22:27:27.000000000 +0300
+++ 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttividdec_mine.c
        2009-07-06 16:27:06.000000000 +0300
@@ -1102,6 +1102,10 @@
             if (!gst_tividdec_exit_video(viddec)) {
                 return GST_STATE_CHANGE_FAILURE;
             }
+            /* stop codec engine */
+            if (gst_tividdec_codec_stop(viddec) < 0) {
+                GST_ERROR("failed to stop codec\n");
+            }
             break;
 
         case GST_STATE_CHANGE_READY_TO_PAUSED:
@@ -1437,11 +1441,6 @@
         gst_ticircbuffer_data_consumed(viddec->circBuf, encDataWindow, 0);
     }
 
-    /* stop codec engine */
-    if (gst_tividdec_codec_stop(viddec) < 0) {
-        GST_ERROR("failed to stop codec\n");
-    }
-
     /* Notify main thread if it is waiting on decode thread shut-down */
     viddec->decodeDrained = TRUE;
     Rendezvous_force(viddec->waitOnDecodeDrain);
--- 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttividenc1.c
    2009-06-15 22:27:27.000000000 +0300
+++ 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttividenc1_mine.c
       2009-07-06 16:28:25.000000000 +0300
@@ -1077,6 +1077,10 @@
             if (!gst_tividenc1_exit_video(videnc1)) {
                 return GST_STATE_CHANGE_FAILURE;
             }
+            /* Stop codec engine */
+            if (gst_tividenc1_codec_stop(videnc1) < 0) {
+                GST_ERROR("failed to stop codec\n");
+            }
             break;
 
         default:
@@ -1532,11 +1536,6 @@
         Ccv_delete(hCcv);
     }
 
-    /* Stop codec engine */
-    if (gst_tividenc1_codec_stop(videnc1) < 0) {
-        GST_ERROR("failed to stop codec\n");
-    }
-
     videnc1->encodeDrained = TRUE;
     Rendezvous_force(videnc1->waitOnEncodeDrain);
     Rendezvous_force(videnc1->waitOnQueueThread);
--- 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttividenc.c 
    2009-06-15 22:27:27.000000000 +0300
+++ 
gstreamer-ti-0+svnr225-r0/gstreamer_ti/ti_build/ticodecplugin/src/gsttividenc_mine.c
        2009-07-06 16:29:48.000000000 +0300
@@ -1022,6 +1022,10 @@
             if (!gst_tividenc_exit_video(videnc)) {
                 return GST_STATE_CHANGE_FAILURE;
             }
+            /* Stop codec engine */
+            if (gst_tividenc_codec_stop(videnc) < 0) {
+                GST_ERROR("failed to stop codec\n");
+            }
             break;
 
         default:
@@ -1376,11 +1380,6 @@
         gst_ticircbuffer_data_consumed(videnc->circBuf, encDataWindow, 0);
     }
 
-    /* Stop codec engine */
-    if (gst_tividenc_codec_stop(videnc) < 0) {
-        GST_ERROR("failed to stop codec\n");
-    }
-
     /* Notify main thread if it is waiting on decode thread shut-down */
     videnc->encodeDrained = TRUE;
     Rendezvous_force(videnc->waitOnQueueThread);

> 2009/9/14, Caglar Akyuz <[email protected]>:
> > On Monday 14 September 2009 13:09:28 yang shaobo wrote:
> >> Dear Uppal :
> >> Thanks for your quick reply.
> >> I modified the $(DSPLINK)/config/all/CFG_Linux.c.
> >> I changed the Handle signals for cleanup from "TRUE" to "FALSE".
> >> And , changed NUMSIGNALS from "8u" to "0u".
> >> Then , I recompiled the dsplink , cmem ,dmai , codec_combos, also
> >> gstreamer ( I use gstreamer as my application ).
> >> However , the gstreamer still can not receive signals.
> >> It seemed that nothing is changed.
> >> So, I am confused .
> >
> > I guess you need to change your codec engine configuration. AFAIK,
> >  $(DSPLINK)/config/all/CFG_Linux.c file is not used while using codec
> > engine,
> > instead CE provides its own configuration. It was far easier for me to
> > modify
> > dsplink code for this. Please see the attached patch.
> >
> > I hope this helps,
> > Caglar
> >
> > ---
> > ti-codec-engine-221-r0/codec_engine_2_21/cetools/packages/dsplink/gpp/src
> >/api/Linux/drv_api.c 2008-10-24 22:58:03.000000000 +0300
> > +++
> > ti-codec-engine-221-r0/codec_engine_2_21/cetools/packages/dsplink/gpp/src
> >/api/Linux/drv_api_mine.c    2009-07-06 15:45:50.000000000 +0300
> > @@ -477,7 +477,8 @@
> >          status = DSP_EPOINTER ;
> >          SET_FAILURE_REASON ;
> >      }
> > -
> > +
> > +    DSPLINK_atExitHandler();
> >      TRC_1LEAVE ("DRV_Finalize", status) ;
> >
> >      return status ;
> > @@ -1606,7 +1607,7 @@
> >
> >      TRC_1ENTER ("DRV_installCleanupRoutines", linkCfgPtr) ;
> >
> > -    if (linkCfgPtr->gppObject->gppOsObject->handleSignals == TRUE) {
> > +    if (0 && linkCfgPtr->gppObject->gppOsObject->handleSignals == TRUE)
> > { sigemptyset (&block_mask) ;
> >          sigfillset( &block_mask) ;
> >          sigInstaller.sa_flags   = (SA_RESTART) ;
> > @@ -1619,7 +1620,6 @@
> >                         NULL) ;
> >          }
> >      }
> > -    atexit (&DSPLINK_atExitHandler) ;
> >
> >      TRC_0LEAVE ("DRV_installCleanupRoutines") ;
> >  }



_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to