All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [RFC PATCH 0/2] RFC: Detect and reject new ad-hoc CONFIG options
@ 2016-08-24 14:44 Simon Glass
  2016-08-24 14:44 ` [U-Boot] [RFC PATCH 1/2] Kconfig: Add a whitelist of " Simon Glass
  2016-08-24 14:44 ` [U-Boot] [RFC PATCH 2/2] Makefile: Give a build error if ad-hoc CONFIG options are added Simon Glass
  0 siblings, 2 replies; 8+ messages in thread
From: Simon Glass @ 2016-08-24 14:44 UTC (permalink / raw)
  To: u-boot

Despite the availability of Kconfig, the number of ad-hoc CONFIG options in
U-Boot is still at over 8000.

In February 2015 (commit 741e58e0) there were around 8387 ad-hoc CONFIGs.
As of this patch there are about 8336, only a little fewer.

One problem is that new ones are still being added, admittedly at a slower
rate. This series adds a Makefile check to detect that and produce a build
error. This provides immediate feedback that new CONFIG options should go in
Kconfig.

NOTE: This series needs work since there are some existing CONFIG options
that are not included in the whitelist. Assuming that the approach makes
sense, I will tweak my scripts to sort this out.


Simon Glass (2):
  Kconfig: Add a whitelist of ad-hoc CONFIG options
  Makefile: Give a build error if ad-hoc CONFIG options are added

 Makefile                     |   10 +-
 scripts/check-config.sh      |   55 +
 scripts/config_whitelist.txt | 9035 ++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 9099 insertions(+), 1 deletion(-)
 create mode 100755 scripts/check-config.sh
 create mode 100644 scripts/config_whitelist.txt

-- 
2.8.0.rc3.226.g39d4020

^ permalink raw reply	[flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-09-12  4:19 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-24 14:44 [U-Boot] [RFC PATCH 0/2] RFC: Detect and reject new ad-hoc CONFIG options Simon Glass
2016-08-24 14:44 ` [U-Boot] [RFC PATCH 1/2] Kconfig: Add a whitelist of " Simon Glass
2016-08-24 16:16   ` Stephen Warren
2016-08-24 14:44 ` [U-Boot] [RFC PATCH 2/2] Makefile: Give a build error if ad-hoc CONFIG options are added Simon Glass
2016-08-24 16:24   ` Stephen Warren
2016-08-25 12:04     ` Simon Glass
2016-08-29 19:38     ` Simon Glass
2016-09-12  4:19     ` Simon Glass

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.