* [PATCH] staging: lustre: lustre: libcfs: Added a blank line
@ 2015-09-20 17:05 Anjali Menon
2015-09-21 1:48 ` Greg KH
0 siblings, 1 reply; 2+ messages in thread
From: Anjali Menon @ 2015-09-20 17:05 UTC (permalink / raw)
To: gregkh
Cc: andreas.dilger, oleg.drokin, linux-kernel, devel, HPDD-discuss,
Anjali Menon
Added a blank line after declaration. Coding style
warning detected by checkpatch.pl
WARNING: Missing a blank line after declarations
Signed-off-by: Anjali Menon <cse.anjalimenon@gmail.com>
---
drivers/staging/lustre/lustre/libcfs/module.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/lustre/lustre/libcfs/module.c b/drivers/staging/lustre/lustre/libcfs/module.c
index f0ee76a..7d0427b 100644
--- a/drivers/staging/lustre/lustre/libcfs/module.c
+++ b/drivers/staging/lustre/lustre/libcfs/module.c
@@ -334,6 +334,7 @@ static int libcfs_ioctl_int(struct cfs_psdev_file *pfile, unsigned long cmd,
default: {
struct libcfs_ioctl_handler *hand;
+
err = -EINVAL;
down_read(&ioctl_list_sem);
list_for_each_entry(hand, &ioctl_list, item) {
--
1.9.1
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH] staging: lustre: lustre: libcfs: Added a blank line
2015-09-20 17:05 [PATCH] staging: lustre: lustre: libcfs: Added a blank line 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: andreas.dilger, oleg.drokin, linux-kernel, devel, HPDD-discuss
On Sun, Sep 20, 2015 at 10:35:01PM +0530, Anjali Menon wrote:
> Added a blank line after declaration. Coding style
> warning detected by checkpatch.pl
>
> WARNING: Missing a blank line after declarations
>
> Signed-off-by: Anjali Menon <cse.anjalimenon@gmail.com>
> ---
> drivers/staging/lustre/lustre/libcfs/module.c | 1 +
> 1 file changed, 1 insertion(+)
What tree are you making this against, it's not quite "clean" and should
be rebased.
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:05 [PATCH] staging: lustre: lustre: libcfs: Added a blank line 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®