From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Glass <sjg@chromium.org>
Date: Sun, 11 Sep 2016 22:19:55 -0600
Subject: [U-Boot] [RFC PATCH 2/2] Makefile: Give a build error if ad-hoc
 CONFIG options are added
In-Reply-To: <169169d2-61fc-cec2-7d02-87e7e5c6c99d@wwwdotorg.org>
References: <1472049854-4891-1-git-send-email-sjg@chromium.org>
 <1472049854-4891-3-git-send-email-sjg@chromium.org>
 <169169d2-61fc-cec2-7d02-87e7e5c6c99d@wwwdotorg.org>
Message-ID: <CAPnjgZ3HLZ1mWWcLEf-BTvujX7kmuCQRVjT3q-mvpL-xqgsiiA@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Stephen,

On 24 August 2016 at 10:24, Stephen Warren <swarren@wwwdotorg.org> wrote:
> On 08/24/2016 08:44 AM, Simon Glass wrote:
>>
>> New CONFIG options should be added via Kconfig. To help prevent new ad-hoc
>> CONFIGs from being added, give a build error when these are detected.
>
>
> Acked-by: Stephen Warren <swarren@nvidia.com>
> Tested-by: Stephen Warren <swarren@nvidia.com>
>
> I worried about the increased build time due to find/grep on all Kconfig
> files, but at least with a hot disk cache it seems to make no noticeable
> difference:-)

Yes it was pretty slow at first, but I fiddled with it to sort that.

The current approach is based on files that are already generated by
the build. There is a very small overhead but it is small enough that
I cannot reliably measure it.

[...]

Regards,
Simon