[PATCH 1/5] mtd: m25p80: fix allocation size
Marek Vasut
marex at denx.de
Sun Oct 27 09:30:11 PDT 2013
Hi Brian,
[...]
> @@ -1137,14 +1135,10 @@ static int m25p_probe(struct spi_device *spi)
> static int m25p_remove(struct spi_device *spi)
> {
> struct m25p *flash = spi_get_drvdata(spi);
> - int status;
>
> /* Clean up MTD stuff. */
> - status = mtd_device_unregister(&flash->mtd);
> - if (status == 0) {
> - kfree(flash->command);
> - kfree(flash);
> - }
> + mtd_device_unregister(&flash->mtd);
> +
> return 0;
> }
I wonder if we shouldn't return "status" in here in case mtd_device_unregister()
failed.
Best regards,
Marek Vasut
More information about the linux-mtd
mailing list