Skip to content

Staging: cx25821: Fix style issues

Leonid V. Fedorenchik lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest44 patches 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/44Staging: cx25821: Fix labels' placement in cx25821-video.cdrivers/staging/cx25821/cx25821-video.c+1 -10 replies
2/44Staging: cx25821: Fix function signaturesdrivers/staging/cx25821/cx25821-video.c+32 -170 replies
3/44Staging: cx25821: Fix obscure line endingsdrivers/staging/cx25821/cx25821-video.c+1 -20 replies
4/44Staging: cx25821: Fix indent in commentdrivers/staging/cx25821/cx25821-video.c+11 -110 replies
5/44Staging: cx25821: Change indent with spaces to tabs 1drivers/staging/cx25821/cx25821-video.c+10 -100 replies
6/44Staging: cx25821: Change indent with spaces to tabs 2drivers/staging/cx25821/cx25821-video.c+11 -120 replies
7/44Staging: cx25821: Change indent with spaces to tabs 3drivers/staging/cx25821/cx25821-video.c+17 -180 replies
8/44Staging: cx25821: Change indent with spaces to tabs 4drivers/staging/cx25821/cx25821-video.c+13 -140 replies
9/44Staging: cx25821: Change indent with spaces to tabs 5drivers/staging/cx25821/cx25821-video.c+10 -100 replies
10/44Staging: cx25821: Change indent with spaces to tabs 6drivers/staging/cx25821/cx25821-video.c+10 -100 replies
11/44Staging: cx25821: Change indent with spaces to tabs 7drivers/staging/cx25821/cx25821-video.c+31 -340 replies
12/44Staging: cx25821: Change indent with spaces to tabs 8drivers/staging/cx25821/cx25821-video.c+18 -200 replies
13/44Staging: cx25821: Change indent with spaces to tabs 9drivers/staging/cx25821/cx25821-video.c+21 -220 replies
14/44Staging: cx25821: Change indent with spaces to tabs 10drivers/staging/cx25821/cx25821-video.c+13 -130 replies
15/44Staging: cx25821: Change indent with spaces to tabs 11drivers/staging/cx25821/cx25821-video.c+16 -170 replies
16/44Staging: cx25821: Change indent with spaces to tabs 12drivers/staging/cx25821/cx25821-video.c+12 -120 replies
17/44Staging: cx25821: Change indent with spaces to tabs 13drivers/staging/cx25821/cx25821-video.c+27 -270 replies
18/44Staging: cx25821: Change indent with spaces to tabs 14drivers/staging/cx25821/cx25821-video.c+18 -180 replies
19/44Staging: cx25821: Change indent with spaces to tabs 15drivers/staging/cx25821/cx25821-video.c+10 -100 replies
20/44Staging: cx25821: Change indent with spaces to tabs 16drivers/staging/cx25821/cx25821-video.c+13 -130 replies
21/44Staging: cx25821: Change indent with spaces to tabs 17drivers/staging/cx25821/cx25821-video.c+10 -100 replies
22/44Staging: cx25821: Change indent with spaces to tabs 18drivers/staging/cx25821/cx25821-video.c+11 -110 replies
23/44Staging: cx25821: Change indent with spaces to tabs 19drivers/staging/cx25821/cx25821-video.c+15 -150 replies
24/44Staging: cx25821: Change indent with spaces to tabs 20drivers/staging/cx25821/cx25821-video.c+16 -160 replies
25/44Staging: cx25821: Change indent with spaces to tabs 21drivers/staging/cx25821/cx25821-video.c+10 -100 replies
26/44Staging: cx25821: Change indent with spaces to tabs 22drivers/staging/cx25821/cx25821-video.c+10 -110 replies
27/44Staging: cx25821: Change indent with spaces to tabs 23drivers/staging/cx25821/cx25821-video.c+13 -130 replies
28/44Staging: cx25821: Change indent with spaces to tabs 24drivers/staging/cx25821/cx25821-video.c+11 -110 replies
29/44Staging: cx25821: Change indent with spaces to tabs 25drivers/staging/cx25821/cx25821-video.c+14 -150 replies
30/44Staging: cx25821: Change indent with spaces to tabs 26drivers/staging/cx25821/cx25821-video.c+9 -90 replies
31/44Staging: cx25821: Change indent with spaces to tabs 27drivers/staging/cx25821/cx25821-video.c+14 -150 replies
32/44Staging: cx25821: Change indent with spaces to tabs 28drivers/staging/cx25821/cx25821-video.c+9 -90 replies
33/44Staging: cx25821: Change indent with spaces to tabs 29drivers/staging/cx25821/cx25821-video.c+23 -240 replies
34/44Staging: cx25821: Change indent with spaces to tabs 30drivers/staging/cx25821/cx25821-video.c+11 -110 replies
35/44Staging: cx25821: Change indent with spaces to tabs 31drivers/staging/cx25821/cx25821-video.c+17 -180 replies
36/44Staging: cx25821: Change indent with spaces to tabs 32drivers/staging/cx25821/cx25821-video.c+56 -570 replies
37/44Staging: cx25821: Change indent with spaces to tabs 33drivers/staging/cx25821/cx25821-video.c+14 -140 replies
38/44Staging: cx25821: Change indent with spaces to tabs 34drivers/staging/cx25821/cx25821-video.c+38 -380 replies
39/44Staging: cx25821: Change indent with spaces to tabs 35drivers/staging/cx25821/cx25821-video.c+5 -50 replies
40/44Staging: cx25821: Fix braces around statements in cx25821-video.cdrivers/staging/cx25821/cx25821-video.c+2 -40 replies
41/44Staging: cx25821: Replace :? by if-else in cx25821-video.cdrivers/staging/cx25821/cx25821-video.c+20 -120 replies
42/44Staging: cx25821: cx25821-video.h: Line up constantdrivers/staging/cx25821/cx25821-video.h+1 -10 replies
43/44Staging: cx25821: cx25821-video.h: Fix indentdrivers/staging/cx25821/cx25821-video.h+15 -150 replies
44/44Staging: cx25821: cx25821-video.h: Fix long linesdrivers/staging/cx25821/cx25821-video.h+29 -180 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 00/44] Staging: cx25821: Fix style issues

This patch series fixes up style issues in cx25821-video.{c,h} files in drivers/staging/cx25821

What the heck with patches 5-39? ... Well, in cx25821-video.c file there are lots of lines starting with 7 spaces where clearly this intented to be a tab; that is reported by checkpatch.pl script. And even more lines that start with one or more tabs and 7 spaces; this is intented to be 2 or more tabs instead and this is not reported by checkpatch.pl script (obviously). So, first, I fix up all that kind of wrong indents and resulting patch was too big and unreviewable. (around 2K lines) So, since I haven't thought of better way to make the patch reviewable, I decided to split that huge patch into small reviewable patches. The result is 35 patches. %-) If you can think of a better way to deal with this, let me know.

Leonid V. Fedorenchik (44):
Staging: cx25821: Fix labels' placement in cx25821-video.c
Staging: cx25821: Fix function signatures
Staging: cx25821: Fix obscure line endings
Staging: cx25821: Fix indent in comment
Staging: cx25821: Change indent with spaces to tabs 1
Staging: cx25821: Change indent with spaces to tabs 2
Staging: cx25821: Change indent with spaces to tabs 3
Staging: cx25821: Change indent with spaces to tabs 4
Staging: cx25821: Change indent with spaces to tabs 5
Staging: cx25821: Change indent with spaces to tabs 6
Staging: cx25821: Change indent with spaces to tabs 7
Staging: cx25821: Change indent with spaces to tabs 8
Staging: cx25821: Change indent with spaces to tabs 9
Staging: cx25821: Change indent with spaces to tabs 10
Staging: cx25821: Change indent with spaces to tabs 11
Staging: cx25821: Change indent with spaces to tabs 12
Staging: cx25821: Change indent with spaces to tabs 13
Staging: cx25821: Change indent with spaces to tabs 14
Staging: cx25821: Change indent with spaces to tabs 15
Staging: cx25821: Change indent with spaces to tabs 16
Staging: cx25821: Change indent with spaces to tabs 17
Staging: cx25821: Change indent with spaces to tabs 18
Staging: cx25821: Change indent with spaces to tabs 19
Staging: cx25821: Change indent with spaces to tabs 20
Staging: cx25821: Change indent with spaces to tabs 21
Staging: cx25821: Change indent with spaces to tabs 22
Staging: cx25821: Change indent with spaces to tabs 23
Staging: cx25821: Change indent with spaces to tabs 24
Staging: cx25821: Change indent with spaces to tabs 25
Staging: cx25821: Change indent with spaces to tabs 26
Staging: cx25821: Change indent with spaces to tabs 27
Staging: cx25821: Change indent with spaces to tabs 28
Staging: cx25821: Change indent with spaces to tabs 29
Staging: cx25821: Change indent with spaces to tabs 30
Staging: cx25821: Change indent with spaces to tabs 31
Staging: cx25821: Change indent with spaces to tabs 32
Staging: cx25821: Change indent with spaces to tabs 33
Staging: cx25821: Change indent with spaces to tabs 34
Staging: cx25821: Change indent with spaces to tabs 35
Staging: cx25821: Fix braces around statements in cx25821-video.c
Staging: cx25821: Replace :? by if-else in cx25821-video.c
Staging: cx25821: cx25821-video.h: Line up constant
Staging: cx25821: cx25821-video.h: Fix indent
Staging: cx25821: cx25821-video.h: Fix long lines

 drivers/staging/cx25821/cx25821-video.c | 1244 ++++++++++++++++---------------
 drivers/staging/cx25821/cx25821-video.h |   79 ++-
 2 files changed, 669 insertions(+), 654 deletions(-)