* [PATCH] staging: lustre: lustre: libcfs: Removed a space
@ 2015-09-20 17:29 Anjali Menon
2015-09-21 1:48 ` Greg KH
0 siblings, 1 reply; 2+ messages in thread
From: Anjali Menon @ 2015-09-20 17:29 UTC (permalink / raw)
To: gregkh
Cc: oleg.drokin, andreas.dilger, HPDD-discuss, devel, linux-kernel,
Anjali Menon
Removed space between function name and open paranthesis
detected by checkpatch.pl
112: WARNING: space prohibited between function name and open
parenthesis '('
152: WARNING: space prohibited between function name and open
parenthesis '('
164: WARNING: space prohibited between function name and open
parenthesis '('
165: WARNING: space prohibited between function name and open
parenthesis '('
345: WARNING: space prohibited between function name and open
parenthesis '('
447: WARNING: space prohibited between function name and open
parenthesis '('
Signed-off-by: Anjali Menon <cse.anjalimenon@gmail.com>
---
drivers/staging/lustre/lustre/libcfs/module.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/lustre/lustre/libcfs/module.c b/drivers/staging/lustre/lustre/libcfs/module.c
index 7d0427b..4216592 100644
--- a/drivers/staging/lustre/lustre/libcfs/module.c
+++ b/drivers/staging/lustre/lustre/libcfs/module.c
@@ -109,7 +109,7 @@ enum {
PSDEV_LNET_FAIL_VAL, /* userdata for fail loc */
};
-static void kportal_memhog_free (struct libcfs_device_userstate *ldu)
+static void kportal_memhog_free(struct libcfs_device_userstate *ldu)
{
struct page **level0p = &ldu->ldu_memhog_root_page;
struct page **level1p;
@@ -149,7 +149,7 @@ static void kportal_memhog_free (struct libcfs_device_userstate *ldu)
*level0p = NULL;
}
- LASSERT (ldu->ldu_memhog_pages == 0);
+ LASSERT(ldu->ldu_memhog_pages == 0);
}
static int kportal_memhog_alloc(struct libcfs_device_userstate *ldu, int npages,
@@ -161,8 +161,8 @@ static int kportal_memhog_alloc(struct libcfs_device_userstate *ldu, int npages,
int count1;
int count2;
- LASSERT (ldu->ldu_memhog_pages == 0);
- LASSERT (ldu->ldu_memhog_root_page == NULL);
+ LASSERT(ldu->ldu_memhog_pages == 0);
+ LASSERT(ldu->ldu_memhog_root_page == NULL);
if (npages < 0)
return -EINVAL;
@@ -342,7 +342,7 @@ static int libcfs_ioctl_int(struct cfs_psdev_file *pfile, unsigned long cmd,
if (err != -EINVAL) {
if (err == 0)
err = libcfs_ioctl_popdata(arg,
- data, sizeof (*data));
+ data, sizeof(*data));
break;
}
}
@@ -444,7 +444,7 @@ static int init_libcfs_module(void)
goto cleanup_crypto;
}
- CDEBUG (D_OTHER, "portals setup OK\n");
+ CDEBUG(D_OTHER, "portals setup OK\n");
return 0;
cleanup_crypto:
cfs_crypto_unregister();
--
1.9.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] staging: lustre: lustre: libcfs: Removed a space
2015-09-20 17:29 [PATCH] staging: lustre: lustre: libcfs: Removed a space Anjali Menon
@ 2015-09-21 1:48 ` Greg KH
0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2015-09-21 1:48 UTC (permalink / raw)
To: Anjali Menon
Cc: oleg.drokin, andreas.dilger, HPDD-discuss, devel, linux-kernel
On Sun, Sep 20, 2015 at 10:59:16PM +0530, Anjali Menon wrote:
> Removed space between function name and open paranthesis
> detected by checkpatch.pl
>
> 112: WARNING: space prohibited between function name and open
> parenthesis '('
Please don't wrap error/warning messages like these.
And this patch doesn't apply at all to my tree, what are you making it
against?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-09-21 4:38 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-20 17:29 [PATCH] staging: lustre: lustre: libcfs: Removed a space Anjali Menon
2015-09-21 1:48 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®