From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx3.mail.elte.hu ([157.181.1.138]:34152 "EHLO mx3.mail.elte.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932199Ab1EMKCA (ORCPT ); Fri, 13 May 2011 06:02:00 -0400 Date: Fri, 13 May 2011 12:01:52 +0200 From: Ingo Molnar Subject: Re: [PATCH v2] kconfig: autogenerated config_is_xxx macro Message-ID: <20110513100152.GN13647@elte.hu> References: <1304658229-30820-1-git-send-email-plagnioj@jcrosoft.com> <20110507015041.GA21017@game.jcrosoft.org> <4DC7AB57.9050002@suse.cz> <20110513080909.GO18952@game.jcrosoft.org> <20110513083013.GC13647@elte.hu> <20110513083652.GP18952@game.jcrosoft.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20110513083652.GP18952@game.jcrosoft.org> Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Jean-Christophe PLAGNIOL-VILLARD Cc: Michal Marek , Arnaud Lacombe , linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org, x86@kernel.org, Ingo Molnar * Jean-Christophe PLAGNIOL-VILLARD wrote: > On 10:30 Fri 13 May , Ingo Molnar wrote: > > > > * Jean-Christophe PLAGNIOL-VILLARD wrote: > > > > > -#ifdef CONFIG_PCI_BIOS > > > - if (!rt->signature) { > > > + if (config_is_pci_bios() && !rt->signature) { > > > > Makes sense - but please name it in a more obvious way, such as: > > > > pci_bios_enabled() > the idea to generate the macro via Kconfig Okay, and there we are stuck with whatever the Kconfig name is. (we could rename that but not needed really) Why not the canonical config_pci_bios() variant? It's the shortest one to write. The '_is' looks pretty superfluous to me. Hm, i guess it could be mixed up with a function that configures the pci_bios. I guess since i don't have any better idea config_is_pci_bios() sounds like a good choice after all. Thanks, Ingo