Re: [PATCH v2 00/15] tidspbridge driver MMU-related cleanups

2012-09-21 Thread Ramirez Luna, Omar
Hi Laurent, On Wed, Sep 19, 2012 at 7:06 AM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hello, Here's the second version of my tidspbridge MMU-related cleanup patches. The first version has been sent privately only, don't try to search the mailing list archive for it :-)

Re: [PATCH v2 4/8] ARM: OMAP2+: hwmod: revise hardreset behavior

2012-04-19 Thread Ramirez Luna, Omar
Hi Paul, On Thu, Apr 19, 2012 at 2:46 PM, Paul Walmsley p...@pwsan.com wrote: Hi Benoît, On Thu, 19 Apr 2012, Cousson, Benoit wrote: On 4/19/2012 7:17 PM, Paul Walmsley wrote: On Thu, 19 Apr 2012, Cousson, Benoit wrote: The concern of the people doing SW in these embedded processors

Re: [PATCH v3 4/4] ARM: omap: pass minimal SoC/board data for UART from dt

2012-04-11 Thread Ramirez Luna, Omar
On Wed, Apr 11, 2012 at 3:39 AM, Cousson, Benoit b-cous...@ti.com wrote: Yes, it is a known issue and the fix was unfortunately was not merged during -rc phases but is fixed in 3.4 and should be fixed in 3.3 stable branch as well. I received the notification from Greg KH 2 weeks ago about the

Re: [PATCH v3 4/4] ARM: omap: pass minimal SoC/board data for UART from dt

2012-04-10 Thread Ramirez Luna, Omar
Hi, On Wed, Dec 14, 2011 at 5:55 AM, Rajendra Nayak rna...@ti.com wrote: Pass minimal data needed for console boot, from dt, for OMAP4 panda/sdp and OMAP3 beagle boards, and get rid of the static initialization from generic board file. ... diff --git a/arch/arm/mach-omap2/board-generic.c

Re: [PATCH v2 5/8] ARM: OMAP2+: hwmod: ensure that SYSCONFIG bits are reprogrammed after a reset

2012-03-15 Thread Ramirez Luna, Omar
On Thu, Mar 15, 2012 at 1:25 AM, Paul Walmsley p...@pwsan.com wrote: Hola Omar, Hola :) On Wed, 14 Mar 2012, Ramirez Luna, Omar wrote: If we reached here the reset lines will be asserted, and then the code below touches sysc registers on a module under reset. Do you know of any case where

Re: [PATCH v2 5/8] ARM: OMAP2+: hwmod: ensure that SYSCONFIG bits are reprogrammed after a reset

2012-03-14 Thread Ramirez Luna, Omar
Hi Paul, 2012/2/27 Paul Walmsley p...@pwsan.com: diff --git a/arch/arm/mach-omap2/omap_hwmod.c b/arch/arm/mach-omap2/omap_hwmod.c index db4ad41..aeb6f4c 100644 --- a/arch/arm/mach-omap2/omap_hwmod.c +++ b/arch/arm/mach-omap2/omap_hwmod.c @@ -1490,13 +1490,22 @@ static int _reset(struct

Re: [PATCH] ARM: OMAP2: fix mailbox init code

2012-02-23 Thread Ramirez Luna, Omar
Hi, On Thu, Feb 23, 2012 at 3:18 AM, Bedia, Vaibhav vaibhav.be...@ti.com wrote: On Thu, Feb 23, 2012 at 14:23:35, Ohad Ben-Cohen wrote: [...] Which happens on CONFIG_ARCH_OMAP2 !CONFIG_SOC_OMAP2420, due to missing omap2_mboxes declaration. [...] -struct omap_mbox *omap2_mboxes[] = {

Re: Latest OMAP randconfig build error

2012-02-22 Thread Ramirez Luna, Omar
On Wed, Feb 22, 2012 at 11:58 AM, Tony Lindgren t...@atomide.com wrote: * Ohad Ben-Cohen o...@wizery.com [120222 01:30]: + Tony, Suman On Wed, Feb 22, 2012 at 10:51 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: arch/arm/mach-omap2/mailbox.c: In function 'omap2_mbox_probe':

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-14 Thread Ramirez Luna, Omar
On Tue, Feb 14, 2012 at 10:23 AM, Felipe Contreras felipe.contre...@gmail.com wrote: When that case is applicable, we should first modify the loader code or prepare the baseimages to be common so we can get rid of specific loaders and just dump them into memory. I'd say the less workarounds,

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-13 Thread Ramirez Luna, Omar
On Feb 11, 2012 3:03 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Sat, Feb 11, 2012 at 9:19 PM, Ramirez Luna, Omar omar.rami...@ti.com wrote: On Feb 10, 2012 12:44 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Fri, Feb 10, 2012 at 10:35 PM, Dan Carpenter

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-13 Thread Ramirez Luna, Omar
On Thu, Feb 9, 2012 at 9:18 PM, Greg KH gre...@linuxfoundation.org wrote: On Thu, Feb 09, 2012 at 04:45:00PM -0800, Ramirez Luna, Omar wrote: Hi, On Thu, Feb 9, 2012 at 3:30 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Again, I'm totally confused as to _WHY_ this needs to be y

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-09 Thread Ramirez Luna, Omar
Hi, On Thu, Feb 9, 2012 at 3:30 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Again, I'm totally confused as to _WHY_ this needs to be y.  What is causing this oops without it?  If an oops is happening, then shouldn't this be a strict dependancy?  Why allow it to be disabled at all if

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-01 Thread Ramirez Luna, Omar
On Wed, Feb 1, 2012 at 2:11 AM, Felipe Contreras felipe.contre...@gmail.com wrote: On Wed, Feb 1, 2012 at 5:22 AM, Justin P. Mattock justinmatt...@gmail.com wrote: not sure if I see this warning or not on my machine, but is there a fix for the warning? I would rather see that, than hide it

Re: [PATCH 2/2] staging: tidspbridge: fix incorrect free to drv_datap

2012-01-31 Thread Ramirez Luna, Omar
On Tue, Jan 31, 2012 at 2:21 AM, Dan Carpenter dan.carpen...@oracle.com wrote: On Mon, Jan 30, 2012 at 07:20:18PM -0600, Omar Ramirez Luna wrote: This structure is still used after it has been freed, since it is being allocated in probe, calls to free it have been moved to module's remove

Re: [PATCH 1/5] staging: tidspbridge: more readable code

2012-01-30 Thread Ramirez Luna, Omar
2012/1/24 Felipe Contreras felipe.contre...@gmail.com: 2012/1/23 Víctor Manuel Jáquez Leal vjaq...@igalia.com: Uppercase function names are not pretty. Also the code flow readability is enhanced. Looks good to me. FWIW, I agree. Regards, Omar -- To unsubscribe from this list: send the line

Re: [PATCH 2/5] staging: tidspbridge: remove unused header

2012-01-30 Thread Ramirez Luna, Omar
2012/1/24 Felipe Contreras felipe.contre...@gmail.com: 2012/1/23 Víctor Manuel Jáquez Leal vjaq...@igalia.com: No functional changes. The header file drv_interface.h was only used locally, hence there's no need to have it. Also the only prototyped functions were the file_operations

Re: [PATCH 3/5] staging: tidspbridge: Lindent to drv_interface.c

2012-01-30 Thread Ramirez Luna, Omar
2012/1/23 Víctor Manuel Jáquez Leal vjaq...@igalia.com: No functional changes. According to Lindent, the file drv_internface.c had some lines with bad indentation. This commit is the output of Lindent. Usually lindent tends to do whatever it wants, unless carefully configured... ... @@

Re: [PATCH 4/5] staging: tidspbridge: silence the compiler

2012-01-30 Thread Ramirez Luna, Omar
2012/1/23 Víctor Manuel Jáquez Leal vjaq...@igalia.com: Silence the warning when compiling drv_interface.c Signed-off-by: Víctor Manuel Jáquez Leal vjaq...@igalia.com ---  drivers/staging/tidspbridge/rmgr/drv_interface.c |    5 +++--  1 files changed, 3 insertions(+), 2 deletions(-) diff

Re: [PATCH] tidspbridge: Rename module from bridgedriver to tidspbridge

2012-01-30 Thread Ramirez Luna, Omar
On Tue, Jan 24, 2012 at 3:25 PM, Joe Perches j...@perches.com wrote: tidspbridge when built as a module is named bridgedriver. bridgedriver is not a particularly good module name. tidspbridge is what the source is named.  That seems a more appropriate module name too as it describes the

Re: 3.3-rc1 console lag (was: Re: [PATCH v8 00/20] OMAP2+: UART: Runtime adaptation + cleanup)

2012-01-26 Thread Ramirez Luna, Omar
Hi Paul, On Wed, Jan 25, 2012 at 9:00 PM, Paul Walmsley p...@pwsan.com wrote: ... Ensure CONFIG_OMAP_PRM is set while testing irq_chaining with uart. And for pm_qos usage ensure CONFIG_CPU_IDLE is selected other wise console might be sluggish. There is console lag for omap2plus_defconfig

3.3-rc1 console lag (was: Re: [PATCH v8 00/20] OMAP2+: UART: Runtime adaptation + cleanup)

2012-01-25 Thread Ramirez Luna, Omar
Hi, On Fri, Nov 11, 2011 at 3:57 AM, Govindraj.R govindraj.r...@ti.com wrote: Converting uart driver to adapt to pm runtime API's. Code re-org + cleanup. Moving some functionality from serial.c to omap-serial.c ... Ensure CONFIG_OMAP_PRM is set while testing irq_chaining with uart. And for

Re: Bad page state in tidspbridge driver

2012-01-09 Thread Ramirez Luna, Omar
Hi Laurent, On Sun, Jan 8, 2012 at 12:17 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi, I'm hitting what seems to be a tidspbridge driver issue when using gst-dsp with the DSP JPEG encoder. My application captures images from the OMAP3 ISP directly to frame buffer memory

Re: [PATCH v4 4/4] OMAP3/4: iommu: adapt to runtime pm

2012-01-05 Thread Ramirez Luna, Omar
On Tue, Dec 27, 2011 at 3:41 AM, Felipe Contreras felipe.contre...@gmail.com wrote: On Sun, Dec 25, 2011 at 2:03 AM, Ramirez Luna, Omar omar.rami...@ti.com wrote: On Fri, Dec 23, 2011 at 11:04 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Which omap-iommu? The platform driver

Re: [PATCH v4 1/4] OMAP3: hwmod data: add mmu data for iva and isp

2012-01-05 Thread Ramirez Luna, Omar
Hi Laurent On Sun, Dec 25, 2011 at 3:08 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: I'm not sure how this clock stuff works, but I'm guessing the device is supposed to go to sleep at some points in time, and with your patch OMAP3/4: iommu: adapt to runtime pm it won't, as

Re: [PATCH v4 4/4] OMAP3/4: iommu: adapt to runtime pm

2011-12-24 Thread Ramirez Luna, Omar
On Fri, Dec 23, 2011 at 11:04 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Which omap-iommu? The platform driver, or the device stuff? The device stuff is always built-in, but not the platform driver (drivers/iommu/omap-iommu.c), that can be a module. Both, I can't recall exactly

Re: [PATCH v4 1/4] OMAP3: hwmod data: add mmu data for iva and isp

2011-12-23 Thread Ramirez Luna, Omar
On Mon, Dec 19, 2011 at 10:11 AM, Felipe Contreras felipe.contre...@gmail.com wrote: On Fri, Dec 16, 2011 at 4:01 AM, Ramirez Luna, Omar omar.rami...@ti.com wrote: On Thu, Dec 15, 2011 at 6:39 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Are you sure you are not missing something

Re: [PATCH v4 4/4] OMAP3/4: iommu: adapt to runtime pm

2011-12-23 Thread Ramirez Luna, Omar
On Mon, Dec 19, 2011 at 10:27 AM, Felipe Contreras felipe.contre...@gmail.com wrote: On Fri, Dec 16, 2011 at 5:18 AM, Ramirez Luna, Omar omar.rami...@ti.com wrote: On Thu, Dec 15, 2011 at 6:53 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Thu, Dec 15, 2011 at 6:18 AM, Omar

Re: [PATCH v4 1/4] OMAP3: hwmod data: add mmu data for iva and isp

2011-12-15 Thread Ramirez Luna, Omar
On Thu, Dec 15, 2011 at 6:39 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Thu, Dec 15, 2011 at 6:18 AM, Omar Ramirez Luna omar.rami...@ti.com wrote: +/* l4_core - isp mmu */ +static struct omap_hwmod_ocp_if omap3xxx_l4_core__isp_mmu = { +       .master         =

Re: [PATCH v4 4/4] OMAP3/4: iommu: adapt to runtime pm

2011-12-15 Thread Ramirez Luna, Omar
On Thu, Dec 15, 2011 at 6:33 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Thu, Dec 15, 2011 at 6:18 AM, Omar Ramirez Luna omar.rami...@ti.com wrote: @@ -123,11 +123,10 @@ static int iommu_enable(struct omap_iommu *obj)        if (!arch_iommu)                return -ENODEV; -  

Re: [PATCH v4 4/4] OMAP3/4: iommu: adapt to runtime pm

2011-12-15 Thread Ramirez Luna, Omar
On Thu, Dec 15, 2011 at 6:53 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Thu, Dec 15, 2011 at 6:18 AM, Omar Ramirez Luna omar.rami...@ti.com wrote: Use runtime PM functionality interfaced with hwmod enable/idle functions, to replace direct clock operations, reset and sysconfig

Re: [PATCH] ARM: OMAP: mcbsp: Fix possible memory corruption

2011-12-14 Thread Ramirez Luna, Omar
On Mon, Dec 12, 2011 at 12:33 PM, Tony Lindgren t...@atomide.com wrote: Applying into fixes. FWIW, I was bisecting this issue, the patch fixes the following slab corruptions on boot log: ... hw-breakpoint: found 5 (+1 reserved) breakpoint and 1 watchpoint registers. hw-breakpoint: maximum

Re: [PATCH] ARM: remove header files included more than once

2011-12-12 Thread Ramirez Luna, Omar
Hi Haojian, On Sun, Dec 11, 2011 at 7:10 PM, Haojian Zhuang haojian.zhu...@gmail.com wrote: ... Hi Omar, Thanks for your patch. Could you help to split your patches into small pieces? Since different people are handling for different files. For example, I can help to handle files in

Re: [PATCH 1/2] ARM: OMAP: dmtimer: fix sleeping function called from invalid context

2011-12-12 Thread Ramirez Luna, Omar
On Mon, Dec 12, 2011 at 5:08 PM, Tony Lindgren t...@atomide.com wrote: ... @@ -134,22 +134,13 @@ static void omap_dm_timer_reset(struct omap_dm_timer *timer)  int omap_dm_timer_prepare(struct omap_dm_timer *timer)  { ... -       ret = omap_dm_timer_set_source(timer, OMAP_TIMER_SRC_32_KHZ);

Re: [PATCH v2] staging: tidspbridge: request dmtimer clocks on init

2011-12-10 Thread Ramirez Luna, Omar
On Fri, Dec 9, 2011 at 4:38 PM, Greg KH g...@kroah.com wrote: How about for 3.2.1?  I'll try to get this into 3.2, but I can't guarantee anything so late in the release cycle, sorry. Sounds fair, fingers crossed for 3.2 ;) Thanks, Omar -- To unsubscribe from this list: send the line

Re: [PATCH 1/2] ARM: OMAP: dmtimer: fix sleeping function called from invalid context

2011-12-09 Thread Ramirez Luna, Omar
On Fri, Dec 9, 2011 at 3:34 PM, Tony Lindgren t...@atomide.com wrote: +     ret = omap_dm_timer_set_source(timer, OMAP_TIMER_SRC_32_KHZ); ...  EXPORT_SYMBOL_GPL(omap_dm_timer_request); This does not seem right.. It seems that you're hardcoding the source clock to 32 KiHz clock while other

Re: [PATCH v2] staging: tidspbridge: request dmtimer clocks on init

2011-12-08 Thread Ramirez Luna, Omar
On Wed, Dec 7, 2011 at 4:11 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Sat, Nov 19, 2011 at 12:18 AM, Omar Ramirez Luna omar.rami...@ti.com wrote: Given that dm timer framework doesn't support request of clocks by soft | hard irqs because some recent changes, tidspbridge needs

Re: [PATCH v2] staging: tidspbridge: request dmtimer clocks on init

2011-12-08 Thread Ramirez Luna, Omar
On Thu, Dec 8, 2011 at 1:50 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Thu, Dec 8, 2011 at 9:10 PM, Ramirez Luna, Omar omar.rami...@ti.com wrote: On Wed, Dec 7, 2011 at 4:11 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Sat, Nov 19, 2011 at 12:18 AM, Omar Ramirez

Re: [PATCH v2] staging: tidspbridge: request dmtimer clocks on init

2011-12-08 Thread Ramirez Luna, Omar
On Thu, Dec 8, 2011 at 2:11 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Sat, Nov 19, 2011 at 12:18 AM, Omar Ramirez Luna omar.rami...@ti.com wrote: Given that dm timer framework doesn't support request of clocks by soft | hard irqs because some recent changes, tidspbridge needs

Re: [PATCH v2] staging: tidspbridge: request dmtimer clocks on init

2011-12-08 Thread Ramirez Luna, Omar
On Thu, Dec 8, 2011 at 6:07 PM, Greg KH g...@kroah.com wrote: Definitely 3.2-fix material IMO :) I believe it was pushed to staging-next... I'm rooting so it can be pushed to staging-linus :) As it wasn't originally stated that this was a problem that needed to be fixed for 3.2, can it

Re: [PATCH] ARM: OMAP: dmtimer: fix missing content/correction in low-power mode support

2011-11-21 Thread Ramirez Luna, Omar
Hi, On Tue, Nov 8, 2011 at 12:00 AM, Tarun Kanti DebBarma tarun.ka...@ti.com wrote: Since omap_dm_timer_write_reg/__omap_dm_timer_write is now modified to use timer-func_base OCP_CFG should not use this wrapper anymore. Instead use __raw_writel() directly and use timer-io_base instead to

Re: [PATCH] staging: tidspbridge: request dmtimer clocks on init

2011-11-18 Thread Ramirez Luna, Omar
Hi Greg, On Fri, Nov 18, 2011 at 3:54 PM, Omar Ramirez Luna omar.rami...@ti.com wrote: diff --git a/drivers/staging/tidspbridge/core/dsp-clock.c b/drivers/staging/tidspbridge/core/dsp-clock.c index 3d1279c..1ba10ae 100644 --- a/drivers/staging/tidspbridge/core/dsp-clock.c +++

Re: [PATCH v3 2/4] OMAP4: hwmod data: add mmu hwmod for ipu and dsp

2011-11-08 Thread Ramirez Luna, Omar
Hi, On Fri, Nov 4, 2011 at 6:23 PM, Kevin Hilman khil...@ti.com wrote: +     .flags          = HWMOD_INIT_NO_RESET, Why is this needed? ... +     .flags          = HWMOD_INIT_NO_RESET, And this? I have this because the hwmod complains about a failure in hard reset, even though the reset

Re: [PATCH v3 4/4] OMAP3/4: iommu: adapt to runtime pm

2011-11-07 Thread Ramirez Luna, Omar
Hi, On Fri, Nov 4, 2011 at 6:27 PM, Kevin Hilman khil...@ti.com wrote: @@ -821,9 +820,7 @@ static irqreturn_t iommu_fault_handler(int irq, void *data)       if (!obj-refcount)               return IRQ_NONE; -     clk_enable(obj-clk);       errs = iommu_report_fault(obj, da); -    

Re: [PATCH v3 0/4] OMAP: iommu: hwmod support and runtime PM

2011-11-02 Thread Ramirez Luna, Omar
+ Joerg, iommu-list On Tue, Nov 1, 2011 at 5:15 PM, Omar Ramirez Luna omar.rami...@ti.com wrote: Introduced hwmod support for OMAP3 (iva, isp) and OMAP4 (ipu, dsp), along with the corresponding runtime PM routines to deassert reset lines, enable/disable clocks and configure sysc registers.

Re: [PATCH v3 4/4] OMAP3/4: iommu: adapt to runtime pm

2011-11-02 Thread Ramirez Luna, Omar
Hi MyungJoo, On Wed, Nov 2, 2011 at 5:16 AM, MyungJoo Ham myungjoo@gmail.com wrote: On Wed, Nov 2, 2011 at 7:15 AM, Omar Ramirez Luna omar.rami...@ti.com wrote: Use runtime PM functionality interfaced with hwmod enable/idle functions, to replace direct clock operations, reset and sysconfig

Re: [PATCH v3 4/4] OMAP3/4: iommu: adapt to runtime pm

2011-11-02 Thread Ramirez Luna, Omar
On Wed, Nov 2, 2011 at 10:21 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Tue, Nov 01, 2011 at 05:15:52PM -0500, Omar Ramirez Luna wrote: Use runtime PM functionality interfaced with hwmod enable/idle functions, to replace direct clock operations, reset and sysconfig handling.

Re: tidspbridge issue with omap_dm_timer_free

2011-08-23 Thread Ramirez Luna, Omar
Hi, On Sun, Aug 14, 2011 at 2:44 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Yeah, but with the current approach it would be possible that everything works fine, and then the DSP goes to hibernation, a module is loaded that uses one dm timer, and then when the DSP wakes up, that

Re: tidspbridge issue with omap_dm_timer_free

2011-08-13 Thread Ramirez Luna, Omar
On Wed, Aug 10, 2011 at 1:42 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Hi, I am trying to use a more recent version of the tidspbridge code in the Nokia N9, but I'm stuck with this warning that is caused by using the dm timer framework. [   30.883636] BUG: sleeping function

Re: tidspbridge issue with omap_dm_timer_free

2011-08-13 Thread Ramirez Luna, Omar
On Wed, Aug 10, 2011 at 6:27 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Wed, Aug 10, 2011 at 9:42 PM, Felipe Contreras felipe.contre...@gmail.com wrote: I am trying to use a more recent version of the tidspbridge code in the Nokia N9, but I'm stuck with this warning that is

Re: [PATCH 0/8] staging: tidspbridge: PM routines cleanup

2011-04-29 Thread Ramirez Luna, Omar
On Wed, Mar 23, 2011 at 1:49 PM, Omar Ramirez Luna omar.rami...@ti.com wrote: Cleanup and fixes for tidspbridge's PM routines. The most significant ones are: - Protection of critical sections when disabling clocks which  could cause issues during PM transitions. - Fix of resume path from RET

Re: [PATCH v2] OMAP: iovmm: fix SW flags passed by user

2011-04-15 Thread Ramirez Luna, Omar
Hi Hiroshi, On Fri, Mar 25, 2011 at 3:04 PM, Omar Ramirez Luna omar.rami...@ti.com wrote: Commit d038aee24dcd5a2a0d8547f5396f67ae9698ac8e omap: iovmm: don't check 'da' to set IOVMF_DA_FIXED flag, changes iovmm to receive flags specified by user, however the upper 16 bits of the flags are

Re: HELP:dspbridge:how to configure GPT timer

2011-04-11 Thread Ramirez Luna, Omar
Hi, On Sat, Apr 9, 2011 at 7:38 AM, onlyfever onlyfe...@gmail.com wrote: Hi all! currently dsp make use of gpt 5, 6, 7 or 8. I'm using gpt 8 for something else. Is it possible to configure dsp-bridge to use other gpt timer? I want to change gpt 8 to gpt 9,or gpt 7. Please give me some

Re: [PATCH 0/5] OMAP: l3: fixes and cleanup

2011-03-31 Thread Ramirez Luna, Omar
On Wed, Mar 30, 2011 at 1:45 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote:   OMAP3: l3: fix for irq 10: nobody cared message   OMAP3: l3: fix omap3_l3_probe error path   OMAP3: l3: minor cleanup for error message, parenthesis and extra     lines   OMAP4: l3: fix omap4_l3_probe error

Re: [PATCH] OMAP3: l3: fix for irq 10: nobody cared message

2011-03-28 Thread Ramirez Luna, Omar
Hi, On Mon, Mar 28, 2011 at 12:38 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: If an error occurs in the L3 on any other initiator than MPU, the interrupt goes unhandled given that the 'base' register was calculated with the initialized err_base value (which coincidentally points to

Re: [PATCH 0/4] iommu: Prevent oops in iommu_get() and while arch_iommu is in use

2011-03-27 Thread Ramirez Luna, Omar
Hi, On Sun, Mar 27, 2011 at 12:27 PM, Sakari Ailus sakari.ai...@maxwell.research.nokia.com wrote: Ramirez Luna, Omar wrote: On Fri, Mar 25, 2011 at 10:13 AM, Sakari Ailus sakari.ai...@maxwell.research.nokia.com wrote: This patchset is aimed to fix a problem in arch_iommu implementation

Re: [PATCH] OMAP3: l3: fix for irq 10: nobody cared message

2011-03-27 Thread Ramirez Luna, Omar
On Sun, Mar 27, 2011 at 8:30 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Sat, Mar 26, 2011 at 2:29 AM, Omar Ramirez Luna omar.rami...@ti.com wrote: If an error occurs in the L3 on any other initiator than MPU, the interrupt goes unhandled given that the 'base' register was

Re: [PATCH] OMAP3: l3: fix for irq 10: nobody cared message

2011-03-27 Thread Ramirez Luna, Omar
On Sat, Mar 26, 2011 at 4:38 PM, Sergei Shtylyov sshtyl...@mvista.com wrote: Hello. On 26-03-2011 3:29, Omar Ramirez Luna wrote: If an error occurs in the L3 on any other initiator than MPU, the interrupt goes unhandled given that the 'base' register was calculated with the initialized

Re: [PATCH] OMAP: iovmm: fix SW flags passed by user

2011-03-25 Thread Ramirez Luna, Omar
On Fri, Mar 25, 2011 at 12:30 PM, Sergei Shtylyov sshtyl...@mvista.com wrote: Hello. Omar Ramirez Luna wrote: Commit d038aee24dcd changes iovmm to receive flags specified by user,   Please also specify that commit's summary for the human readers. Bare ID is only usable to gitweb... Here

Re: [PATCH 0/4] iommu: Prevent oops in iommu_get() and while arch_iommu is in use

2011-03-25 Thread Ramirez Luna, Omar
On Fri, Mar 25, 2011 at 10:13 AM, Sakari Ailus sakari.ai...@maxwell.research.nokia.com wrote: Hi, This patchset is aimed to fix a problem in arch_iommu implementation references. When an actual arch_iommu implementation is not loaded while iommu_get() is being called results to a kernel oops,

Re: [PATCH] OMAP: iovmm: fix SW flags passed by user

2011-03-25 Thread Ramirez Luna, Omar
On Fri, Mar 25, 2011 at 3:05 PM, Russell King - ARM Linux li...@arm.linux.org.uk Commit d038aee24dcd changes iovmm to receive flags specified by user,   Please also specify that commit's summary for the human readers. Bare ID is only usable to gitweb... Here it is the patch, introducing

Re: [PATCH] OMAP3: l3: fix for irq 10: nobody cared message

2011-03-25 Thread Ramirez Luna, Omar
On Fri, Mar 25, 2011 at 7:29 PM, Omar Ramirez Luna omar.rami...@ti.com wrote: If an error occurs in the L3 on any other initiator than MPU, the interrupt goes unhandled given that the 'base' register was calculated with the initialized err_base value (which s/err_base/err_source/

Re: [RESEND PATCH] staging: tidspbridge: protect dmm_map properly

2011-03-11 Thread Ramirez Luna, Omar
Hi Felipe, On Fri, Mar 11, 2011 at 11:38 AM, Felipe Contreras felipe.contre...@gmail.com wrote: On Fri, Mar 11, 2011 at 7:30 PM, Felipe Contreras felipe.contre...@nokia.com wrote: We need to protect not only the dmm_map list, but the individual map_obj's, otherwise, we might be building the

Re: [RESEND PATCH] staging: tidspbridge: protect dmm_map properly

2011-03-11 Thread Ramirez Luna, Omar
Hi, On Fri, Mar 11, 2011 at 12:31 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Well, not crashing is better than crashing, regardless of the status of the driver. And there's no reason not to include the fix. No reason indeed. But if you send it to Greg, then it's Greg's call. Ok

Re: [PATCH v2 2/4] OMAP4: hwmod data: add mmu hwmod for ipu and dsp

2011-03-08 Thread Ramirez Luna, Omar
Hi Benoit, On Tue, Mar 8, 2011 at 3:29 PM, Cousson, Benoit b-cous...@ti.com wrote: What I meant is that the mmu does not have any explicit PRCM module enable control. The PRCM does control only the DSP as a whole. Since hwmods are generated with a PRCM granularity, any sub module will not be

Re: [PATCH v2 2/4] OMAP4: hwmod data: add mmu hwmod for ipu and dsp

2011-03-07 Thread Ramirez Luna, Omar
Hi Benoit, On Mon, Mar 7, 2011 at 6:55 AM, Cousson, Benoit b-cous...@ti.com wrote: Hi Omar, I have some concern about the introduction of a hwmod that does not match the actual HW capability. MMU does exist, but there is no SW control for it. Maybe I'm missing something, but iommu (driver)

Re: [PATCH] staging: tidspbridge: protect dmm_map properly

2011-03-07 Thread Ramirez Luna, Omar
Hi Felipe, On Mon, Mar 7, 2011 at 12:02 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Mon, Dec 20, 2010 at 7:12 PM, Felipe Contreras felipe.contre...@nokia.com wrote: We need to protect not only the dmm_map list, but the individual map_obj's, otherwise, we might be building the

Re: [PATCH] OMAP2+: hwmod: use status bit info for reset line

2011-03-06 Thread Ramirez Luna, Omar
Hi Paul On Fri, Mar 4, 2011 at 3:36 PM, Paul Walmsley p...@pwsan.com wrote: It's after our -rc6 cutoff, but I'll take it anyway for 2.6.39.  Made a few minor changes -- updated patch below -- if you have a chance, please give it a quick test.  The 'integration-2.6.39' branch at

Re: [PATCH v3 2/2] OMAP: IOMMU: add support to callback during fault handling

2011-02-21 Thread Ramirez Luna, Omar
Hi, On Wed, Feb 16, 2011 at 1:35 PM, David Cohen daco...@gmail.com wrote: diff --git a/arch/arm/mach-omap2/iommu2.c b/arch/arm/mach-omap2/iommu2.c index 49a1e5e..adb083e 100644 --- a/arch/arm/mach-omap2/iommu2.c +++ b/arch/arm/mach-omap2/iommu2.c ...        da = iommu_read_reg(obj,

Re: [PATCH v3 2/2] OMAP: IOMMU: add support to callback during fault handling

2011-02-21 Thread Ramirez Luna, Omar
Hi, On Mon, Feb 21, 2011 at 3:12 PM, David Cohen daco...@gmail.com wrote: Generic errors codes make easier to threat possible IOMMU users which have (partially or totally) common drivers for different OMAP versions. Agree then. +       /* Fault callback or TLB/PTE Dynamic loading */ +      

Re: [PATCH v5 3/5] OMAP4: hwmod data: add mailbox data

2011-02-17 Thread Ramirez Luna, Omar
Hi Tony, On Thu, Feb 17, 2011 at 5:39 PM, Tony Lindgren t...@atomide.com wrote: * Ramirez Luna, Omar omar.rami...@ti.com [110215 16:11]: If you have to do several platform_get_irq_byname to get this one, I'd prefer to get rid of that name for OMAP4. It will make mailbox irq consistent

Re: [PATCH v5 0/5] omap: mailbox: hwmod support

2011-02-15 Thread Ramirez Luna, Omar
Hi Kevin, Benoit, On Fri, Feb 11, 2011 at 5:01 PM, Kevin Hilman khil...@ti.com wrote: Omar Ramirez Luna omar.rami...@ti.com writes: Mailbox hwmod support for OMAP 2,3,4. This was tested on OMAP3 (3430, 3630), minor testing was made on OMAP4. No testing on OMAP2 since I don't have the

Re: [PATCH v5 3/5] OMAP4: hwmod data: add mailbox data

2011-02-15 Thread Ramirez Luna, Omar
Hi Benoit, On Mon, Feb 14, 2011 at 9:00 AM, Cousson, Benoit b-cous...@ti.com wrote: +static struct omap_hwmod_irq_info omap44xx_mailbox_irqs[] = { +       { .name = mbox, .irq = 26 + OMAP44XX_IRQ_GIC_START, }, The original entry was unnamed since it is an unique entry and thus does not need

Re: [PATCH v5 3/5] OMAP4: hwmod data: add mailbox data

2011-02-15 Thread Ramirez Luna, Omar
Hi, On Tue, Feb 15, 2011 at 4:05 PM, Cousson, Benoit b-cous...@ti.com wrote: It is this way instead of plain platform_get_irq because omap2420 has two interrupt sources to MPU and mailbox driver uses platform_get_irq_byname to get the irq number. This is what I was thinking, except that on

Re: [PATCH 2/2] OMAP: Cleanup IOMMU error messages

2011-01-31 Thread Ramirez Luna, Omar
Hi David, On Mon, Jan 31, 2011 at 11:25 AM, David Cohen david.co...@nokia.com wrote: IOMMU error messages are duplicated. They're printed on IOMMU specific layer for OMAP2,3 and once again on the above layer. With this patch, the error message is printed on the above layer only. So, you say

Re: [PATCH 00/12]staging:tidspbridge - Remove hungarian notation from structures

2011-01-27 Thread Ramirez Luna, Omar
On Mon, Jan 17, 2011 at 9:19 PM, Armando Uribe x0095...@ti.com wrote: This series of patches remove the hungarian notation from the structure's elements. Also the whole patch series has been rebased to the latest of tidspbridge tree, and minor corrections were made from the original version

Re: [PATCHv2] staging: tidspbridge: configure full L1 MMU range

2011-01-13 Thread Ramirez Luna, Omar
Hi, On Wed, Jan 5, 2011 at 3:25 PM, Fernando Guzman Lugo fernando.l...@ti.com wrote: From: Guzman Lugo, Fernando fernando.l...@ti.com IVA MMU can manage up to 4GB of address space through its page tables, given that it's L1 is divided into 1MB sections it requires at least 16KB for its table

Re: [PATCH] staging: tidspbridge - configure full L1 MMU range

2011-01-05 Thread Ramirez Luna, Omar
Hi, On Wed, Jan 5, 2011 at 12:20 AM, Fernando Guzman Lugo fernando.l...@ti.com wrote: Otherwise a virtual address beyond of the L1 size is used, the MMU hardware will look into a memory that does not belong to L1 translation tables. IOW; the MMU would allow to access any memory, configured or

Re: [PATCH 1/5] staging:tidspbridge - Remove unused defined constants

2010-12-23 Thread Ramirez Luna, Omar
Hi, On Thu, Dec 16, 2010 at 7:18 PM, Armando Uribe x0095...@ti.com wrote: Remove defined constants not being used. Signed-off-by: Armando Uribe x0095...@ti.com Pushed the entire series to dspbridge. Regards, Omar -- To unsubscribe from this list: send the line unsubscribe linux-omap in the

Re: [PATCH] staging: tidspbridge: remove code referred by OPT_ZERO_COPY_LOADER

2010-12-23 Thread Ramirez Luna, Omar
Hi, On Mon, Dec 20, 2010 at 3:23 PM, Ernesto Ramos erne...@ti.com wrote: Remove code referred by OPT_ZERO_COPY_LOADER since it is not used. Signed-off-by: Ernesto Ramos erne...@ti.com ---  drivers/staging/tidspbridge/dynload/cload.c |   60 +--  1 files changed, 20

Re: [PATCH 1/2] staging: tidspbridge: rmgr/node.c code cleanup

2010-12-22 Thread Ramirez Luna, Omar
On Thu, Dec 9, 2010 at 3:47 PM, Ionut Nicu ionut.n...@gmail.com wrote: Reorganized some code in rmgr/node.c to increase its readability. Most of the changes reduce the code indentation level and simplifiy the code. No functional changes were done. Signed-off-by: Ionut Nicu

Re: [PATCH 2/2] staging: tidspbridge: rmgr code cleanup

2010-12-22 Thread Ramirez Luna, Omar
Hi Ionut, On Thu, Dec 9, 2010 at 3:47 PM, Ionut Nicu ionut.n...@gmail.com wrote: Reorganized some code in the rmgr module to increase its readability. No functional changes were done. Signed-off-by: Ionut Nicu ionut.n...@mindbit.ro ---  drivers/staging/tidspbridge/rmgr/drv.c  |  171

Re: [PATCH] dspbridge: Fix atoi to support hexadecimal numbers correctly

2010-12-22 Thread Ramirez Luna, Omar
On Sun, Dec 12, 2010 at 7:39 AM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: For some strange reason, the DSP base image node/object properties description string stores hexadecimal numbers with a 'h' or 'H' suffix instead of a '0x' prefix. This causes parsing issue because the

Re: [PATCH] dspbridge: Fix atoi to support hexadecimal numbers correctly

2010-12-22 Thread Ramirez Luna, Omar
On Wed, Dec 22, 2010 at 7:00 PM, Ramirez Luna, Omar omar.rami...@ti.com wrote: On Sun, Dec 12, 2010 at 7:39 AM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: For some strange reason, the DSP base image node/object properties description string stores hexadecimal numbers with a 'h

Re: [PATCH v4 5/5] OMAP: mailbox: use runtime pm for clk and sysc handling

2010-12-17 Thread Ramirez Luna, Omar
On Fri, Dec 17, 2010 at 10:28 AM, Cousson, Benoit b-cous...@ti.com wrote:  /* SYSCONFIG: register bit definition */ -#define AUTOIDLE       (1  0)  #define SOFTRESET      (1  1) -#define SMARTIDLE      (2  3)  #define OMAP4_SOFTRESET        (1  0) -#define OMAP4_NOIDLE   (1  2) -#define

Re: [PATCH v4 5/5] OMAP: mailbox: use runtime pm for clk and sysc handling

2010-12-16 Thread Ramirez Luna, Omar
Hi, On Thu, Dec 16, 2010 at 2:28 AM, Varadarajan, Charulatha ch...@ti.com wrote: On Thu, Dec 16, 2010 at 12:17, Omar Ramirez Luna omar.rami...@ti.com wrote: Use runtime pm APIs to enable/disable mailbox clocks and to configure SYSC register. Based on the patch sent by Felipe Contreras:

Re: [PATCH v4 4/5] OMAP: mailbox: build device using omap_device/omap_hwmod

2010-12-16 Thread Ramirez Luna, Omar
On Thu, Dec 16, 2010 at 2:44 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Thu, Dec 16, 2010 at 02:08:11PM +0530, Varadarajan, Charulatha wrote: +       oh = omap_hwmod_lookup(mailbox); +       if (!oh) { +               pr_err(%s: unable to find hwmod\n, __func__); +      

Re: [PATCH v4 5/5] OMAP: mailbox: use runtime pm for clk and sysc handling

2010-12-16 Thread Ramirez Luna, Omar
Hi, On Thu, Dec 16, 2010 at 10:32 AM, Kanigeri, Hari h-kanige...@ti.com wrote: @@ -130,12 +120,6 @@ static int omap2_mbox_startup(struct omap_mbox *mbox)        l = mbox_read_reg(MAILBOX_REVISION);        pr_debug(omap mailbox rev %d.%d\n, (l 0xf0) 4, (l 0x0f)); -       if

Re: [PATCH 13/14] OMAP2/3: PRCM: split OMAP2/3-specific PRCM code into OMAP2/3-specific files

2010-12-15 Thread Ramirez Luna, Omar
On Tue, Dec 14, 2010 at 10:50 PM, Paul Walmsley p...@pwsan.com wrote: Hi, On Mon, 6 Dec 2010, Paul Walmsley wrote: In preparation for adding OMAP4-specific PRCM accessor/mutator functions, split the existing OMAP2/3 PRCM code into OMAP2/3-specific files.  Most of what was in

Re: [PATCH] OMAP: omap_device: use pdev as parameter to get rt_va

2010-12-13 Thread Ramirez Luna, Omar
Hi Benoit, On Mon, Dec 13, 2010 at 2:12 AM, Cousson, Benoit b-cous...@ti.com wrote: Hi Omar, On 12/11/2010 12:45 AM, Ramirez Luna, Omar wrote: Make the parameter received by omap_device_get_mpu_rt_va consistent with the functions meant to be called by drivers. Also move its header

Re: [PATCH] staging: tidspbridge: use the right type for list_is_last

2010-12-13 Thread Ramirez Luna, Omar
On Sun, Dec 12, 2010 at 6:02 AM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Omar, On Wednesday 08 December 2010 23:20:23 Omar Ramirez Luna wrote: Removes the following warning:   CC [M]  drivers/staging/tidspbridge/rmgr/rmm.o drivers/staging/tidspbridge/rmgr/rmm.c: In

Re: [PATCH 1/2] staging: tidspbridge: rmgr/node.c code cleanup

2010-12-13 Thread Ramirez Luna, Omar
Hi, On Sun, Dec 12, 2010 at 8:20 AM, Ionut Nicu ionut.n...@mindbit.ro wrote: Are the other modes (rdma, zerocopy) not supported at all? No, those modes are not supported... the only one working for dsp streams is proc-copy. Is there any plan to make them work? If not, maybe the code that

Re: [PATCH 1/2] staging: tidspbridge: rmgr/node.c code cleanup

2010-12-09 Thread Ramirez Luna, Omar
Hi Ionut, On Thu, Dec 9, 2010 at 4:22 PM, Ionut Nicu ionut.n...@mindbit.ro wrote: Hi Omar, On Thu, 2010-12-09 at 23:47 +0200, Ionut Nicu wrote: Reorganized some code in rmgr/node.c to increase its readability. Most of the changes reduce the code indentation level and simplifiy the code. No

Re: [PATCH] staging: tidspbridge: remove file handling functions for loader

2010-12-08 Thread Ramirez Luna, Omar
Hi, On Wed, Dec 8, 2010 at 4:26 PM, Greg KH g...@kroah.com wrote: On Tue, Dec 07, 2010 at 12:09:06AM -0600, Omar Ramirez Luna wrote: Instead use request_firmware and friends to get a valid firmware image. Right now the image is supplied dynamically through udev and the following rule:

Re: [PATCH] staging: tidspbridge: remove file handling functions for loader

2010-12-08 Thread Ramirez Luna, Omar
On Wed, Dec 8, 2010 at 5:08 PM, Greg KH g...@kroah.com wrote: On Wed, Dec 08, 2010 at 05:02:20PM -0600, Ramirez Luna, Omar wrote: Hi, On Wed, Dec 8, 2010 at 4:26 PM, Greg KH g...@kroah.com wrote: On Tue, Dec 07, 2010 at 12:09:06AM -0600, Omar Ramirez Luna wrote: Instead use

Re: [PATCH 1/6] OMAP: device: make rt_va easily avaialable to drivers

2010-12-08 Thread Ramirez Luna, Omar
On Wed, Dec 8, 2010 at 5:54 PM, Omar Ramirez Luna omar.rami...@ti.com wrote: Patch OMAP: hwmod/device: add omap_{device, hwmod}_get_mpu_rt_va[1], introduces omap_device_get_rt_va which is meant to be called by drivers to retrieve the _mpu_rt_va, however this function receives a pointer to an

Re: [PATCH 8/8] staging: tidspbridge - make sync_wait_on_event interruptible

2010-12-06 Thread Ramirez Luna, Omar
On Mon, Oct 25, 2010 at 5:17 PM, Guzman Lugo, Fernando fernando.l...@ti.com wrote: So that avoid nonkillable process. Signed-off-by: Fernando Guzman Lugo x0095...@ti.com --- .../staging/tidspbridge/include/dspbridge/sync.h | 13 +++-- 1 files changed, 11 insertions(+), 2

Re: [PATCH] staging: tidspbridge: overwrite DSP error codes

2010-12-06 Thread Ramirez Luna, Omar
On Wed, Nov 3, 2010 at 7:31 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 v4 1/3] staging: tidspbridge: fix mgr_enum_node_info

2010-12-06 Thread Ramirez Luna, Omar
On Fri, Nov 5, 2010 at 12:01 PM, Ionut Nicu ionut.n...@gmail.com wrote: From: Felipe Contreras felipe.contre...@gmail.com The current code was always returning a non-zero status value to userspace applications when this ioctl was called. The error code was ENODATA, which isn't actually an

Re: [PATCH v3 0/3] staging: tidspbridge: bugfixes

2010-12-06 Thread Ramirez Luna, Omar
On Fri, Nov 5, 2010 at 11:31 AM, Ionut Nicu ionut.n...@gmail.com wrote: Changes since v1: * Split the mgr_enum_node_info patch into two patches: one that fixes the issue and one that reorganizes the code. Ionut Nicu (3):  staging: tidspbridge: fix mgr_enum_node_info  staging: tidspbridge:

Re: [PATCH v3 00/12] staging: tidspbridge: various cleanups

2010-12-06 Thread Ramirez Luna, Omar
Hi Ionut, On Sun, Nov 21, 2010 at 4:46 AM, Ionut Nicu ionut.n...@gmail.com wrote: This set of patches replaces some of the redundant components of the tidspbridge driver, such as: * wrapper functions for kmalloc/kfree * custom bitmap implementation * custom linked list implementation

  1   2   3   4   >