From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Shilimkar, Santosh" Subject: Re: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default Date: Wed, 18 Jul 2012 12:34:29 +0530 Message-ID: References: <1342485005-5178-1-git-send-email-javier@dowhile0.org> <20120718065951.GH6522@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: Received: from na3sys009aog107.obsmtp.com ([74.125.149.197]:43212 "EHLO na3sys009aog107.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751287Ab2GRHEv (ORCPT ); Wed, 18 Jul 2012 03:04:51 -0400 Received: by ghbf14 with SMTP id f14so2068178ghb.8 for ; Wed, 18 Jul 2012 00:04:50 -0700 (PDT) In-Reply-To: <20120718065951.GH6522@atomide.com> Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Tony Lindgren Cc: Javier Martinez Canillas , Kevin.Hilman.khilman@ti.com, Russell King , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Chris Ball , linux-mmc@vger.kernel.org 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 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 support DMA devices nor OMAP DMA support, it fails when trying >> >> to obtain a DMA channel which leads to the following error on an OMAP3 >> >> IGEPv2 Rev.C board (and probably on most OMAP boards with MMC support): >> >> >> >> [ 2.199981] omap_hsmmc omap_hsmmc.1: unable to obtain RX DMA engine channel 48 >> >> [ 2.215087] omap_hsmmc omap_hsmmc.0: unable to obtain RX DMA engine channel 62 >> >> >> >> selecting automatically CONFIG_DMADEVICES and CONFIG_DMA_OMAP solves it. >> >> >> >> Signed-off-by: Javier Martinez Canillas >> >> --- >> > Considering, we are updating drivers to select the DMA engine, can you >> > also include >> > "drivers/spi/spi-omap2-mcspi.c" which is also updated for DMA engine. >> > >> > Regards >> > Santosh >> >> Hi Santosh, >> >> Ok, I'll send a v2 now which includes spi-omap2-mcspi then. > > I don't think we should do this, the drivers should work with and without > dma. This just needs to be added to the omap2plus_defconfig. > Well this was not decided based on any DMA CONFIG option before for the subject drivers. It is already by default enabled if the DMA is supported by the driver IP. There is a possibility to disable it from driver platform/dt data so that still remains. Regards Santosh From mboxrd@z Thu Jan 1 00:00:00 1970 From: santosh.shilimkar@ti.com (Shilimkar, Santosh) Date: Wed, 18 Jul 2012 12:34:29 +0530 Subject: [PATCH 1/1] mmc: host: enable OMAP DMA engine support for omap hosts by default In-Reply-To: <20120718065951.GH6522@atomide.com> References: <1342485005-5178-1-git-send-email-javier@dowhile0.org> <20120718065951.GH6522@atomide.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 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 support DMA devices nor OMAP DMA support, it fails when trying >> >> to obtain a DMA channel which leads to the following error on an OMAP3 >> >> IGEPv2 Rev.C board (and probably on most OMAP boards with MMC support): >> >> >> >> [ 2.199981] omap_hsmmc omap_hsmmc.1: unable to obtain RX DMA engine channel 48 >> >> [ 2.215087] omap_hsmmc omap_hsmmc.0: unable to obtain RX DMA engine channel 62 >> >> >> >> selecting automatically CONFIG_DMADEVICES and CONFIG_DMA_OMAP solves it. >> >> >> >> Signed-off-by: Javier Martinez Canillas >> >> --- >> > Considering, we are updating drivers to select the DMA engine, can you >> > also include >> > "drivers/spi/spi-omap2-mcspi.c" which is also updated for DMA engine. >> > >> > Regards >> > Santosh >> >> Hi Santosh, >> >> Ok, I'll send a v2 now which includes spi-omap2-mcspi then. > > I don't think we should do this, the drivers should work with and without > dma. This just needs to be added to the omap2plus_defconfig. > Well this was not decided based on any DMA CONFIG option before for the subject drivers. It is already by default enabled if the DMA is supported by the driver IP. There is a possibility to disable it from driver platform/dt data so that still remains. Regards Santosh