Re: [U-Boot] [PATCH 2/2] config_cmd_default.h: wrap each definition in "ifndef"

Hi Alexey, On Wed, 14 Jan 2015 06:23:55 + Alexey Brodkin wrote: > Hi Masahiro-san, > > On Wed, 2015-01-14 at 13:12 +0900, Masahiro Yamada wrote: > > On Fri, 9 Jan 2015 21:21:20 + > > Alexey Brodkin wrote: > > I think this change is harmless, but > > > > #define CONFIG_CMD_BDI 1 >

Re: [U-Boot] [PATCH 2/2] config_cmd_default.h: wrap each definition in "ifndef"

Hi Masahiro-san, On Wed, 2015-01-14 at 13:12 +0900, Masahiro Yamada wrote: > On Fri, 9 Jan 2015 21:21:20 + > Alexey Brodkin wrote: > I think this change is harmless, but > > #define CONFIG_CMD_BDI 1 > > is simpler than > > #ifndef CONFIG_CMD_BDI > #define CONFIG_CMD_BDI > #endif > > >

Re: [U-Boot] [PATCH 2/2] config_cmd_default.h: wrap each definition in "ifndef"

On Fri, 9 Jan 2015 21:21:20 + Alexey Brodkin wrote: > Hi all, > > On Sat, 2015-01-03 at 16:20 +0300, Alexey Brodkin wrote: > > Now with switch to Kconfig it's possible to select commands via > > menuconfig or defconfigs. > > > > Moreover some commands are enabled by default in both > > "co

Re: [U-Boot] [PATCH 2/2] config_cmd_default.h: wrap each definition in "ifndef"

Hi all, On Sat, 2015-01-03 at 16:20 +0300, Alexey Brodkin wrote: > Now with switch to Kconfig it's possible to select commands via > menuconfig or defconfigs. > > Moreover some commands are enabled by default in both > "config_cmd_default.h" as a legacy way and in "common/Kconfig". > > And on at

[U-Boot] [PATCH 2/2] config_cmd_default.h: wrap each definition in "ifndef"

Now with switch to Kconfig it's possible to select commands via menuconfig or defconfigs. Moreover some commands are enabled by default in both "config_cmd_default.h" as a legacy way and in "common/Kconfig". And on attempt to build U-Boot following warnings are printed for definitions made in bot