Hi Masahiro, On 23 September 2016 at 03:32, Masahiro Yamada <[email protected]> wrote: > 2016-09-23 13:16 GMT+09:00 Simon Glass <[email protected]>: >> On 19 September 2016 at 12:20, Stephen Warren <[email protected]> wrote: >>> From: Stephen Warren <[email protected]> >>> >>> cmd_cpp_cfg generates a dependency output, but because it's invoked using >>> if_changed rather than if_changed_dep, that dependency file is ignored. >>> This results in Kbuild not knowing about which files u-boot.cfg depends >>> on, so it may not be rebuilt when required. >>> >>> A practical result of this is that u-boot.cfg may continue to reference >>> CONFIG_ options that no longer exist in the source tree, and this can >>> cause the adhoc config options check to fail. >>> >>> This change modifies Makefile to use if_changed_dep, which in turn causes >>> all dependencies to be known to the next make invocation. >>> >>> Signed-off-by: Stephen Warren <[email protected]> > > > In the first place, why do we need u-boot.cfg? > > CONFIG options from board headers are already collected in > include/autoconf.mk. > > I think the most correct way is to rework check-config.sh > to parse include/autoconf.mk (I can do this). > > Then, remove all of the u-boot.cfg rules.
This file is intended to allow buildman to see all the CONFIG options that are set, whether in #defines or in Kconfig. Regards, Simon _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
