[PATCH] mtd: m25p80: fix missing MTD module owner

Marek Vasut marex at denx.de
Mon Oct 8 09:09:44 EDT 2012


Dear Grégor Boirie,

> Marek,
> 
> Sorry about that. I didn't really mean to submit an official patch at
> first...

No no, nothing to be sorry about actually, we're really grateful you spotted the 
issue and reported it!

> Nevermind, please find one just below.

Actually, can you try one more time and after commiting the change to your local 
git tree, do "git send-email --to="linux-mtd at lists.infradead.org" HEAD~1" 
please?

Also, it is a good idea to add "Cc: Name Surname <e at ma.il>" lines to the patch 
description just before the Signed-off-by line so that the people who will end 
up applying it are CCed. That's at least Artem.

> Regards,
> gregor
> 
> ---
>  drivers/mtd/devices/m25p80.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/mtd/devices/m25p80.c b/drivers/mtd/devices/m25p80.c
> index 1924d24..06beb3f 100644
> --- a/drivers/mtd/devices/m25p80.c
> +++ b/drivers/mtd/devices/m25p80.c
> @@ -873,6 +873,7 @@ static int __devinit m25p_probe(struct spi_device *spi)
>  	else
>  		flash->mtd.name = dev_name(&spi->dev);
>  +	flash->mtd.owner = THIS_MODULE;
>  	flash->mtd.type = MTD_NORFLASH;
>  	flash->mtd.writesize = 1;
>  	flash->mtd.flags = MTD_CAP_NORFLASH;

Best regards,
Marek Vasut



More information about the linux-mtd mailing list