From: Felipe Balbi <balbi@ti.com>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>,
linux-kernel <linux-kernel@vger.kernel.org>,
<linux-kbuild@vger.kernel.org>, "Balbi, Felipe" <balbi@ti.com>
Subject: Re: Kconfig "softdepends" idea
Date: Tue, 30 Apr 2013 13:33:18 +0300 [thread overview]
Message-ID: <20130430103318.GB12161@arwen.pp.htv.fi> (raw)
In-Reply-To: <20130430102440.GA15522@merkur.ravnborg.org>
[-- Attachment #1: Type: text/plain, Size: 1753 bytes --]
Hi,
On Tue, Apr 30, 2013 at 12:24:40PM +0200, Sam Ravnborg wrote:
> > So, my idea is to have a new kind of Kconfig dependency. I'll call it
> > "softdepends" in lack of better name. A driver maintainer could use
> > "softdepends on ARCH_OMAP", instead of "depends on ARCH_OMAP", to say
> > that this driver does not actually build depend on ARCH_OMAP, but for
> > all normal purposes it does.
> >
> > Normally, this would result in the same behavior as the normal
> > "depends", and Linus would not get a questions whether he wants to
> > enable this OMAP specific driver or not, and the driver would not be
> > visible on the menuconfig.
> >
> > But the user could enable the driver if he explicitly so wants. Perhaps
> > a Kconfig option such as "ignore softdepends", enabling of which would
> > allow the user to enable the drivers that use softdepends. Or, maybe
> > just require the user to add the config option manually into his .config.
>
> If I understand you correct this is already possible today without
> extending the kconfig language.
>
> config OMAP_SPECIFIC_DRIVER
> depends on ARCH_OMAP || SHOW_ALL_DRIVERS
>
>
> If SHOW_ALL_DRIVERS are "y" then user can say "y" to the OMAP specific
> driver too, even without ARCH_OMAP being selected.
SHOW_ALL_DRIVERS might create another set of issues. If Linus isn't
taking patches removing 'depends on ARCH_OMAP' would he take patches
adding SHOW_ALL_DRIVERS ?
I think something like 'available on ARCH_OMAP' might be useful,
however. We could start converting depends on ARCH_* into available on
ARCH_* and when build testing we could:
$ make DEBUG_IGNORE_ARCH=y allmodconfig
$ make drivers/usb/host/
Would this be acceptable ?
--
balbi
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
next prev parent reply other threads:[~2013-04-30 10:34 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-30 9:52 Tomi Valkeinen
2013-04-30 10:24 ` Sam Ravnborg
2013-04-30 10:33 ` Felipe Balbi [this message]
2013-04-30 11:10 ` Tomi Valkeinen
2013-04-30 11:40 ` Sam Ravnborg
2013-04-30 12:01 ` Tomi Valkeinen
2013-04-30 11:36 ` Sam Ravnborg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130430103318.GB12161@arwen.pp.htv.fi \
--to=balbi@ti.com \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sam@ravnborg.org \
--cc=tomi.valkeinen@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®