Re: [PATCH] mmc: dw_mmc: use resource_size_t to store physical address

2015-11-18 Thread Andy Shevchenko
On Wed, Nov 18, 2015 at 2:38 PM, Arnd Bergmann wrote: > On Wednesday 18 November 2015 11:35:27 Andy Shevchenko wrote: >> On Fri, Nov 13, 2015 at 11:35 AM, Arnd Bergmann wrote: >> > On Friday 13 November 2015 03:10:13 Andy Shevchenko wrote: >> >> On Thu, Nov 12, 2015

Re: [PATCH] mmc: dw_mmc: use resource_size_t to store physical address

2015-11-18 Thread Arnd Bergmann
On Wednesday 18 November 2015 17:29:19 Andy Shevchenko wrote: > > I understand most of the things here, what I don't is how a platform > is supposed to work if you have the following: > a) HW, that uses register space let's say higher than 32-bit; > b) DMA engine, which should provide a DMA

Re: [PATCH] mmc: dw_mmc: use resource_size_t to store physical address

2015-11-18 Thread Andy Shevchenko
On Wed, Nov 18, 2015 at 5:45 PM, Arnd Bergmann wrote: > On Wednesday 18 November 2015 17:29:19 Andy Shevchenko wrote: >> >> I understand most of the things here, what I don't is how a platform >> is supposed to work if you have the following: >> a) HW, that uses register space

Re: [PATCH] mmc: dw_mmc: use resource_size_t to store physical address

2015-11-18 Thread Arnd Bergmann
On Wednesday 18 November 2015 18:17:32 Andy Shevchenko wrote: > On Wed, Nov 18, 2015 at 5:45 PM, Arnd Bergmann wrote: > > On Wednesday 18 November 2015 17:29:19 Andy Shevchenko wrote: > >> For me it clearly looks like a platform (HW / SW) configuration issue. > > > > I think some

Re: [PATCH] mmc: dw_mmc: use resource_size_t to store physical address

2015-11-18 Thread Andy Shevchenko
On Wed, Nov 18, 2015 at 6:22 PM, Arnd Bergmann wrote: > On Wednesday 18 November 2015 18:17:32 Andy Shevchenko wrote: >> On Wed, Nov 18, 2015 at 5:45 PM, Arnd Bergmann wrote: >> > On Wednesday 18 November 2015 17:29:19 Andy Shevchenko wrote: > >> >> For me it

Re: [PATCH] mmc: dw_mmc: use resource_size_t to store physical address

2015-11-18 Thread Arnd Bergmann
On Wednesday 18 November 2015 11:35:27 Andy Shevchenko wrote: > On Fri, Nov 13, 2015 at 11:35 AM, Arnd Bergmann wrote: > > On Friday 13 November 2015 03:10:13 Andy Shevchenko wrote: > >> On Thu, Nov 12, 2015 at 4:14 PM, Arnd Bergmann wrote: > >> > The dw_mmc driver

Re: [PATCH] mmc: dw_mmc: use resource_size_t to store physical address

2015-11-18 Thread Andy Shevchenko
On Fri, Nov 13, 2015 at 11:35 AM, Arnd Bergmann wrote: > On Friday 13 November 2015 03:10:13 Andy Shevchenko wrote: >> On Thu, Nov 12, 2015 at 4:14 PM, Arnd Bergmann wrote: >> > The dw_mmc driver stores the physical address of the MMIO registers >> > in a pointer,

Re: [PATCH] mmc: dw_mmc: use resource_size_t to store physical address

2015-11-17 Thread Jaehoon Chung
Dear, Arnd. On 11/13/2015 06:35 PM, Arnd Bergmann wrote: > On Friday 13 November 2015 03:10:13 Andy Shevchenko wrote: >> On Thu, Nov 12, 2015 at 4:14 PM, Arnd Bergmann wrote: >>> The dw_mmc driver stores the physical address of the MMIO registers >>> in a pointer, which requires

Re: [PATCH] mmc: dw_mmc: use resource_size_t to store physical address

2015-11-13 Thread Arnd Bergmann
On Friday 13 November 2015 03:10:13 Andy Shevchenko wrote: > On Thu, Nov 12, 2015 at 4:14 PM, Arnd Bergmann wrote: > > The dw_mmc driver stores the physical address of the MMIO registers > > in a pointer, which requires the use of type casts, and is actually > > broken if anyone

Re: [PATCH] mmc: dw_mmc: use resource_size_t to store physical address

2015-11-12 Thread Andy Shevchenko
On Thu, Nov 12, 2015 at 4:14 PM, Arnd Bergmann wrote: > The dw_mmc driver stores the physical address of the MMIO registers > in a pointer, which requires the use of type casts, and is actually > broken if anyone ever has this device on a 32-bit SoC in registers > above 4GB. Gcc