Re: staging: unisys: fix else statement in visornic_main.c

2016-02-16 Thread Arfvidson, Erik
On 2/16/2016 4:02 AM, Joe Perches wrote: > On Tue, 2016-02-16 at 10:31 +0300, Dan Carpenter wrote: > > [ checkpatch.pl told someone to introduce a bug and they did... ] > > Yeah, it happens. > > People should not take a message like "generally" as > an absolute. > > "else is not generally useful

Re: [PATCH 14/15] staging: unisys: fix alignment in visornic_main.c

2015-12-03 Thread Arfvidson, Erik
On 12/2/15 8:22 AM, Dan Carpenter wrote: > On Mon, Nov 30, 2015 at 02:46:14PM -0500, Benjamin Romer wrote: >> @@ -1651,7 +1651,7 @@ service_resp_queue(struct uiscmdrsp *cmdrsp, struct >> visornic_devdata *devdata, >> * netif_wake_queue() >>