linux-kernel - Re: [PATCH] staging: android: fix Missing a blank line after declarations warnings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20140531041948.GA10014@kroah.com>
Date:	Fri, 30 May 2014 21:19:48 -0700
From:	Greg KH <gregkh@...uxfoundation.org>
To:	Scott Weir <sjw0410@...il.com>
Cc:	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: android: fix Missing a blank line after
 declarations warnings

On Fri, May 30, 2014 at 03:56:44PM +1000, Scott Weir wrote:
> On Wed, May 28, 2014 at 02:30:27PM -0700, Greg KH wrote:
> > On Wed, May 28, 2014 at 06:52:51PM +1000, Scott Weir wrote:
> > > This patch corrects coding style issue:
> > > WARNING: Missing a black line after declarations
> > > 
> > > Signed-off-by: Scott Weir <sjw0410@...il.com>
> > > ---
> > >  drivers/staging/android/sync_debug.c | 4 ++++
> > 
> > This file isn't in my tree.  Always work against linux-next, not Linus's
> > tree for new patches.
> > 
> > thanks,
> > 
> > greg k-h
> 
> This patch was made against linux-next. Is it better to use your staging tree?

Yes, but normally it should also be fine to use linux-next.

I don't know where that sync_debug.c file is coming from, it's not in my
staging branch, some other tree must be creating it, which is
troublesome to me...

Anyway, sorry, but yes, work off of the staging-next branch of
staging.git on git.kernel.org and you should be fine.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/