Hi, Chetan,
On Fri, Feb 17, 2017 at 02:51:11PM +0900, chetan wrote:
Is this fix really necessary?
This way of fixing is seen across multiple places in the kernel, so it
seems standard way of handling these kind of errors from checkpatch.
That's fair. I suppose the correct way to go about it w
Hi Matthew,
> Is this fix really necessary?
This way of fixing is seen across multiple places in the kernel, so it
seems standard way of handling these kind of errors from checkpatch.
> On that note though, if it is going to be added I think it would make
> sense to get the indentation correct wi
Hi Chetan,
On Fri, Feb 17, 2017 at 12:08:02PM +0900, Chetan Sethi wrote:
+do {if (KS_WLAN_DEBUG > (n)) \
+ printk(KERN_NOTICE "%s: "fmt, __FUNCTION__, ## args); \
+ } while (0)
Is this fix really necessary? I know checkpatch is complaining, b
This is 02nd patch to file ks_wlan.h file fixing error of enclosing
complex macro value in parentheses and do-while statement has been used
to prevent further warnings from checkpatch.pl
Signed-off-by: Chetan Sethi
---
drivers/staging/ks7010/ks_wlan.h | 5 +++--
1 file changed, 3 insertions(+),
