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(-) _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel