Skip to contentSkip to conversation

mtd: partitions: kill unused ecclayout struct

PATCH In review Brian Norris 3 participants 5 messages Nov 12, 2015 – Nov 16, 2015 Memory technology devices (MTD)

lore
Share & tools
lore

Brian Norris

author
#0 lore raw

PATCH 1/1 mtd: partitions: kill unused ecclayout struct

This field is not used.

Reported here:

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

Reported-by Brian Foster <[email protected]>
Cc Brian Foster <[email protected]>
Signed-off-by Brian Norris <[email protected]>

---
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(-)
include/linux/mtd/partitions.h +0 -1
index 282644c0c5c0..34388ced421a 100644
@@ -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 */
 };
Signature · 1 line
2.6.0.rc2.230.g3dd15c0
Reviewed-by on v1 1/1
#2 lore raw
In reply to Brian Norris
Quoted 9 lines · On Wed, 11 Nov 2015 17:05:56 -0800 Brian Norris <[email protected]>…

This field is not used.

Reported here:

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

Reported-by Brian Foster <[email protected]>
Cc Brian Foster <[email protected]>
Signed-off-by Brian Norris <[email protected]>
Reviewed-by Boris Brezillon <[email protected]>
Quoted 20 lines

---
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(-)
include/linux/mtd/partitions.h +0 -1
index 282644c0c5c0..34388ced421a 100644
@@ -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 */
 };
Signature · 3 lines
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
author
#3 lore raw
Quoted 7 lines · Boris Brezillon
Quoted 4 lines · On Wed, 11 Nov 2015 17:05:56 -0800 Brian Norris <[email protected]>…

This field is not used.

Reported here:

Quoted 7 lines
Quoted 5 lines

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

Reported-by Brian Foster <[email protected]>
Cc Brian Foster <[email protected]>
Signed-off-by Brian Norris <[email protected]>
Reviewed-by Boris Brezillon <[email protected]>

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

#4 lore raw
In reply to Brian Norris
Quoted 6 lines · 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

Quoted 7 lines
Reported-by Brian Foster <[email protected]>
Cc Brian Foster <[email protected]>
Signed-off-by Brian Norris <[email protected]>

---
Depends on this for killing its last use:

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

Looks Ok to me.

cheers!

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