Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error
- From: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
- Date: Mon, 5 Feb 2007 08:32:50 -0800 (PST)
On Mon, 5 Feb 2007, David Woodhouse wrote:
Secondly, please don't _ever_ use 'select'.
No, David.
I don't know why you keep repeating this mantra, when it's WRONG.
Using "select" is a lot more sane and intelligent than assuming that users
know what dependencies they want.
The Kconfig files should ask about *end-user* visible features. They
should say "do you want to support X".
If "X" then needs Y, Z and something else to actually compile, then that
Kconfig file should DAMN WELL use "select". Stop claiming anything else!
The user shouldn't know that they should say that they need some library Y
in order to even see the question for "X". It's not a sane thing to ask
them to know and care about. They care about the devices or capabilities
they want to support, not about the fact that a USB storage device needs
the SCSI core layer, for example.
So stop saying "don't use select".
Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
- Follow-Ups:
- Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error
- From: David Woodhouse
- Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error
- From: Russell King
- Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error
- References:
- Super Kernel Sunday!
- From: Linus Torvalds
- [patch] MTD: fix DOC2000/2001/2001PLUS build error
- From: Ingo Molnar
- Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error
- From: David Woodhouse
- Super Kernel Sunday!
- Prev by Date: Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error
- Next by Date: Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error
- Previous by thread: Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error
- Next by thread: Re: [patch] MTD: fix DOC2000/2001/2001PLUS build error
- Index(es):
Relevant Pages
|