Re: [PATCHv3] staging: vt6656: removed erroneous else statement

2015-01-31 Thread Greg Kroah-Hartman
On Sat, Jan 31, 2015 at 02:05:21PM -0500, Derrick Greenspan wrote: > This patch fixes the checkpatch.pl warning: > > WARNING: else is not generally useful after a break or return > 559: FILE: drivers/staging/vt6656/rxtx.c:559: > return vnt_rxtx_datahead_g_fb(tx_context, &buf->data_head); > } else

Re: [PATCHv3] staging: vt6656: removed erroneous else statement

2015-01-31 Thread Malcolm Priestley
On 31/01/15 19:05, Derrick Greenspan wrote: This patch fixes the checkpatch.pl warning: WARNING: else is not generally useful after a break or return 559: FILE: drivers/staging/vt6656/rxtx.c:559: Checkpatch does not check that the if and else has different function scope. It isn't erroneous

[PATCHv3] staging: vt6656: removed erroneous else statement

2015-01-31 Thread Derrick Greenspan
This patch fixes the checkpatch.pl warning: WARNING: else is not generally useful after a break or return 559: FILE: drivers/staging/vt6656/rxtx.c:559: return vnt_rxtx_datahead_g_fb(tx_context, &buf->data_head); } else { Signed-off-by: Derrick Greenspan --- Changes in v2: - Hopefully fixed whit