Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-15 Thread Sergei Shtylyov
Hello. On 14-04-2011 1:16, Janusz Krzysztofik wrote: Janusz Krzysztofik wrote: After switching from mem->dma_handle to virt_to_phys(mem->vaddr) used for obtaining page frame number passed to remap_pfn_range() (commit 35d9f510b67b10338161aba6229d4f55b4000f5b), videobuf-dma-contig Please

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-14 Thread Sergei Shtylyov
Hello. On 14-04-2011 1:01, Janusz Krzysztofik wrote: After switching from mem->dma_handle to virt_to_phys(mem->vaddr) used for obtaining page frame number passed to remap_pfn_range() (commit 35d9f510b67b10338161aba6229d4f55b4000f5b), videobuf-dma-contig Please specify the commit summary

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-13 Thread Janusz Krzysztofik
Dnia czwartek 14 kwiecień 2011 o 00:00:08 Russell King - ARM Linux napisał(a): > On Wed, Apr 13, 2011 at 10:56:39PM +0200, Janusz Krzysztofik wrote: > > Dnia środa 13 kwiecień 2011 o 20:32:31 Russell King - ARM Linux > > > > napisał(a): > > > On Wed, Apr 13, 2011 at 12:52:31PM +0200, Janusz Krzys

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-13 Thread Russell King - ARM Linux
On Wed, Apr 13, 2011 at 10:56:39PM +0200, Janusz Krzysztofik wrote: > Dnia środa 13 kwiecień 2011 o 20:32:31 Russell King - ARM Linux > napisał(a): > > On Wed, Apr 13, 2011 at 12:52:31PM +0200, Janusz Krzysztofik wrote: > > > Taking into account that I'm just trying to fix a regression, and > > >

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-13 Thread Janusz Krzysztofik
Dnia środa 13 kwiecień 2011 o 23:01:55 Janusz Krzysztofik napisał(a): > Dnia środa 13 kwiecień 2011 o 19:36:30 Sergei Shtylyov napisał(a): > > Hello. > > > > Janusz Krzysztofik wrote: > > >>> After switching from mem->dma_handle to > > >>> virt_to_phys(mem->vaddr) used for obtaining page frame num

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-13 Thread Janusz Krzysztofik
Dnia środa 13 kwiecień 2011 o 19:36:30 Sergei Shtylyov napisał(a): > Hello. > > Janusz Krzysztofik wrote: > >>> After switching from mem->dma_handle to virt_to_phys(mem->vaddr) > >>> used for obtaining page frame number passed to remap_pfn_range() > >>> (commit 35d9f510b67b10338161aba6229d4f55b400

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-13 Thread Janusz Krzysztofik
Dnia środa 13 kwiecień 2011 o 20:32:31 Russell King - ARM Linux napisał(a): > On Wed, Apr 13, 2011 at 12:52:31PM +0200, Janusz Krzysztofik wrote: > > Taking into account that I'm just trying to fix a regression, and > > not invent a new, long term solution: are you able to name an ARM > > based bo

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-13 Thread Russell King - ARM Linux
On Wed, Apr 13, 2011 at 12:52:31PM +0200, Janusz Krzysztofik wrote: > Taking into account that I'm just trying to fix a regression, and not > invent a new, long term solution: are you able to name an ARM based > board which a) is already supported in 2.6.39, b) is (or can be) > equipped with a d

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-13 Thread Sergei Shtylyov
Hello. Janusz Krzysztofik wrote: After switching from mem->dma_handle to virt_to_phys(mem->vaddr) used for obtaining page frame number passed to remap_pfn_range() (commit 35d9f510b67b10338161aba6229d4f55b4000f5b), videobuf-dma-contig Please specify the commit summary -- for the human rea

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-13 Thread Janusz Krzysztofik
Dnia środa 13 kwiecień 2011 o 14:03:44 Sergei Shtylyov napisał(a): > Hello. > > On 13-04-2011 1:06, Janusz Krzysztofik wrote: > > After switching from mem->dma_handle to virt_to_phys(mem->vaddr) > > used for obtaining page frame number passed to remap_pfn_range() > > (commit 35d9f510b67b10338161ab

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-13 Thread Sergei Shtylyov
Hello. On 13-04-2011 1:06, Janusz Krzysztofik wrote: After switching from mem->dma_handle to virt_to_phys(mem->vaddr) used for obtaining page frame number passed to remap_pfn_range() (commit 35d9f510b67b10338161aba6229d4f55b4000f5b), videobuf-dma-contig Please specify the commit summary --

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-13 Thread Janusz Krzysztofik
On Tue 12 Apr 2011 at 23:40:11 Russell King - ARM Linux wrote: > On Tue, Apr 12, 2011 at 11:06:34PM +0200, Janusz Krzysztofik wrote: > > The patch tries to solve this regression by using > > virt_to_phys(bus_to_virt(mem->dma_handle)) instead of problematic > > virt_to_phys(mem->vaddr). > > Who say

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-12 Thread Theodore Kilgore
On Tue, 12 Apr 2011, Russell King - ARM Linux wrote: > On Tue, Apr 12, 2011 at 11:06:34PM +0200, Janusz Krzysztofik wrote: > > The patch tries to solve this regression by using > > virt_to_phys(bus_to_virt(mem->dma_handle)) instead of problematic > > virt_to_phys(mem->vaddr). > > Who says tha

Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-12 Thread Russell King - ARM Linux
On Tue, Apr 12, 2011 at 11:06:34PM +0200, Janusz Krzysztofik wrote: > The patch tries to solve this regression by using > virt_to_phys(bus_to_virt(mem->dma_handle)) instead of problematic > virt_to_phys(mem->vaddr). Who says that DMA handles are bus addresses in the strictest sense? DMA handles

[PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM

2011-04-12 Thread Janusz Krzysztofik
After switching from mem->dma_handle to virt_to_phys(mem->vaddr) used for obtaining page frame number passed to remap_pfn_range() (commit 35d9f510b67b10338161aba6229d4f55b4000f5b), videobuf-dma-contig stopped working on my ARM based board. The ARM architecture maintainer, Russell King, confirmed