Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-08-25 Thread Russell King - ARM Linux
On Fri, Aug 24, 2012 at 02:10:38PM +0200, Peter Meerwald wrote: > with Linux 3.6-rc3: > > CONFIG_DMA_OMAP=m > CONFIG_DMA_ENGINE=y > CONFIG_DMA_VIRTUAL_CHANNELS=m > CONFIG_MMC_OMAP_HS=y > CONFIG_SPI_OMAP24XX=y > CONFIG_MTD_NAND_OMAP2=y > > LD init/built-in.o > drivers/built-in.o: In functio

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-08-24 Thread Peter Meerwald
> >> > > the commit just sets CONFIG_DMA_OMAP=y and CONFIG_DMADEVICES=y in > >> > > omap2plus_defconfig; this does not help people updating the kernel > >> > > while > >> > > keeping the config, nor does it help people in configuring the kernel > >> > > > >> > > there is a dependency (at least fo

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-08-24 Thread Shilimkar, Santosh
On Fri, Aug 24, 2012 at 4:09 PM, Russell King - ARM Linux wrote: > On Fri, Aug 24, 2012 at 03:51:26PM +0530, Shilimkar, Santosh wrote: >> On Fri, Aug 24, 2012 at 3:12 PM, Russell King - ARM Linux >> wrote: >> > >> > On Fri, Aug 24, 2012 at 09:51:15AM +0200, Peter Meerwald wrote: >> > > the commit

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-08-24 Thread Russell King - ARM Linux
On Fri, Aug 24, 2012 at 03:51:26PM +0530, Shilimkar, Santosh wrote: > On Fri, Aug 24, 2012 at 3:12 PM, Russell King - ARM Linux > wrote: > > > > On Fri, Aug 24, 2012 at 09:51:15AM +0200, Peter Meerwald wrote: > > > the commit just sets CONFIG_DMA_OMAP=y and CONFIG_DMADEVICES=y in > > > omap2plus_d

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-08-24 Thread Shilimkar, Santosh
On Fri, Aug 24, 2012 at 3:12 PM, Russell King - ARM Linux wrote: > > On Fri, Aug 24, 2012 at 09:51:15AM +0200, Peter Meerwald wrote: > > the commit just sets CONFIG_DMA_OMAP=y and CONFIG_DMADEVICES=y in > > omap2plus_defconfig; this does not help people updating the kernel while > > keeping the co

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-08-24 Thread Russell King - ARM Linux
On Fri, Aug 24, 2012 at 09:51:15AM +0200, Peter Meerwald wrote: > the commit just sets CONFIG_DMA_OMAP=y and CONFIG_DMADEVICES=y in > omap2plus_defconfig; this does not help people updating the kernel while > keeping the config, nor does it help people in configuring the kernel > > there is a de

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-08-24 Thread Peter Meerwald
On Fri, 24 Aug 2012, Shilimkar, Santosh wrote: > On Fri, Aug 24, 2012 at 2:30 AM, Peter Meerwald wrote: > > > > On Wed, 18 Jul 2012, Javier Martinez Canillas wrote: > > > > > On Wed, Jul 18, 2012 at 10:36 AM, Shilimkar, Santosh > > > wrote: > > > > On Wed, Jul 18, 2012 at 1:14 PM, S, Venkatraman

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-08-24 Thread Shilimkar, Santosh
On Fri, Aug 24, 2012 at 2:30 AM, Peter Meerwald wrote: > > On Wed, 18 Jul 2012, Javier Martinez Canillas wrote: > > > On Wed, Jul 18, 2012 at 10:36 AM, Shilimkar, Santosh > > wrote: > > > On Wed, Jul 18, 2012 at 1:14 PM, S, Venkatraman > > > wrote: > > >> On Wed, Jul 18, 2012 at 12:40 PM, Tony L

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-08-23 Thread Peter Meerwald
On Wed, 18 Jul 2012, Javier Martinez Canillas wrote: > On Wed, Jul 18, 2012 at 10:36 AM, Shilimkar, Santosh > wrote: > > On Wed, Jul 18, 2012 at 1:14 PM, S, Venkatraman wrote: > >> On Wed, Jul 18, 2012 at 12:40 PM, Tony Lindgren wrote: > >>> * Shilimkar, Santosh [120718 00:09]: > On Wed,

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-20 Thread S, Venkatraman
On Fri, Jul 20, 2012 at 2:58 PM, S, Venkatraman wrote: > On Thu, Jul 19, 2012 at 5:02 PM, Tony Lindgren wrote: >> * Shilimkar, Santosh [120718 02:49]: >>> >>> "The patch simply make them depend on DMA_OMAP since DMA_OMAP >>> will select DMA_ENGINE automatically" >>> >>> This won't be true if the

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-20 Thread S, Venkatraman
On Thu, Jul 19, 2012 at 5:02 PM, Tony Lindgren wrote: > * Shilimkar, Santosh [120718 02:49]: >> >> "The patch simply make them depend on DMA_OMAP since DMA_OMAP >> will select DMA_ENGINE automatically" >> >> This won't be true if the DMA selection are not done >> at ARCH_OMAP level as discussed i

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-19 Thread Tony Lindgren
* Shilimkar, Santosh [120718 02:49]: > > "The patch simply make them depend on DMA_OMAP since DMA_OMAP > will select DMA_ENGINE automatically" > > This won't be true if the DMA selection are not done > at ARCH_OMAP level as discussed in this thread. > > Having said that, I think Russell and Ton

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-19 Thread Tony Lindgren
* Javier Martinez Canillas [120718 01:31]: > That was my first approach in fact, I sent this patch: > https://patchwork.kernel.org/patch/1198401/ > > But then was told by Kevin Hilman that the right solution was to do > the select on the drives to fix not only omap2plus_defconfig but any > config

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread Shilimkar, Santosh
On Wed, Jul 18, 2012 at 3:08 PM, S, Venkatraman wrote: > On Wed, Jul 18, 2012 at 2:46 PM, Javier Martinez Canillas > wrote: [...] And what about enabling it at the SoC config level but making the drivers dependant on CONFIG_DMADEVICES and CONFIG_DMA_OMAP? If you agree I can

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread S, Venkatraman
On Wed, Jul 18, 2012 at 2:46 PM, Javier Martinez Canillas wrote: > On Wed, Jul 18, 2012 at 11:11 AM, Shilimkar, Santosh > wrote: >> On Wed, Jul 18, 2012 at 2:19 PM, Javier Martinez Canillas >> wrote: >>> On Wed, Jul 18, 2012 at 10:36 AM, Shilimkar, Santosh >>> wrote: On Wed, Jul 18, 2012 a

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread Javier Martinez Canillas
On Wed, Jul 18, 2012 at 11:11 AM, Shilimkar, Santosh wrote: > On Wed, Jul 18, 2012 at 2:19 PM, Javier Martinez Canillas > wrote: >> On Wed, Jul 18, 2012 at 10:36 AM, Shilimkar, Santosh >> wrote: >>> On Wed, Jul 18, 2012 at 1:14 PM, S, Venkatraman wrote: On Wed, Jul 18, 2012 at 12:40 PM, To

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread Shilimkar, Santosh
On Wed, Jul 18, 2012 at 2:19 PM, Javier Martinez Canillas wrote: > On Wed, Jul 18, 2012 at 10:36 AM, Shilimkar, Santosh > wrote: >> On Wed, Jul 18, 2012 at 1:14 PM, S, Venkatraman wrote: >>> On Wed, Jul 18, 2012 at 12:40 PM, Tony Lindgren wrote: * Shilimkar, Santosh [120718 00:09]: >

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread Javier Martinez Canillas
On Wed, Jul 18, 2012 at 10:36 AM, Shilimkar, Santosh wrote: > On Wed, Jul 18, 2012 at 1:14 PM, S, Venkatraman wrote: >> On Wed, Jul 18, 2012 at 12:40 PM, Tony Lindgren wrote: >>> * Shilimkar, Santosh [120718 00:09]: On Wed, Jul 18, 2012 at 12:29 PM, Tony Lindgren wrote: > * Javier Ma

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread Shilimkar, Santosh
On Wed, Jul 18, 2012 at 1:14 PM, S, Venkatraman wrote: > On Wed, Jul 18, 2012 at 12:40 PM, Tony Lindgren wrote: >> * Shilimkar, Santosh [120718 00:09]: >>> On Wed, Jul 18, 2012 at 12:29 PM, Tony Lindgren wrote: >>> > * Javier Martinez Canillas [120716 23:56]: >>> >> On Tue, Jul 17, 2012 at 8:4

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread Javier Martinez Canillas
On Wed, Jul 18, 2012 at 8:59 AM, Tony Lindgren wrote: > * Javier Martinez Canillas [120716 23:56]: >> On Tue, Jul 17, 2012 at 8:45 AM, Shilimkar, Santosh >> wrote: >> > Hi, >> > >> > On Tue, Jul 17, 2012 at 6:00 AM, Javier Martinez Canillas >> > wrote: >> >> The OMAP MMC and OMAP High Speed MMC

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread S, Venkatraman
On Wed, Jul 18, 2012 at 12:40 PM, Tony Lindgren wrote: > * Shilimkar, Santosh [120718 00:09]: >> On Wed, Jul 18, 2012 at 12:29 PM, Tony Lindgren wrote: >> > * Javier Martinez Canillas [120716 23:56]: >> >> On Tue, Jul 17, 2012 at 8:45 AM, Shilimkar, Santosh >> >> wrote: >> >> > Hi, >> >> > >>

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread Tony Lindgren
* Shilimkar, Santosh [120718 00:09]: > On Wed, Jul 18, 2012 at 12:29 PM, Tony Lindgren wrote: > > * Javier Martinez Canillas [120716 23:56]: > >> On Tue, Jul 17, 2012 at 8:45 AM, Shilimkar, Santosh > >> wrote: > >> > Hi, > >> > > >> > On Tue, Jul 17, 2012 at 6:00 AM, Javier Martinez Canillas >

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread Shilimkar, Santosh
On Wed, Jul 18, 2012 at 12:29 PM, Tony Lindgren wrote: > * Javier Martinez Canillas [120716 23:56]: >> On Tue, Jul 17, 2012 at 8:45 AM, Shilimkar, Santosh >> wrote: >> > Hi, >> > >> > On Tue, Jul 17, 2012 at 6:00 AM, Javier Martinez Canillas >> > wrote: >> >> The OMAP MMC and OMAP High Speed MM

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-18 Thread Tony Lindgren
* Javier Martinez Canillas [120716 23:56]: > On Tue, Jul 17, 2012 at 8:45 AM, Shilimkar, Santosh > wrote: > > Hi, > > > > On Tue, Jul 17, 2012 at 6:00 AM, Javier Martinez Canillas > > wrote: > >> The OMAP MMC and OMAP High Speed MMC hosts now use entirely the DMA > >> engine API instead of the p

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-16 Thread Javier Martinez Canillas
On Tue, Jul 17, 2012 at 8:45 AM, Shilimkar, Santosh wrote: > Hi, > > On Tue, Jul 17, 2012 at 6:00 AM, Javier Martinez Canillas > wrote: >> The OMAP MMC and OMAP High Speed MMC hosts now use entirely the DMA >> engine API instead of the previous private DMA API implementation. >> >> So, if the ker

Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default

2012-07-16 Thread Shilimkar, Santosh
Hi, On Tue, Jul 17, 2012 at 6:00 AM, Javier Martinez Canillas wrote: > The OMAP MMC and OMAP High Speed MMC hosts now use entirely the DMA > engine API instead of the previous private DMA API implementation. > > So, if the kernel is built with support for any of these hosts but it > doesn't suppo