* Re: [PATCH RESEND] mtd: fix kfree bcm47xxsflash
2013-05-22 3:51 ` [PATCH RESEND] " Libo Chen
@ 2013-05-22 5:32 ` Rafał Miłecki
2013-05-28 12:40 ` Libo Chen
2013-05-28 15:29 ` Andy Shevchenko
2 siblings, 0 replies; 8+ messages in thread
From: Rafał Miłecki @ 2013-05-22 5:32 UTC (permalink / raw)
To: Libo Chen
Cc: Li Zefan, artem.bityutskiy, LKML, wfp5p, linux-mtd, hauke,
Andrew Morton, dwmw2
2013/5/22 Libo Chen <clbchenlibo.chen@huawei.com>:
> mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its member.
Thanks!
--
Rafał
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH RESEND] mtd: fix kfree bcm47xxsflash
2013-05-22 3:51 ` [PATCH RESEND] " Libo Chen
2013-05-22 5:32 ` Rafał Miłecki
@ 2013-05-28 12:40 ` Libo Chen
2013-05-28 12:49 ` Rafał Miłecki
2013-05-28 15:29 ` Andy Shevchenko
2 siblings, 1 reply; 8+ messages in thread
From: Libo Chen @ 2013-05-28 12:40 UTC (permalink / raw)
To: dwmw2
Cc: Li Zefan, Greg KH, artem.bityutskiy, zajec5, LKML, wfp5p,
linux-mtd, hauke, Andrew Morton
ping...
On 2013/5/22 11:51, Libo Chen wrote:
>
> mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its member.
>
> Signed-off-by: Libo Chen <libo.chen@huawei.com>
> Acked-by: Rafał Miłecki <zajec5@gmail.com>
> ---
> drivers/mtd/devices/bcm47xxsflash.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> * add Acked-by: Rafał Miłecki <zajec5@gmail.com>
> * change send list
>
> diff --git a/drivers/mtd/devices/bcm47xxsflash.c b/drivers/mtd/devices/bcm47xxsflash.c
> index 9526628..4aa9141 100644
> --- a/drivers/mtd/devices/bcm47xxsflash.c
> +++ b/drivers/mtd/devices/bcm47xxsflash.c
> @@ -76,7 +76,7 @@ static int bcm47xxsflash_bcma_probe(struct platform_device *pdev)
> return 0;
>
> err_dev_reg:
> - kfree(&b47s->mtd);
> + kfree(b47s);
> out:
> return err;
> }
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH RESEND] mtd: fix kfree bcm47xxsflash
2013-05-28 12:40 ` Libo Chen
@ 2013-05-28 12:49 ` Rafał Miłecki
0 siblings, 0 replies; 8+ messages in thread
From: Rafał Miłecki @ 2013-05-28 12:49 UTC (permalink / raw)
To: Libo Chen
Cc: Li Zefan, Greg KH, artem.bityutskiy, LKML, wfp5p, linux-mtd,
hauke, Andrew Morton, dwmw2
2013/5/28 Libo Chen <clbchenlibo.chen@huawei.com>:
> ping...
It takes a lot of time for someone to pick up mtd patches, just be patient ;)
--
Rafał
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH RESEND] mtd: fix kfree bcm47xxsflash
2013-05-22 3:51 ` [PATCH RESEND] " Libo Chen
2013-05-22 5:32 ` Rafał Miłecki
2013-05-28 12:40 ` Libo Chen
@ 2013-05-28 15:29 ` Andy Shevchenko
2013-05-29 2:40 ` Libo Chen
2 siblings, 1 reply; 8+ messages in thread
From: Andy Shevchenko @ 2013-05-28 15:29 UTC (permalink / raw)
To: Libo Chen
Cc: Li Zefan, artem.bityutskiy, zajec5, LKML, Bill Pemberton,
linux-mtd, hauke, Andrew Morton, dwmw2
This is good fix, but I rather prefer not to see it in this shape.
See comments below.
On Wed, May 22, 2013 at 6:51 AM, Libo Chen <clbchenlibo.chen@huawei.com> wrote:
>
> mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its member.
>
> Signed-off-by: Libo Chen <libo.chen@huawei.com>
> Acked-by: Rafał Miłecki <zajec5@gmail.com>
> ---
> drivers/mtd/devices/bcm47xxsflash.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> * add Acked-by: Rafał Miłecki <zajec5@gmail.com>
> * change send list
>
> diff --git a/drivers/mtd/devices/bcm47xxsflash.c b/drivers/mtd/devices/bcm47xxsflash.c
> index 9526628..4aa9141 100644
> --- a/drivers/mtd/devices/bcm47xxsflash.c
> +++ b/drivers/mtd/devices/bcm47xxsflash.c
> @@ -76,7 +76,7 @@ static int bcm47xxsflash_bcma_probe(struct platform_device *pdev)
> return 0;
>
> err_dev_reg:
> - kfree(&b47s->mtd);
> + kfree(b47s);
> out:
> return err;
> }
What about to switch to devm_* and remove those labels and kfrees at all?
Additionally you can add another patch to use module_platform_driver()
macro instead of module_init/exit.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH RESEND] mtd: fix kfree bcm47xxsflash
2013-05-28 15:29 ` Andy Shevchenko
@ 2013-05-29 2:40 ` Libo Chen
0 siblings, 0 replies; 8+ messages in thread
From: Libo Chen @ 2013-05-29 2:40 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Li Zefan, artem.bityutskiy, zajec5, LKML, Bill Pemberton,
linux-mtd, hauke, Andrew Morton, dwmw2
On 2013/5/28 23:29, Andy Shevchenko wrote:
> This is good fix, but I rather prefer not to see it in this shape.
> See comments below.
>
> On Wed, May 22, 2013 at 6:51 AM, Libo Chen <clbchenlibo.chen@huawei.com> wrote:
>>
>> mtd is just member of bcm47xxsflash, so we should free bcm47xxsflash not its member.
>>
>> Signed-off-by: Libo Chen <libo.chen@huawei.com>
>> Acked-by: Rafał Miłecki <zajec5@gmail.com>
>> ---
>> drivers/mtd/devices/bcm47xxsflash.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> * add Acked-by: Rafał Miłecki <zajec5@gmail.com>
>> * change send list
>>
>> diff --git a/drivers/mtd/devices/bcm47xxsflash.c b/drivers/mtd/devices/bcm47xxsflash.c
>> index 9526628..4aa9141 100644
>> --- a/drivers/mtd/devices/bcm47xxsflash.c
>> +++ b/drivers/mtd/devices/bcm47xxsflash.c
>> @@ -76,7 +76,7 @@ static int bcm47xxsflash_bcma_probe(struct platform_device *pdev)
>> return 0;
>>
>> err_dev_reg:
>> - kfree(&b47s->mtd);
>> + kfree(b47s);
>> out:
>> return err;
>> }
>
> What about to switch to devm_* and remove those labels and kfrees at all?
>
> Additionally you can add another patch to use module_platform_driver()
> macro instead of module_init/exit.
>
ok, I will update later.
thanks,
Libo
> --
> With Best Regards,
> Andy Shevchenko
>
> .
>
^ permalink raw reply [flat|nested] 8+ messages in thread