mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
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 --]

  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®