Hi Paul,
On Tue, May 14, 2013 at 3:28 PM, Paul Bolle pebo...@tiscali.nl wrote:
0) I ran into two Kconfig oddities with the Dynamic Voltage and
Frequency Scaling (DVFS) option for the DSP Bridge driver (ie, Kconfig
symbol TIDSPBRIDGE_DVFS).
1) That Kconfig symbol depends on OMAP_PM_SRF. But
Hi,
On Thu, Feb 28, 2013 at 11:51 AM, Pali Rohár pali.ro...@gmail.com wrote:
Fix includes and use clk_prepare_enable/clk_disable_unprepare
Signed-off-by: Pali Rohár pali.ro...@gmail.com
Signed-off-by: Joni Lapilainen joni.lapilai...@gmail.com
---
Hi Greg,
On Thu, Jan 17, 2013 at 6:47 PM, Greg Kroah-Hartman
gre...@linuxfoundation.org wrote:
On Thu, Jan 10, 2013 at 03:36:57AM -0600, Omar Ramirez Luna wrote:
Patches for staging-next, fixing comments and suggestions provided
by Chen Gang.
There is an additional scm patch, that removes
Hi Tony,
On Thu, Jan 17, 2013 at 8:01 PM, Tony Lindgren t...@atomide.com wrote:
* Greg Kroah-Hartman gre...@linuxfoundation.org [130117 16:51]:
On Thu, Jan 10, 2013 at 03:36:57AM -0600, Omar Ramirez Luna wrote:
Patches for staging-next, fixing comments and suggestions provided
by Chen Gang
Patches for staging-next, fixing comments and suggestions provided
by Chen Gang.
There is an additional scm patch, that removes hardcoded defines
related to direct register handling for SCM, it was dependent on
changes that already made it to mainline.
Omar Ramirez Luna (5):
staging
, because the driver expects the NULL
ending to be among the 255 char limit.
Reported-by: Chen Gang gang.c...@asianux.com
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
---
drivers/staging/tidspbridge/rmgr/proc.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
The value allocated doesn't match the one that is meant to be
stored, resulting in corruption of memory for longer strings
that can't be held in such space.
Fix by allocating the correct byte value for the string meant to
be stored.
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
On both counts, sym_name could be printed uninitialized, this
is solved by moving the pr_* statement to be triggered if the
value is assigned.
Reported-by: Chen Gang gang.c...@asianux.com
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
---
drivers/staging/tidspbridge/rmgr/nldr.c |6
Instead of ioremapping SCM registers, use the correspondent layer
to write into them.
This allows us to get rid of a layer violation, since the registers
are no longer touched by driver code.
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
---
drivers/staging/tidspbridge/core
On Mon, Jan 7, 2013 at 5:03 PM, Greg Kroah-Hartman
gre...@linuxfoundation.org wrote:
On Mon, Dec 24, 2012 at 08:10:24AM -0600, Omar Ramirez Luna wrote:
3.8-rc1 introduced changes in the clock management header files,
this resulted in compilation breakages for this driver.
Define this locally
: error:
'OMAP3430_CM_AUTOIDLE_PLL' undeclared (first use in this function)
drivers/staging/tidspbridge/core/tiomap_io.c:416:13: error:
'OMAP3430_CM_CLKEN_PLL' undeclared (first use in this function)
Reported-by: Chen Gang gang.c...@asianux.com
Signed-off-by: Omar Ramirez Luna omar.rami
in the correct
order while handling clocks.
Code path to enable/disable dsp clocks can still be reached from an
atomic context, hence we can't use clk_prepare_enable and
clk_disable_unprepare yet.
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
---
drivers/staging/tidspbridge/core/dsp-clock.c
Hi Greg,
On 30 October 2012 16:02, Greg Kroah-Hartman gre...@linuxfoundation.org wrote:
OK.
Greg, do these patches look OK to you to move to live under
drivers/mailbox?
Um, I don't know, I wasn't paying attention here, sorry.
As part of plat-omap code cleanup, I was planning to move
Tony,
On 29 October 2012 12:52, Tony Lindgren t...@atomide.com wrote:
--- /dev/null
+++ b/include/linux/platform_data/omap_mailbox.h
@@ -0,0 +1,105 @@
This file should only contain pure platform data needed
by the core omap code to pass to the mailbox driver.
Ok, looking at it closely,
Ramirez Luna omar.rami...@copitl.com
---
Intended for 3.7 due to code changes during rc1.
.../tidspbridge/include/dspbridge/host_os.h|4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/tidspbridge/include/dspbridge/host_os.h
b/drivers/staging
is of type 'const
void *'
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
---
Intended for 3.7 due to code changes during rc1.
drivers/staging/tidspbridge/hw/hw_mmu.c | 40 +++
drivers/staging/tidspbridge/hw/hw_mmu.h | 28 +++---
2 files changed
writeback addressing modes for
__raw_ accessors, so the build system was filled with warnings from
the old parameter usage.
Omar Ramirez Luna (6):
staging: tidspbridge: request the right irq for mmu
staging: tidspbridge: drop const from custom mmu implementation
staging: tidspbridge: change type
from integer
without a cast
../io.h:88: note: expected 'volatile void *' but argument is of type 'u32'
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
---
Intended for 3.7 due to code changes during rc1.
.../tidspbridge/include/dspbridge/cfgdefs.h|4 ++--
drivers/staging
without a cast
../io.h:88: note: expected 'volatile void *' but argument is of type 'u32'
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
---
Intended for 3.7 due to code changes during rc1.
drivers/staging/tidspbridge/core/tiomap3430.c | 37 +
1 file changed, 26
Due to data type change, readl can no longer receive a u32.
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
---
Intended for 3.7 due to code changes during rc1.
drivers/staging/tidspbridge/rmgr/node.c | 21 +++--
1 file changed, 15 insertions(+), 6 deletions(-)
diff
From: Omar Ramirez Luna omar.l...@linaro.org
This should get rid of warnings of the type:
warning: passing argument 1 of '' discards qualifiers from pointer target type
note: expected 'void *' but argument is of type 'const void *'
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
Hi,
On Wed, Oct 24, 2012 at 5:28 PM, Greg Kroah-Hartman
gre...@linuxfoundation.org wrote:
On Wed, Oct 24, 2012 at 05:09:14PM -0500, Omar Ramirez Luna wrote:
With 3.7-rc1 changes:
- New irq numbering in OMAP3 broke the driver request for a mmu irq,
until this is migrated to the common iommu
Hi,
On Mon, Sep 24, 2012 at 1:54 PM, selso selso.liber...@gmail.com wrote:
From: sli sli@SLI-V420.(none)
Signed-off-by: sli sli@SLI-V420.(none)
---
drivers/staging/tidspbridge/core/dsp-clock.c |3 ++
drivers/staging/tidspbridge/core/tiomap3430.c |4 ++
by the firmware. E.g.:
tidspbridge sees: 24BC8D90-BB45-11D4-B756-006008BDB66F
firmware expects: .24BC8D90_BB45_11D4_B756_006008BDB66F
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
CC: Andy Shevchenko andriy.shevche...@linux.intel.com
---
drivers/staging/tidspbridge/rmgr/dbdcd.c | 27
This fixes the compilation warnings that have been showing up lately.
Omar Ramirez Luna (5):
staging: tidspbridge: add pud code
staging: tidspbridge: split bridge_io_on_loaded
staging: tidspbridge: dynamically allocate ibuf in dload_data
staging: tidspbridge: dynamically allocate
*' but argument is of
type 'pmdval_t (*)[2]'
While at it, eliminate 'if' nesting to increase readability.
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
---
drivers/staging/tidspbridge/core/tiomap3430.c | 27 +++-
1 files changed, 17 insertions(+), 10 deletions
Ramirez Luna omar.rami...@copitl.com
---
drivers/staging/tidspbridge/core/io_sm.c | 704 +++---
1 files changed, 343 insertions(+), 361 deletions(-)
diff --git a/drivers/staging/tidspbridge/core/io_sm.c
b/drivers/staging/tidspbridge/core/io_sm.c
index c51f651..480a384
Dynamically allocate ibuf to silence the following warning:
drivers/staging/tidspbridge/dynload/cload.c:
In function 'dload_data':
drivers/staging/tidspbridge/dynload/cload.c:1337:
warning: the frame size of 1216 bytes is larger than 1024 bytes
Signed-off-by: Omar Ramirez Luna omar.rami
Dynamically allocate my_sym_buf to silence the following warning:
drivers/staging/tidspbridge/dynload/cload.c:
In function 'dload_symbols':
drivers/staging/tidspbridge/dynload/cload.c:890:
warning: the frame size of 1040 bytes is larger than 1024 bytes
Signed-off-by: Omar Ramirez Luna
it was replaced for kstrtol
as the former is obsolete now.
Signed-off-by: Omar Ramirez Luna omar.rami...@copitl.com
---
drivers/staging/tidspbridge/pmgr/dbll.c |7 +--
1 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/tidspbridge/pmgr/dbll.c
b/drivers/staging
2012 01:07
Reported-by: Tony Lindgren t...@atomide.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/core/tiomap3430.c | 20
drivers/staging/tidspbridge/core/wdt.c|8 +++-
2 files changed, 19 insertions(+), 9 deletions
seen by IVA_SS at address 0
Omar Ramirez Luna (2):
staging: tidspbridge: always compile dsp wdt code
staging: tidspbridge: detect wdt3 feature at runtime
drivers/staging/tidspbridge/Kconfig| 16 -
drivers/staging/tidspbridge/core/tiomap3430.c | 10
In order to detect it at runtime, we need the code handling wdt
clock available at runtime to decide whether to enable or disable
based on the baseimage symbols. Default timeout has been set to 5
seconds.
Downside is that we will lose the option to set a custom timeout
for overflow, but than can
...
That occurs because the dsp tries to access wdt3 registers when the clock
for those registers is not enabled.
Reported-by: Felipe Contreras felipe.contre...@gmail.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/core/tiomap3430.c | 10 +++---
1 files
to take (between this and
default y for CONFIG_TIDSPBRIDGE_WDT3[1]).
[1] http://www.mail-archive.com/linux-omap@vger.kernel.org/msg62119.html
Omar Ramirez Luna (2):
staging: tidspbridge: always compile dsp wdt code
staging: tidspbridge: detect wdt3 feature at runtime
drivers/staging
In order to detect it at runtime, we need the code handling wdt
clock available at runtime to decide whether to enable or disable
based on the baseimage symbols. Default timeout has been set to 5
seconds.
Downside is that we will lose the option to set a custom timeout
for overflow, but than can
...
That occurs because the dsp tries to access wdt3 registers when the clock
for those registers is not enabled.
Reported-by: Felipe Contreras felipe.contre...@gmail.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/core/tiomap3430.c | 10 +++---
1 files
On Tue, Jan 31, 2012 at 2:17 AM, Dan Carpenter dan.carpen...@oracle.com wrote:
On Mon, Jan 30, 2012 at 07:20:17PM -0600, Omar Ramirez Luna wrote:
There are two members of pr_ctxt allocated during bridge_open that
are never freed resulting in memory leaks, these are stream_id and
node_id
triggered
error paths during module removal leaking some memory.
When accepted they must be propagated to stable kernel releases.
Omar Ramirez Luna (2):
staging: tidspbridge: fix bridge_open memory leaks
staging: tidspbridge: fix incorrect free to drv_datap
drivers/staging/tidspbridge/core
variables
could result in memory leaking due to missing handling of failure
scenarios. While at it, the indentation changes were introduced to
avoid interleaved goto statements inside big if blocks.
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/rmgr/drv_interface.c
drv_get_first_dev_extension: Failed to retrieve the object handle
drv_destroy: Failed to store DRV object
mgr_destroy: Failed to store MGR object
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/core/tiomap3430.c|2 --
drivers/staging/tidspbridge/rmgr
Given that dm timer framework doesn't support request of clocks
by soft | hard irqs because some recent changes, tidspbridge needs
to request its clocks on init and enable/disable them on demand.
This was first seen on 3.2-rc1.
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers
Given that dm timer framework doesn't support request of clocks
by soft | hard irqs because some recent changes, tidspbridge needs
to request its clocks on init and enable/disable them on demand.
This was first seen on 3.2-rc1.
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers
/drv_interface.c:359: error: for each function
it appears in.)
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/rmgr/drv_interface.c |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/tidspbridge/rmgr/drv_interface.c
b/drivers
Seen on v3.1-rc3, patch:
omap: mcbsp: Drop in-driver transfer support
bafe2721a0fbd1cc1af04384133684f660f3658e
Removed code that now cause tidspbridge to break while compiling.
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/core/dsp-clock.c |1 -
1
Create function find_containing_mapping_da to search mapping objects
given the dsp va instead of the mpu va, in preparation to delete dmm
functions altogether.
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/rmgr/proc.c | 41
Guzman Lugo x0095...@ti.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/Kconfig| 10 +
drivers/staging/tidspbridge/core/_deh.h|2 +
drivers/staging/tidspbridge/core/_tiomap.h | 16 +-
drivers/staging/tidspbridge
...@ti.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/Kconfig|1 +
drivers/staging/tidspbridge/core/tiomap3430.c | 94 +++-
.../tidspbridge/include/dspbridge/dspdefs.h|3 +-
drivers/staging/tidspbridge/rmgr
:
OMAP: iovmm: fix SW flags passed by user[2].
[1] http://bit.ly/lUxbRO
[2] http://bit.ly/mgSrr8
Fernando Guzman Lugo (3):
staging: tidspbridge: remove custom mmu code
staging: tidspbridge: remove dmm module
staging: tidspbridge: decouple mmu functions from regular code
Omar Ramirez Luna
if dynamic debug
is enabled.
Signed-off-by: Fernando Guzman Lugo x0095...@ti.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/Makefile |2 +-
drivers/staging/tidspbridge/core/tiomap3430.c |1 -
.../staging/tidspbridge/include/dspbridge
-off-by: Fernando Guzman Lugo x0095...@ti.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/Makefile |2 +-
drivers/staging/tidspbridge/core/_deh.h|3 -
drivers/staging/tidspbridge/core/dsp_mmu.c | 396
staging: tidspbridge: rmgr/node.c code cleanup
Laurent Pinchart (1):
staging: tidspbridge: Fix atoi to support hexadecimal numbers correctly
Omar Ramirez Luna (1):
staging: tidspbridge: use the right type for list_is_last
Ramos Falcon, Ernesto (1):
staging: tidspbridge: remove
-by: Hari Kanigeri h-kanige...@ti.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/core/io_sm.c |8 +++---
drivers/staging/tidspbridge/core/tiomap3430.c | 10 +---
.../staging/tidspbridge/include/dspbridge/io_sm.h | 21
, Fernando (1):
staging: tidspbridge: configure full L1 MMU range
Omar Ramirez Luna (1):
staging: tidspbridge: replace mbox callback with notifier_call
drivers/staging/tidspbridge/core/tiomap3430.c | 15 +++
1 files changed, 7 insertions(+), 8 deletions(-)
Regards,
Omar
Set of important changes for 2.6.38, to fix a critical bug (1) and a
compile break (2).
Guzman Lugo, Fernando (1):
staging: tidspbridge: configure full L1 MMU range
Omar Ramirez Luna (1):
staging: tidspbridge: replace mbox callback with notifier_call
drivers/staging/tidspbridge/core
-by: Felipe Contreras felipe.contre...@nokia.com
Signed-off-by: Fernando Guzman Lugo fernando.l...@ti.com
Signed-off-by: Felipe Contreras felipe.contre...@nokia.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/core/tiomap3430.c |5 +
1 files changed, 1
Lately, mailbox callbacks have been replaced by notifier block
call chains, this needs to be changed in the users of mailbox,
otherwise compilation will break due to missing parameters.
Reported-by: Hari Kanigeri h-kanige...@ti.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers
Lately, mailbox callbacks have been replaced by notifier block
call chains, this needs to be changed in the users of mailbox,
otherwise compilation will break due to missing parameters.
Reported-by: Hari Kanigeri h-kanige...@ti.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
correctly
Omar Ramirez Luna (1):
staging: tidspbridge: use the right type for list_is_last
Ramos Falcon, Ernesto (1):
staging: tidspbridge: remove code referred by OPT_ZERO_COPY_LOADER
Sapiens, Rene (1):
staging: tidspbridge: overwrite DSP error codes
drivers/staging
$FIRMWARE /sys/$DEVPATH/data; \
echo 0 /sys/$DEVPATH/loading'
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
.../tidspbridge/include/dspbridge/dbldefs.h| 10 --
.../staging/tidspbridge/include/dspbridge/dbll.h |7 ++
.../tidspbridge/include
Hi Greg,
Please consider the following patch set for tidspbridge driver in
staging tree, the following has been tested on an omap3430 based board
+ gst-dsp environment. Many thanks to Felipe Contreras for piling this up,
as I'm just propagating his patches.
First two patches:
1. omap: dsp:
, but needed until
omap_ctrl_set_dsp_boot*() are provided.
Signed-off-by: Felipe Contreras felipe.contre...@gmail.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/core/tiomap3430.c | 14 +-
1 files changed, 13 insertions(+), 1 deletions(-)
diff --git
: 'MEMBLOCK_REAL_LIMIT'
undeclared (first use in this function)
Signed-off-by: Felipe Contreras felipe.contre...@gmail.com
Acked-by: Tony Lindgren t...@atomide.com
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
arch/arm/plat-omap/devices.c |4 +++-
1 files changed, 3 insertions(+), 1
Since the device name has changed use the new name to
bind the driver to it.
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
drivers/staging/tidspbridge/rmgr/drv_interface.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/tidspbridge/rmgr
Rebased version of the patches that couldn't be applied.
Original set:
[1] http://marc.info/?l=linux-kernelm=127897548706686w=2
[2] http://marc.info/?l=linux-kernelm=127982129204138w=2
Fixed unexpected change while replacing IN modifier
[3] http://marc.info/?l=linux-kernelm=127972475532227w=2
From: Menon, Nishanth n...@ti.com
Remove custom globaltypes.h header
Signed-off-by: Nishanth Menon n...@ti.com
---
drivers/staging/tidspbridge/hw/GlobalTypes.h | 291 --
drivers/staging/tidspbridge/hw/MMURegAcM.h |1 -
drivers/staging/tidspbridge/hw/hw_defs.h
From: Menon, Nishanth n...@ti.com
Signed-off-by: Nishanth Menon n...@ti.com
---
drivers/staging/tidspbridge/core/chnl_sm.c |4 ++--
drivers/staging/tidspbridge/core/io_sm.c |2 +-
drivers/staging/tidspbridge/core/msg_sm.c |2 +-
From: Menon, Nishanth n...@ti.com
OPTIONAL modifier makes no sense in linux kernel
Signed-off-by: Nishanth Menon n...@ti.com
---
drivers/staging/tidspbridge/core/chnl_sm.c |2 +-
.../staging/tidspbridge/include/dspbridge/cod.h|2 +-
From: Menon, Nishanth n...@ti.com
Remove yet another custom definition header
Signed-off-by: Nishanth Menon n...@ti.com
---
.../staging/tidspbridge/include/dspbridge/dbdefs.h |1 -
.../staging/tidspbridge/include/dspbridge/dbtype.h | 69
69 matches
Mail list logo