Re: [PATCH] cfg80211 debugfs: Cleanup some checkpatch issues

2017-01-28 Thread Dmitriy Pichugin
On Fri, Jan 27, 2017 at 11:48:35AM -0800, Joe Perches wrote: > On Fri, 2017-01-27 at 22:26 +0300, Pichugin Dmitry wrote: > > This fixes the checkpatch.pl warnings: > > * Macros should not use a trailing semicolon. > > * Spaces required around that '='. > > * Symbolic permissions 'S_IRUGO' are not p

Re: [PATCH] cfg80211 debugfs: Cleanup some checkpatch issues

2017-01-27 Thread Joe Perches
On Fri, 2017-01-27 at 22:00 +0100, Johannes Berg wrote: > On Fri, 2017-01-27 at 22:26 +0300, Pichugin Dmitry wrote: > > This fixes the checkpatch.pl warnings: > > * Macros should not use a trailing semicolon. > > * Spaces required around that '='. > > * Symbolic permissions 'S_IRUGO' are not prefer

Re: [PATCH] cfg80211 debugfs: Cleanup some checkpatch issues

2017-01-27 Thread Johannes Berg
On Fri, 2017-01-27 at 22:26 +0300, Pichugin Dmitry wrote: > This fixes the checkpatch.pl warnings: > * Macros should not use a trailing semicolon. > * Spaces required around that '='. > * Symbolic permissions 'S_IRUGO' are not preferred. > * Macro argument reuse 'buflen' - possible side-effects I

Re: [PATCH] cfg80211 debugfs: Cleanup some checkpatch issues

2017-01-27 Thread Joe Perches
On Fri, 2017-01-27 at 22:26 +0300, Pichugin Dmitry wrote: > This fixes the checkpatch.pl warnings: > * Macros should not use a trailing semicolon. > * Spaces required around that '='. > * Symbolic permissions 'S_IRUGO' are not preferred. OK > * Macro argument reuse 'buflen' - possible side-effect

[PATCH] cfg80211 debugfs: Cleanup some checkpatch issues

2017-01-27 Thread Pichugin Dmitry
This fixes the checkpatch.pl warnings: * Macros should not use a trailing semicolon. * Spaces required around that '='. * Symbolic permissions 'S_IRUGO' are not preferred. * Macro argument reuse 'buflen' - possible side-effects Signed-off-by: Pichugin Dmitry --- net/wireless/debugfs.c | 15 +