/* CONFIG_8723AU_AP_MODE */ (line 1763)
The inner #ifdefs are unnecessary as they depend on the
same condition as the outer #ifdef and can thus be removed.
Signed-off-by: Andreas Ruprecht andreas.rupre...@fau.de
---
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 6 --
1 file changed, 6 deletions
?
[...]
As this is the module init function, it can (and should) be declared
static.
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre/osc/osc_request.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/lustre/osc/osc_request.c
b
On 02.02.2015 15:16, Al Viro wrote:
On Mon, Feb 02, 2015 at 02:36:43PM +0100, Andreas Ruprecht wrote:
When running sparse on the osc/ subdirectory, it shows the
following warning:
andreas@workbox:~/linux-next$ make C=1 M=drivers/staging/lustre/lustre/osc/
[...]
drivers/staging/lustre/lustre
/lustre/lustre/osc/
[...]
drivers/staging/lustre/lustre/osc/osc_request.c:3335:12: warning:
symbol 'osc_init' was not declared. Should it be static?
[...]
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre/osc/osc_request.c | 2 +-
1 file changed, 1 insertion(+), 1
checkpatch complains about three places where a space is prohibited
before the braces for an #if defined() check. This patch removes
the spaces.
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre/fid/fid_internal.h | 2 +-
drivers/staging/lustre/lustre/fid
checkpatch complains about an unneeded return statement in the
function search_granted_lock(), which is removed by this patch.
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging
checkpatch finds two unnecessary line continuations in ldlm_lockd.c
which are removed by this patch.
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging
checkpatch complains about static variables being initialized to 0
which is unnecessary. This patch removes the initialization.
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c | 2 +-
drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 2
checkpatch complains about three places where braces are not
necessary. This patch removes the braces.
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre/ldlm/ldlm_pool.c| 8 +++-
drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 3 +--
2 files
A space which was forgotten in a CDEBUG output is added by this patch.
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_request.c
checkpatch complains about a number of lines which are over the
80 character line limit. This patch changes most of them to comply,
the few remaining lines with warnings are not changed for readability
reasons.
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre
. Some overlong lines are
still present for readability reasons, and a few warnings about breaks
not being useful also still remain.
Andreas Ruprecht (10):
staging: lustre: ldlm: Add missing newlines after declarations
staging: lustre: ldlm: Fix overlong lines
staging: lustre: ldlm: Fix warning
checkpatch complains about two places where a space is prohibited
before the braces for in #if defined() check. This patch removes
these warnings.
Signed-off-by: Andreas Ruprecht rup...@einserver.de
---
drivers/staging/lustre/lustre/ldlm/ldlm_pool.c | 2 +-
drivers/staging/lustre/lustre/ldlm
13 matches
Mail list logo