[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.00.1101261158490.13163@chino.kir.corp.google.com>
Date: Wed, 26 Jan 2011 12:00:08 -0800 (PST)
From: David Rientjes <rientjes@...gle.com>
To: Russell King - ARM Linux <linux@....linux.org.uk>
cc: Andrew Morton <akpm@...ux-foundation.org>,
Ralf Baechle <ralf@...ux-mips.org>,
Kyle McMartin <kyle@...hat.com>,
James Bottomley <James.Bottomley@...senpartnership.com>,
Benjamin Herrenschmidt <benh@...nel.crashing.org>,
Thomas Gleixner <tglx@...utronix.de>,
"H. Peter Anvin" <hpa@...or.com>, Ingo Molnar <mingo@...hat.com>,
"David S. Miller" <davem@...emloft.net>,
Greg Kroah-Hartman <gregkh@...e.de>,
Randy Dunlap <randy.dunlap@...cle.com>, x86@...nel.org,
linux-arch@...r.kernel.org, netdev@...r.kernel.org,
linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [patch] kconfig: unify GENERIC_ISA_DMA and ISA_DMA_API
On Wed, 26 Jan 2011, Russell King - ARM Linux wrote:
> On Tue, Jan 25, 2011 at 05:06:28PM -0800, David Rientjes wrote:
> > CONFIG_GENERIC_ISA_DMA and CONFIG_ISA_DMA_API usually have dependencies
> > on one another depending on the architecture and generic kernel code uses
> > either to determine whether an ISA-style DMA API is configured.
>
> Wrong.
>
> GENERIC_ISA_DMA enables support for the standard ISA DMA allocator found
> in kernel/dma.c
>
> ISA_DMA_API says that a platform supports the ISA DMA interfaces.
>
> An architecture can provide the ISA DMA interfaces, but not use the
> standard ISA DMA allocator found in kernel/dma.c. Such as the one in
> arch/arm/kernel/dma.c.
>
> So on ARM, we have platforms where ISA_DMA_API=y but GENERIC_ISA_DMA=n.
>
Ok, fair enough. Thanks for pointing that out. I'll work to resolve the
existing dependency chain on x86 that cause build failures when one is
disabled and another is not and add Kconfig dependencies on drivers that
require GENERIC_ISA_DMA.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists