Re: [PATCH v2 1/3] checkpatch: add Kconfig 'default n' test

2016-05-03 Thread Yingjoe Chen
On Fri, 2016-04-22 at 22:32 +0800, Yingjoe Chen wrote: > If a Kconfig config option doesn't specify 'default', the default > will be n. Adding 'default n' is unnecessary. > > Add a test to warn about this. > > Signed-off-by: Yingjoe Chen > --- > Change in v2: > -

Re: [PATCH v2 1/3] checkpatch: add Kconfig 'default n' test

2016-05-03 Thread Yingjoe Chen
On Fri, 2016-04-22 at 22:32 +0800, Yingjoe Chen wrote: > If a Kconfig config option doesn't specify 'default', the default > will be n. Adding 'default n' is unnecessary. > > Add a test to warn about this. > > Signed-off-by: Yingjoe Chen > --- > Change in v2: > - Change according to Joe

[PATCH v2 1/3] checkpatch: add Kconfig 'default n' test

2016-04-22 Thread Yingjoe Chen
If a Kconfig config option doesn't specify 'default', the default will be n. Adding 'default n' is unnecessary. Add a test to warn about this. Signed-off-by: Yingjoe Chen --- Change in v2: - Change according to Joe Perches' suggestion scripts/checkpatch.pl | 7

[PATCH v2 1/3] checkpatch: add Kconfig 'default n' test

2016-04-22 Thread Yingjoe Chen
If a Kconfig config option doesn't specify 'default', the default will be n. Adding 'default n' is unnecessary. Add a test to warn about this. Signed-off-by: Yingjoe Chen --- Change in v2: - Change according to Joe Perches' suggestion scripts/checkpatch.pl | 7 +++ 1 file changed, 7