linux-kernel - Re: [RESEND PATCH 00/11] staging:speakup: Multiple Checkpatch issues.

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 for Android: free password hash cracker in your pocket

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

Date:   Wed, 22 Mar 2017 21:42:49 +0100
From:   Greg KH <gregkh@...uxfoundation.org>
To:     Arushi Singhal <arushisinghal19971997@...il.com>
Cc:     outreachy-kernel <outreachy-kernel@...glegroups.com>,
        w.d.hubbs@...il.com, chris@...-brannons.com, kirk@...sers.ca,
        samuel.thibault@...-lyon.org, speakup@...ux-speakup.org,
        devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [RESEND PATCH 00/11] staging:speakup: Multiple Checkpatch issues.

On Wed, Mar 22, 2017 at 12:20:23PM -0700, Arushi Singhal wrote:
> On Tuesday, March 21, 2017 at 5:12:41 PM UTC+5:30, Arushi Singhal wrote:
> 
>     Improve readability by fixing multiple checkpatch.pl
>     issues in speakup driver.
> 
>     Arushi Singhal (11):
>       staging: speakup: Moved logical to previous line.
>       staging: speakup: Remove multiple assignments
>       staging: speakup: Simplify "NULL" comparisons
>       staging: speakup: fixes braces {} should be used on all arms of this
>         statement
>       staging: speakup: Remove multiple assignments
>       staging: speakup: Moved OR operator to previous line.
>       staging: speakup: spaces preferred around operator
>       staging: speakup: Removed Unnecessary parentheses.
>       staging: speakup: Simplify the NULL comparisons
>       staging: speakup: Match alignment with open parenthesis.
>       staging: speakup: Fix alignment with parenthesis.
> 
>      drivers/staging/speakup/fakekey.c        |   2 +-
>      drivers/staging/speakup/kobjects.c       |   2 +-
>      drivers/staging/speakup/main.c           | 110
>     +++++++++++++++++--------------
>      drivers/staging/speakup/selection.c      |   4 +-
>      drivers/staging/speakup/serialio.c       |   2 +-
>      drivers/staging/speakup/speakup.h        |  12 ++--
>      drivers/staging/speakup/speakup_acntpc.c |   8 +--
>      drivers/staging/speakup/speakup_apollo.c |   2 +-
>      drivers/staging/speakup/speakup_decext.c |  10 +--
>      drivers/staging/speakup/speakup_decpc.c  |  26 ++++----
>      drivers/staging/speakup/speakup_dectlk.c |   6 +-
>      drivers/staging/speakup/speakup_dtlk.c   |   2 +-
>      drivers/staging/speakup/speakup_keypc.c  |   6 +-
>      drivers/staging/speakup/speakup_ltlk.c   |   2 +-
>      drivers/staging/speakup/varhandlers.c    |  18 ++---
>      15 files changed, 114 insertions(+), 98 deletions(-)
> 
> 
> Hi
> Any update on these patches?

One day?  Please give us a chance, realize that it's normally a 2 week
timeframe to get a patch reviewed...

thanks,

greg k-h