On Mon, Jan 18, 2010 at 2:50 AM, <[email protected]> wrote: > From: Jassi Brar <[email protected]> > > Add precautionary check before releasing memory region. > > Signed-off-by: Jassi Brar <[email protected]> > --- > drivers/spi/spi_s3c64xx.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/drivers/spi/spi_s3c64xx.c b/drivers/spi/spi_s3c64xx.c > index 3acf381..0e883f9 100644 > --- a/drivers/spi/spi_s3c64xx.c > +++ b/drivers/spi/spi_s3c64xx.c > @@ -1097,7 +1097,8 @@ static int s3c64xx_spi_remove(struct platform_device > *pdev) > iounmap((void *) sdd->regs); > > mem_res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - release_mem_region(mem_res->start, resource_size(mem_res)); > + if (mem_res != NULL) > + release_mem_region(mem_res->start, resource_size(mem_res));
request_mem_region() in in an unconditional path in the .probe() hook. Are you seeing failure on the remove path? g. ------------------------------------------------------------------------------ Throughout its 18-year history, RSA Conference consistently attracts the world's best and brightest in the field, creating opportunities for Conference attendees to learn about information security's most important issues through interactions with peers, luminaries and emerging and established companies. http://p.sf.net/sfu/rsaconf-dev2dev _______________________________________________ spi-devel-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/spi-devel-general
