[PATCH 2/5] [WATCHDOG] s3c2410_wdt - Avoid freeing memory, if memory region could not be allocated
Kukjin Kim
kgene.kim at samsung.com
Thu May 20 06:32:40 EDT 2010
Sergei Shtylyov wrote:
>
> Hello.
>
> Kukjin Kim wrote:
>
> > From: Banajit Goswami <banajit.g at samsung.com>
>
> > In case the WDT device memory allocation fails, this patch modifies the
>
> This is not memory allocation.
Ok, will fix it.
>
> > driver such that, it does not try to free the memory on exit.
>
> > Signed-off-by: Banajit Goswami <banajit.g at samsung.com>
> > Signed-off-by: Kukjin Kim <kgene.kim at samsung.com>
> > ---
> > drivers/watchdog/s3c2410_wdt.c | 3 +--
> > 1 files changed, 1 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/watchdog/s3c2410_wdt.c
b/drivers/watchdog/s3c2410_wdt.c
> > index e4cebef..d421258 100644
> > --- a/drivers/watchdog/s3c2410_wdt.c
> > +++ b/drivers/watchdog/s3c2410_wdt.c
> > @@ -426,8 +426,7 @@ static int __devinit s3c2410wdt_probe(struct
> platform_device *pdev)
> > wdt_mem = request_mem_region(res->start, size, pdev->name);
> > if (wdt_mem == NULL) {
> > dev_err(dev, "failed to get memory region\n");
> > - ret = -ENOENT;
> > - goto err_req;
> > + return -ENOMEM;
>
> It should be EBUSY actually...
Ok, will fix it.
>
>
> WBR, Sergei
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
More information about the linux-arm-kernel
mailing list