[PATCH v5 4/5] staging: ks7010: fix coding style issue of using __func__ instead of __FUNCTION__

This patch fixes coding style issue of using __func__ instead of gcc specific __FUNCTION__, warning as issued by checkpatch Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - mo

[PATCH v5 5/5] staging: ks7010: fix coding style issue of aligning comments properly

This patch fixes coding style issue of having block comments using a trailing */ on a separate line, warning as issued by checkpatch Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subj

[PATCH v5 3/5] staging: ks7010: fix coding style issue of using pr_notice instead of printk

Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - modified description to exclude mention of patch number from changelog - updated subject to include driver name v5: - updated

[PATCH v5 2/5] staging: ks7010: fix coding style issue of enclosing complex macro value in parentheses

This patch fixes error of enclosing complex macro value in parentheses, error as issued by checkpatch Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject - incorporated review comment of c

[PATCH v5 1/5] staging: ks7010: fixed warning of avoiding line over 80 characters

This patch fixes warning of line over 80 characters, as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - modified description to exclude mention of patch

[PATCH v4 6/6] staging: ks7010: fix coding style issue of aligning comments properly

This patch fixes coding style issue of having block comments using a trailing */ on a separate line, warning as issued by checkpatch Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subj

[PATCH v4 5/6] staging: ks7010: fix coding style issue of using __func__ instead of __FUNCTION__

This patch fixes coding style issue of using __func__ instead of gcc specific __FUNCTION__, warning as issued by checkpatch Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - mo

[PATCH v4 4/6] staging: ks7010: fix coding style issue of using pr_notice instead of printk

This patch fixes coding style issue of using pr_notice instead of printk, warning as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - modified descr

[PATCH v4 2/6] staging: ks7010: fix coding style issue of enclosing complex macro value in parentheses

This patch fixes error of enclosing complex macro value in parentheses, error as issued by checkpatch Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject - incorporated review comment of c

[PATCH v4 3/6] staging: ks7010: fix coding style issue of using tabs instead of spaces

This patch fixes coding style issue of using tabs instead of spaces at start of line, error as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - mo

[PATCH v4 1/6] staging: ks7010: fixed warning of avoiding line over 80 characters

This patch fixes warning of line over 80 characters, as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject v4: - modified description to exclude mention of patch

Re: [PATCH v3 2/6] staging: ks7010: fix coding style issue of enclosing complex macro value in parentheses

it has been ensured that no new issues are introduced in the process of fixing the original ones. I will fix up all other comments and resend complete series of patches. thanks, chetan On 2017年02月25日 02:07, Greg KH wrote: > On Fri, Feb 17, 2017 at 06:47:39PM +0900, Chetan Sethi wrote: >&g

[PATCH v3 6/6] drivers: staging: fix coding style issue of aligning comments properly

This is 06th patch to file ks_wlan.h in order to fix coding style issue of having block comments using a trailing */ on a separate line, warning as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patc

[PATCH v3 5/6] drivers: staging: fix coding style issue of using __func__ instead of __FUNCTION__

This is 05th patch to file ks_wlan.h which fixes coding style issue of using __func__ instead of gcc specific __FUNCTION__, warning as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned

[PATCH v3 4/6] drivers: staging: fix coding style issue of using pr_notice instead of printk

This is 04th patch to ks_wlan.h fixing coding style issue of using pr_notice instead of printk, warning as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject d

[PATCH v3 3/6] staging: ks7010: fix coding style issue of using tabs instead of spaces

This is 03rd patch to file ks_wlan.h fixing coding style issue of using tabs instead of spaces at start of line, error as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch re

[PATCH v3 2/6] staging: ks7010: fix coding style issue of enclosing complex macro value in parentheses

This is 02nd patch to file ks_wlan.h file fixing error of enclosing complex macro value in parentheses Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject - incorporated review comment of c

[PATCH v3 1/6] staging: ks7010: fixed warning of avoiding line over 80 characters

This is patch 01 to ks_wlan.h file in order to fix warning of line over 80 characters, as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- v2: - split multiple changes across different patches v3: - mentioned patch revision in subject drivers/staging/

Re: [PATCH 2/6] staging: ks7010: fix coding style issue of enclosing complex macro value in parentheses

n correct with the `do {`: Sure, I will update and resend v2 version of all the patches. thanks, chetan On 2017年02月17日 13:36, Matthew Kilgore wrote: > Hi Chetan, > > On Fri, Feb 17, 2017 at 12:08:02PM +0900, Chetan Sethi wrote: >> +do {if (KS_WLAN_DEBUG > (n

[PATCH 6/6] drivers: staging: fix coding style issue of aligning comments properly

This is 06th patch to file ks_wlan.h in order to fix coding style issue of having block comments using a trailing */ on a separate line, warning as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- drivers/staging/ks7010/ks_wlan.h | 3 ++- 1 file chan

[PATCH 5/6] drivers: staging: fix coding style issue of using __func__ instead of __FUNCTION__

This is 05th patch to file ks_wlan.h which fixes coding style issue of using __func__ instead of gcc specific __FUNCTION__, warning as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- drivers/staging/ks7010/ks_wlan.h | 2 +- 1 file changed, 1 insertion

[PATCH 4/6] drivers: staging: fix coding style issue of using pr_notice instead of printk

This is 04th patch to ks_wlan.h fixing coding style issue of using pr_notice instead of printk, warning as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- drivers/staging/ks7010/ks_wlan.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/d

[PATCH 3/6] staging: ks7010: fix coding style issue of using tabs instead of spaces

This is 03rd patch to file ks_wlan.h fixing coding style issue of using tabs instead of spaces at start of line, error as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- drivers/staging/ks7010/ks_wlan.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 2/6] staging: ks7010: fix coding style issue of enclosing complex macro value in parentheses

This is 02nd patch to file ks_wlan.h file fixing error of enclosing complex macro value in parentheses and do-while statement has been used to prevent further warnings from checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- drivers/staging/ks7010/ks_wlan.h | 5 +++--

[PATCH 1/6] staging: ks7010: fixed warning of avoiding line over 80 characters

This is patch 01 to ks_wlan.h file in order to fix warning of line over 80 characters, as issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- drivers/staging/ks7010/ks_wlan.h | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/d

[PATCH] staging: ks7010: fix coding style issues reported by checkpatch.pl

This is patch to ks_wlan.h which fixes various coding style errors and warnings issued by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- drivers/staging/ks7010/ks_wlan.h | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/drivers/s

[PATCH] fix coding style issue use octal permission instead of decimal

This is a patch to the dim2_hdm.c file that fixes coding style error found by checkpatch.pl Signed-off-by: Chetan Sethi <cpsethi...@gmail.com> --- drivers/staging/most/hdm-dim2/dim2_hdm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/most/hd