Re: [PATCH] drivers: staging: ft1000: ft1000-usb: initialize 'status' with STATUS_SUCCESS in request_code_segment()

2013-12-03 Thread Chen Gang
Oh, another member has already fixed it (found earlier than me), and integrated it into next-20131203 tree, so this patch is obsoleted. The related git commit is "8aced95 staging: ft1000: fix use of potentially uninitialized variable" Thanks. On 11/27/2013 05:27 PM, Chen Gang wrote: > On

Re: [PATCH] drivers: staging: ft1000: ft1000-usb: initialize 'status' with STATUS_SUCCESS in request_code_segment()

2013-12-03 Thread Chen Gang
Oh, another member has already fixed it (found earlier than me), and integrated it into next-20131203 tree, so this patch is obsoleted. The related git commit is 8aced95 staging: ft1000: fix use of potentially uninitialized variable Thanks. On 11/27/2013 05:27 PM, Chen Gang wrote: On

Re: [PATCH] drivers: staging: ft1000: ft1000-usb: initialize 'status' with STATUS_SUCCESS in request_code_segment()

2013-11-27 Thread Chen Gang
On 11/27/2013 05:18 PM, Josh Triplett wrote: > On Wed, Nov 27, 2013 at 11:01:18AM +0800, Chen Gang wrote: >> If "!bool_case", it returns unexpected value instead of STATUS_SUCCESS, >> so need fix it, the related warning (with allmodconfig under hexagon): >> >> CC [M]

Re: [PATCH] drivers: staging: ft1000: ft1000-usb: initialize 'status' with STATUS_SUCCESS in request_code_segment()

2013-11-27 Thread Josh Triplett
On Wed, Nov 27, 2013 at 11:01:18AM +0800, Chen Gang wrote: > If "!bool_case", it returns unexpected value instead of STATUS_SUCCESS, > so need fix it, the related warning (with allmodconfig under hexagon): > > CC [M] drivers/staging/ft1000/ft1000-usb/ft1000_download.o >

Re: [PATCH] drivers: staging: ft1000: ft1000-usb: initialize 'status' with STATUS_SUCCESS in request_code_segment()

2013-11-27 Thread Josh Triplett
On Wed, Nov 27, 2013 at 11:01:18AM +0800, Chen Gang wrote: If !bool_case, it returns unexpected value instead of STATUS_SUCCESS, so need fix it, the related warning (with allmodconfig under hexagon): CC [M] drivers/staging/ft1000/ft1000-usb/ft1000_download.o

Re: [PATCH] drivers: staging: ft1000: ft1000-usb: initialize 'status' with STATUS_SUCCESS in request_code_segment()

2013-11-27 Thread Chen Gang
On 11/27/2013 05:18 PM, Josh Triplett wrote: On Wed, Nov 27, 2013 at 11:01:18AM +0800, Chen Gang wrote: If !bool_case, it returns unexpected value instead of STATUS_SUCCESS, so need fix it, the related warning (with allmodconfig under hexagon): CC [M]

[PATCH] drivers: staging: ft1000: ft1000-usb: initialize 'status' with STATUS_SUCCESS in request_code_segment()

2013-11-26 Thread Chen Gang
If "!bool_case", it returns unexpected value instead of STATUS_SUCCESS, so need fix it, the related warning (with allmodconfig under hexagon): CC [M] drivers/staging/ft1000/ft1000-usb/ft1000_download.o drivers/staging/ft1000/ft1000-usb/ft1000_download.c: In function

[PATCH] drivers: staging: ft1000: ft1000-usb: initialize 'status' with STATUS_SUCCESS in request_code_segment()

2013-11-26 Thread Chen Gang
If !bool_case, it returns unexpected value instead of STATUS_SUCCESS, so need fix it, the related warning (with allmodconfig under hexagon): CC [M] drivers/staging/ft1000/ft1000-usb/ft1000_download.o drivers/staging/ft1000/ft1000-usb/ft1000_download.c: In function 'request_code_segment':