[PATCH resend] OMAP: DSS2: Replace strncmp() with sysfs_streq() in overlay_manager_store()

2010-07-28 Thread Archit Taneja
In the function overlay_manager_store, the length of the string buf is used to comapre the 2 strings, there is a possibility of false positive match if buf is a prefix string of another manager. The use of sysfs_streq() fixes this. Signed-off-by: Archit Taneja arc...@ti.com ---

[PATCH] twl4030 reboot workaround

2010-07-28 Thread Mikko Rapeli
From: Mikko Rapeli ext-mikko.rap...@nokia.com Original patch: http://marc.info/?l=linux-omapm=126522625032441w=2 Removes TWL4030 sleep script prior to rebooting, only on OMAP3. This is necessary since DPLL3 reset causes SYS_OFFMODE pin to go low, resulting in the sleep script being executed on

Re: [RFC 1/8] TILER-DMM: DMM-PAT driver for TI TILER

2010-07-28 Thread Laurent Pinchart
Hi, On Tuesday 27 July 2010 21:05:11 Sin, David wrote: On Tuesday 27 July 2010 13:38:00 Hiremath, Vaibhav wrote: On Saturday 24 July 2010 04:52:00 Sin, David wrote: This patch adds support for DMM-PAT initialization and programming. [snip] diff --git

Re: [RFC 1/8] TILER-DMM: DMM-PAT driver for TI TILER

2010-07-28 Thread Laurent Pinchart
On Tuesday 27 July 2010 21:53:28 Hiremath, Vaibhav wrote: On Wednesday 28 July 2010 00:35:00 Sin, David wrote: On Tuesday 27 July 2010 01:38:00 Hiremath, Vaibhav wrote: On Saturday 24 July 2010 04:52:00 Sin, David wrote: [snip] [Hiremath, Vaibhav] If I understand correctly, DMM stands

RE: [PATCH] twl4030 reboot workaround

2010-07-28 Thread Gopinath, Thara
-Original Message- From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Mikko Rapeli Sent: Wednesday, July 28, 2010 3:12 PM To: linux-omap@vger.kernel.org Cc: Turquette, Mike; sa...@linux.intel.com Subject: [PATCH] twl4030 reboot workaround

Re: [PATCH] twl4030 reboot workaround

2010-07-28 Thread Peter 'p2' De Schrijver
On Wed, Jul 28, 2010 at 11:41:32AM +0200, Rapeli Mikko (EXT-Ixonos/Oulu) wrote: From: Mikko Rapeli ext-mikko.rap...@nokia.com Original patch: http://marc.info/?l=linux-omapm=126522625032441w=2 Removes TWL4030 sleep script prior to rebooting, only on OMAP3. This is necessary since DPLL3

Re: OMAP2 DSS Linux Driver

2010-07-28 Thread Tomi Valkeinen
On Wed, 2010-07-28 at 10:55 +0200, ext Nicholas Beck wrote: Hi Tomi, Firstly, I apologise if you would prefer this on a maillist rather than a direct mail. Please let me know if you would prefer this on a maillist I will post it there instead. cc'ing linux-omap mailing list is always good

Re: [PATCH 2/2] omap:mailbox-provide multiple reader support

2010-07-28 Thread Ohad Ben-Cohen
Hi Hari, On Wed, Jul 28, 2010 at 1:02 AM, Ohad Ben-Cohen o...@wizery.com wrote: On Wed, Jul 21, 2010 at 12:41 AM, Hari Kanigeri h-kanige...@ti.com wrote: This patch provides mutiple readers support for a mailbox instance. Signed-off-by: Hari Kanigeri h-kanige...@ti.com ---  

Re: OMAP2 DSS Linux Driver

2010-07-28 Thread Nicholas Beck
On 28/07/10 11:32, Tomi Valkeinen wrote: On Wed, 2010-07-28 at 10:55 +0200, ext Nicholas Beck wrote: Hi Tomi, Firstly, I apologise if you would prefer this on a maillist rather than a direct mail. Please let me know if you would prefer this on a maillist I will post it there instead.

RE: [PATCH resend] OMAP: DSS2: Replace strncmp() with sysfs_streq() in overlay_manager_store()

2010-07-28 Thread Premi, Sanjeev
-Original Message- From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Taneja, Archit Sent: Wednesday, July 28, 2010 11:52 AM To: tomi.valkei...@nokia.com Cc: linux-omap@vger.kernel.org; Taneja, Archit Subject: [PATCH resend] OMAP: DSS2:

RE: [PATCH 2/5] omap:hwspinlock-define HWSPINLOCK base address

2010-07-28 Thread Premi, Sanjeev
-Original Message- From: Kanigeri, Hari Sent: Tuesday, July 27, 2010 10:20 PM To: Premi, Sanjeev; Linux Omap; Tony Lindgren Cc: Shilimkar, Santosh; Cousson, Benoit; Que, Simon Subject: RE: [PATCH 2/5] omap:hwspinlock-define HWSPINLOCK base address Sanjeev, @@ -49,5

RE: [PATCH 2/5] omap:hwspinlock-define HWSPINLOCK base address

2010-07-28 Thread Kanigeri, Hari
Sanjeev, -Original Message- From: Premi, Sanjeev Sent: Wednesday, July 28, 2010 9:12 AM To: Kanigeri, Hari; Linux Omap; Tony Lindgren Cc: Shilimkar, Santosh; Cousson, Benoit; Que, Simon Subject: RE: [PATCH 2/5] omap:hwspinlock-define HWSPINLOCK base address -Original

RE: [PATCH 2/5] omap:hwspinlock-define HWSPINLOCK base address

2010-07-28 Thread Premi, Sanjeev
-Original Message- From: Kanigeri, Hari Sent: Wednesday, July 28, 2010 7:53 PM To: Premi, Sanjeev; Linux Omap; Tony Lindgren Cc: Shilimkar, Santosh; Cousson, Benoit; Que, Simon Subject: RE: [PATCH 2/5] omap:hwspinlock-define HWSPINLOCK base address [snip] Sanjeev,

[PATCH 0/5] staging:ti dspbridge: Checkpatch errors cleanup series

2010-07-28 Thread Ernesto Ramos
These patches are targetted to remove checkpatch errors within dspbridge driver. Ernesto Ramos (5): staging:ti dspbridge: remove unused typedef REG16 staging:ti dspbridge: remove unnecessary check for NULL pointer in cmm.c staging:ti dspbridge: remove function delete_strm_mgr

[PATCH 1/5] staging:ti dspbridge: remove unused typedef REG16

2010-07-28 Thread Ernesto Ramos
Remove unused typedef REG16 in dspbridge. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/core/_tiomap.h |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/drivers/staging/tidspbridge/core/_tiomap.h b/drivers/staging/tidspbridge/core/_tiomap.h

RE: [PATCH resend] OMAP: DSS2: Replace strncmp() with sysfs_streq() in overlay_manager_store()

2010-07-28 Thread Taneja, Archit
-Original Message- From: Premi, Sanjeev Sent: Wednesday, July 28, 2010 7:28 PM To: Taneja, Archit; tomi.valkei...@nokia.com Cc: linux-omap@vger.kernel.org Subject: RE: [PATCH resend] OMAP: DSS2: Replace strncmp() with sysfs_streq() in overlay_manager_store() -Original

[PATCH 4/5] staging:ti dspbridge: remove unnecessary volatile variables

2010-07-28 Thread Ernesto Ramos
Remove unnecessary volatile variables; use accessor functions __raw_readl/__raw_writel instead when applicable. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/core/tiomap3430.c |8 drivers/staging/tidspbridge/dynload/tramp.c |4 ++--

[PATCH 2/5] staging:ti dspbridge: remove unnecessary check for NULL pointer in cmm.c

2010-07-28 Thread Ernesto Ramos
Remove unnecessary check for NULL pointer in cmm.c. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/pmgr/cmm.c |8 ++-- 1 files changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/staging/tidspbridge/pmgr/cmm.c

[PATCH 3/5] staging:ti dspbridge: remove function delete_strm_mgr

2010-07-28 Thread Ernesto Ramos
Remove function delete_strm_mgr in strm.c and use kfree instead. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/rmgr/strm.c | 16 ++-- 1 files changed, 2 insertions(+), 14 deletions(-) diff --git a/drivers/staging/tidspbridge/rmgr/strm.c

[PATCH 5/5] staging:ti dspbridge: replace simple_strtoul by strict_strtoul

2010-07-28 Thread Ernesto Ramos
Replace simple_strtoul by strict_strtoul in atoi function. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/rmgr/dbdcd.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/drivers/staging/tidspbridge/rmgr/dbdcd.c

RE: [RFC 1/8] TILER-DMM: DMM-PAT driver for TI TILER

2010-07-28 Thread Sin, David
-Original Message- From: Laurent Pinchart [mailto:laurent.pinch...@ideasonboard.com] Sent: Wednesday, July 28, 2010 5:00 AM To: linux-arm-ker...@lists.infradead.org Cc: Hiremath, Vaibhav; Sin, David; linux-omap@vger.kernel.org; Tony Lindgren; Russell King; Ohad Ben-Cohen;

[PATCH 04/10] staging:ti dspbridge: remove DSP_SUCCEEDED macro from services

2010-07-28 Thread Ernesto Ramos
Since status succeeded is now 0 macro DSP_SUCCEEDED is not necessary anymore. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/services/cfg.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/tidspbridge/services/cfg.c

[PATCH 00/10] staging:ti dspbridge: Remove DSP_FAILED and DSP_SUCCEEDED macros

2010-07-28 Thread Ernesto Ramos
This series of patches is targetted to remove macros DSP_FAILED and DSP_SUCCEEDED since bridge driver currently uses standard kernel error codes. Ernesto Ramos (10): staging:ti dspbridge: remove DSP_SUCCEEDED macro from core staging:ti dspbridge: remove DSP_SUCCEEDED macro from pmgr

[PATCH 09/10] staging:ti dspbridge: remove DSP_FAILED macro from services

2010-07-28 Thread Ernesto Ramos
Since status succeeded is 0, DSP_FAILED macro is not necessary anymore. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/services/cfg.c | 13 ++--- 1 files changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers/staging/tidspbridge/services/cfg.c

[PATCH 06/10] staging:ti dspbridge: remove DSP_FAILED macro from core

2010-07-28 Thread Ernesto Ramos
Since status succeeded is 0, DSP_FAILED macro is not necessary anymore. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/core/chnl_sm.c| 20 +++--- drivers/staging/tidspbridge/core/io_sm.c | 64 ++--

[PATCH 01/10] staging:ti dspbridge: remove DSP_SUCCEEDED macro from core

2010-07-28 Thread Ernesto Ramos
Since status succeeded is now 0 macro DSP_SUCCEEDED is not necessary anymore. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/core/chnl_sm.c| 22 drivers/staging/tidspbridge/core/dsp-clock.c |4 +- drivers/staging/tidspbridge/core/io_sm.c

[PATCH 08/10] staging:ti dspbridge: remove DSP_FAILED macro from rmgr

2010-07-28 Thread Ernesto Ramos
Since status succeeded is 0, DSP_FAILED macro is not necessary anymore. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/rmgr/dbdcd.c | 24 drivers/staging/tidspbridge/rmgr/disp.c | 28 drivers/staging/tidspbridge/rmgr/drv.c

[PATCH 05/10] staging:ti dspbridge: remove DSP_SUCCEEDED macro definition

2010-07-28 Thread Ernesto Ramos
Since status succeeded is now 0 macro DSP_SUCCEEDED is not necessary anymore. This patch removes this define. Signed-off-by: Ernesto Ramos erne...@ti.com --- .../staging/tidspbridge/include/dspbridge/dbdefs.h |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git

[PATCH 07/10] staging:ti dspbridge: remove DSP_FAILED macro from pmgr

2010-07-28 Thread Ernesto Ramos
Since status succeeded is 0, DSP_FAILED macro is not necessary anymore. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/pmgr/chnl.c |2 +- drivers/staging/tidspbridge/pmgr/cmm.c|2 +- drivers/staging/tidspbridge/pmgr/cod.c|8 +++---

[PATCH 02/10] staging:ti dspbridge: remove DSP_SUCCEEDED macro from pmgr

2010-07-28 Thread Ernesto Ramos
Since status succeeded is now 0 macro DSP_SUCCEEDED is not necessary anymore. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/pmgr/chnl.c |8 +- drivers/staging/tidspbridge/pmgr/cmm.c| 135 +

[PATCH 10/10] staging:ti dspbridge: remove DSP_FAILED macro definition

2010-07-28 Thread Ernesto Ramos
Since status succeeded is now 0 macro DSP_FAILED is not necessary anymore. This patch removes this define. Signed-off-by: Ernesto Ramos erne...@ti.com --- .../staging/tidspbridge/include/dspbridge/dbdefs.h |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git

[PATCH 3/4] staging:ti dspbridge: make sure IVA2 is OFF when dev is created

2010-07-28 Thread Ernesto Ramos
Regardless the IVA2 power state before bridgedriver is installed, the driver must ensure that IVA2 power state is OFF when the device driver is created. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/core/tiomap3430.c |4 +++- 1 files changed, 3 insertions(+), 1

[PATCH 0/4] staging:ti dspbridge: Fix IVA2 power OFF series

2010-07-28 Thread Ernesto Ramos
This series of patches fixes issues that prevent IVA2 to turn OFF when there are issues to load the base image and makes sure IVA2 switched OFF when dsp bridge driver is installed. Ernesto Ramos (4): staging:ti dspbridge: fix bridge_brd_stop so IVA2 is set OFF staging:ti dspbridge:

[PATCH 4/4] staging:ti dspbridge: remove bridge_brd_delete function

2010-07-28 Thread Ernesto Ramos
From: Ernesto Ramos erne...@ernesto-desktop.(none) Remove bridge_brd_delete() function since it is doing the same that bridge_brd_stop(). Signed-off-by: Ernesto Ramos erne...@ernesto-desktop.(none) --- drivers/staging/tidspbridge/core/tiomap3430.c | 57 + 1 files

[PATCH 1/4] staging:ti dspbridge: fix bridge_brd_stop so IVA2 is set OFF

2010-07-28 Thread Ernesto Ramos
right now, bridge_brd_stop is not changing the IVA2 power state to OFF since PM_PWSTST_IVA2 is not 0 after calling this function. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/core/tiomap3430.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff

[PATCH 2/4] staging:ti dspbridge: proc_load/start should set IVA2 to OFF in case of failure

2010-07-28 Thread Ernesto Ramos
When a base image is being loaded or started and by some reason the process fails, the IVA2 should be switched OFF. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/rmgr/proc.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git

[PATCH] staging:ti dspbridge: remove find_gcf from nldr.c

2010-07-28 Thread Ernesto Ramos
Remove find_gcf from nldr.c and use kernel function gcd(). Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/rmgr/nldr.c | 20 ++-- 1 files changed, 2 insertions(+), 18 deletions(-) diff --git a/drivers/staging/tidspbridge/rmgr/nldr.c

RE: [PATCH 2/5] omap:hwspinlock-define HWSPINLOCK base address

2010-07-28 Thread Kanigeri, Hari
Sanjeev, @@ -49,5 +49,10 @@ #define OMAP44XX_MAILBOX_BASE (L4_44XX_BASE + 0xF4000) #define OMAP44XX_HSUSB_OTG_BASE (L4_44XX_BASE + 0xAB000) +#define OMAP4_MMU1_BASE0x55082000 +#define OMAP4_MMU2_BASE

Re: [PATCH 4/5] staging:ti dspbridge: remove unnecessary volatile variables

2010-07-28 Thread Nishanth Menon
Ramos Falcon, Ernesto had written, on 07/28/2010 09:40 AM, the following: Remove unnecessary volatile variables; use accessor functions __raw_readl/__raw_writel instead when applicable. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/core/tiomap3430.c |8

Re: [PATCH 2/5] staging:ti dspbridge: remove unnecessary check for NULL pointer in cmm.c

2010-07-28 Thread Nishanth Menon
Ramos Falcon, Ernesto had written, on 07/28/2010 09:40 AM, the following: Remove unnecessary check for NULL pointer in cmm.c. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/pmgr/cmm.c |8 ++-- 1 files changed, 2 insertions(+), 6 deletions(-) diff --git

Re: [PATCH 5/5] staging:ti dspbridge: replace simple_strtoul by strict_strtoul

2010-07-28 Thread Greg KH
On Wed, Jul 28, 2010 at 09:40:52AM -0500, Ernesto Ramos wrote: Replace simple_strtoul by strict_strtoul in atoi function. Why not use the built-in kernel function instead of having your own atoi function? thanks, greg k-h -- To unsubscribe from this list: send the line unsubscribe linux-omap

RE: [RFC 0/8] TI TILER-DMM driver

2010-07-28 Thread Sin, David
-Original Message- From: Hans Verkuil [mailto:hverk...@xs4all.nl] Sent: Saturday, July 24, 2010 6:13 AM To: Sin, David Cc: linux-arm-ker...@lists.arm.linux.org.uk; linux-omap@vger.kernel.org; Tony Lindgren; Russell King; Kanigeri, Hari; Ohad Ben-Cohen; Hiremath, Vaibhav;

Re: [PATCH 2/5] staging:ti dspbridge: remove unnecessary check for NULL pointer in cmm.c

2010-07-28 Thread Nishanth Menon
Nishanth Menon had written, on 07/28/2010 10:04 AM, the following: Ramos Falcon, Ernesto had written, on 07/28/2010 09:40 AM, the following: Remove unnecessary check for NULL pointer in cmm.c. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/pmgr/cmm.c |8

RE: [RFC 1/8] TILER-DMM: DMM-PAT driver for TI TILER

2010-07-28 Thread Hiremath, Vaibhav
-Original Message- From: Sin, David Sent: Wednesday, July 28, 2010 8:09 PM To: Laurent Pinchart; linux-arm-ker...@lists.infradead.org Cc: Hiremath, Vaibhav; linux-omap@vger.kernel.org; Tony Lindgren; Russell King; Ohad Ben-Cohen; Kanigeri, Hari; Shilimkar, Santosh; Molnar, Lajos

Re: [PATCH] twl4030 reboot workaround

2010-07-28 Thread Mike Turquette
Mikko Rapeli wrote: From: Mikko Rapeli ext-mikko.rap...@nokia.com Original patch: http://marc.info/?l=linux-omapm=126522625032441w=2 I forgot about this one... Removes TWL4030 sleep script prior to rebooting, only on OMAP3. This is necessary since DPLL3 reset causes SYS_OFFMODE pin to go

[PATCH] staging:ti dspbridge: avoid possible NULL dereference panic

2010-07-28 Thread Ernesto Ramos
When dsp_notifications array is received from user, dspbridge verifies the array has valid pointers and dsp_notification structures. However, these structures contain pointers that need to be checked for valid handles. Signed-off-by: Ernesto Ramos erne...@ti.com ---

RE: [PATCH 2/5] staging:ti dspbridge: remove unnecessary check for NULL pointer in cmm.c

2010-07-28 Thread Ramos Falcon, Ernesto
-Original Message- From: Menon, Nishanth Sent: Wednesday, July 28, 2010 10:32 AM To: Ramos Falcon, Ernesto Cc: gre...@suse.de; Ramirez Luna, Omar; o...@wizery.com; ameya.pala...@nokia.com; felipe.contre...@nokia.com; Guzman Lugo, Fernando; linux-ker...@vger.kernel.org;

RE: [PATCH 3/5] omap:hwspinlock-added hwspinlock driver

2010-07-28 Thread Premi, Sanjeev
-Original Message- From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Kanigeri, Hari Sent: Monday, July 19, 2010 10:20 PM To: Linux Omap; Tony Lindgren Cc: Shilimkar, Santosh; Cousson, Benoit; Que, Simon; Kanigeri, Hari Subject: [PATCH 3/5]

RE: [PATCH 5/5] omap:hwspinlocks-ensure the order of registration

2010-07-28 Thread Premi, Sanjeev
-Original Message- From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Kanigeri, Hari Sent: Monday, July 19, 2010 10:20 PM To: Linux Omap; Tony Lindgren Cc: Shilimkar, Santosh; Cousson, Benoit; Que, Simon; Kanigeri, Hari Subject: [PATCH

RE: [PATCH resend] OMAP: DSS2: Replace strncmp() with sysfs_streq() in overlay_manager_store()

2010-07-28 Thread Premi, Sanjeev
-Original Message- From: Taneja, Archit Sent: Wednesday, July 28, 2010 8:06 PM To: Premi, Sanjeev; tomi.valkei...@nokia.com Cc: linux-omap@vger.kernel.org Subject: RE: [PATCH resend] OMAP: DSS2: Replace strncmp() with sysfs_streq() in overlay_manager_store()

RE: [PATCH 5/5] omap:hwspinlocks-ensure the order of registration

2010-07-28 Thread Kanigeri, Hari
Sanjeev, Thanks for your comments. -Original Message- From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Kanigeri, Hari Sent: Monday, July 19, 2010 10:20 PM To: Linux Omap; Tony Lindgren Cc: Shilimkar, Santosh; Cousson, Benoit;

Re: [PATCH 2/5] staging:ti dspbridge: remove unnecessary check for NULL pointer in cmm.c

2010-07-28 Thread Felipe Contreras
Hi Ernesto, On Wed, Jul 28, 2010 at 7:53 PM, Ramos Falcon, Ernesto erne...@ti.com wrote: here is a better approach: remove cmm_xlator_delete altogether [...] I considered this approach before but in terms of maintainability I thought it was easier to locate where translator tables are

RE: [PATCH 2/5] staging:ti dspbridge: remove unnecessary check for NULL pointer in cmm.c

2010-07-28 Thread Ramos Falcon, Ernesto
Agree! I'll send new version of this patch. -Original Message- From: Felipe Contreras [mailto:felipe.contre...@gmail.com] Sent: Wednesday, July 28, 2010 12:12 PM To: Ramos Falcon, Ernesto Cc: Menon, Nishanth; gre...@suse.de; Ramirez Luna, Omar; o...@wizery.com; ameya.pala...@nokia.com;

RE: [RFC 0/8] TI TILER-DMM driver

2010-07-28 Thread Hiremath, Vaibhav
-Original Message- From: Sin, David Sent: Wednesday, July 28, 2010 8:53 PM To: Hans Verkuil Cc: linux-omap@vger.kernel.org; Tony Lindgren; Russell King; Kanigeri, Hari; Ohad Ben-Cohen; Hiremath, Vaibhav; Shilimkar, Santosh; linux-arm- ker...@lists.infradead.org Subject: RE: [RFC

Re: [PATCH 5/5] staging:ti dspbridge: replace simple_strtoul by strict_strtoul

2010-07-28 Thread Andy Shevchenko
On Wed, Jul 28, 2010 at 6:09 PM, Greg KH gre...@suse.de wrote: On Wed, Jul 28, 2010 at 09:40:52AM -0500, Ernesto Ramos wrote: Replace simple_strtoul by strict_strtoul in atoi function. Why not use the built-in kernel function instead of having your own atoi function? The commit message left

Re: [PATCH 5/5] staging:ti dspbridge: replace simple_strtoul by strict_strtoul

2010-07-28 Thread Andy Shevchenko
On Wed, Jul 28, 2010 at 5:40 PM, Ernesto Ramos erne...@ti.com wrote: Replace simple_strtoul by strict_strtoul in atoi function. In general the question what is the purpose here to do the change? @@ -1023,7 +1025,9 @@ static s32 atoi(char *psz_buf)                base = 16;        } -      

Re: [PATCH 5/5] staging:ti dspbridge: replace simple_strtoul by strict_strtoul

2010-07-28 Thread Greg KH
On Wed, Jul 28, 2010 at 09:40:19PM +0300, Andy Shevchenko wrote: On Wed, Jul 28, 2010 at 6:09 PM, Greg KH gre...@suse.de wrote: On Wed, Jul 28, 2010 at 09:40:52AM -0500, Ernesto Ramos wrote: Replace simple_strtoul by strict_strtoul in atoi function. Why not use the built-in kernel

RE: [RFC 0/8] TI TILER-DMM driver

2010-07-28 Thread Sin, David
-Original Message- From: Hiremath, Vaibhav Sent: Wednesday, July 28, 2010 12:31 PM To: Sin, David; Hans Verkuil Cc: linux-omap@vger.kernel.org; Tony Lindgren; Russell King; Kanigeri, Hari; Ohad Ben-Cohen; Shilimkar, Santosh; linux-arm-ker...@lists.infradead.org Subject: RE: [RFC

Re: [PATCH v2 03/20] mmc: support embedded data field in mmc_host

2010-07-28 Thread Vitaly Wool
Hi Ohad, On Wed, Jul 21, 2010 at 7:33 PM, Ohad Ben-Cohen o...@wizery.com wrote: Add support to set/get mmc_host private embedded data. This is needed to allow software to dynamically create (and remove) SDIO functions which represents embedded SDIO devices. snip @@ -209,6 +209,8 @@ struct

[PATCH 0/5] use idr kernel library instead of handles

2010-07-28 Thread Ernesto Ramos
From: Rene Sapiens rene.sapi...@ti.com Use idr kernel library to send/receive node and stream ids to the user instead of kernel address. This id will be use to access the node and stream handles at the kernel side, if id does not match to any handle error -EFAULT is returned. For processor

[PATCH] staging:ti dspbridge: use stream id instead of kernel address

2010-07-28 Thread Ernesto Ramos
Send stream ids to the user instead of handles, then when the id is coming from user dspbridge can retrive the handle using id and avoid using invalid handles. Signed-off-by: Ernesto Ramos erne...@ti.com --- .../staging/tidspbridge/include/dspbridge/drv.h|5 +-

[PATCH] staging:ti dspbridge: use node id instead of kernel address

2010-07-28 Thread Ernesto Ramos
Use idr kernel library to send/receive node ids to the user instead of kernel address. This id will be use to access the node handles at the kernel side, if id does not match to any handle error -EFAULT is returned. Signed-off-by: Ernesto Ramos erne...@ti.com ---

[PATCH] staging:ti dspbridge: avoid errors if node handle is zero

2010-07-28 Thread Ernesto Ramos
As 'zero' can be a perfectly good id, it can be picked up as a NULL from userspace, avoid issues in API and user apps if node handle is zero. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/pmgr/dspapi.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-)

[PATCH] staging:ti dspbridge: avoid errors if stream id is zero

2010-07-28 Thread Ernesto Ramos
As 'zero' can be a perfectly good id, it can be picked up as a NULL from userspace, avoid issues in API and user apps if stream handle is zero. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/pmgr/dspapi.c |8 ++-- 1 files changed, 6 insertions(+), 2

[PATCH] staging:ti dspbridge: use processor handle from context instead of user's

2010-07-28 Thread Ernesto Ramos
Make sure dspbridge driver uses a valid processor handle by using the handle stored in process context. Signed-off-by: Ernesto Ramos erne...@ti.com --- drivers/staging/tidspbridge/pmgr/dspapi.c | 50 + 1 files changed, 29 insertions(+), 21 deletions(-) diff --git

RE: [PATCH resend] OMAP: DSS2: Replace strncmp() with sysfs_streq() in overlay_manager_store()

2010-07-28 Thread Taneja, Archit
-Original Message- From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Taneja, Archit Sent: Wednesday, July 28, 2010 11:52 AM To: tomi.valkei...@nokia.com Cc: linux-omap@vger.kernel.org; Taneja, Archit Subject:

RE: [PATCH 3/5] omap:hwspinlock-added hwspinlock driver

2010-07-28 Thread Marathe, Yogesh
-Original Message- From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap- ow...@vger.kernel.org] On Behalf Of Premi, Sanjeev Sent: Wednesday, July 28, 2010 10:28 PM To: Kanigeri, Hari; Linux Omap; Tony Lindgren Cc: Shilimkar, Santosh; Cousson, Benoit; Que, Simon Subject: RE:

RE: [PATCH resend] OMAP: DSS2: Replace strncmp() with sysfs_streq() in overlay_manager_store()

2010-07-28 Thread Premi, Sanjeev
-Original Message- From: Taneja, Archit Sent: Thursday, July 29, 2010 9:16 AM To: Premi, Sanjeev; tomi.valkei...@nokia.com Cc: linux-omap@vger.kernel.org Subject: RE: [PATCH resend] OMAP: DSS2: Replace strncmp() with sysfs_streq() in overlay_manager_store() -Original

RE: [PATCH resend] OMAP: DSS2: Replace strncmp() with sysfs_streq() in overlay_manager_store()

2010-07-28 Thread Taneja, Archit
-Original Message- From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Taneja, Archit Sent: Wednesday, July 28, 2010 11:52 AM To: tomi.valkei...@nokia.com Cc: linux-omap@vger.kernel.org; Taneja, Archit