public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] Compiling u-boot (sunxi/next)
@ 2015-09-18  8:22 Olliver Schinagl
  2015-09-18  8:32 ` Ian Campbell
  0 siblings, 1 reply; 3+ messages in thread
From: Olliver Schinagl @ 2015-09-18  8:22 UTC (permalink / raw)
  To: u-boot

Hey all,

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?

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] Compiling u-boot (sunxi/next)
  2015-09-18  8:22 [U-Boot] Compiling u-boot (sunxi/next) Olliver Schinagl
@ 2015-09-18  8:32 ` Ian Campbell
  2015-09-18  8:38   ` Olliver Schinagl
  0 siblings, 1 reply; 3+ messages in thread
From: Ian Campbell @ 2015-09-18  8:32 UTC (permalink / raw)
  To: u-boot

On Fri, 2015-09-18 at 10:22 +0200, Olliver Schinagl wrote:
> Hey all,

Did you CC the wrong Hans by mistake?

> 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.

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.

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
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] Compiling u-boot (sunxi/next)
  2015-09-18  8:32 ` Ian Campbell
@ 2015-09-18  8:38   ` Olliver Schinagl
  0 siblings, 0 replies; 3+ messages in thread
From: Olliver Schinagl @ 2015-09-18  8:38 UTC (permalink / raw)
  To: u-boot

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
>>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-09-18  8:38 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-18  8:22 [U-Boot] Compiling u-boot (sunxi/next) Olliver Schinagl
2015-09-18  8:32 ` Ian Campbell
2015-09-18  8:38   ` Olliver Schinagl

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox