[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAB3L5oz_yXuwddZ3O2yaZ3xkJkbntN6+U1bt=Vj3hO49cvy6ig@mail.gmail.com>
Date: Sat, 3 Jun 2017 23:28:31 +0530
From: srishti sharma <srishtishar@...il.com>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: Gilad Ben-Yossef <gilad@...yossef.com>, devel@...verdev.osuosl.org,
driverdev-devel@...uxdriverproject.org,
linux-kernel@...r.kernel.org, linux-crypto@...r.kernel.org
Subject: Re: [PATCH] Staging: ccree: ssi_aead.h: Fixed a pointer declaration error.
Hey,
checkpatch.pl generated two errors , because the dereferencing
operator was placed next to the structure name instead of being placed
with the pointer .
for eg:
struct scatterlist* srcSgl; (this was giving an error)
whereas
struct scatterlist *srcSgl; (this did not give an error)
Both of them will compile , but the second one is a better
representation of it and does not produce an error on running
checkpatch.pl .
Regards,
Srishti
On Sat, Jun 3, 2017 at 2:07 PM, Greg KH <gregkh@...uxfoundation.org> wrote:
> On Sat, Jun 03, 2017 at 04:15:17AM +0530, srishti sharma wrote:
>> Fixed a pointer declaration error , the dereferencing operator was misplaced.
>
> Odd use of a ,
>
> Also, I don't understand what was "misplaced" here? There does not seem
> to be a "error" fixed here...
>
Powered by blists - more mailing lists