Hey Ian,

On 18-09-15 10:32, Ian Campbell wrote:
On Fri, 2015-09-18 at 10:22 +0200, Olliver Schinagl wrote:
Hey all,
Did you CC the wrong Hans by mistake?
Yeah, curse you auto-complete :S
just working on a _defconfig for our board and noticed that u-boot fails
to compile if my defconfig doesn't have:
# CONFIG_CMD_IMLS is not set

What supprised me is that it has to be in there as a comment to unset
it. I would have imaged that it would be unset by default and only be
available if set in the defconfig, or rather, how can commenting it in
the defconfig unset it?
That's how Kconfig works, an option is either set =m or =y or is a comment
saying "# FOO is not set", which is equivalent to the (nonexistent) =n
state.
ah, well I knew that the 'non-existant' variable would take the default value, maybe I just assumed the default value should be unset :)

If a value isn't present at all then it takes on its default value, as
specified in the Kconfig file (default default is off, "default y" makes it
y etc), as modified by select from or depends on other options.

I'm not sure about the actual build failure. It's probably just a bug in
either the code or the Kconfig, but I don't know what IMLS even is.
IMLS, IMage LiSt, but only for NOR, the NAND variant has some more letters appended :)

Ian.

I've added the u-boot list to the CC, but afaik it's not an open list
and my registrations have failed on a number of occasions ...

Not that it should fail to compile either way, here's the output:

In file included from
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/common/cmd_bootm.c:35:0:
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/include/mtd/cfi_flash.h:168:22:
error: unknown type name ‘flash_info_t’
   void flash_write_cmd(flash_info_t * info, flash_sect_t sect,
                        ^
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/include/mtd/cfi_flash.h:168:43:
error: unknown type name ‘flash_sect_t’
   void flash_write_cmd(flash_info_t * info, flash_sect_t sect,
                                             ^
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/include/mtd/cfi_flash.h:172:22:
error: unknown type name ‘flash_info_t’
   void flash_cmd_reset(flash_info_t *info);
                        ^
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/common/cmd_bootm.c:36:8:
error: unknown type name ‘flash_info_t’
   extern flash_info_t flash_info[]; /* info for FLASH chips */
          ^
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot-sunxi/common/cmd_bootm.c: In
function ‘do_imls_nor’:
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/common/cmd_bootm.c:321:2:
error: unknown type name ‘flash_info_t’
    flash_info_t *info;
    ^
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/common/cmd_bootm.c:326:7:
error: ‘CONFIG_SYS_MAX_FLASH_BANKS’ undeclared (first use in this
function)
     i < CONFIG_SYS_MAX_FLASH_BANKS; ++i, ++info) {
         ^
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/common/cmd_bootm.c:326:7:
note: each undeclared identifier is reported only once for each function
it appears in
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/common/cmd_bootm.c:328:11:
error: request for member ‘flash_id’ in something not a structure or
union
     if (info->flash_id == FLASH_UNKNOWN)
             ^
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/common/cmd_bootm.c:328:25:
error: ‘FLASH_UNKNOWN’ undeclared (first use in this function)
     if (info->flash_id == FLASH_UNKNOWN)
                           ^
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/common/cmd_bootm.c:330:23:
error: request for member ‘sector_count’ in something not a structure or
union
     for (j = 0; j < info->sector_count; ++j) {
                         ^
/home/oliver/Ultimaker/dev/sunxi-bsp/u-boot
-sunxi/common/cmd_bootm.c:332:22:
error: request for member ‘start’ in something not a structure or union
      hdr = (void *)info->start[j];


Olliver


_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to