Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn()

2016-09-28 Thread Russell King - ARM Linux
On Wed, Sep 28, 2016 at 10:53:49AM +0300, Roger Quadros wrote: > Hi, > > On 12/09/16 14:38, Roger Quadros wrote: > > Hi Santosh & Russell, > > > > On 19/08/16 19:38, Santosh Shilimkar wrote: > >> > >> On 8/19/2016 12:30 AM, Roger Quadros wrote: > >>> Hi Santosh, > >>> > >> > > So I'm 99.9% co

Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn()

2016-09-28 Thread Roger Quadros
Hi, On 12/09/16 14:38, Roger Quadros wrote: > Hi Santosh & Russell, > > On 19/08/16 19:38, Santosh Shilimkar wrote: >> >> On 8/19/2016 12:30 AM, Roger Quadros wrote: >>> Hi Santosh, >>> >> > So I'm 99.9% convinced that the proposed change is correct. > I will got with that then :-) a

Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn()

2016-09-12 Thread Roger Quadros
Hi Santosh & Russell, On 19/08/16 19:38, Santosh Shilimkar wrote: > > On 8/19/2016 12:30 AM, Roger Quadros wrote: >> Hi Santosh, >> > So I'm 99.9% convinced that the proposed change is correct. >>> I will got with that then :-) and take my objection back. Just >>> saying that if there

Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn()

2016-08-19 Thread Santosh Shilimkar
On 8/19/2016 12:30 AM, Roger Quadros wrote: Hi Santosh, So I'm 99.9% convinced that the proposed change is correct. I will got with that then :-) and take my objection back. Just saying that if there other breakages which I can't recollect now, those drivers needs to be patched as well.

Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn()

2016-08-19 Thread Roger Quadros
Hi Santosh, On 19/08/16 05:01, Santosh Shilimkar wrote: > On 8/18/2016 4:07 PM, Russell King - ARM Linux wrote: >> On Thu, Aug 18, 2016 at 09:55:55AM -0700, Santosh Shilimkar wrote: >>> Hi Russell, >>> >>> On 8/18/2016 7:24 AM, Russell King - ARM Linux wrote: On Wed, Aug 17, 2016 at 03:05:17P

Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn()

2016-08-18 Thread Santosh Shilimkar
On 8/18/2016 4:07 PM, Russell King - ARM Linux wrote: On Thu, Aug 18, 2016 at 09:55:55AM -0700, Santosh Shilimkar wrote: Hi Russell, On 8/18/2016 7:24 AM, Russell King - ARM Linux wrote: On Wed, Aug 17, 2016 at 03:05:17PM +0300, Roger Quadros wrote: Since commit 6ce0d2001692 ("ARM: dma: Use d

Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn()

2016-08-18 Thread Santosh Shilimkar
Hi Russell, On 8/18/2016 7:24 AM, Russell King - ARM Linux wrote: On Wed, Aug 17, 2016 at 03:05:17PM +0300, Roger Quadros wrote: Since commit 6ce0d2001692 ("ARM: dma: Use dma_pfn_offset for dma address translation"), dma_to_pfn() already returns the PFN with the physical memory start offset so

Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn()

2016-08-18 Thread Russell King - ARM Linux
On Thu, Aug 18, 2016 at 09:55:55AM -0700, Santosh Shilimkar wrote: > Hi Russell, > > On 8/18/2016 7:24 AM, Russell King - ARM Linux wrote: > >On Wed, Aug 17, 2016 at 03:05:17PM +0300, Roger Quadros wrote: > >>Since commit 6ce0d2001692 ("ARM: dma: Use dma_pfn_offset for dma address > >>translation

Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn()

2016-08-18 Thread Russell King - ARM Linux
On Wed, Aug 17, 2016 at 03:05:17PM +0300, Roger Quadros wrote: > Since commit 6ce0d2001692 ("ARM: dma: Use dma_pfn_offset for dma address > translation"), > dma_to_pfn() already returns the PFN with the physical memory start offset > so we don't need to add it again. > > This fixes USB mass stora