Re: [PATCH 05/12] staging:tidspbridge - set5 remove hungarian from structs

2010-12-23 Thread Sapiens, Rene
Hi Dan, On Wed, Dec 15, 2010 at 2:53 AM, Dan Carpenter wrote: > On Tue, Dec 14, 2010 at 02:33:11PM -0600, Rene Sapiens wrote: >> -                                 pfn_unload(pnode->nldr_node_obj, >> +                                 load(pnode->nldr_node_obj, >>                                  

Re: DSP Bridge video decode of above VGA videos

2010-11-23 Thread Sapiens, Rene
Hi James, > The second error log is the stack trace when the watchdog timer calls > SYNC_EnterCS. Actually is not the WDT calling the function to get into the critical section, it is the WDT being disabled when the DSP is going to OFF state. This should fine. > The stack trace is shown below. >

Re: DSP Bridge video decode of above VGA videos

2010-11-23 Thread Sapiens, Rene
Hi James, On Tue, Nov 23, 2010 at 8:34 AM, James Adams wrote: > A bit more information about why videos are not playing in a port of > Zoom2 to gumstix: > > 2) > If I use the OpenCORE unit tests I can successfully decode: >   VGA movies with software codec >   1280*544 movie with software codec >

Re: DSP Bridge video decode of above VGA videos

2010-11-22 Thread Sapiens, Rene
James, On Mon, Nov 22, 2010 at 11:12 AM, James Adams wrote: > Hi Rene, > > We have got the watchdog timer enabled at the moment (interval 5 seconds). > The code does indeed point to the line as you say. > I tried using dump_stack and it was getting triggered by an interrupt > calling handle_hiber

Re: DSP Bridge video decode of above VGA videos

2010-11-22 Thread Sapiens, Rene
Hi James, On Mon, Nov 22, 2010 at 9:26 AM, James Adams wrote: > D/TI_Video_Decoder(  934): VIDDEC_InitDSP_H264Dec():6573 After Rounding: > nFrameW > idth = 1280, nFrameHeight = 544 > drivers/dsp/bridge/services/sync.c, line 357: Assertion (0) failed. > drivers/dsp/bridge/services/sync.c, line 357

Re: [PATCH v2 08/12] staging: tidspbridge: convert rmgr to list_head

2010-11-08 Thread Sapiens, Rene
Hi Ionut, On Sat, Nov 6, 2010 at 12:18 PM, Ionut Nicu wrote: > Hi Rene, > > On Fri, 2010-11-05 at 18:07 -0600, Sapiens, Rene wrote: >> Hi Ionut, >> >> On Fri, Nov 5, 2010 at 9:13 AM, Ionut Nicu wrote: >> > Convert the rmgr module of the tidspbridge driver >

Re: [PATCH v2 07/12] staging: tidspbridge: convert pmgr to list_head

2010-11-08 Thread Sapiens, Rene
Hi Ionut, On Sun, Nov 7, 2010 at 7:39 AM, Ionut Nicu wrote: > Hi Rene, > > On Fri, 2010-11-05 at 16:41 -0600, Sapiens, Rene wrote: >> Hi Ionut, >> >> On Fri, Nov 5, 2010 at 9:13 AM, Ionut Nicu wrote: >> > Convert the pmgr module of the tidspbridge driver >

Re: [PATCH v2 06/12] staging: tidspbridge: convert core to list_head

2010-11-08 Thread Sapiens, Rene
Hi Ionut, On Sat, Nov 6, 2010 at 11:31 AM, Ionut Nicu wrote: > Hi Rene, > > On Fri, 2010-11-05 at 16:12 -0600, Sapiens, Rene wrote: >> Hi Ionut, >> >> On Fri, Nov 5, 2010 at 9:13 AM, Ionut Nicu wrote: >> > Convert the core module of the tidspbridge driver >

Re: [PATCH v2 06/12] staging: tidspbridge: convert core to list_head

2010-11-08 Thread Sapiens, Rene
Hi Ionut, On Sat, Nov 6, 2010 at 11:21 AM, Ionut Nicu wrote: > Hi Rene, > > On Fri, 2010-11-05 at 15:07 -0600, Sapiens, Rene wrote: >> Hi Ionut, >> >> On Fri, Nov 5, 2010 at 9:13 AM, Ionut Nicu wrote: >> > Convert the core module of the tidspbridge driver >

Re: [PATCH v2 08/12] staging: tidspbridge: convert rmgr to list_head

2010-11-05 Thread Sapiens, Rene
Hi Ionut, On Fri, Nov 5, 2010 at 9:13 AM, Ionut Nicu wrote: > Convert the rmgr module of the tidspbridge driver > to use struct list_head instead of struct lst_list. > > Signed-off-by: Ionut Nicu > diff --git a/drivers/staging/tidspbridge/rmgr/drv.c > b/drivers/staging/tidspbridge/rmgr/drv.c

Re: [PATCH v2 07/12] staging: tidspbridge: convert pmgr to list_head

2010-11-05 Thread Sapiens, Rene
Hi Ionut, On Fri, Nov 5, 2010 at 9:13 AM, Ionut Nicu wrote: > Convert the pmgr module of the tidspbridge driver > to use struct list_head instead of struct lst_list. > + * Memory is coalesced back to the appropriate heap when a buffer is What is being fixed here? >  * freed. >  * >  * Notes:

Re: [PATCH v2 06/12] staging: tidspbridge: convert core to list_head

2010-11-05 Thread Sapiens, Rene
Hi Ionut, On Fri, Nov 5, 2010 at 9:13 AM, Ionut Nicu wrote: > Convert the core module of the tidspbridge driver > to use struct list_head instead of struct lst_list. > > Signed-off-by: Ionut Nicu > diff --git a/drivers/staging/tidspbridge/core/io_sm.c > b/drivers/staging/tidspbridge/core/io_

Re: [PATCH v2 06/12] staging: tidspbridge: convert core to list_head

2010-11-05 Thread Sapiens, Rene
Hi Ionut, On Fri, Nov 5, 2010 at 9:13 AM, Ionut Nicu wrote: > Convert the core module of the tidspbridge driver > to use struct list_head instead of struct lst_list. > >        if (!status) { >                /* Get a free chirp: */ > -               chnl_packet_obj = > -                   (st

Re: [PATCH 1/8] staging: tidspbridge: overwrite DSP error codes

2010-11-03 Thread Sapiens, Rene
On Wed, Nov 3, 2010 at 5:36 PM, Rene Sapiens wrote: > When calling the DSP's remote functions, the DSP returns error > codes different from the ones managed by the kernel, the > function's return value is shared with the MPU using a shared > structure. This patch overwrites those error codes by ke

RE: [PATCH 3/4] staging: tidspbridge: remove gb bitmap implementation

2010-10-23 Thread Sapiens, Rene
Hi Ionut, On Friday, October 22, 2010 9:09 AM Ionut Nicu wrote: > Most likely I will have to break patch 4/4 from this series (which I > also believe is way too big) into multiple patches and re-submit. > > I'm expecting some advices from the maintainers/list on how to split > patch 4 (lst_list re

RE: [PATCH 4/7] omap:mailbox-send message in process context

2010-10-21 Thread Sapiens, Rene
Hari, On Thursday, October 21, 2010 3:49 PM Hari Kanigeri wrote: > Rene, > > Thanks for your comment. > > >>> @@ -92,20 +92,25 @@ int omap_mbox_msg_send(struct omap_mbox *mbox, >>>       mbox_msg_t msg) struct omap_mbox_queue *mq = mbox->txq; >>>       int ret = 0, len; >>> >>> -     spin_lock

RE: [PATCH 4/7] omap:mailbox-send message in process context

2010-10-21 Thread Sapiens, Rene
Hi Hari, On Thursday, October 14, 2010 9:13 PM Kanigeri, Hari wrote: > Schedule the Tasklet to send only when mailbox fifo is full, else > send the message in the Process context. This would avoid > needless scheduling of Tasklet for every message transfer > > Signed-off-by: Hari Kanigeri > ---

RE: [PATCHv2] mailbox: change full flag per mailbox queue instead of global

2010-08-19 Thread Sapiens, Rene
Hi Ohad, This patch already contains the missed changes due to previous rebases. Thanks. Regards, Rene > -Original Message- > From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap- > ow...@vger.kernel.org] On Behalf Of Guzman Lugo, Fernando > Sent: Tuesday, August 10, 2010 8:13 PM >

RE: [PATCH 09/10] omap: mailbox: convert block api to kfifo

2010-08-10 Thread Sapiens, Rene
Hi Ohad, Sure I will do it. Regards, Rene > -Original Message- > From: Ohad Ben-Cohen [mailto:o...@wizery.com] > Sent: Tuesday, August 10, 2010 9:43 AM > To: Guzman Lugo, Fernando; Sapiens, Rene > Cc: Hiroshi DOYU; linux-omap@vger.kernel.org; linux-arm- > ker...@l

RE: Serial console not working after waking up from sleep

2010-06-16 Thread Sapiens, Rene
You can do a telnet to the device... you should be able to work with it but your serial session will show the garbage. Regards, Rene > -Original Message- > From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap- > ow...@vger.kernel.org] On Behalf Of Michael Trimarchi > Sent: Wednesday

RE: [PATCH v3 4/4] omap: mailbox: convert block api to kfifo

2010-06-14 Thread Sapiens, Rene
Hi Hiroshi > -Original Message- > From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap- > ow...@vger.kernel.org] On Behalf Of Hiroshi DOYU > Sent: Monday, June 14, 2010 3:58 AM > To: Guzman Lugo, Fernando; o...@wizery.com > Cc: Chitriki Rudramuni, Deepak; Ramirez Luna, Omar; Kanigeri,

RE: [PATCH 09/10] omap: mailbox: convert block api to kfifo

2010-06-08 Thread Sapiens, Rene
Hi Ohad, In mbox_rx_work() you are removing the lines that enable back the mbox irq for the RX case, but inside __mbox_rx_interrupt() this interrupt is disabled in the case that the kfifo for Rx mailbox gets full. So I think that we need to enable it back as soon as there is space in this kf