Re: [PATCH v2] staging: rtl8188eu: remove not necessary braces {} (checkpatch fix)

2017-02-01 Thread Martin Karamihov
On 02/01/2017 08:11 PM, Joe Perches wrote: ...and clarifying the code for a human reader is much more important than making a file not have any checkpatch warnings. I agree. I respect the developers' own coding style and believe that some things (>80 characters long lines, name conventions,

Re: [PATCH v2] staging: rtl8188eu: remove not necessary braces {} (checkpatch fix)

2017-02-01 Thread Martin Karamihov
On 02/01/2017 08:11 PM, Joe Perches wrote: ...and clarifying the code for a human reader is much more important than making a file not have any checkpatch warnings. I agree. I respect the developers' own coding style and believe that some things (>80 characters long lines, name conventions,

Re: [PATCH v2] staging: rtl8188eu: remove not necessary braces {} (checkpatch fix)

2017-02-01 Thread Martin Karamihov
On 02/01/2017 12:39 AM, Joe Perches wrote: On Tue, 2017-01-31 at 21:24 +0200, Martin Karamihov wrote: This is checkpatch fix for hal/bb_cfg.c file: remove not necessary braces {} Signed-off-by: Martin Karamihov <martino...@gmail.com> --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 3 +

Re: [PATCH v2] staging: rtl8188eu: remove not necessary braces {} (checkpatch fix)

2017-02-01 Thread Martin Karamihov
On 02/01/2017 12:39 AM, Joe Perches wrote: On Tue, 2017-01-31 at 21:24 +0200, Martin Karamihov wrote: This is checkpatch fix for hal/bb_cfg.c file: remove not necessary braces {} Signed-off-by: Martin Karamihov --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 3 +-- 1 file changed, 1

[PATCH v2] staging: rtl8188eu: remove not necessary braces {} (checkpatch fix)

2017-01-31 Thread Martin Karamihov
This is checkpatch fix for hal/bb_cfg.c file: remove not necessary braces {} Signed-off-by: Martin Karamihov <martino...@gmail.com> --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/d

[PATCH v2] staging: rtl8188eu: remove not necessary braces {} (checkpatch fix)

2017-01-31 Thread Martin Karamihov
This is checkpatch fix for hal/bb_cfg.c file: remove not necessary braces {} Signed-off-by: Martin Karamihov --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal

[PATCH] staging: rtl8188eu: checkpatch fixes: removed not necessary braces {}

2017-01-28 Thread Martin Karamihov
Signed-off-by: Martin Karamihov <martino...@gmail.com> --- checkpatch fixes: remove not necessary braces drivers/staging/rtl8188eu/hal/bb_cfg.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal/bb

[PATCH] staging: rtl8188eu: checkpatch fixes: removed not necessary braces {}

2017-01-28 Thread Martin Karamihov
Signed-off-by: Martin Karamihov --- checkpatch fixes: remove not necessary braces drivers/staging/rtl8188eu/hal/bb_cfg.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal/bb_cfg.c index 134fa6c595a8

[PATCH] staging: rtl8188eu: checkpatch fixes: removed not necessary braces {}

2017-01-28 Thread Martin Karamihov
Signed-off-by: Martin Karamihov <martino...@gmail.com> --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal/bb_cfg.c index 134fa6c595a8..26e0ef224299

[PATCH] staging: rtl8188eu: checkpatch fixes: removed not necessary braces {}

2017-01-28 Thread Martin Karamihov
Signed-off-by: Martin Karamihov --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal/bb_cfg.c index 134fa6c595a8..26e0ef224299 100644 --- a/drivers/staging

[PATCH] staging: rtl8188eu: checkpatch fixes: removed not necessary braces

2017-01-28 Thread Martin Karamihov
checkpatch fixes: removes not necessary braces

[PATCH] staging: rtl8188eu: checkpatch fixes: removed not necessary braces

2017-01-28 Thread Martin Karamihov
checkpatch fixes: removes not necessary braces