On Fri, Aug 17, 2007 at 08:59:29AM -0700, Nelson, Shannon wrote:
> >From: Adrian Bunk [mailto:[email protected]]
> >Sent: Thursday, August 16, 2007 5:48 PM
> >To: Nelson, Shannon
> >Cc: [email protected]; [email protected];
> >Williams, Dan J; [email protected]
> >Subject: Re: [-mm PATCH] DMA engine kconfig improvements (rev2)
> >
> >On Thu, Aug 16, 2007 at 05:10:25PM -0700, Shannon Nelson wrote:
> >> From: Adrian Bunk <[email protected]>
> >>
> >> This patch contains the following changes to the DMA engine menus:
> >>...
> >> - make it clear in the INTEL_IOATDMA help text that this
> >driver is for
> >> rare hardware the user most likely doesn't has
> >>...
> >> +config INTEL_IOATDMA
> >> + tristate "Intel I/OAT DMA support"
> >> + depends on PCI && X86
> >> + select DMA_ENGINE
> >> + select DCA
> >> + help
> >> + Enable support for the Intel(R) I/OAT DMA engine present
> >> + in recent chipsets.
> >>...
> >
> >It seems the string "Xeon" somehow got lost, or is it now also
> >available with other hardware?
> >
> >"in recent Intel Xeon chipsets"?
> >
> >cu
> >Adrian
>
> Good catch - however, it is probably best not to limit these things.
Since the effects of CONFIG_NET_DMA are considered big enough to warrant
all these #ifdef's in the net code it should be made clear for the user
that this is only for hardware he most likely doesn't has.
And it's not about any kind of kconfig dependency, it's simply about the
help text.
> sln
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
