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

Wolfgang Denk wd at denx.de
Fri Mar 20 22:04:14 CET 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>
> ---
> v4:
>   - Keep config_cmd_all.h sorted
>   - Add "mtd_" to the names of some now non-static functions
>   - Cleanup of "jffs2" names and comments in cmd_mtdparts.c
> 
> v3:
>   - Keep lists sorted (README, Makefile) as pointed out by Wolfgang
>   - Generated without find-copies-harder
> 
> v2:
>   This patch is based in the original version by Kyungmin Park with some
>   additional changes by Jean-Christophe Plagniol-Villard. Changes are
>   the patch description and removal of the compilation errors/warnings.
> 
>  README                                 |    1 +
>  board/fads/fads.h                      |    4 +-
>  common/Makefile                        |    1 +
>  common/cmd_flash.c                     |   18 +-
>  common/cmd_jffs2.c                     | 1731 +-------------------------------
>  common/{cmd_jffs2.c => cmd_mtdparts.c} |  432 +--------
>  common/cmd_nand.c                      |   10 +-
>  include/config_cmd_all.h               |    1 +
>  include/configs/ADNPESC1.h             |    4 +-
>  include/configs/Alaska8220.h           |    4 +-
>  include/configs/BAB7xx.h               |    4 +-
>  include/configs/BC3450.h               |    2 +-
>  include/configs/CATcenter.h            |    4 +-
>  include/configs/DB64360.h              |    4 +-
>  include/configs/DB64460.h              |    4 +-
>  include/configs/DK1C20.h               |    4 +-
>  include/configs/DK1S10.h               |    4 +-
>  include/configs/ELPPC.h                |    4 +-
>  include/configs/FPS850L.h              |    2 +-
>  include/configs/FPS860L.h              |    2 +-
>  include/configs/LANTEC.h               |    4 +-
>  include/configs/MHPC.h                 |    4 +-
>  include/configs/MIP405.h               |    4 +-
>  include/configs/ML2.h                  |    4 +-
>  include/configs/MPC8266ADS.h           |    4 +-
>  include/configs/NC650.h                |    4 +-
>  include/configs/NETTA.h                |    4 +-
>  include/configs/PPChameleonEVB.h       |    4 +-
>  include/configs/R360MPI.h              |    4 +-
>  include/configs/RBC823.h               |    4 +-
>  include/configs/Rattler.h              |    4 +-
>  include/configs/SIMPC8313.h            |    2 +-
>  include/configs/SXNI855T.h             |    4 +-
>  include/configs/TB5200.h               |    2 +-
>  include/configs/TQM5200.h              |    2 +-
>  include/configs/TQM823L.h              |    2 +-
>  include/configs/TQM823M.h              |    2 +-
>  include/configs/TQM834x.h              |    2 +-
>  include/configs/TQM850L.h              |    2 +-
>  include/configs/TQM850M.h              |    2 +-
>  include/configs/TQM855L.h              |    2 +-
>  include/configs/TQM855M.h              |    2 +-
>  include/configs/TQM85xx.h              |    4 +-
>  include/configs/TQM860L.h              |    2 +-
>  include/configs/TQM860M.h              |    2 +-
>  include/configs/TQM862L.h              |    2 +-
>  include/configs/TQM862M.h              |    2 +-
>  include/configs/TQM866M.h              |    2 +-
>  include/configs/ZUMA.h                 |    4 +-
>  include/configs/apollon.h              |    2 +-
>  include/configs/cm5200.h               |    2 +-
>  include/configs/debris.h               |    4 +-
>  include/configs/ep7312.h               |    4 +-
>  include/configs/ep8260.h               |    4 +-
>  include/configs/fx12mm.h               |    2 +-
>  include/configs/hymod.h                |    4 +-
>  include/configs/idmr.h                 |    2 +-
>  include/configs/impa7.h                |    4 +-
>  include/configs/imx31_litekit.h        |    2 +-
>  include/configs/imx31_phycore.h        |    2 +-
>  include/configs/incaip.h               |    4 +-
>  include/configs/innokom.h              |    4 +-
>  include/configs/microblaze-generic.h   |    2 +-
>  include/configs/modnet50.h             |    4 +-
>  include/configs/motionpro.h            |    2 +-
>  include/configs/mx1fs2.h               |    4 +-
>  include/configs/mx31ads.h              |    2 +-
>  include/configs/netstar.h              |    2 +-
>  include/configs/omap2420h4.h           |    4 +-
>  include/configs/qong.h                 |    2 +-
>  include/configs/sc3.h                  |    2 +-
>  include/configs/sc520_cdp.h            |    4 +-
>  include/configs/sc520_spunk.h          |    4 +-
>  include/configs/smmaco4.h              |    2 +-
>  include/configs/trab.h                 |    2 +-
>  include/configs/v37.h                  |    4 +-
>  include/configs/vct.h                  |    2 +-
>  include/configs/virtlab2.h             |    2 +-
>  include/configs/voiceblue.h            |    2 +-
>  include/configs/xilinx-ppc.h           |    4 +-
>  include/configs/xsengine.h             |    4 +-
>  81 files changed, 164 insertions(+), 2256 deletions(-)
>  copy common/{cmd_jffs2.c => cmd_mtdparts.c} (82%)

Thanks. Applied to "next" branch. 

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
A witty saying proves nothing, but saying  something  pointless  gets
people's attention.


More information about the U-Boot mailing list