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. -- Best Regards Masahiro Yamada _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
