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
---
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
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
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
-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
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
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
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
---
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.
-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:
-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
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
-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,
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
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
-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
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 ++--
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
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
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
-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;
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
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
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
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 ++--
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
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
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
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 +++---
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 +
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
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
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:
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
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
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
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
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
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
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
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
-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;
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
-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
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
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
---
-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;
-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]
-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
-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()
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;
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
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;
-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
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
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;
}
-
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
-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
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
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
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 +-
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
---
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(-)
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
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
-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:
-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:
-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
-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
69 matches
Mail list logo