From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH] OMAP: SPI: Fix the trying to free nonexistent resource error Date: Mon, 24 Oct 2011 13:45:07 +0200 Message-ID: <20111024114507.GK8708@ponder.secretlab.ca> References: <1319451864-8212-1-git-send-email-shubhrajyoti@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: spi-devel-general@lists.sourceforge.net, linux-kernel@vger.kernel.org, stable@vger.kernel.org To: Shubhrajyoti D Return-path: Content-Disposition: inline In-Reply-To: <1319451864-8212-1-git-send-email-shubhrajyoti@ti.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-spi.vger.kernel.org On Mon, Oct 24, 2011 at 03:54:24PM +0530, Shubhrajyoti D wrote: > Currently there is a request_mem_region(r->start, .. > followed by r->start += pdata->regs_offset; > > And then in remove > > r = platform_get_resource(pdev, IORESOURCE_MEM, 0); > release_mem_region(r->start, resource_size(r)); > > Here the offset addition is not taken care. Fix the code for the > same. > > Signed-off-by: Shubhrajyoti D Applied, thanks. g. > --- > drivers/spi/spi-omap2-mcspi.c | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/spi/spi-omap2-mcspi.c b/drivers/spi/spi-omap2-mcspi.c > index 06dc9a9..471b0f3 100644 > --- a/drivers/spi/spi-omap2-mcspi.c > +++ b/drivers/spi/spi-omap2-mcspi.c > @@ -1123,15 +1123,15 @@ static int __init omap2_mcspi_probe(struct platform_device *pdev) > status = -ENODEV; > goto err1; > } > + r->start += pdata->regs_offset; > + r->end += pdata->regs_offset; > + mcspi->phys = r->start; > if (!request_mem_region(r->start, resource_size(r), > dev_name(&pdev->dev))) { > status = -EBUSY; > goto err1; > } > > - r->start += pdata->regs_offset; > - r->end += pdata->regs_offset; > - mcspi->phys = r->start; > mcspi->base = ioremap(r->start, resource_size(r)); > if (!mcspi->base) { > dev_dbg(&pdev->dev, "can't ioremap MCSPI\n"); > -- > 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