[PATCH v2] M25p80 little bugs
wangyuhang
wangyuhang2014 at gmail.com
Fri Jul 12 02:33:54 EDT 2013
Hi, Marek
Really sorry for the previous deformed patch.
And you are right, using devm_kzalloc looks better.
I have resended the patch, please help me review.
Thanks.
Signed-off-by: wangyuhang <wangyuhang2014 at gmail.com>
---
drivers/mtd/devices/m25p80.c | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/drivers/mtd/devices/m25p80.c b/drivers/mtd/devices/m25p80.c
index 5b6b072..70f3c54 100644
--- a/drivers/mtd/devices/m25p80.c
+++ b/drivers/mtd/devices/m25p80.c
@@ -955,15 +955,9 @@ static int m25p_probe(struct spi_device *spi)
}
}
- flash = kzalloc(sizeof *flash, GFP_KERNEL);
+ flash = devm_kzalloc(&spi->dev, sizeof *flash, GFP_KERNEL);
if (!flash)
return -ENOMEM;
- flash->command = kmalloc(MAX_CMD_SIZE + (flash->fast_read ? 1 : 0),
- GFP_KERNEL);
- if (!flash->command) {
- kfree(flash);
- return -ENOMEM;
- }
flash->spi = spi;
mutex_init(&flash->lock);
@@ -1032,6 +1026,11 @@ static int m25p_probe(struct spi_device *spi)
flash->fast_read = true;
#endif
+ flash->command = devm_kzalloc(&spi->dev,
+ MAX_CMD_SIZE + (flash->fast_read ? 1 : 0), GFP_KERNEL);
+ if (!flash->command)
+ return -ENOMEM;
+
if (info->addr_width)
flash->addr_width = info->addr_width;
else {
--
1.7.9.5
More information about the linux-mtd
mailing list