On Tue, Feb 24, 2015 at 09:56:39AM -0800, Greg KH wrote:
> On Tue, Feb 24, 2015 at 05:53:55PM +0000, Daniel Lockyer wrote:
> > Fixed a coding style issue.
> > 
> > Signed-off-by: Daniel Lockyer <thisisdaniellock...@gmail.com>
> > ---
> >  drivers/staging/wlan-ng/hfa384x.h | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/staging/wlan-ng/hfa384x.h 
> > b/drivers/staging/wlan-ng/hfa384x.h
> > index 8f20910..3bb8c1b 100644
> > --- a/drivers/staging/wlan-ng/hfa384x.h
> > +++ b/drivers/staging/wlan-ng/hfa384x.h
> > @@ -1166,6 +1166,7 @@ typedef struct hfa384x_usbctlx {
> >     CTLX_STATE state;       /* Tracks running state */
> >  
> >     struct completion done;
> > +
> >     volatile int reapable;  /* Food for the reaper task */
> >  
> >     ctlx_cmdcb_t cmdcb;     /* Async command callback */
> 
> What coding style issue does this fix?  It looks wrong to me.
> 
> thanks,
> 
> greg k-h

I used checkpatch.pl on the file and it returns "WARNING: Missing a blank line 
after declarations".

I can't see anything specific in the CodingStyle file. Is this an error on 
checkpatch.pl's behalf?

Thanks,

Daniel L

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to