Staging: cx25821: Fix style issues
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest44 patches 1 replyIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/44 | Staging: cx25821: Fix labels' placement in cx25821-video.cdrivers/ | +1 | 0 replies | |
| 2/44 | Staging: cx25821: Fix function signaturesdrivers/ | +32 | 0 replies | |
| 3/44 | Staging: cx25821: Fix obscure line endingsdrivers/ | +1 | 0 replies | |
| 4/44 | Staging: cx25821: Fix indent in commentdrivers/ | +11 | 0 replies | |
| 5/44 | Staging: cx25821: Change indent with spaces to tabs 1drivers/ | +10 | 0 replies | |
| 6/44 | Staging: cx25821: Change indent with spaces to tabs 2drivers/ | +11 | 0 replies | |
| 7/44 | Staging: cx25821: Change indent with spaces to tabs 3drivers/ | +17 | 0 replies | |
| 8/44 | Staging: cx25821: Change indent with spaces to tabs 4drivers/ | +13 | 0 replies | |
| 9/44 | Staging: cx25821: Change indent with spaces to tabs 5drivers/ | +10 | 0 replies | |
| 10/44 | Staging: cx25821: Change indent with spaces to tabs 6drivers/ | +10 | 0 replies | |
| 11/44 | Staging: cx25821: Change indent with spaces to tabs 7drivers/ | +31 | 0 replies | |
| 12/44 | Staging: cx25821: Change indent with spaces to tabs 8drivers/ | +18 | 0 replies | |
| 13/44 | Staging: cx25821: Change indent with spaces to tabs 9drivers/ | +21 | 0 replies | |
| 14/44 | Staging: cx25821: Change indent with spaces to tabs 10drivers/ | +13 | 0 replies | |
| 15/44 | Staging: cx25821: Change indent with spaces to tabs 11drivers/ | +16 | 0 replies | |
| 16/44 | Staging: cx25821: Change indent with spaces to tabs 12drivers/ | +12 | 0 replies | |
| 17/44 | Staging: cx25821: Change indent with spaces to tabs 13drivers/ | +27 | 0 replies | |
| 18/44 | Staging: cx25821: Change indent with spaces to tabs 14drivers/ | +18 | 0 replies | |
| 19/44 | Staging: cx25821: Change indent with spaces to tabs 15drivers/ | +10 | 0 replies | |
| 20/44 | Staging: cx25821: Change indent with spaces to tabs 16drivers/ | +13 | 0 replies | |
| 21/44 | Staging: cx25821: Change indent with spaces to tabs 17drivers/ | +10 | 0 replies | |
| 22/44 | Staging: cx25821: Change indent with spaces to tabs 18drivers/ | +11 | 0 replies | |
| 23/44 | Staging: cx25821: Change indent with spaces to tabs 19drivers/ | +15 | 0 replies | |
| 24/44 | Staging: cx25821: Change indent with spaces to tabs 20drivers/ | +16 | 0 replies | |
| 25/44 | Staging: cx25821: Change indent with spaces to tabs 21drivers/ | +10 | 0 replies | |
| 26/44 | Staging: cx25821: Change indent with spaces to tabs 22drivers/ | +10 | 0 replies | |
| 27/44 | Staging: cx25821: Change indent with spaces to tabs 23drivers/ | +13 | 0 replies | |
| 28/44 | Staging: cx25821: Change indent with spaces to tabs 24drivers/ | +11 | 0 replies | |
| 29/44 | Staging: cx25821: Change indent with spaces to tabs 25drivers/ | +14 | 0 replies | |
| 30/44 | Staging: cx25821: Change indent with spaces to tabs 26drivers/ | +9 | 0 replies | |
| 31/44 | Staging: cx25821: Change indent with spaces to tabs 27drivers/ | +14 | 0 replies | |
| 32/44 | Staging: cx25821: Change indent with spaces to tabs 28drivers/ | +9 | 0 replies | |
| 33/44 | Staging: cx25821: Change indent with spaces to tabs 29drivers/ | +23 | 0 replies | |
| 34/44 | Staging: cx25821: Change indent with spaces to tabs 30drivers/ | +11 | 0 replies | |
| 35/44 | Staging: cx25821: Change indent with spaces to tabs 31drivers/ | +17 | 0 replies | |
| 36/44 | Staging: cx25821: Change indent with spaces to tabs 32drivers/ | +56 | 0 replies | |
| 37/44 | Staging: cx25821: Change indent with spaces to tabs 33drivers/ | +14 | 0 replies | |
| 38/44 | Staging: cx25821: Change indent with spaces to tabs 34drivers/ | +38 | 0 replies | |
| 39/44 | Staging: cx25821: Change indent with spaces to tabs 35drivers/ | +5 | 0 replies | |
| 40/44 | Staging: cx25821: Fix braces around statements in cx25821-video.cdrivers/ | +2 | 0 replies | |
| 41/44 | Staging: cx25821: Replace :? by if-else in cx25821-video.cdrivers/ | +20 | 0 replies | |
| 42/44 | Staging: cx25821: cx25821-video.h: Line up constantdrivers/ | +1 | 0 replies | |
| 43/44 | Staging: cx25821: cx25821-video.h: Fix indentdrivers/ | +15 | 0 replies | |
| 44/44 | Staging: cx25821: cx25821-video.h: Fix long linesdrivers/ | +29 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
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(-)