Re: [PATCH] staging: lustre: ptlrpc: add missing include directive

On Fri, 14 Aug 2015 18:50:24 -0700 Greg KH wrote: > On Fri, Aug 14, 2015 at 12:57:06PM +0300, Ioan-Adrian Ratiu wrote: > > Without including ptlrpc_internal.h, GCC gives prototype warnings > > "pack_generic.c:642:5: warning: no previous prototype for ..." > > It

[PATCH] staging: lustre: ptlrpc: add missing include directive

Without including ptlrpc_internal.h, GCC gives prototype warnings "pack_generic.c:642:5: warning: no previous prototype for ..." and sparse also complains "pack_generic.c:642:5: warning: symbol 'lustre_unpack_req_ptlrpc_body' was not declared. ..." Signed-off-by: I

[PATCH v2] staging: rtl8192e: rtllib: fix macro style issue

Remove macro and use explicit case statements. Code is a little longer but clearer. Checkpatch.pl does not complain anymore. Signed-off-by: Ioan-Adrian Ratiu --- drivers/staging/rtl8192e/rtllib_rx.c | 80 +++- 1 file changed, 52 insertions(+), 28 deletions

[PATCH] staging: rtl8192e: rtllib: fix macro style issue

Enclose defined macro in paranthesis to avoid checkpatch complaint "ERROR: Macros with complex values should be enclosed in parentheses" Signed-off-by: Ioan-Adrian Ratiu --- drivers/staging/rtl8192e/rtllib_rx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drive

[PATCH v2] staging: lirc: sasem: fix whitespace style issue

From: Adi Ratiu checkpatch.pl gives an error on line 188 because it uses more than 8 spaces indentation. This patch converts the 8 spaces to a tab. Signed-off-by: Adi Ratiu --- drivers/staging/media/lirc/lirc_sasem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/s