Re: [PATCH 0/2] staging: ccree: Fix coding style and remove warnings

From: Gilad Ben-Yossef
Date: Wed Jun 28 2017 - 05:45:32 EST


Thank you Karthik,

On Wed, Jun 28, 2017 at 12:37 PM, <karthik@xxxxxxxxxxxx> wrote:
> From: Karthik Tummala <karthik@xxxxxxxxxxxx>
>
> 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?

>
> 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

--
Gilad Ben-Yossef
Chief Coffee Drinker

"If you take a class in large-scale robotics, can you end up in a
situation where the homework eats your dog?"
-- Jean-Baptiste Queru