Code cleanup minor changes to fix some compilation warnings.
- Indent style changes
- Fixed undefined LEAD3 compilation flag
- Fixed ISO C90 forbids mixed declarations and code
- Fixed unused variables
---
Index:
Remove the compilation flags to make Dynamic loader optimizations default in
the code. Modified Kbuild so these flags are not defined anymore.
---
Index: omapkernel/drivers/dsp/dspbridge/mpu_driver/inc/dbtype.h
===
---
I'll be splitting these to fit the size limit, hope this is ok.
Regards,
omar
-Original Message-
From: Ramirez Luna, Omar
Sent: Monday, August 04, 2008 9:54 AM
To: 'linux-omap@vger.kernel.org'
Subject: [PATCH 00/07][GIT 3/4+] Bridge cleanup patches
Hi,
The following set of patches
Hi,
The following set of patches, are used to fix some of the warnings and errors
reported from various code check tools, like checkpatch and sparse.
They also remove some code that is now obsolete and try to make bridge code
more understandable and clean, these patches are placed under:
Hi,
These are the latest set of patches that contain changes to dspbridge code.
They are used to fix some of the warnings and errors reported from various code
check tools, like checkpatch and sparse.
They also remove some code that is now obsolete and try to make bridge code
more
Hi,
This patch replaces bool typedefs with the ones from the kernel, as reported by
checkpatch tool. It should be applied on top of the latest set of cleanup
patches (BRD001 to BRD007) and after Felipe's Kbuild patch (BRD008).
As it is quite big, you can found it at:
Hi,
The following patch fixes warnings present at building stages.
Cleaning up the code to remove minor issues and an unnecessary header file.
Placed under:
http://omapzoom.org/gf/project/omapbridge/frs/?action=FrsReleaseBrowsefrs_package_id=16
Named: BRD010-ARM-OMAP-cleanup_warnings.patch
-
Hi,
I would like to announce that the latest patches for dspbridge code are now
uploaded, you can find them in Gforge OmapZoom site at:
http://omapzoom.org/gf/project/omapbridge/frs/?action=FrsReleaseViewrelease_id=119
Also, update your API and samples code when using the latest patches or
Hi,
I updated Gforge site with some more patches they are now a total of 17
uploaded, the ones that were added are applied on top of the previous ones, you
can find them at:
http://omapzoom.org/gf/project/omapbridge/frs/?action=FrsReleaseViewrelease_id=119
I remind you to update your API and
Although this is a set of two patches they are next in patch order to the ones
stored on OmapZoom:
http://omapzoom.org/gf/project/omapbridge/frs/?action=FrsReleaseViewrelease_id=119
- omar
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to
Fixed cosmetic changes reported from checkpatch
Signed-off-by: Omar Ramirez Luna [EMAIL PROTECTED]
Acked-by: Hari Kanigeri [EMAIL PROTECTED]
---
Index: omapkernel/drivers/dsp/bridge/gen/gs.c
===
---
Adding description to bridge debug option and indenting Kconfig lines
Signed-off-by: Omar Ramirez Luna [EMAIL PROTECTED]
Acked-by: Hari Kanigeri [EMAIL PROTECTED]
---
Index: omapkernel/drivers/dsp/bridge/Kconfig
===
---
Please ignore the previous patch.
Adding description to bridge debug option and indenting Kconfig lines
Signed-off-by: Omar Ramirez Luna [EMAIL PROTECTED]
Acked-by: Hari Kanigeri [EMAIL PROTECTED]
---
Index: omapkernel/drivers/dsp/bridge/Kconfig
Hi all,
The following patches were submitted to the Omap 3/4 Linux Kernel tree, please
find them at the following link:
http://omapzoom.org/gf/project/omapbridge/frs/?action=FrsReleaseViewrelease_id=119
Patch order:
1. 0020-BRIDGE-use-likely-unlikely-for-status-checking.patch (Hiroshi.DOYU)
2.
Hi,
You can find dspbridge binaries uploaded into omapzoom site:
http://omapzoom.org/gf/project/omapbridge/frs/?action=FrsReleaseViewrelease_id=132
Base images (.dof64P) are statically linked DSP executables loaded on the DSP
when system boots up, dynamic libraries (.dll64P) are DSP object
Hi,
From: linux-omap-ow...@vger.kernel.org
[mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Cyril
Chemparathy
Davinci's MDIO controller is present on other TI devices, without an
accompanying EMAC. For example, on tnetv107x, the same MDIO module is used in
conjunction with a 3-port
Hi,
From: linux-omap-ow...@vger.kernel.org
[mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of
Chemparathy, Cyril
In addition to being embedded into the EMAC controller, the CPDMA hardware
block is used in TI's CPSW switch controller. Fortunately, the programming
interface to this hardware
venki kaps wrote:
...
My problem is resolved.GPtimer7 was not reset during the MMU FAULT
occurrence before the first power cycle. So this pending interrupt is
preventing the system sleep entry.
Now it works fine after resetting Gptimer7 in pm suspend path.
That's doesn't sound right, why
Hi Sergio,
Aguirre, Sergio wrote:
Hi Omar,
...
+#if defined(CONFIG_ARCH_OMAP2)
+else if (cpu_is_omap2430()) {
+list = omap2_mboxes;
+
+list[0]-irq = platform_get_irq_byname(pdev, dsp);
+} else if (cpu_is_omap2420()) {
Isn't both 2430 and 2420 doing the
Ramirez Luna, Omar wrote:
Fix the mailbox support detection for OMAP3630, 3530/25 and 2430.
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
- Testing was made under 3630 and 3430 boards.
- Given that 2430 uses similar initialization than OMAP3, changes
to handle this case
Kevin Hilman wrote:
Ramirez Luna, Omar omar.rami...@ti.com writes:
Ramirez Luna, Omar wrote:
Fix the mailbox support detection for OMAP3630, 3530/25 and 2430.
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
- Testing was made under 3630 and 3430 boards.
- Given that 2430 uses
Felipe Contreras wrote:
On Thu, Sep 23, 2010 at 7:11 PM, Ramirez Luna, Omar
omar.rami...@ti.com wrote:
Code is not the same, it is 2 line which apply for both but couldn't
find an easy way of making them share the request for dsp mailbox
without changing more code, perhaps a macro to detect
Hi,
Felipe Contreras wrote:
Hi,
I was trying the iommu patches but I noticed the tidspbridge driver
doesn't work at all, I found two critical issues:
First, the platform device is missing, so there's no way the driver
can work. It seems to have been dropped from the migration to staging
Felipe Contreras wrote:
...
Anyway, first, I think I tried a version similar to that one, and
doesn't apply on mainline any more, also, bootmem doesn't seem to be
working on ARM either, so I had to move to memblock, and third, I
don't think that platform device is correct since it depends on
Felipe Contreras wrote:
...
Well, I thought it would be nice if the driver actually worked on
.36, but anyway, I'll rebase the rest of my cleanup patches and send
them. Hopefully I find a way to get rid of the ioremap() issue.
phys_to_virt instead of ioremap should work fine, still need
Felipe Contreras wrote:
...
phys_to_virt instead of ioremap should work fine, still need to try
it though.
I tried that... Didn't work. I guess we need to flush at some points
(since the memory is cacheable).
Hmmm, because my other option was to move the reserved memory outside the
Hi Tony,
On Fri, Nov 5, 2010 at 12:53 PM, Felipe Contreras
felipe.contre...@gmail.com wrote:
On Fri, Nov 5, 2010 at 6:34 PM, Tony Lindgren t...@atomide.com wrote:
* Felipe Contreras felipe.contre...@gmail.com [101019 00:28]:
Also, don't be picky about the location.
As this code is in
On Fri, Nov 5, 2010 at 8:19 PM, Omar Ramirez Luna omar.rami...@ti.com wrote:
Boot tested on omap 3430 and 3630, functionality on iva2 only.
I meant functionally tested on iva2 only.
Regards,
Omar
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to
Hi Benoit,
On Sat, Nov 6, 2010 at 12:08 PM, Cousson, Benoit b-cous...@ti.com wrote:
+/* l4_core - mailbox */
+static struct omap_hwmod_ocp_if omap2420_l4_core__mailbox = {
+ .master =omap2420_l4_core_hwmod,
+ .slave =omap2420_mailbox_hwmod,
+ .addr
defined you need to scroll beyond the supposed first
reference in omap_hwmod (right now above ocp_if)
On 11/5/2010 9:17 PM, Ramirez Luna, Omar wrote:
+/* mailbox */
The original comment is missing.
quote
/*
* 'mailbox' class
* mailbox module allowing communication between the on-chip
On Sat, Nov 6, 2010 at 1:11 PM, Cousson, Benoit b-cous...@ti.com wrote:
-#if defined(CONFIG_ARCH_OMAP3430)
+#if defined(CONFIG_ARCH_OMAP3)
Ideally you should get rid of all the CONFIG_ARCH_OMAPXXX or cpu_is_omap in
that code. This is a driver, it should be generic.
If you have to handle
On Sat, Nov 6, 2010 at 1:21 PM, Cousson, Benoit b-cous...@ti.com wrote:
On 11/5/2010 9:17 PM, Ramirez Luna, Omar wrote:
Remove unreachable return statement.
Signed-off-by: Omar Ramirez Lunaomar.rami...@ti.com
---
arch/arm/mach-omap2/mailbox.c | 2 --
1 files changed, 0 insertions(+), 2
On Sat, Nov 6, 2010 at 1:32 PM, Cousson, Benoit b-cous...@ti.com wrote:
On 11/5/2010 9:17 PM, Ramirez Luna, Omar wrote:
Could you please try to use a consistent naming in the subject?
You use OMAP3, OMAP4, OMAP2 so it should be OMAP and not omap.
Yes, I'll change it.
Regards,
Omar
PM, Ramirez Luna, Omar wrote:
Boot tested on omap 3430 and 3630, functionality on iva2 only.
Introduced hwmod data and support for omap3 (iva2 mmu isp mmu) and
omap4 (ducati mmu tesla mmu).
Minor cleanup due to this changes.
There is an issue (present without patches too):
Doing a cycle
On Sat, Nov 6, 2010 at 2:11 PM, Cousson, Benoit b-cous...@ti.com wrote:
On 11/5/2010 9:19 PM, Ramirez Luna, Omar wrote:
iommu driver is meant to provide control of mmu hardware blocks
A dot is missing here, and a capital letter should follow.
Actually it is a comma, it is meant to be part
On Sat, Nov 6, 2010 at 2:15 PM, Cousson, Benoit b-cous...@ti.com wrote:
On 11/5/2010 9:19 PM, Ramirez Luna, Omar wrote:
Add mmu hwmod data for iva2 and isp.
s/iva2/iva/
Where this terminology can be found? I'm basing this naming on what
was there or TRM. For omap3 there is no such thing
On Sat, Nov 6, 2010 at 3:47 PM, Cousson, Benoit b-cous...@ti.com wrote:
On 11/5/2010 9:19 PM, Ramirez Luna, Omar wrote:
Add mmu hwmod data for ducati and tesla.
s/ducati/ipu/
s/tesla/dsp/
Please do not use internal codename.
Tried to avoid confusion with what was originally in the driver
On Sat, Nov 6, 2010 at 4:05 PM, Cousson, Benoit b-cous...@ti.com wrote:
static struct iommu_device omap3_devices[] = {
You should not need that at all, you are just duplicating the data that are
already in hwmod, and then you are going to create real platform_device.
ok, will try that.
On Sat, Nov 6, 2010 at 4:17 PM, Cousson, Benoit b-cous...@ti.com wrote:
You should use runtime PM directly. That omap_device step is useless.
Moreover, this patch should be merged with the previous one.
Agree, will check on runtime PM.
Regards,
Omar
--
To unsubscribe from this list: send the
On Sat, Nov 6, 2010 at 4:28 PM, Cousson, Benoit b-cous...@ti.com wrote:
On 11/5/2010 9:19 PM, Ramirez Luna, Omar wrote:
Since omap-iommu is now using hwmod, there are structures and
arrays that can be cleaned up to increase the readability of
the code.
This patch should be merged
On Sun, Nov 7, 2010 at 3:05 PM, Felipe Contreras
felipe.contre...@gmail.com wrote:
Huh? The mailbox is working for me on tidspbridge on 2.6.37-rc1.
Are you using an omap based board with a 3630 chip?
Regards,
Omar
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the
Hi Benoit,
On Mon, Nov 8, 2010 at 2:56 AM, Cousson, Benoit b-cous...@ti.com wrote:
no, apart from the ordering of structure members, that I will change,
If you do want to change a patch already submitted to loml, the least you
can do is to comment on the mailing list, and afaik, you didn't
On Mon, Nov 8, 2010 at 5:21 PM, Cousson, Benoit b-cous...@ti.com wrote:
I wrote this one from scratch, I didn't see that there were pieces to
handle some stuff since the code is buried in a private tree.
Not true at all... It was sent to l-o:
On Mon, Nov 8, 2010 at 5:05 PM, Cousson, Benoit b-cous...@ti.com wrote:
Add mmu hwmod data for iva2 and isp.
s/iva2/iva/
Where this terminology can be found? I'm basing this naming on what
was there or TRM. For omap3 there is no such thing as iva, it is iva2.
IVA2 means IVA v2, before we
Hi,
Seriously, I'm getting very confused here, and am very annoyed by the
whole thing.
Here's what I don't like:
- the original driver didn't even seem to work properly
- people sent me patches they never tested and broke things even worse
- some people have no respect for the omap
On Tue, Nov 9, 2010 at 5:48 PM, Kevin Hilman
khil...@deeprootsystems.com wrote:
#if defined(CONFIG_OMAP_MBOX_FWK) || defined(CONFIG_OMAP_MBOX_FWK_MODULE)
+static struct omap_device_pm_latency mbox_latencies[] = {
+ [0] = {
+ .activate_func = omap_device_enable_clocks,
+
On Tue, Nov 9, 2010 at 5:38 PM, Kevin Hilman
khil...@deeprootsystems.com wrote:
---
arch/arm/mach-omap2/devices.c | 100
---
arch/arm/mach-omap2/mailbox.c | 1 +
arch/arm/mach-omap2/omap_hwmod_3xxx_data.c | 54 +++
please
On Wed, Nov 24, 2010 at 2:47 AM, Tomi Valkeinen
tomi.valkei...@nokia.com wrote:
Hi,
On Tue, 2010-11-23 at 19:00 -0600, ext Omar Ramirez Luna wrote:
From: Yanick Saugy yanick.sa...@heig-vd.ch
The PIN which controls LCD's reset signal (VGA_nRESET/h_GPIO_96) was not
configured as a GPIO. We
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
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
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
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:
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
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 ionut.n...@mindbit.ro
---
drivers/staging/tidspbridge/rmgr/node.c | 607
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:
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
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
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
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
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
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
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
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:
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__);
+
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
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
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
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
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
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
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
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
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
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
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
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
From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/07/2010 07:00 PM, the following:
This patch creates a new sysfs entry to query the driver
state without making any bridge direct call, this is
useful to monitor driver state without creating any handle
to bridge driver.
why sysfs
From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/07/2010 07:00 PM, the following:
Include dspbridge compilation whenever bridge driver is selected to
be compiled, either as a module or part of the kernel. This will
initialize platform data specific PM functions.
?? are you intending
From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/07/2010 07:01 PM, the following:
- Use standard convention to define ioctls.
- Removed runtime check for ioctl matching table number.
- Added __deprectaed marker to functions that are not used anymore.
Currently 'DB' is used
From: Menon, Nishanth
Apologies on the spam, but a few points below for preventing a future
ioctl change:
Menon, Nishanth had written, on 01/07/2010 08:59 PM, the following:
Ramirez Luna, Omar had written, on 01/07/2010 07:01 PM, the following:
- Use standard convention to define ioctls
From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/08/2010 10:48 AM, the following:
From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/07/2010 07:00 PM, the following:
Include dspbridge compilation whenever bridge driver is selected to
be compiled, either as a module
From: Menon, Nishanth
Omar Ramirez Luna had written, on 01/07/2010 07:00 PM, the following:
This patch removes an unused variable in bridge_open
and bridge_release functions, which reduces indentation.
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
From: Felipe Contreras [mailto:felipe.contre...@nokia.com]
On Fri, Jan 08, 2010 at 02:00:26AM +0100, Omar Ramirez Luna wrote:
This patch cleans SSI clock preparation and also removes
the dependency on IO_ADDRESS variants for easier maintenance.
So this patch is:
Cleaning ssi handling
*
From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/11/2010 11:50 AM, the following:
This is the official 0.1 dspbridge version.
CC: Felipe Contreras felipe.contre...@nokia.com
CC: Hiroshi Doyu hiroshi.d...@nokia.com
CC: Ameya Palande ameya.pala...@nokia.com
CC: Nishant Menon n
From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/11/2010 12:46 PM, the following:
From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/11/2010 11:50 AM, the following:
This is the official 0.1 dspbridge version.
CC: Felipe Contreras felipe.contre...@nokia.com
CC: Hiroshi
Resending with u8 instead of short for size_cmd array
From: Ramirez Luna, Omar
- Use standard convention to define ioctls.
Currently 0xDB is used as identifier for dspbridge.
Discussion:
http://marc.info/?l=linux-omapm=126297631603035w=2
WARNING:
An update in api ioctl definitions is required
From: Ramirez Luna, Omar
This is the official 0.1 dspbridge version.
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
CC: Felipe Contreras felipe.contre...@nokia.com
CC: Hiroshi Doyu hiroshi.d...@nokia.com
CC: Ameya Palande ameya.pala...@nokia.com
CC: Nishant Menon n...@ti.com
---
drivers
From: Ramirez Luna, Omar
Sent: Monday, January 11, 2010 7:01 PM
To: linux-omap
Cc: Ramirez Luna, Omar; Tony Lindgren; Menon, Nishanth; Hiroshi Doyu; Ameya
Palande; Felipe Contreras
Subject: [PATCH v3 1/7] DSPBRIDGE: replace IO_ADDRESS with ioremap
Use ioremap() instead of using IO_ADDRESS macro
From: Ramirez Luna, Omar
Sent: Monday, January 11, 2010 7:01 PM
To: linux-omap
Cc: Ramirez Luna, Omar; Menon, Nishanth; Hiroshi Doyu; Ameya Palande; Felipe
Contreras
Subject: [PATCH v3 2/7] DSPBRIDGE: Replace magic numbers in SSI configuration
Rename ssi_sysconfig variable and replace the usage
From: Ramirez Luna, Omar
Sent: Monday, January 11, 2010 7:01 PM
To: linux-omap
Cc: Ramirez Luna, Omar; Menon, Nishanth; Hiroshi Doyu; Ameya Palande; Felipe
Contreras
Subject: [PATCH v3 3/7] DSPBRIDGE: Mark unused ioctls as deprecated
Mark unused ioctls as deprecated for future removal
and print
From: Ramirez Luna, Omar
Sent: Monday, January 11, 2010 7:01 PM
To: linux-omap
Cc: Ramirez Luna, Omar; Menon, Nishanth; Hiroshi Doyu; Ameya Palande; Felipe
Contreras
Subject: [PATCH v3 4/7] DSPBRIDGE: remove ioctl runtime check
This patch removes the runtime check of device
ioctl array.
Signed
From: Ramirez Luna, Omar
Sent: Monday, January 11, 2010 7:01 PM
To: linux-omap
Cc: Ramirez Luna, Omar; Menon, Nishanth; Hiroshi Doyu; Ameya Palande; Felipe
Contreras
Subject: [PATCH v3 5/7] DSPBRIDGE: Rename pResult to result for
WCD_CallDevIOCtl
Trivial cleanup, rename pResult to result
From: Ramirez Luna, Omar
Sent: Monday, January 11, 2010 7:01 PM
To: linux-omap
Cc: Ramirez Luna, Omar; Menon, Nishanth; Hiroshi Doyu; Ameya Palande; Felipe
Contreras
Subject: [PATCH v3 6/7] DSPBRIDGE: Use _IOxx macro to define ioctls
- Use standard convention to define ioctls.
Currently 0xDB
Hi Ameya,
Hi Omar,
On Tue, 2010-01-12 at 02:00 +0100, ext Omar Ramirez Luna wrote:
Ioctl numbers has changed, this will require an update on libbridge
and clients accesing directly to the driver
- IO_ADDRESS is replaced for ioremap.
*v3
- modified:
DSPBRIDGE: replace IO_ADDRESS
From: Ameya Palande on Monday, January 18, 2010 3:55 AM
On Sat, 2010-01-16 at 03:01 +0100, ext Omar Ramirez Luna wrote:
Fix header locations, replaced mach to plat
Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
arch/arm/mach-omap2/dspbridge.c|2 +-
From: Ameya Palande on Monday, January 18, 2010 4:20 AM
On Sat, 2010-01-16 at 03:01 +0100, ext Omar Ramirez Luna wrote:
From: Fernando Guzman Lugo x0095...@ti.com
This patch changes clk_notifier_unregister with
cpufreq_unregister_notifier, which is used for OPP change
notifications
From: Ameya Palande on Monday, January 18, 2010 4:32 AM
On Sat, 2010-01-16 at 03:01 +0100, ext Omar Ramirez Luna wrote:
Include dspbridge compilation whenever bridge driver is selected to
be compiled, either as a module or part of the kernel. This will
initialize platform data specific PM
From: Ramirez Luna, Omar on Wednesday, January 13, 2010 7:11 PM
From: Ernesto Ramos erne...@ti.com
Release resources allocated during MAP, Node Allocation,
STRM Buf Allocation in the case of cp_to_usr fails.
Signed-off-by: Ernesto Ramos erne...@ti.com
CC: Nishanth Menon n...@ti.com
CC: Hiroshi
From: Ramirez Luna, Omar on Wednesday, January 13, 2010 7:11 PM
From: Ernesto Ramos erne...@ti.com
Add more error checking and range to address usecases
accessing kernel APIs directly.
Signed-off-by: Ernesto Ramos erne...@ti.com
CC: Nishanth Menon n...@ti.com
CC: Hiroshi Doyu hiroshi.d
From: Omar Ramirez Luna on Wednesday, January 13, 2010 7:11 PM
From: Ernesto Ramos erne...@ti.com
This patch takes care of the possible null pointers
dereferenced within dsp bridge driver.
Signed-off-by: Ernesto Ramos erne...@ti.com
CC: Nishanth Menon n...@ti.com
CC: Hiroshi Doyu
1 - 100 of 314 matches
Mail list logo