All of lore.kernel.org
 help / color / mirror / Atom feed
From: Brice Goglin <brice@myri.com>
To: "Brandeburg, Jesse" <jesse.brandeburg@intel.com>
Cc: Adrian Bunk <bunk@kernel.org>,
	e1000-devel@lists.sourceforge.net, netdev@vger.kernel.org,
	"Sosnowski, Maciej" <maciej.sosnowski@intel.com>,
	linux-kernel@vger.kernel.org,
	Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>,
	linuxppc-dev@ozlabs.org, "Kirsher,
	Jeffrey T" <jeffrey.t.kirsher@intel.com>,
	sam@ravnborg.org, Jeff Garzik <jgarzik@redhat.com>
Subject: Re: [BUILD_FAILURE] 2.6.27-git2 - allyesconfig on powerpc selectsCONFIG_INTEL_IOATDMA=y
Date: Mon, 13 Oct 2008 20:21:45 +0200	[thread overview]
Message-ID: <48F391B9.1000206@myri.com> (raw)
In-Reply-To: <36D9DB17C6DE9E40B059440DB8D95F52066E2EAE@orsmsx418.amr.corp.intel.com>

Brandeburg, Jesse wrote:
> What we want, is myri10ge and ixgbe drivers that can build whether or not CONFIG_INTEL_IOATDMA is enabled.  IF CONFIG_INTEL_IOATDMA *is* enabled (which it should not be on PPC) then there are several cases we want to work:
> CONFIG_INTEL_IOATDMA=m  ---> CONFIG_IXGBE=[m|n]
> CONFIG_INTEL_IOATDMA=y  ---> CONFIG_IXGBE=[m|y|n]
> CONFIG_INTEL_IOATDMA=n  ---> CONFIG_IXGBE=[m|y|n]
> CONFIG_INTEL_IOATDMA depends on X86
>   

I am not sure I want to prevent myri10ge=y just because ioatdma=m.

I would vote for adding some Kconfig stuff to define CONFIG_MYRI10GE_DCA
as boolean set to yes if (IOATDMA=y and MYRI10GE=y/m) or (IOATDMA=m and
MYRI10GE=m). And then use #ifdef CONFIG_MYRI10GE_DCA in the driver source.

Brice

WARNING: multiple messages have this Message-ID (diff)
From: Brice Goglin <brice@myri.com>
To: "Brandeburg, Jesse" <jesse.brandeburg@intel.com>
Cc: Adrian Bunk <bunk@kernel.org>,
	Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>,
	Jeff Garzik <jgarzik@redhat.com>,
	"Kirsher, Jeffrey T" <jeffrey.t.kirsher@intel.com>,
	linux-kernel@vger.kernel.org, apw@shadowen.org,
	linuxppc-dev@ozlabs.org, sam@ravnborg.org,
	e1000-devel@lists.sourceforge.net, netdev@vger.kernel.org,
	"Sosnowski, Maciej" <maciej.sosnowski@intel.com>
Subject: Re: [BUILD_FAILURE] 2.6.27-git2 - allyesconfig on powerpc selectsCONFIG_INTEL_IOATDMA=y
Date: Mon, 13 Oct 2008 20:21:45 +0200	[thread overview]
Message-ID: <48F391B9.1000206@myri.com> (raw)
In-Reply-To: <36D9DB17C6DE9E40B059440DB8D95F52066E2EAE@orsmsx418.amr.corp.intel.com>

Brandeburg, Jesse wrote:
> What we want, is myri10ge and ixgbe drivers that can build whether or not CONFIG_INTEL_IOATDMA is enabled.  IF CONFIG_INTEL_IOATDMA *is* enabled (which it should not be on PPC) then there are several cases we want to work:
> CONFIG_INTEL_IOATDMA=m  ---> CONFIG_IXGBE=[m|n]
> CONFIG_INTEL_IOATDMA=y  ---> CONFIG_IXGBE=[m|y|n]
> CONFIG_INTEL_IOATDMA=n  ---> CONFIG_IXGBE=[m|y|n]
> CONFIG_INTEL_IOATDMA depends on X86
>   

I am not sure I want to prevent myri10ge=y just because ioatdma=m.

I would vote for adding some Kconfig stuff to define CONFIG_MYRI10GE_DCA
as boolean set to yes if (IOATDMA=y and MYRI10GE=y/m) or (IOATDMA=m and
MYRI10GE=m). And then use #ifdef CONFIG_MYRI10GE_DCA in the driver source.

Brice


WARNING: multiple messages have this Message-ID (diff)
From: Brice Goglin <brice@myri.com>
To: "Brandeburg, Jesse" <jesse.brandeburg@intel.com>
Cc: Adrian Bunk <bunk@kernel.org>,
	e1000-devel@lists.sourceforge.net, netdev@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>,
	linuxppc-dev@ozlabs.org, "Kirsher,
	Jeffrey T" <jeffrey.t.kirsher@intel.com>,
	apw@shadowen.org, sam@ravnborg.org,
	Jeff Garzik <jgarzik@redhat.com>
Subject: Re: [BUILD_FAILURE] 2.6.27-git2 - allyesconfig on powerpc selectsCONFIG_INTEL_IOATDMA=y
Date: Mon, 13 Oct 2008 20:21:45 +0200	[thread overview]
Message-ID: <48F391B9.1000206@myri.com> (raw)
In-Reply-To: <36D9DB17C6DE9E40B059440DB8D95F52066E2EAE@orsmsx418.amr.corp.intel.com>

Brandeburg, Jesse wrote:
> What we want, is myri10ge and ixgbe drivers that can build whether or not CONFIG_INTEL_IOATDMA is enabled.  IF CONFIG_INTEL_IOATDMA *is* enabled (which it should not be on PPC) then there are several cases we want to work:
> CONFIG_INTEL_IOATDMA=m  ---> CONFIG_IXGBE=[m|n]
> CONFIG_INTEL_IOATDMA=y  ---> CONFIG_IXGBE=[m|y|n]
> CONFIG_INTEL_IOATDMA=n  ---> CONFIG_IXGBE=[m|y|n]
> CONFIG_INTEL_IOATDMA depends on X86
>   

I am not sure I want to prevent myri10ge=y just because ioatdma=m.

I would vote for adding some Kconfig stuff to define CONFIG_MYRI10GE_DCA
as boolean set to yes if (IOATDMA=y and MYRI10GE=y/m) or (IOATDMA=m and
MYRI10GE=m). And then use #ifdef CONFIG_MYRI10GE_DCA in the driver source.

Brice


-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/

  reply	other threads:[~2008-10-13 18:26 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-13 10:15 [BUILD_FAILURE] 2.6.27-git2 - allyesconfig on powerpc selects CONFIG_INTEL_IOATDMA=y Kamalesh Babulal
2008-10-13 10:15 ` Kamalesh Babulal
2008-10-13 11:26 ` Adrian Bunk
2008-10-13 11:26   ` Adrian Bunk
2008-10-13 11:26   ` Adrian Bunk
2008-10-13 12:02   ` Brice Goglin
2008-10-13 12:02     ` Brice Goglin
2008-10-13 18:05   ` [BUILD_FAILURE] 2.6.27-git2 - allyesconfig on powerpc selectsCONFIG_INTEL_IOATDMA=y Brandeburg, Jesse
2008-10-13 18:05     ` Brandeburg, Jesse
2008-10-13 18:05     ` Brandeburg, Jesse
2008-10-13 18:21     ` Brice Goglin [this message]
2008-10-13 18:21       ` Brice Goglin
2008-10-13 18:21       ` Brice Goglin
2008-10-13 18:35       ` Adrian Bunk
2008-10-13 18:35         ` Adrian Bunk
2008-10-13 18:35         ` Adrian Bunk
2008-10-14  9:12         ` Brice Goglin
2008-10-14  9:12           ` Brice Goglin
2008-10-14  9:12           ` Brice Goglin
2008-10-14  9:35           ` Adrian Bunk
2008-10-14  9:35             ` Adrian Bunk
2008-10-14 16:10           ` Brandeburg, Jesse
2008-10-14 16:10             ` Brandeburg, Jesse
2008-10-14 17:58             ` Adrian Bunk
2008-10-14 17:58               ` Adrian Bunk
2008-10-15  7:53               ` Kamalesh Babulal
2008-10-15  7:53                 ` Kamalesh Babulal

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=48F391B9.1000206@myri.com \
    --to=brice@myri.com \
    --cc=bunk@kernel.org \
    --cc=e1000-devel@lists.sourceforge.net \
    --cc=jeffrey.t.kirsher@intel.com \
    --cc=jesse.brandeburg@intel.com \
    --cc=jgarzik@redhat.com \
    --cc=kamalesh@linux.vnet.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=maciej.sosnowski@intel.com \
    --cc=netdev@vger.kernel.org \
    --cc=sam@ravnborg.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.