Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S938994AbcKWP4Z (ORCPT <rfc822;w@1wt.eu>);
        Wed, 23 Nov 2016 10:56:25 -0500
Received: from mail.linuxfoundation.org ([140.211.169.12]:39526 "EHLO
        mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S934495AbcKWP4X (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Wed, 23 Nov 2016 10:56:23 -0500
Date: Wed, 23 Nov 2016 16:56:32 +0100
From: Greg KH <gregkh@linuxfoundation.org>
To: Fleetwood Farm <waltfeasel@gmail.com>
Cc: Samuel Thibault <samuel.thibault@ens-lyon.org>,
        devel@driverdev.osuosl.org, kirk@reisers.ca,
        linux-kernel@vger.kernel.org, w.d.hubbs@gmail.com,
        chris@the-brannons.com, speakup@linux-speakup.org
Subject: Re: [PATCH v2 0/6] staging: speakup: speakup_soft.c checkpatch
 modifications
Message-ID: <20161123155632.GC8719@kroah.com>
References: <1479741565-19419-1-git-send-email-waltfeasel@gmail.com>
 <20161123090905.GA31482@kroah.com>
 <CACfmeC=-NKgX4QKF0Btj6OTvOxSjZgcoQ4BrNLq5NEpPSZ+1Ww@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <CACfmeC=-NKgX4QKF0Btj6OTvOxSjZgcoQ4BrNLq5NEpPSZ+1Ww@mail.gmail.com>
User-Agent: Mutt/1.7.1 (2016-10-04)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 1712
Lines: 45

On Wed, Nov 23, 2016 at 05:52:25AM -0500, Fleetwood Farm wrote:
> On Nov 23, 2016 4:08 AM, "Greg KH" <gregkh@linuxfoundation.org> wrote:
> >
> > On Mon, Nov 21, 2016 at 10:19:19AM -0500, Walt Feasel wrote:
> > > Make Linux kernel style modifications for speak_soft.c to include:
> > >
> > > Space around operator
> > > Logical continuation
> > > Multiple blank lines
> > > Blank line after {
> > > comment modifications
> > > Align parenthesis
> >
> > Ok, now that you have a bunch of cleanup patches accepted, why not work
> > on some "real" code changes?? There's no need to fix all of the coding
> > style issues in drivers/staging/ at once, leave some for other people to
> > learn on :)
> >
> > thanks,
> >
> > greg k-h
> 
> Well, I have around 40 more patches already prepared waiting for my limit to
> clear (which just did) and about 12 more files left to checkpatch in speakup.
> I am currently waiting on the newbies administer to clear my sent emails asking
> advice on if some of my tougher challenges (octals and CamelCase) are done
> correctly.

"newbies administrator"?

> I am also currently waiting on the u-boot mailing list administer to do the
> same on some of my patches.
> I also am doing some online 'c' courses to increase (get) my skills set.
> What type of "real" code changes did you have in mind with my limited skill
> set?

I'd strongly suggest working on some userspace programs to get a
less-limited skill set :)  Then find some area of the kernel you are
interested in, and help out there.  Speakup really can use the help, and
the community of users of their devices is very good, so any real work
you can do there would always be greatly appreciated.

best of luck,

greg k-h