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 erro...@gmail.com 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, On Tue, Nov 23, 2010 at 8:34 AM, James Adams james.r.ad...@gmail.com 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

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. 1)

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 james.r.ad...@gmail.com 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.

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 james.r.ad...@gmail.com 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

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 ionut.n...@mindbit.ro 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 ionut.n...@gmail.com wrote: Convert the core module of the tidspbridge driver to use

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 ionut.n...@mindbit.ro 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 ionut.n...@gmail.com wrote: Convert the core module of the tidspbridge driver to use

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 ionut.n...@mindbit.ro 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 ionut.n...@gmail.com wrote: Convert the pmgr module of the tidspbridge driver to use

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 ionut.n...@mindbit.ro 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 ionut.n...@gmail.com wrote: Convert the rmgr module of the tidspbridge driver to use

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 ionut.n...@gmail.com wrote: Convert the core module of the tidspbridge driver to use struct list_head instead of struct lst_list. snip        if (!status) {                /* Get a free chirp: */ -               chnl_packet_obj = -    

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 ionut.n...@gmail.com wrote: Convert the core module of the tidspbridge driver to use struct list_head instead of struct lst_list. Signed-off-by: Ionut Nicu ionut.n...@mindbit.ro snip diff --git a/drivers/staging/tidspbridge/core/io_sm.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 ionut.n...@gmail.com wrote: Convert the pmgr module of the tidspbridge driver to use struct list_head instead of struct lst_list. snip + * Memory is coalesced back to the appropriate heap when a buffer is What is being fixed here?  *

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 ionut.n...@gmail.com wrote: Convert the rmgr module of the tidspbridge driver to use struct list_head instead of struct lst_list. Signed-off-by: Ionut Nicu ionut.n...@mindbit.ro snip diff --git a/drivers/staging/tidspbridge/rmgr/drv.c

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 rene.sapi...@ti.com 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

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: [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: [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(mq-lock); +    

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 To:

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...@lists.infradead.org; Kanigeri

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