[PATCH] mtd: partitions: kill unused ecclayout struct

[PATCH] mtd: partitions: kill unused ecclayout struct

[Brian Norris]

This field is not used.

Reported here:

http://lists.infradead.org/pipermail/linux-mtd/2015-October/062417.html

Reported-by: Brian Foster <brian.foster@maximintegrated.com>
Cc: Brian Foster <brian.foster@maximintegrated.com>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
---
Depends on this for killing its last use:

  http://patchwork.ozlabs.org/patch/543159/

 include/linux/mtd/partitions.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/include/linux/mtd/partitions.h b/include/linux/mtd/partitions.h
index 282644c0c5c0..34388ced421a 100644
--- a/include/linux/mtd/partitions.h
+++ b/include/linux/mtd/partitions.h
@@ -41,7 +41,6 @@ struct mtd_partition {
 	uint64_t size;			/* partition size */
 	uint64_t offset;		/* offset within the master MTD space */
 	uint32_t mask_flags;		/* master MTD flags to mask out for this partition */
-	struct nand_ecclayout *ecclayout;	/* out of band layout for this partition (NAND only) */
 	struct device_node *of_node;	/* OF node attached to the partition */
 };
 
-- 
2.6.0.rc2.230.g3dd15c0

Re: [PATCH] mtd: partitions: kill unused ecclayout struct

[Brian Norris]

On Wed, Nov 11, 2015 at 05:05:56PM -0800, Brian Norris wrote:
> Depends on this for killing its last use:
> 
>   http://patchwork.ozlabs.org/patch/543159/

Also depends on this, for context:

  http://patchwork.ozlabs.org/patch/543161/

I should have just made these a series...

Re: [PATCH] mtd: partitions: kill unused ecclayout struct

[Boris Brezillon]

On Wed, 11 Nov 2015 17:05:56 -0800
Brian Norris <computersforpeace@gmail.com> wrote:

> This field is not used.
> 
> Reported here:
> 
> http://lists.infradead.org/pipermail/linux-mtd/2015-October/062417.html
> 
> Reported-by: Brian Foster <brian.foster@maximintegrated.com>
> Cc: Brian Foster <brian.foster@maximintegrated.com>
> Signed-off-by: Brian Norris <computersforpeace@gmail.com>

Reviewed-by: Boris Brezillon <boris.brezillon@free-electrons.com>

> ---
> Depends on this for killing its last use:
> 
>   http://patchwork.ozlabs.org/patch/543159/
> 
>  include/linux/mtd/partitions.h | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/include/linux/mtd/partitions.h b/include/linux/mtd/partitions.h
> index 282644c0c5c0..34388ced421a 100644
> --- a/include/linux/mtd/partitions.h
> +++ b/include/linux/mtd/partitions.h
> @@ -41,7 +41,6 @@ struct mtd_partition {
>  	uint64_t size;			/* partition size */
>  	uint64_t offset;		/* offset within the master MTD space */
>  	uint32_t mask_flags;		/* master MTD flags to mask out for this partition */
> -	struct nand_ecclayout *ecclayout;	/* out of band layout for this partition (NAND only) */
>  	struct device_node *of_node;	/* OF node attached to the partition */
>  };
>  



-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Re: [PATCH] mtd: partitions: kill unused ecclayout struct

[Brian Norris]

On Thu, Nov 12, 2015 at 09:42:41AM +0100, Boris Brezillon wrote:
> On Wed, 11 Nov 2015 17:05:56 -0800
> Brian Norris <computersforpeace@gmail.com> wrote:
> 
> > This field is not used.
> > 
> > Reported here:
> > 
> > http://lists.infradead.org/pipermail/linux-mtd/2015-October/062417.html
> > 
> > Reported-by: Brian Foster <brian.foster@maximintegrated.com>
> > Cc: Brian Foster <brian.foster@maximintegrated.com>
> > Signed-off-by: Brian Norris <computersforpeace@gmail.com>
> 
> Reviewed-by: Boris Brezillon <boris.brezillon@free-electrons.com>

Adjusted the context a bit and pushed to l2-mtd.git/next

Re: [PATCH] mtd: partitions: kill unused ecclayout struct

[Brian Foster]

On Wednesday 11-November-2015 17:05:56 Brian Norris wrote:
> This field is not used.
> 
> Reported here:
> 
> http://lists.infradead.org/pipermail/linux-mtd/2015-October/062417.html
> 
> Reported-by: Brian Foster <brian.foster@maximintegrated.com>
> Cc: Brian Foster <brian.foster@maximintegrated.com>
> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
> ---
> Depends on this for killing its last use:
> 
>   http://patchwork.ozlabs.org/patch/543159/

  Looks Ok to me.

cheers!
	-blf-

-- 
Brian Foster
Principal MTS, Software        |  La Ciotat, France
Maxim Integrated               |  http://www.maximintegrated.com/