[PATCH 00/36] Staging: cx25821: Clean up patch series

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux