Re: [upstream] mtd/ifc: fix ifc driver memory release issue

2013-05-10 Thread Artem Bityutskiy
On Thu, 2013-03-28 at 21:28 -0700, Brian Norris wrote: > On Wed, Mar 27, 2013 at 5:25 AM, Roy Zang wrote: > > memory is allocated by devm_kzalloc, so release it using > > devm_kfree() instead kfree(); > > You are correct that it should not be freed with kfree(). But the > correct solution is that

Re: [upstream] mtd/ifc: fix ifc driver memory release issue

2013-03-28 Thread Brian Norris
On Wed, Mar 27, 2013 at 5:25 AM, Roy Zang wrote: > memory is allocated by devm_kzalloc, so release it using > devm_kfree() instead kfree(); You are correct that it should not be freed with kfree(). But the correct solution is that it does not need to be freed (explicitly) at all. That's the whole

[upstream] mtd/ifc: fix ifc driver memory release issue

2013-03-26 Thread Roy Zang
memory is allocated by devm_kzalloc, so release it using devm_kfree() instead kfree(); Signed-off-by: Li Hao Signed-off-by: Cao Yonghua Signed-off-by: Roy Zang --- drivers/mtd/nand/fsl_ifc_nand.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/mtd/nand/fsl_ifc_