[PATCH v4 4/5] staging: rtl8712: style fix unneeded else

2017-12-19 Thread Martin Homuth
This patch fixes a coding style issues in the rtl8712 module as noted by checkpatch.pl where an unnecessary else is used. It fixes the following checkpatch.pl warning: WARNING: else is not generally useful after a break or return Signed-off-by: Martin Homuth <mar...@martinhomuth

[PATCH v4 0/5] staging: rtl8712: fix several checkpatch style issues

2017-12-19 Thread Martin Homuth
- missed one line length warning ([1/5]) v4: - fix remaining reference simplifications ([2/5]) Martin Homuth (5): staging: rtl8712: style fix over 80 characters warnings staging: rtl8712: style fix multiple line dereferences staging: rtl8712: style fix indentation staging: rtl8712: style

[PATCH v4 1/5] staging: rtl8712: style fix over 80 characters warnings

2017-12-19 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl by reducing the characters per line to under 80. It fixes the following checkpatch.pl warning: WARNING: line over 80 characters Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- drivers/s

[PATCH v4 5/5] staging: rtl8712: style fix returned error code

2017-12-19 Thread Martin Homuth
This patch fixes a coding style issues in the rtl8712 module as noted by checkpatch.pl regarding the returned error code. It fixes the following checkpatch.pl warning: WARNING: ENOSYS means 'invalid syscall nr' and nothing else Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- d

[PATCH v4 2/5] staging: rtl8712: style fix multiple line dereferences

2017-12-19 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl related to dereferencing over multiple lines. It fixes the following checkpatch.pl warning: WARNING: Avoid multiple line dereference - prefer %s Signed-off-by: Martin Homuth <mar...@martinhomuth

[PATCH v4 3/5] staging: rtl8712: style fix indentation

2017-12-19 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl related to indentation. It fixes the following checkpatch.pl warning: WARNING: suspect code indent for conditional statements Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- drivers/s

Re: [PATCH v3 2/5] staging: rtl8712: style fix multiple line dereferences

2017-12-19 Thread Martin Homuth
On 12/19/2017 05:41 PM, Joe Perches wrote: > On Tue, 2017-12-19 at 17:32 +0100, Martin Homuth wrote: >> This patch fixes various coding style issues in the rtl8712 module as >> noted by checkpatch.pl related to dereferencing over multiple lines. > [] >> diff --git a

[PATCH v3 1/5] staging: rtl8712: style fix over 80 characters warnings

2017-12-19 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl by reducing the characters per line to under 80. It fixes the following checkpatch.pl warning: WARNING: line over 80 characters Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- drivers/s

[PATCH v3 4/5] staging: rtl8712: style fix unneeded else

2017-12-19 Thread Martin Homuth
This patch fixes a coding style issues in the rtl8712 module as noted by checkpatch.pl where an unnecessary else is used. It fixes the following checkpatch.pl warning: WARNING: else is not generally useful after a break or return Signed-off-by: Martin Homuth <mar...@martinhomuth

[PATCH v3 3/5] staging: rtl8712: style fix indentation

2017-12-19 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl related to indentation. It fixes the following checkpatch.pl warning: WARNING: suspect code indent for conditional statements Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- drivers/s

[PATCH v3 2/5] staging: rtl8712: style fix multiple line dereferences

2017-12-19 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl related to dereferencing over multiple lines. It fixes the following checkpatch.pl warning: WARNING: Avoid multiple line dereference - prefer %s Signed-off-by: Martin Homuth <mar...@martinhomuth

[PATCH v3 5/5] staging: rtl8712: style fix returned error code

2017-12-19 Thread Martin Homuth
This patch fixes a coding style issues in the rtl8712 module as noted by checkpatch.pl regarding the returned error code. It fixes the following checkpatch.pl warning: WARNING: ENOSYS means 'invalid syscall nr' and nothing else Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- d

[PATCH v3 0/5] staging: rtl8712: fix several checkpatch style issues

2017-12-19 Thread Martin Homuth
- missed one line length warning ([1/5]) Martin Homuth (5): staging: rtl8712: style fix over 80 characters warnings staging: rtl8712: style fix multiple line dereferences staging: rtl8712: style fix indentation staging: rtl8712: style fix unneeded else staging: rtl8712: style fix returned

[PATCH v2 5/5] staging: rtl8712: style fix returned error code

2017-12-18 Thread Martin Homuth
This patch fixes a coding style issues in the rtl8712 module as noted by checkpatch.pl regarding the returned error code. It fixes the following checkpatch.pl warning: WARNING: ENOSYS means 'invalid syscall nr' and nothing else Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- d

[PATCH v2 3/5] staging: rtl8712: style fix indentation

2017-12-18 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl related to indentation. It fixes the following checkpatch.pl warning: WARNING: suspect code indent for conditional statements Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- drivers/s

[PATCH v2 4/5] staging: rtl8712: style fix unneeded else

2017-12-18 Thread Martin Homuth
This patch fixes a coding style issues in the rtl8712 module as noted by checkpatch.pl where an unnecessary else is used. It fixes the following checkpatch.pl warning: WARNING: else is not generally useful after a break or return Signed-off-by: Martin Homuth <mar...@martinhomuth

[PATCH v2 1/5] staging: rtl8712: style fix over 80 characters warnings

2017-12-18 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl by reducing the characters per line to under 80. It fixes the following checkpatch.pl warning: WARNING: line over 80 characters Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- drivers/s

[PATCH v2 0/5] staging: rtl8712: fix several checkpatch style issues

2017-12-18 Thread Martin Homuth
This patch series fixes some coding style issues reported by checkpatch.pl. It is based on next-20171218 v2: - avoid line-wrapping by thunderbird - use temporary pointers to avoid index in separate line ([2/5]) Martin Homuth (5): staging: rtl8712: style fix over 80 characters warnings

[PATCH v2 2/5] staging: rtl8712: style fix multiple line dereferences

2017-12-18 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl related to dereferencing over multiple lines. It fixes the following checkpatch.pl warning: WARNING: Avoid multiple line dereference - prefer %s Signed-off-by: Martin Homuth <mar...@martinhomuth

[PATCH 5/5] staging: rtl8712: style fix returned error code

2017-12-18 Thread Martin Homuth
This patch fixes a coding style issues in the rtl8712 module as noted by checkpatch.pl regarding the returned error code. It fixes the following checkpatch.pl warning: WARNING: ENOSYS means 'invalid syscall nr' and nothing else Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- d

[PATCH 2/5] staging: rtl8712: style fix multiple line dereferences

2017-12-18 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl related to dereferencing over multiple lines. It fixes the following checkpatch.pl warning: WARNING: Avoid multiple line dereference - prefer %s Signed-off-by: Martin Homuth <mar...@martinhomuth

[PATCH 4/5] staging: rtl8712: style fix unneeded else

2017-12-18 Thread Martin Homuth
This patch fixes a coding style issues in the rtl8712 module as noted by checkpatch.pl where an unnecessary else is used. It fixes the following checkpatch.pl warning: WARNING: else is not generally useful after a break or return Signed-off-by: Martin Homuth <mar...@martinhomuth

[PATCH 3/5] staging: rtl8712: style fix indentation

2017-12-18 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl related to indentation. It fixes the following checkpatch.pl warning: WARNING: suspect code indent for conditional statements Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- drivers/s

[PATCH 0/5] staging: rtl8712: fix several checkpatch style issues

2017-12-18 Thread Martin Homuth
This patch series fixes some coding style issues reported by checkpatch.pl. It is based on next-20171218 Martin Homuth (5): staging: rtl8712: style fix over 80 characters warnings staging: rtl8712: style fix multiple line dereferences staging: rtl8712: style fix indentation staging

[PATCH 1/5] staging: rtl8712: style fix over 80 characters warnings

2017-12-18 Thread Martin Homuth
This patch fixes various coding style issues in the rtl8712 module as noted by checkpatch.pl by reducing the characters per line to under 80. It fixes the following checkpatch.pl warning: WARNING: line over 80 characters Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- drivers/s

[PATCH Resend] staging: media: cxd2099: style fix - replace hard-coded function names

2017-11-29 Thread Martin Homuth
ion's name, in a string Signed-off-by: Martin Homuth <martin.hom...@martinhomuth.de> --- drivers/staging/media/cxd2099/cxd2099.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media/cxd2099/cxd2099.c b/drivers/staging/media/cxd2099/cxd2099.c index 3e30f48

Re: [PATCH Resend] staging: media: cxd2099: style fix - replace hard-coded function names

2017-11-29 Thread Martin Homuth
ion's name, in a string Signed-off-by: Martin Homuth <martin.hom...@martinhomuth.de> --- drivers/staging/media/cxd2099/cxd2099.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media/cxd2099/cxd2099.c b/drivers/staging/media/cxd2099/cxd2099.c index 3e30f48

[PATCH] staging: media: cxd2099: style fix - replace hard-coded function names

2017-11-29 Thread Martin Homuth
n a string Signed-off-by: Martin Homuth <martin.hom...@martinhomuth.de> --- drivers/staging/media/cxd2099/cxd2099.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media/cxd2099/cxd2099.c b/drivers/staging/media/cxd2099/cxd2099.c index 3e30f48..6641dd2 1006

[PATCH Resend] staging: media: lirc: style fix - replace hard-coded function names

2017-11-28 Thread Martin Homuth
n a string Signed-off-by: Martin Homuth <mar...@martinhomuth.de> --- drivers/staging/media/lirc/lirc_zilog.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/media/lirc/lirc_zilog.c b/drivers/staging/media/lirc/lirc_zilog.c index 6bd0717bf76e..be6

[PATCH] staging/media: lirc: style fix - replace hard-coded function names

2017-11-26 Thread Martin Homuth
From: Martin Homuth <martin.hom...@emlix.com> Date: Sun, 26 Nov 2017 20:14:33 +0100 Subject: [PATCH] lirc: style fix - replace hard-coded function names Instead of hard coding the function name the __func__ variable should be used. Signed-off-by: Martin Homuth <martin.hom...@emlix.com