[PATCH v2 1/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (missing blank line)

Resolved: "CHECK: Please use a blank line after.." from checkpatch.pl Signed-off-by: Chandra Annamaneni --- Previous versions of these patches were not split into different patches, did not have different patch numbers and did not have the keyword staging. drivers/staging/kpc2000/kpc

[PATCH v2 3/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (alignment)

Resolved: "CHECK: Alignment should match open parenthesis" from checkpatch Signed-off-by: Chandra Annamaneni --- Previous versions of these patches were not split into different patches, did not have different patch numbers and did not have the keyword staging. drivers/stagi

[PATCH v2 4/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (Unnecessary parenthesis)

Resolved: CHECK: Unnecessary parentheses around table[i] Signed-off-by: Chandra Annamaneni --- Previous versions of these patches were not split into different patches, did not have different patch numbers and did not have the keyword staging. drivers/staging/kpc2000/kpc2000_spi.c | 2 +- 1

[PATCH v2 2/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (misaligned brace)

Resolved: ERROR: else should follow close brace '}' Signed-off-by: Chandra Annamaneni --- Previous versions of these patches were not split into different patches, did not have different patch numbers and did not have the keyword staging. The previous version of this patch had

[PATCH 2/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (misaligned brace)

Resolved: ERROR: else should follow close brace '}' Signed-off-by: Chandra Annamaneni --- Previous versions of these patches were not split into different patches, did not have different patch numbers and did not have the keyword staging. The previous version of this patch had

[PATCH 2/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (alignment)

Resolved: "CHECK: Alignment should match open parenthesis" from checkpatch Signed-off-by: Chandra Annamaneni --- Previous versions of these patches were not split into different patches, did not have different patch numbers and did not have the keyword staging. drivers/stagi

[PATCH 3/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (alignment)

Resolved: "CHECK: Alignment should match open parenthesis" from checkpatch Signed-off-by: Chandra Annamaneni --- Previous versions of these patches were not split into different patches, did not have different patch numbers and did not have the keyword staging. drivers/stagi

[PATCH 4/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (Unnecessary parenthesis)

Resolved: CHECK: Unnecessary parentheses around table[i] Signed-off-by: Chandra Annamaneni --- Previous versions of these patches were not split into different patches, did not have different patch numbers and did not have the keyword staging. drivers/staging/kpc2000/kpc2000_spi.c | 2 +- 1

[PATCH 1/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (missing blank line)

Resolved: "CHECK: Please use a blank line after.." from checkpatch.pl Signed-off-by: Chandra Annamaneni --- Previous versions of these patches were not split into different patches, did not have different patch numbers and did not have the keyword staging. drivers/staging/kpc2000/kpc

[PATCH 3/5] KPC2000: kpc2000_spi.c: Fix style issues (misaligned brace)

Resolved: ERROR: else should follow close brace '}' Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/staging/kpc2000/kpc2000_spi.c index d1

[PATCH 1/5] KPC2000: kpc2000_spi.c: Fix style issues (line length)

Resoved: "WARNING: line over 80 characters" from checkpatch.pl Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/stagi

[PATCH 2/5] KPC2000: kpc2000_spi.c: Fix style issues (missing blank line)

Resolved: "CHECK: Please use a blank line after.." from checkpatch.pl Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/staging/kpc2000/kpc2000_spi.c index 81

[PATCH 5/5] KPC2000: kpc2000_spi.c: Fix style issues (Unnecessary parenthesis)

Resolved: CHECK: Unnecessary parentheses around table[i] Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/staging/kpc2000/kpc2000_spi.c index

[PATCH 4/5] KPC2000: kpc2000_spi.c: Fix style issues (alignment)

Resolved: "CHECK: Alignment should match open parenthesis" from checkpatch Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drive

[PATCH] KPC2000: kpc2000_spi.c: Fix style issues (Unecessary parenthesis)

Resolved: CHECK: Unnecessary parentheses around table[i] Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/staging/kpc2000/kpc2000_spi.c index 2082d86

[PATCH] KPC2000: kpc2000_spi.c: Fix style issues (misaligned brace)

Resolved: ERROR: else should follow close brace '}' Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/staging/kpc2000/kpc2000_spi.c ind

[PATCH] KPC2000: kpc2000_spi.c: Fix style issues (alignment)

Resolved: "CHECK: Alignment should match open parenthesis" from checkpatch.pl Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drive

[PATCH] KPC2000: kpc2000_spi.c: Fix style issues (missing blank line)

Resolved: "CHECK: Please use a blank line after.." from checkpatch.pl Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/staging/kpc2000/kpc2000_spi.c ind

[PATCH] KPC2000: kpc2000_spi.c: Fix style issues (line length)

Resoved: "WARNING: line over 80 characters" from checkpatch.pl Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/stagi

[PATCH] KPC2000: kpc2000_spi.c: Fix alignment and style problems.

Fixed alignment and style issues raised by checkpatch.pl Signed-off-by: Chandra Annamaneni --- drivers/staging/kpc2000/kpc2000_spi.c | 49 --- 1 file changed, 28 insertions(+), 21 deletions(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers

[PATCH v5] Staging: most: use __func__ instead of the function name

Change video.c to use %s, __func__ instead of function names. Warnings flagged by checkpatch.pl Signed-off-by: Chandra Annamaneni diff --git a/drivers/staging/most/aim-v4l2/video.c b/drivers/staging/most/aim-v4l2/video.c index e074841..59e861e 100644 --- a/drivers/staging/most/aim-v4l2

Re: [PATCH v4] Staging: most: use __func__ instead of the function name

Sorry! And thanks for pointing it out. regards Chandra On Wed, 26 Apr 2017, Dan Carpenter wrote: > On Wed, Apr 26, 2017 at 06:02:29AM -0700, Chandra Annamaneni wrote: > > > > Change video.c to use %s, __func__ instead of function names. > > Warnings flagged by checkpatch.p

[PATCH v4] Staging: most: use __func__ instead of the function name

Change video.c to use %s, __func__ instead of function names. Warnings flagged by checkpatch.pl Signed-off-by: Chandra Annamaneni diff --git a/drivers/staging/most/aim-v4l2/video.c b/drivers/staging/most/aim-v4l2/video.c index 59e861e..e074841 100644 --- a/drivers/staging/most/aim-v4l2

Re: [PATCH v3] Staging: most: use __func__ instead of the function name

I assumed the email should be a reply to the previous email so it appears in the thread. As to the leading spaces, it was a problem with my mail client. Ill resend the patch. -- Chandra On Tue, 25 Apr 2017, Greg KH wrote: > On Mon, Apr 24, 2017 at 11:20:22PM -0700, Chandra Annamaneni wr

Re: [PATCH v3] Staging: most: use __func__ instead of the function name

Change video.c to use %s, __func__ instead of function names. Warnings flagged by checkpatch.pl Signed-off-by: Chandra Annamaneni diff --git a/drivers/staging/most/aim-v4l2/video.c b/drivers/staging/most/aim-v4l2/video.c index e074841..59e861e 100644 --- a/drivers/staging/most

Re: [PATCH v3] Staging: most: use __func__ instead of the function name

sting such a bad thing? > A: Top-posting. > Q: What is the most annoying thing in e-mail? > > A: No. > Q: Should I include quotations after my reply? > > > http://daringfireball.net/2007/07/on_top > > On Fri, Apr 21, 2017 at 12:40:25AM -0700, Chandra Annamaneni wrote:

Re: [PATCH v3] Staging: most: use __func__ instead of the function name

Third Reminder. Please review. -- Chandra On Sun, 9 Apr 2017, chandrasekhar annamaneni wrote: > > Second reminder, please review. > > Thanks. > Chandra > > > On Fri, Mar 17, 2017 at 12:49 PM, Chandra Annamaneni > wrote: > > Change video.c to use

[PATCH v3] Staging: most: use __func__ instead of the function name

Change video.c to use %s, __func__ instead of function names. Warnings flagged by checkpatch.pl Signed-off-by: Chandra Annamaneni diff --git a/drivers/staging/most/aim-v4l2/video.c b/drivers/staging/most/aim-v4l2/video.c index e074841..59e861e 100644 --- a/drivers/staging/most/aim-v4l2

[PATCH] Fix style warnings in drivers/staging/most/aim-v4l2

Enclosed is a patch to the file video.c. It only fixes style warning flagged by checkpatch.pl. Please let me know if anything else needs to be done. Signed-off-by: Chandra Annamaneni Thanks. Chandra diff --git a/drivers/staging/most/aim-v4l2/video.c b/drivers/staging/most/aim-v4l2

PATCH for style warnings in drivers/staging/most/aim-v4l2

Enclosed is a patch to the file video.c. It only fixes style warning flagged by checkpatch.pl. Please let me know if anything else needs to be done. Thanks. Chandra diff --git a/drivers/staging/most/aim-v4l2/video.c b/drivers/staging/most/aim-v4l2/video.c index e074841..59e861e 100644 ---