linux-kernel - Re: [PATCH 0/2] staging: ccree: Fix coding style and remove 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] [thread-next>] [day] [month] [year] [list]

Message-ID: <1f681a83-4413-8c9d-ab75-c223b6c8b55b@techveda.org>
Date:   Thu, 29 Jun 2017 16:13:38 +0530
From:   karthik <karthik@...hveda.org>
To:     Gilad Ben-Yossef <gilad@...yossef.com>
Cc:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-crypto@...r.kernel.org,
        driverdev-devel@...uxdriverproject.org, devel@...verdev.osuosl.org,
        Linux kernel mailing list <linux-kernel@...r.kernel.org>,
        sunil.m@...hveda.org
Subject: Re: [PATCH 0/2] staging: ccree: Fix coding style and remove warnings

On Wednesday 28 June 2017 03:15 PM, Gilad Ben-Yossef wrote:
> Thank you Karthik,
> 
> On Wed, Jun 28, 2017 at 12:37 PM,  <karthik@...hveda.org> wrote:
>> From: Karthik Tummala <karthik@...hveda.org>
>>
>> This patch series fixes coding style and removes the following
>> checkpatch.pl warnings:
>>
>> Prefer using '"%s...", __func__' to using the function's name,
>> in a string.
> 
> This one is good.
>>
>> Braces {} are not necessary for single statement blocks.
> 
> I have already sent a patch addressing this and some other things
> yesterday/
> 
>>
>> Rebased on top of next-20170627.
>> Patches were tested and built on next-20170627
> 
> Can you please check patch 1 still applies after yesterdays patch series?
Hi, I have applied your patch series and then tried to apply patch 1 and 
it failed. Should I wait until the patch series gets accepted and then 
continue my work on the new linux-next release ? or is there any other 
option ?
> 
>>
>> Karthik Tummala (2):
>>    staging: ccree: Use __func__ instead of function name
>>    staging: ccree: Remove braces {} for single statement blocks
>>
>>   drivers/staging/ccree/ssi_aead.c | 48 ++++++++++++++++------------------------
>>   1 file changed, 19 insertions(+), 29 deletions(-)
>>
>> --
>> 1.9.1
>>
> 
> Thanks,
> Gilad
> 
Thanks
karthik