Re: [PATCH] libata-sff: trivial corrections to Kconfig help text

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Tejun Heo wrote:
> On 05/29/2010 12:47 PM, Stefan Richter wrote:
>> By the way, is there something that could be said in a few words in the
>> help text when to enable that option and when not?
> 
> Hmmmm... maybe we can emphasize the 'if unsure, say Y' part?

I did follow that recommendation but was still left curious. :-)

>> Or could that option even be hidden and 'select'ed by bus master DMA
>> capable ATA SFF drivers?
> 
> I wanted to make the distinction clear so made the option explicit.
> Opinions on whether that is actually a good idea would differ but at
> the same time, I don't think this would cause any problem, would it?

OK, I first came across it by way of make oldconfig.  Then I read the
changelog and still wondered.  Now I had a look at the menu layout in
gconfig and it makes sense to me now.

Just from the looks of the menu, would calling the prompt

	bool "SFF controllers with bus master DMA"

be sensible?  After all, this option does not just control whether a
certain capability is built into libata, switching it on pulls up the
Kconfig prompts for a whole range of controller drivers.  (Ok, the help
text is saying exactly this, but then the prompt could do so already...)

The 'hidden variable + select' way would be more friendly to "make
oldconfig", but I see now that it would take quite a few select
statements to be added.
-- 
Stefan Richter
-=====-==-=- -=-= ===-=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux