Messages in this thread | | | From | "Leonid V. Fedorenchik" <> | Subject | [PATCH 00/36] Staging: cx25821: Clean up patch series | Date | Sat, 22 Oct 2011 13:43:20 +0800 |
| |
This patch series fixes some style issues in drivers/staging/cx25821 Mostly I was hoping to improve readability and fix some issues found by checkpatch.pl script.
There is one question, however, in patch no. 18: I deleted part of the comment that contradicts to the code and in this case maybe I should edit the code to match the comment instead. I am not sure if in this case it makes sense to check if width > 0.
Leonid V. Fedorenchik (36): Staging: cx25821: cx25821-alsa.c: Line up comments Staging: cx25821: cx25821-alsa.c: Add braces to else clause Staging: cx25821: cx25821-alsa.c: Fix indent Staging: cx25821: cx25821-alsa.c: Change line endings Staging: cx25821: cx25821-audio-upstream.c: Fix indent Staging: cx25821: cx25821-audio-upstream.c: Move operators Staging: cx25821: cx25821-audio-upstream.c: Change line endings Staging: cx25821: cx25821-audio.h: Line up defines Staging: cx25821: cx25821-audio.h: Fix multiline defines Staging: cx25821: cx25821-cards.c: Fix indent Staging: cx25821: cx25821-core.c: Delete empty line Staging: cx25821: cx25821-core.c: Fix indent Staging: cx25821: cx25821-core.c: Change line endings Staging: cx25821: cx25821-i2c.c: Change line endings Staging: cx25821: cx25821-medusa-defines.h: Fix typo Staging: cx25821: cx25821-medusa-defines.h: Line up defines Staging: cx25821: cx25821-medusa-reg.h: Line up defines Staging: cx25821: cx25821-medusa-video.c: Fix comment Staging: cx25821: cx25821-medusa-video.c: Move operators Staging: cx25821: cx25821-medusa-video.c: Change line endings Staging: cx25821: cx25821-video-upstream-ch2.c: Line up comments Staging: cx25821: cx25821-video-upstream-ch2.c: Fix indent Staging: cx25821: cx25821-video-upsstream-ch2.c: Move operators Staging: cx25821: cx25821-video-upstream-ch2.c: Remove braces Staging: cx25821: cx25821-video-upstream-ch2.c: Change line endings Staging: cx25821: cx25821-video-upstream.c: Remove braces Staging: cx25821: cx25821-video-upstream.c: Fix indent Staging: cx25821: cx25821-video-upstream.c: Change line endings Staging: cx25821: cx25821-video.c: Delete empty line Staging: cx25821: cx25821-video.c: Change spaces Staging: cx25821: cx25821-video.c: Fix assignment Staging: cx25821: cx25821-video.c: Fix definitions Staging: cx25821: cx25821-video.c: Move operators Staging: cx25821: cx25821-video.c: Fix indent Staging: cx25821: cx25821-video.c: Change line endings Staging: cx25821: cx25821.h: Line up defines
drivers/staging/cx25821/cx25821-alsa.c | 73 ++-- drivers/staging/cx25821/cx25821-audio-upstream.c | 102 ++-- drivers/staging/cx25821/cx25821-audio.h | 39 +- drivers/staging/cx25821/cx25821-cards.c | 2 +- drivers/staging/cx25821/cx25821-core.c | 57 +-- drivers/staging/cx25821/cx25821-i2c.c | 10 +- drivers/staging/cx25821/cx25821-medusa-defines.h | 6 +- drivers/staging/cx25821/cx25821-medusa-reg.h | 518 ++++++++++---------- drivers/staging/cx25821/cx25821-medusa-video.c | 410 ++++++---------- .../staging/cx25821/cx25821-video-upstream-ch2.c | 126 ++--- drivers/staging/cx25821/cx25821-video-upstream.c | 146 +++---- drivers/staging/cx25821/cx25821-video.c | 145 +++--- drivers/staging/cx25821/cx25821.h | 4 +- 13 files changed, 745 insertions(+), 893 deletions(-)
| |