2014-06-19 11:50:58

by Anil Belur

Subject: [PATCH 1/2] staging: lustre: lustre: fld: lproc_fld.c fixed warning

From: Anil Belur <[email protected]>

fixed warning for line over 80 characters by moving the struct init
onto a diff line.

Signed-off-by: Anil Belur <[email protected]>
---
drivers/staging/lustre/lustre/fld/lproc_fld.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/lustre/lustre/fld/lproc_fld.c b/drivers/staging/lustre/lustre/fld/lproc_fld.c
index b6fff2e..01380b6 100644
--- a/drivers/staging/lustre/lustre/fld/lproc_fld.c
+++ b/drivers/staging/lustre/lustre/fld/lproc_fld.c
@@ -91,10 +91,11 @@ static ssize_t
fld_proc_hash_seq_write(struct file *file, const char *buffer,
size_t count, loff_t *off)
{
- struct lu_client_fld *fld = ((struct seq_file *)file->private_data)->private;
+ struct lu_client_fld *fld;
struct lu_fld_hash *hash = NULL;
int i;

+ fld = ((struct seq_file *)file->private_data)->private;
LASSERT(fld != NULL);

for (i = 0; fld_hash[i].fh_name != NULL; i++) {
--
1.9.0

2014-06-19 11:51:06

by Anil Belur

Subject: [PATCH 2/2] staging: lustre: lustre: fld: lproc_fld.c fixed warning

From: Anil Belur <[email protected]>

fixed warning "WARNING: EXPORT_SYMBOL(foo); should immediately follow its
function/variable"

Signed-off-by: Anil Belur <[email protected]>
---
drivers/staging/lustre/lustre/fld/fld_request.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/staging/lustre/lustre/fld/fld_request.c b/drivers/staging/lustre/lustre/fld/fld_request.c
index 1f8abba..211430d 100644
--- a/drivers/staging/lustre/lustre/fld/fld_request.c
+++ b/drivers/staging/lustre/lustre/fld/fld_request.c
@@ -324,7 +324,6 @@ void fld_client_proc_fini(struct lu_client_fld *fld)
return;
}
#endif
-
EXPORT_SYMBOL(fld_client_proc_fini);

static inline int hash_is_sane(int hash)
--
1.9.0