Hi Laurent,
On Fri, Oct 26, 2012 at 11:35 AM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
That's on my to-do list, as well as porting the OMAP3 ISP driver to videobuf2,
adding DT support, moving to the common clock framework (when that will be
available for the OMAP3), supporting
On Thu, Oct 25, 2012 at 11:39 PM, Tony Lindgren t...@atomide.com wrote:
Joerg and Ohad, do you have any opinions on this?
I agree that there's some merit in having a separate header file for
IOVMM, since it's a different layer from the IOMMU API.
But in reality it's tightly coupled with OMAP's
Hi Marek,
On Mon, Jan 30, 2012 at 9:43 AM, Marek Szyprowski
m.szyprow...@samsung.com wrote:
Did you managed to fix this issue?
Yes -- the recent increase in the vmalloc region triggered a bigger
truncation in the system RAM than we had before, and therefore
conflicted with the previous
Hi Marek,
On Fri, Jan 27, 2012 at 5:17 PM, Marek Szyprowski
m.szyprow...@samsung.com wrote:
There have been some vmalloc layout changes merged to v3.3-rc1.
That was dead-on, thanks a lot!
I did then bump into a different allocation failure which happened
because dma_alloc_from_contiguous()
Hi Marek,
With v19, I can't seem to allocate big regions anymore (e.g. 101MiB).
In particular, this seems to fail:
On Thu, Jan 26, 2012 at 11:00 AM, Marek Szyprowski
m.szyprow...@samsung.com wrote:
+static int cma_activate_area(unsigned long base_pfn, unsigned long count)
+{
+ unsigned
2012/1/27 Marek Szyprowski m.szyprow...@samsung.com:
I've tested it with 256MiB on Exynos4 platform. Could you check if the
problem also appears on 3.2-cma-v19 branch (I've uploaded it a few hours
ago)
Exactly what I needed, thanks :)
Both v18 and v19 seem to work fine with 3.2.
The above
2012/1/27 Marek Szyprowski m.szyprow...@samsung.com:
Ohad, could you tell a bit more about your issue?
Sure, feel free to ask.
Does this 'large region'
is a device private region (declared with dma_declare_contiguous())
Yes, it is.
See omap_rproc_reserve_cma() in:
On Sun, Dec 18, 2011 at 11:47 PM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
Investigation complete.
http://patchwork.linuxtv.org/patch/8451/
Great, thanks a lot Laurent !
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to
On Wed, Nov 23, 2011 at 3:53 AM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
On Sunday 20 November 2011 17:54:26 Dmitry Artamonow wrote:
Fix following build error by explicitely including linux/module.h
header file.
CC drivers/media/video/omap3isp/ispvideo.o
Hi Laurent,
With 3.2-rc2, omap3isp seems to silently hang in my setup (sensor-less OMAP3).
Turning on lockdep yields the below messages, care to take a quick look ?
Thanks!
Ohad.
root@zoom3:~# media-ctl -r -l 'OMAP3 ISP CCP2 input:0-OMAP3 ISP
CCP2:0[1], OMAP3 ISP CCP
2:1-OMAP3 ISP CCDC:0[1],
(and from the number of LWN articles ;) it looks like a
considerable feat.
FWIW, feel free to add my
Tested-by: Ohad Ben-Cohen o...@wizery.com
(small and optional comment: I think it'd be nice if
dma_declare_contiguous would fail if called too late, otherwise users
of that misconfigured device
On Wed, Jun 8, 2011 at 1:46 PM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
On Tuesday 07 June 2011 15:46:26 Ohad Ben-Cohen wrote:
Where do you take care of those potential offsets today ? Or do you
simply ignore the offsets and map the entire page ?
Here http://marc.info/?l
On Tue, Jun 7, 2011 at 10:52 AM, Roedel, Joerg joerg.roe...@amd.com wrote:
Yup. Btw, is there any IOMMU hardware which supports non-natural
alignment? In this case we need to expose these requirements somehow.
Not sure there are. Let's start with natural alignment, and extend it
only if someone
Hi Laurent,
On Tue, Jun 7, 2011 at 12:05 PM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
pgsz isn't used anymore, you can remove it.
Ok.
+ order = get_order(bytes);
Does iommu_map() handle offsets correctly, or does it expect pa to be aligned
to an order (or
On Tue, Jun 7, 2011 at 12:58 PM, Roedel, Joerg joerg.roe...@amd.com wrote:
Btw, mind to split out your changes which move the iommu-api into
drivers/iommu? I can merge them meanwhile into my iommu tree and start
working on a proposal for the generic large page-size support.
Sure, that will be
Hi Laurent,
On Tue, Jun 7, 2011 at 12:22 PM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
+ BUG_ON(!IS_ALIGNED((long)omap_domain-pgtable, IOPGD_TABLE_SIZE));
Either __get_free_pages() guarantees that the allocated memory will be aligned
on an IOPGD_TABLE_SIZE boundary, in
On Tue, Jun 7, 2011 at 2:40 PM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
My point is that if the allocator guarantees the alignment (not as a side
effect of the implementation, but per its API) there's no need to check it
again. As the alignement is required, we need an
Hi Laurent,
On Tue, Jun 7, 2011 at 2:26 PM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
Right now we have a BUG_ON if pa is unaligned, but that can be changed
if needed (do we want it to handle offsets ?).
At least for the OMAP3 ISP we need to, as video buffers don't necessarily
Hi Joerg,
On Mon, Jun 6, 2011 at 1:09 PM, Roedel, Joerg joerg.roe...@amd.com wrote:
The IOMMU-API already supports multiple page-sizes. See the
'order'-parameter of the map/unmap functions.
This is insufficient; users need somehow to tell what page sizes are
supported by the underlying
On Mon, Jun 6, 2011 at 12:10 PM, Arnd Bergmann a...@arndb.de wrote:
I'd still prefer us to take small steps here, and not gate the omap
iommu cleanups with Marek's generic dma_map_ops work though. Let's go
forward and migrate omap's iommu to the generic iommu API, so new code
will be able to
On Mon, Jun 6, 2011 at 10:20 PM, Roedel, Joerg joerg.roe...@amd.com wrote:
Well, it certainly makes sense to have a single implementation for this.
But I want to hide this complexity to the user of the IOMMU-API. The
best choice is to put this into the layer between the IOMMU-API and the
Hi Arnd,
On Fri, Jun 3, 2011 at 6:53 PM, Arnd Bergmann a...@arndb.de wrote:
I think the future of iovmm is looking not so good. Marek Szyprowski
is working on a generic version of the dma-mapping API (dma_map_ops)
based on the iommu API.
Nice! I missed Marek's work somehow.
As far as I can
Hi Kyungmin,
On Fri, Jun 3, 2011 at 2:57 AM, Kyungmin Park kmp...@infradead.org wrote:
Please see the RFC patch, ARM: DMA-mapping IOMMU integration
http://www.spinics.net/lists/linux-mm/msg19856.html
Marek's work somehow escaped me, thanks for the pointers !
Ohad.
--
To unsubscribe from this
- OMAP's iommu code was tested on both OMAP3 and OMAP4
- omap3isp code was tested with a sensor-less OMAP3 (memory-to-memory only)
(thanks Laurent Pinchart for showing me the magic needed to test omap3isp :)
- MSM code was only compile tested
Ohad Ben-Cohen (6):
omap: iommu: generic iommu api
Migrate OMAP's iommu to the generic iommu api, so users can stay
generic, and non-omap-specific code can be removed and eventually
consolidated into a generic framework.
Tested on both OMAP3 and OMAP4.
Signed-off-by: Ohad Ben-Cohen o...@wizery.com
---
arch/arm/plat-omap/Kconfig
differently (or duplicated) in each driver.
Signed-off-by: Ohad Ben-Cohen o...@wizery.com
---
arch/arm/mach-msm/Kconfig |3 ---
arch/arm/plat-omap/Kconfig |3 ---
arch/x86/Kconfig|5 ++---
drivers/Kconfig |2 ++
drivers/Makefile
This should ease finding similarities with other platforms,
with the intention of solving problems once in a generic framework
which everyone can use.
Signed-off-by: Ohad Ben-Cohen o...@wizery.com
---
arch/arm/mach-msm/Kconfig | 12
arch/arm/mach-msm
), and exposing it will ease its generalization,
consolidation, or removal (once a generic virtual memory manager and allocator
would materialize).
Move omap's iommu debug driver to the generic folder as well, for the
same reasons.
Signed-off-by: Ohad Ben-Cohen o...@wizery.com
---
arch/arm/plat
manager framework).
Signed-off-by: Ohad Ben-Cohen o...@wizery.com
---
drivers/media/video/omap3isp/isp.c | 41 +-
drivers/media/video/omap3isp/isp.h |3 ++
drivers/media/video/omap3isp/ispccdc.c | 16 ++--
drivers/media/video/omap3isp
iommu map/unmap api).
Further generalizing of iovmm (or complete removal) should take place
together with broader plans of providing a generic virtual memory manager
and allocation framework (de-coupled from specific mappers).
Signed-off-by: Ohad Ben-Cohen o...@wizery.com
---
arch/arm/plat-omap
Fix a potential NULL pointer dereference by skipping registration of
external entities in case none are provided.
This is useful at least when testing mere memory-to-memory scenarios.
Signed-off-by: Ohad Ben-Cohen o...@wizery.com
---
drivers/media/video/omap3isp/isp.c |2 +-
1 files changed
On Mon, May 30, 2011 at 5:57 PM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
Thanks for the patch, but I've already applied something similar to my tree.
Sorry :-)
np, thanks. I'm just playing with omap3isp a bit and wanted to get rid
of that hey what did I do wrong this time
Trivially fix this:
drivers/media/video/omap3isp/isp.c: In function 'isp_isr_dbg':
drivers/media/video/omap3isp/isp.c:394: warning: zero-length gnu_printf format
string
Signed-off-by: Ohad Ben-Cohen o...@wizery.com
---
drivers/media/video/omap3isp/isp.c |2 +-
1 files changed, 1 insertions
On Thu, Jan 27, 2011 at 11:54 AM, Mark Brown
broo...@opensource.wolfsonmicro.com So what happens when both
drivers are in the system? It sounds like
you've got two different drivers for the same hardware. There must be
some redundancy there if nothing else.
Not really;
TI's 127x/128x devices
On Fri, Dec 3, 2010 at 2:50 PM, manjunatha_ha...@ti.com wrote:
...
Can you please also stage this version? Since the files are becoming big
to be posted as patches?
You should really CC Greg for staging requests...
Thanks Regards,
Manjunatha Halli
Manjunatha Halli (9):
include:linux:
Hi Pavan,
On Wed, Nov 17, 2010 at 6:32 PM, Ohad Ben-Cohen o...@wizery.com wrote:
drivers/staging/ti-st/Kconfig | 10 +
drivers/staging/ti-st/Makefile | 2 +
drivers/staging/ti-st/fmdrv.h | 259
drivers/staging/ti-st/fmdrv_common.c | 2141
Hi Manjunatha,
On Tue, Nov 16, 2010 at 3:18 PM, manjunatha_ha...@ti.com wrote:
drivers/staging/ti-st/Kconfig | 10 +
drivers/staging/ti-st/Makefile | 2 +
drivers/staging/ti-st/fmdrv.h | 259
drivers/staging/ti-st/fmdrv_common.c | 2141
37 matches
Mail list logo
