diff options
| author | Brian Norris <computersforpeace@gmail.com> | 2013-03-28 21:28:56 -0700 |
|---|---|---|
| committer | David Woodhouse <David.Woodhouse@intel.com> | 2013-08-05 18:58:08 +0100 |
| commit | b966a40a3854e1263b3528ef95bc9e6017e8e515 (patch) | |
| tree | dd59d948f14d36b433258101d99d627c70cd7f3a | |
| parent | bddcb5e7caa72890fcd1d6ee7803452f2295a407 (diff) | |
| download | linux-b966a40a3854e1263b3528ef95bc9e6017e8e515.tar.gz linux-b966a40a3854e1263b3528ef95bc9e6017e8e515.tar.xz | |
mtd: fsl_ifc_nand: remove incorrect kfree()
The struct fsl_ifc_mtd is allocated with devm_kzalloc, so its memory
is "managed" automatically by the kernel. That is, we do not need to
free it explicitly; it will be freed when the device is removed. And we
*certainly* shouldn't free it with a regular kfree().
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
| -rw-r--r-- | drivers/mtd/nand/fsl_ifc_nand.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/mtd/nand/fsl_ifc_nand.c b/drivers/mtd/nand/fsl_ifc_nand.c index f1f7f12ab501..180bfa798e13 100644 --- a/drivers/mtd/nand/fsl_ifc_nand.c +++ b/drivers/mtd/nand/fsl_ifc_nand.c @@ -908,7 +908,6 @@ static int fsl_ifc_chip_remove(struct fsl_ifc_mtd *priv) ifc_nand_ctrl->chips[priv->bank] = NULL; dev_set_drvdata(priv->dev, NULL); - kfree(priv); return 0; } |