From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shubhrajyoti Subject: Re: [PATCH 2/3] OMAP: SPI: call pm_runtime_disable in error path and remove Date: Mon, 31 Oct 2011 10:46:50 +0530 Message-ID: <4EAE2F42.1010906@ti.com> References: <1319802259-16565-1-git-send-email-shubhrajyoti@ti.com> <1319802259-16565-3-git-send-email-shubhrajyoti@ti.com> <20111029120655.GE22592@ponder.secretlab.ca> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "Hebbar, Gururaja" To: Grant Likely Return-path: In-Reply-To: <20111029120655.GE22592-e0URQFbLeQY2iJbIjFUEsiwD8/FfD2ys@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: spi-devel-general-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-spi.vger.kernel.org On Saturday 29 October 2011 05:36 PM, Grant Likely wrote: > On Fri, Oct 28, 2011 at 05:14:18PM +0530, Shubhrajyoti D wrote: >> omap mcspi probe() doesnt call pm_runtime disable functions >> in case of failure. remove() doesnt call pm_runtime disable. This could >> lead to warnings as below on subsequent insmod. >> >> ~# insmod spi-omap2-mcspi.ko >> [ 255.383671] omap2_mcspi omap2_mcspi.1: Unbalanced pm_runtime_enable! >> ... >> >> This patch adds the pm_runtime disable() at appropriate stages. >> >> Signed-off-by: Hebbar, Gururaja >> Signed-off-by: Shubhrajyoti D > Nitpick: please use the format "spi/: ...." for the subject > line of spi patches. It's just a consistency thing that I like. Henceforth I will take care > Picked up for v3.3, thanks. > > g. > >> --- >> drivers/spi/spi-omap2-mcspi.c | 2 ++ >> 1 files changed, 2 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/spi/spi-omap2-mcspi.c b/drivers/spi/spi-omap2-mcspi.c >> index 471b0f3..6875a0b 100644 >> --- a/drivers/spi/spi-omap2-mcspi.c >> +++ b/drivers/spi/spi-omap2-mcspi.c >> @@ -1195,6 +1195,7 @@ static int __init omap2_mcspi_probe(struct platform_device *pdev) >> err4: >> spi_master_put(master); >> err3: >> + pm_runtime_disable(&pdev->dev); >> kfree(mcspi->dma_channels); >> err2: >> release_mem_region(r->start, resource_size(r)); >> @@ -1216,6 +1217,7 @@ static int __exit omap2_mcspi_remove(struct platform_device *pdev) >> dma_channels = mcspi->dma_channels; >> >> omap2_mcspi_disable_clocks(mcspi); >> + pm_runtime_disable(&pdev->dev); >> r = platform_get_resource(pdev, IORESOURCE_MEM, 0); >> release_mem_region(r->start, resource_size(r)); >> >> -- >> 1.7.1 >> >> >> ------------------------------------------------------------------------------ >> The demand for IT networking professionals continues to grow, and the >> demand for specialized networking skills is growing even more rapidly. >> Take a complimentary Learning@Cisco Self-Assessment and learn >> about Cisco certifications, training, and career opportunities. >> http://p.sf.net/sfu/cisco-dev2dev >> _______________________________________________ >> spi-devel-general mailing list >> spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org >> https://lists.sourceforge.net/lists/listinfo/spi-devel-general ------------------------------------------------------------------------------ Get your Android app more play: Bring it to the BlackBerry PlayBook in minutes. BlackBerry App World™ now supports Android™ Apps for the BlackBerry® PlayBook™. Discover just how easy and simple it is! http://p.sf.net/sfu/android-dev2dev