Messages in this thread | | | Date | Mon, 31 Oct 2011 10:46:50 +0530 | From | Shubhrajyoti <> | Subject | Re: [PATCH 2/3] OMAP: SPI: call pm_runtime_disable in error path and remove |
| |
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<gururaja.hebbar@ti.com> >> Signed-off-by: Shubhrajyoti D<shubhrajyoti@ti.com> > Nitpick: please use the format "spi/<device>: ...." 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@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/spi-devel-general
| |