[U-Boot] [PATCH v4] Separate mtdparts command from jffs2

Wolfgang Denk wd at denx.de
Fri May 15 17:32:17 CEST 2009


Dear Stefan Roese,

In message <1237465836-26622-1-git-send-email-sr at denx.de> you wrote:
> Currently the mtdparts commands are included in the jffs2 command support.
> This doesn't make sense anymore since other commands (e.g. UBI) use this
> infrastructure as well now. This patch separates the mtdparts commands from
> the jffs2 commands making it possible to only select mtdparts when no JFFS2
> support is needed.
> 
> Signed-off-by: Stefan Roese <sr at denx.de>
> Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>

As it turns out, that split was incomplete or incorrect.

It seems that since commit 68d7d65100 we have two different,
incompatible implementations of mtdparts_init() - one in
common/cmd_jffs2.c and another one in common/cmd_mtdparts.c

When NAND and JFFS2 and MTDPARTS are enabled at the same time, the
mtdparts_init() from common/cmd_mtdparts.c is called, which obviously
does not pereform the required intializations for JFFS2: all JFFS2
related commands just return without any error messages but also
without any useful output either.

Please provide a fix!

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Q:  How many DEC repairman does it take to fix a flat ?
A:  Five; four to hold the car up and one to swap tires.


More information about the U-Boot mailing list