Messages in this thread | | | From | Gilad Ben-Yossef <> | Date | Wed, 28 Jun 2017 12:45:18 +0300 | Subject | Re: [PATCH 0/2] staging: ccree: Fix coding style and remove warnings |
| |
Thank you Karthik,
On Wed, Jun 28, 2017 at 12:37 PM, <karthik@techveda.org> wrote: > From: Karthik Tummala <karthik@techveda.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?
> > 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
| |