]> git.dujemihanovic.xyz Git - u-boot.git/commit
jffs2/mtdparts: Fix problem with usage from JFFS2 and MTDPARTS together
authorStefan Roese <sr@denx.de>
Sat, 16 May 2009 10:04:22 +0000 (12:04 +0200)
committerWolfgang Denk <wd@denx.de>
Thu, 28 May 2009 19:26:00 +0000 (21:26 +0200)
commit76b5883da2cf049cd410901c04ea450e5f5c27c3
tree689898ecf9697436de062a5c7bba0590e88654d2
parentf40f6db278f602b55820693634a7256b0b4e4b80
jffs2/mtdparts: Fix problem with usage from JFFS2 and MTDPARTS together

Currently using JFFS2 with MTDPARTS enabled doesn't work. This is because
mtdparts_init() is available in both files, cmd_mtdparts.c and
cmd_jffs2.c. Please note that in the original cmd_jffs2.c file (before
the jffs2/mtdparts command/file split those 2 different versions
already existed. So this is nothing new. The main problem is that the
variables "current_dev" and "current_partnum" are declared in both
files now. This doesn't work.

This patch now changes the names of those variable to more specific
names: "current_mtd_dev" and "current_mtd_partnum". This is because
this patch also changes the declaration from static to global, so
that they can be used from both files.

Please note that my first tests were not successful. The MTD devices
selected via mtdparts are now accessed but I'm failing to see the
directory listed via the "ls" command. Nothing is displayed. Perhaps
I didn't generate the JFFS2 image correctly (I never used JFFS2 in
U-Boot before). Not sure. Perhaps somebody else could take a look at
this as well. I'll continue looking into this on Monday.

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Wolfgang Denk <wd@denx.de>
Cc: Detlev Zundel <dzu@denx.de>
Cc: Ilya Yanok <yanok@emcraft.com>
Cc: Renaud barbier <renaud.barbier@ge.com>
common/cmd_jffs2.c
common/cmd_mtdparts.c