From mboxrd@z Thu Jan  1 00:00:00 1970
From: greg@kroah.com (Greg KH)
Date: Sat, 26 Nov 2016 21:40:41 +0100
Subject: [PATCH 0/7] security: apparmor: apparmorfs.c Checkpatch mods
In-Reply-To: <20161126180214.GA13382@localhost>
References: <1480149805-18736-1-git-send-email-waltfeasel@gmail.com>
	<20161126110550.GD8388@kroah.com> <20161126165617.GA9754@localhost>
	<20161126171922.GB8450@kroah.com>
	<20161126180214.GA13382@localhost>
Message-ID: <20161126204041.GB9467@kroah.com>
To: kernelnewbies@lists.kernelnewbies.org
List-Id: kernelnewbies.lists.kernelnewbies.org

On Sat, Nov 26, 2016 at 01:02:14PM -0500, Walt Feasel wrote:
> On Sat, Nov 26, 2016 at 06:19:22PM +0100, Greg KH wrote:
> > On Sat, Nov 26, 2016 at 11:56:17AM -0500, Walt Feasel wrote:
> > > On Sat, Nov 26, 2016 at 12:05:50PM +0100, Greg KH wrote:
> > > > On Sat, Nov 26, 2016 at 03:43:18AM -0500, Walt Feasel wrote:
> > > > > Make checkpatch modifications to include:
> > > > > 
> > > > > Multiple blank lines
> > > > > Blank line after declarations
> > > > > Space after cast
> > > > > WARN_ON vs BUG_ON
> > > > > Align parenthesis
> > > > > Comparison to NULL
> > > > > Line continuations
> > > > > 
> > > > > Walt Feasel (7):
> > > > >   security: apparmor: apparmorfs.c Multiple blank lines
> > > > >   security: apparmor: apparmorfs.c Blank line after declarations
> > > > >   security: apparmor: apparmorfs.c Space after cast
> > > > >   security: apparmor: apparmorfs.c WARN_ON vs BUG_ON
> > > > >   security: apparmor: apparmorfs.c Align parenthesis
> > > > >   security: apparmor: apparmorfs.c Comparison to NULL
> > > > >   security: apparmor: apparmorfs.c Line continuations
> > > > 
> > > > Why are you sending apparmor patches to the kernelnewbies list and not
> > > > the maintainers and subsystem list for this code instead?
> > > > 
> > > > thanks,
> > > > 
> > > > greg k-h
> > > Because as stated in my patch notes I am not sure how to
> > > fix certain cases.
> > > I have not been able to find reference material other than
> > > other patches and discussions for certain cases and have
> > > to try and reason why.
> > > You wanted me to move out of staging so I have looked to
> > > other areas but do not wish to send in ridiculous patches
> > > and have me seen as being malicious.
> > > Since this is kernelnewbies mailing list isn't that it's
> > > purpose or am I move on again?
> > > Feel free to skip my post should they trouble you.
> > 
> > Just send the patches to the proper people and list that
> > get_maintainer.pl shows you to, that's all, no need to bother
> > kernelnewbies with it.
> > 
> > good luck!
> > 
> > greg k-h
> So kernelnewbies is not to be used to learn about how to fix
> checkpatch type warning?

Maybe, but really, if you have a well-formed patch, just send it to the
correct maintainers, it's up to them to accept it or not, that's their
job :)

greg k-h