Re: [PATCH] [TRIVIAL] Kconfig: Remove useless "default N" lines

2014-04-15 Thread Paul Bolle
Jiri, On Tue, 2014-02-11 at 13:38 +0100, Paul Bolle wrote: > A number of Kconfig entries default to (uppercase) "N". It was clearly > intended to use "default n". But since (lowercase) "n" is the default > anyway, these lines might as well be removed. > > Signed-off-by: Paul Bolle > --- >[...] >

Re: [PATCH] [TRIVIAL] Kconfig: Remove useless "default N" lines

2014-02-11 Thread Randy Dunlap
On 02/11/2014 08:26 AM, Paul Bolle wrote: > On Tue, 2014-02-11 at 08:03 -0800, Randy Dunlap wrote: >> Yes, I found it in Documentation/SubmittingPatches. >> I find that unfortunate. > > Dan Carpenter and I already suggested updates to the text about getting > permission to add a Reported-by tag in

Re: [PATCH] [TRIVIAL] Kconfig: Remove useless "default N" lines

2014-02-11 Thread Paul Bolle
On Tue, 2014-02-11 at 08:03 -0800, Randy Dunlap wrote: > Yes, I found it in Documentation/SubmittingPatches. > I find that unfortunate. Dan Carpenter and I already suggested updates to the text about getting permission to add a Reported-by tag in https://lkml.org/lkml/2014/2/11/252 and in Dan's re

Re: [PATCH] [TRIVIAL] Kconfig: Remove useless "default N" lines

2014-02-11 Thread Randy Dunlap
On 02/11/2014 08:00 AM, Randy Dunlap wrote: > On 02/11/2014 04:38 AM, Paul Bolle wrote: >> A number of Kconfig entries default to (uppercase) "N". It was clearly >> intended to use "default n". But since (lowercase) "n" is the default >> anyway, these lines might as well be removed. >> >> Signed-of

Re: [PATCH] [TRIVIAL] Kconfig: Remove useless "default N" lines

2014-02-11 Thread Randy Dunlap
On 02/11/2014 04:38 AM, Paul Bolle wrote: > A number of Kconfig entries default to (uppercase) "N". It was clearly > intended to use "default n". But since (lowercase) "n" is the default > anyway, these lines might as well be removed. > > Signed-off-by: Paul Bolle > --- > Tested on a Fedora 20-ba

[PATCH] [TRIVIAL] Kconfig: Remove useless "default N" lines

2014-02-11 Thread Paul Bolle
A number of Kconfig entries default to (uppercase) "N". It was clearly intended to use "default n". But since (lowercase) "n" is the default anyway, these lines might as well be removed. Signed-off-by: Paul Bolle --- Tested on a Fedora 20-based .config, where it was a NOP. But there's a chance th