On Saturday 19 March 2011 05:38:13 Dan Carpenter wrote: > The original code had an off-by-one error so we didn't release the > entire memory region on remove(). > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> Acked-by: Florian Fainelli <florian@xxxxxxxxxxx> > > diff --git a/drivers/vlynq/vlynq.c b/drivers/vlynq/vlynq.c > index f885c86..1d37553 100644 > --- a/drivers/vlynq/vlynq.c > +++ b/drivers/vlynq/vlynq.c > @@ -764,7 +764,7 @@ static int vlynq_remove(struct platform_device *pdev) > > device_unregister(&dev->dev); > iounmap(dev->local); > - release_mem_region(dev->regs_start, dev->regs_end - dev->regs_start); > + release_mem_region(dev->regs_start, dev->regs_end - dev->regs_start + 1); > > kfree(dev); -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html