[U-Boot] [V2] disk/partition: Fix the depend for all the partitions file

Wolfgang Denk wd at denx.de
Sun Sep 2 14:41:56 CEST 2012


Dear Zhong Hongbo,

In message <1345075118-3131-1-git-send-email-bocui107 at gmail.com> you wrote:
> From: Zhong Hongbo <bocui107 at gmail.com>
> 
> all the patitions should depend CONFIG_PARTITIONS, no the
> CONFIG_CMD_$(device). The device symbol may be SATA, SCSI,
> USB, MMC, SYSTEMACE. Or else when the filesystem command be
> enabled, But board's configure file can not include any
> CONFIG_CMD_$(device), The u-boot will fail build. For example:
> CONFIG_CMD_EXT2 is defined in smdk6400.h, but the boards can
> not define any disk device. The error information of build as
> following:
> 
> common/cmd_ext2.c:191: undefined reference to `get_partition_info'
> fs/ext2/libext2fs.o: In function `ext2fs_set_blk_dev':
> fs/ext2/dev.c:44: undefined reference to `get_partition_info'
> 
> Signed-off-by: Zhong Hongbo <bocui107 at gmail.com>
> ---
> Change for V2:
> 	- Add Signed-off-by.
> ---
>  disk/part.c       |   21 +++------------------
>  disk/part_amiga.c |    6 +-----
>  disk/part_dos.c   |    7 +------
>  disk/part_efi.c   |    7 +------
>  disk/part_iso.c   |    7 +------
>  disk/part_mac.c   |    7 +------
>  6 files changed, 8 insertions(+), 47 deletions(-)

Hm.... did you make sure that all related board config files actually
define CONFIG_PARTITIONS ?

I doubt so.

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
COBOL is for morons.                                 -- E.W. Dijkstra


More information about the U-Boot mailing list