Patchwork [2.6.30-rc5+] davinci_nand: cmdlinepart uses MTD IDs

login
register
mail settings
Submitter David Brownell
Date May 12, 2009, 11:23 p.m.
Message ID <200905121623.32809.david-b@pacbell.net>
Download mbox | patch
Permalink /patch/27145/
State New
Headers show

Comments

David Brownell - May 12, 2009, 11:23 p.m.
From: David Brownell <dbrownell@users.sourceforge.net>

Remove some legacy code from the davinci_nand driver, which made
cmdlinepart ignore the the MTD ID passed to it.  Boards can have
multiple NAND chips, and some do (like the DM357 EVM), so this
dated hack is undesirable.

Correct labels a"re like davinci_nand.0" (for chipselect 0).

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
---
 drivers/mtd/nand/davinci_nand.c |   11 -----------
 1 file changed, 11 deletions(-)
Artem Bityutskiy - May 15, 2009, 2:13 p.m.
On Tue, 2009-05-12 at 16:23 -0700, David Brownell wrote:
> From: David Brownell <dbrownell@users.sourceforge.net>
> 
> Remove some legacy code from the davinci_nand driver, which made
> cmdlinepart ignore the the MTD ID passed to it.  Boards can have
> multiple NAND chips, and some do (like the DM357 EVM), so this
> dated hack is undesirable.
> 
> Correct labels a"re like davinci_nand.0" (for chipselect 0).
> 
> Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>

Taken to l2-mtd-2.6.git.

Patch

--- a/drivers/mtd/nand/davinci_nand.c
+++ b/drivers/mtd/nand/davinci_nand.c
@@ -718,19 +718,8 @@  syndrome_done:
 			static const char *probes[] __initconst =
 				{ "cmdlinepart", NULL };
 
-			const char		*master_name;
-
-			/* Set info->mtd.name = 0 temporarily */
-			master_name		= info->mtd.name;
-			info->mtd.name		= (char *)0;
-
-			/* info->mtd.name == 0, means: don't bother checking
-			   <mtd-id> */
 			mtd_parts_nb = parse_mtd_partitions(&info->mtd, probes,
 							    &mtd_parts, 0);
-
-			/* Restore info->mtd.name */
-			info->mtd.name = master_name;
 		}
 
 		if (mtd_parts_nb <= 0) {