Re: [PATCH] checkpatch: stop checking for CONFIG_EXPERIMENTAL

2013-03-18 Thread Paul Bolle
On Sat, 2013-03-16 at 15:38 -0700, Kees Cook wrote: > Hrm, I actually think it would be better to change this to ERROR > instead of WARN. I figured the main reason to add this warning was to prevent people to add dependencies on EXPERIMENTAL while you were cleaning the tree, which could mean you

Re: [PATCH] checkpatch: stop checking for CONFIG_EXPERIMENTAL

2013-03-18 Thread Paul Bolle
On Sat, 2013-03-16 at 15:38 -0700, Kees Cook wrote: Hrm, I actually think it would be better to change this to ERROR instead of WARN. I figured the main reason to add this warning was to prevent people to add dependencies on EXPERIMENTAL while you were cleaning the tree, which could mean you

Re: [PATCH] checkpatch: stop checking for CONFIG_EXPERIMENTAL

2013-03-16 Thread Kees Cook
On Sat, Mar 16, 2013 at 2:30 PM, Paul Bolle wrote: > CONFIG_EXPERIMENTAL has been removed from the tree, in commit > 3d374d09f16f64ab4d71704cbe621514d36cd0b1 ("final removal of > CONFIG_EXPERIMENTAL"). There's no need to test for it in checkpatch > anymore. If it ever pops up again it can be

Re: [PATCH] checkpatch: stop checking for CONFIG_EXPERIMENTAL

2013-03-16 Thread Greg Kroah-Hartman
On Sat, Mar 16, 2013 at 10:30:35PM +0100, Paul Bolle wrote: > 1) A lot of defconfigs still have CONFIG_EXPERIMENTAL in them. How > should that be cleaned up? Just ignore it, when ever those defconfigs ever get used, or updated, it will fall out automatically. thanks, greg k-h -- To unsubscribe

[PATCH] checkpatch: stop checking for CONFIG_EXPERIMENTAL

2013-03-16 Thread Paul Bolle
CONFIG_EXPERIMENTAL has been removed from the tree, in commit 3d374d09f16f64ab4d71704cbe621514d36cd0b1 ("final removal of CONFIG_EXPERIMENTAL"). There's no need to test for it in checkpatch anymore. If it ever pops up again it can be caught when someone feels like cleaning up invalid Kconfig

[PATCH] checkpatch: stop checking for CONFIG_EXPERIMENTAL

2013-03-16 Thread Paul Bolle
CONFIG_EXPERIMENTAL has been removed from the tree, in commit 3d374d09f16f64ab4d71704cbe621514d36cd0b1 (final removal of CONFIG_EXPERIMENTAL). There's no need to test for it in checkpatch anymore. If it ever pops up again it can be caught when someone feels like cleaning up invalid Kconfig symbols

Re: [PATCH] checkpatch: stop checking for CONFIG_EXPERIMENTAL

2013-03-16 Thread Greg Kroah-Hartman
On Sat, Mar 16, 2013 at 10:30:35PM +0100, Paul Bolle wrote: 1) A lot of defconfigs still have CONFIG_EXPERIMENTAL in them. How should that be cleaned up? Just ignore it, when ever those defconfigs ever get used, or updated, it will fall out automatically. thanks, greg k-h -- To unsubscribe

Re: [PATCH] checkpatch: stop checking for CONFIG_EXPERIMENTAL

2013-03-16 Thread Kees Cook
On Sat, Mar 16, 2013 at 2:30 PM, Paul Bolle pebo...@tiscali.nl wrote: CONFIG_EXPERIMENTAL has been removed from the tree, in commit 3d374d09f16f64ab4d71704cbe621514d36cd0b1 (final removal of CONFIG_EXPERIMENTAL). There's no need to test for it in checkpatch anymore. If it ever pops up again it