[PATCH 2/2] Staging: rtl8723bs: os_dep: Invert if selection statement

2019-03-12 Thread Guilherme T Maeoka
From: Guilherme T Maeoka Change 'if (a)' to 'if (!a)' and return. Otherwise, continue with the previouly wrapped block of control. This reduces the indentation level by 2 and 1. I'm not if this commit contributes to the coding style. Signed-off-by: Guilherme T Maeoka --- drivers/staging

[PATCH 1/2] Staging: rtl8723bs: os_dep: Fix several coding style errors

2019-03-12 Thread Guilherme T Maeoka
From: Guilherme T Maeoka Fix coding style errors and warns complained by checkpatck.pl. To list: - remove braces for single statements blocks, - add space required around operators, - replace spaces with tabs to indent, - add blank line after declarations

Re: [PATCH 1/5] Staging: rtl8723bs: os_dep: Fix assignment in if condition

2019-03-09 Thread Guilherme T Maeoka
Thank you for the review Joe. I'll have that in mind in the next patch. On 3/9/19, Joe Perches wrote: > On Sat, 2019-03-09 at 15:30 -0300, Guilherme Tadashi Maeoka wrote: >> Fix an assignment in if condition. >> >> Signed-off-by: Guilherme Tadashi Maeoka >> --- >>