Re: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-24 Thread Federico Vaga
> You can take the patch which adds prepare/finish methods to memory > allocators. It should not have any dependency on the other stuff from > that thread. I'm fine with merging it either together with Your patch > or via Tomasz's patchset, whatever comes first. Thank you. I'll do the job --

RE: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-24 Thread Marek Szyprowski
Hello, On Friday, August 24, 2012 3:23 PM Federico Vaga wrote: > > Getting back to your patch - in your approach cpu cache handling is > > missing. I suspect that it worked fine only because it has been > > tested on some simple platform without any cpu caches (or with very > > small ones). > >

Re: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-24 Thread Federico Vaga
> Getting back to your patch - in your approach cpu cache handling is > missing. I suspect that it worked fine only because it has been > tested on some simple platform without any cpu caches (or with very > small ones). Is missing from the memory allocator because I do it on the device driver.

Re: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-24 Thread Federico Vaga
Getting back to your patch - in your approach cpu cache handling is missing. I suspect that it worked fine only because it has been tested on some simple platform without any cpu caches (or with very small ones). Is missing from the memory allocator because I do it on the device driver. The

RE: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-24 Thread Marek Szyprowski
Hello, On Friday, August 24, 2012 3:23 PM Federico Vaga wrote: Getting back to your patch - in your approach cpu cache handling is missing. I suspect that it worked fine only because it has been tested on some simple platform without any cpu caches (or with very small ones). Is

Re: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-24 Thread Federico Vaga
You can take the patch which adds prepare/finish methods to memory allocators. It should not have any dependency on the other stuff from that thread. I'm fine with merging it either together with Your patch or via Tomasz's patchset, whatever comes first. Thank you. I'll do the job --

RE: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-20 Thread Marek Szyprowski
Hello, I'm really sorry for a late reply, but I was a bit busy with other stuff. On Tuesday, July 31, 2012 10:17 PM Federico Vaga wrote: > Signed-off-by: Federico Vaga > --- > drivers/media/video/Kconfig | 6 +- > drivers/media/video/Makefile | 1 + >

RE: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-20 Thread Marek Szyprowski
Hello, I'm really sorry for a late reply, but I was a bit busy with other stuff. On Tuesday, July 31, 2012 10:17 PM Federico Vaga wrote: Signed-off-by: Federico Vaga federico.v...@gmail.com --- drivers/media/video/Kconfig | 6 +- drivers/media/video/Makefile

[PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-07-31 Thread Federico Vaga
Signed-off-by: Federico Vaga --- drivers/media/video/Kconfig | 6 +- drivers/media/video/Makefile | 1 + drivers/media/video/videobuf2-dma-streaming.c | 187 ++ include/media/videobuf2-dma-streaming.h | 24 4 file

[PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-07-31 Thread Federico Vaga
Signed-off-by: Federico Vaga federico.v...@gmail.com --- drivers/media/video/Kconfig | 6 +- drivers/media/video/Makefile | 1 + drivers/media/video/videobuf2-dma-streaming.c | 187 ++ include/media/videobuf2-dma-streaming.h |