mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Randy Dunlap <randy.dunlap@oracle.com>
To: "Robert P. J. Day" <rpjday@mindspring.com>
Cc: Linux kernel mailing list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] kbuild : Restructure Device Drivers menu for entry selectability
Date: Thu, 7 Dec 2006 12:49:17 -0800	[thread overview]
Message-ID: <20061207124917.7fa95b95.randy.dunlap@oracle.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0612070354200.14891@localhost.localdomain>

On Thu, 7 Dec 2006 04:01:47 -0500 (EST) Robert P. J. Day wrote:

> 
>   Rewrite a number of the Kconfig files under the drivers/ directory
> so that those driver submenus can be selected or de-selected
> directly from the Drivers menu using the kbuild "menuconfig" feature
> without having to enter the submenu itself.
> 
> Signed-off-by: Robert P. J. Day <rpjday@mindspring.com>
> 
> ---
> 
>   NOTE:  Only those drivers submenus for which the Kconfig change was
> *immediately* obvious are included here.  Some of the other Kconfig
> files have "issues" with dependencies so that they'll need some extra
> cleanup before they can be tweaked like this as well.  So this first
> pass of a patch is just the beginning.

InfiniBand is presented in the wrong location in xconfig
with this patch.  After changing this to be
"menuconfig INFINIBAND"
it works as expected and looks good.

Acked-by: Randy Dunlap <randy.dunlap@oracle.com>

> diff --git a/drivers/infiniband/Kconfig b/drivers/infiniband/Kconfig
> index 9edface..25454e8 100644
> --- a/drivers/infiniband/Kconfig
> +++ b/drivers/infiniband/Kconfig
> @@ -1,13 +1,13 @@
> -menu "InfiniBand support"
> -
>  config INFINIBAND
> -	depends on PCI || BROKEN
>  	tristate "InfiniBand support"
> +	depends on PCI || BROKEN
>  	---help---
>  	  Core support for InfiniBand (IB).  Make sure to also select
>  	  any protocols you wish to use as well as drivers for your
>  	  InfiniBand hardware.
> 
> +if INFINIBAND
> +
>  config INFINIBAND_USER_MAD
>  	tristate "InfiniBand userspace MAD support"
>  	depends on INFINIBAND
> @@ -45,4 +45,4 @@ source "drivers/infiniband/ulp/srp/Kconf
> 
>  source "drivers/infiniband/ulp/iser/Kconfig"
> 
> -endmenu
> +endif

---
~Randy

      reply	other threads:[~2006-12-07 20:49 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-07  9:01 Robert P. J. Day
2006-12-07 20:49 ` Randy Dunlap [this message]

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=20061207124917.7fa95b95.randy.dunlap@oracle.com \
    --to=randy.dunlap@oracle.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rpjday@mindspring.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®