Re: [PATCH v2 4/7] staging: rtl8723bs: Move braces to same line as conditional

2017-05-06 Thread Bastien Nocera
On Fri, 2017-05-05 at 16:10 -0600, Justin Vreeland wrote: > > Sorry about blank email. > > I'm not 100% happy with it either.  Larry Finger suggested to fix the > overly long lines so I tried to find a decent way to shorten them.  I > think the best way would be to change the function names or p

Re: [PATCH v2 4/7] staging: rtl8723bs: Move braces to same line as conditional

2017-05-05 Thread Justin Vreeland
Bastien Nocera writes: > On Mon, 2017-05-01 at 18:52 -0600, Justin Vreeland wrote: >> Ensure checkpatch compliance >> >> > Signed-off-by: Justin Vreeland >> --- >> v2: >>   - Added commit message >>   - Fixed overly long lines >> >>  drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c | 18 ++

Re: [PATCH v2 4/7] staging: rtl8723bs: Move braces to same line as conditional

2017-05-05 Thread Justin Vreeland
Bastien Nocera writes: > On Mon, 2017-05-01 at 18:52 -0600, Justin Vreeland wrote: >> Ensure checkpatch compliance >> >> > Signed-off-by: Justin Vreeland >> --- >> v2: >>   - Added commit message >>   - Fixed overly long lines >> >>  drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c | 18 +++

Re: [PATCH v2 4/7] staging: rtl8723bs: Move braces to same line as conditional

2017-05-04 Thread Bastien Nocera
On Mon, 2017-05-01 at 18:52 -0600, Justin Vreeland wrote: > Ensure checkpatch compliance > > > Signed-off-by: Justin Vreeland > --- > v2: >   - Added commit message >   - Fixed overly long lines > >  drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c | 18 -- >  drivers/staging/rtl87

[PATCH v2 4/7] staging: rtl8723bs: Move braces to same line as conditional

2017-05-01 Thread Justin Vreeland
Ensure checkpatch compliance Signed-off-by: Justin Vreeland --- v2: - Added commit message - Fixed overly long lines drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c | 18 -- drivers/staging/rtl8723bs/hal/rtl8723b_rf6052.c | 12 ++-- drivers/staging/rtl8723bs/hal/rtl8