[PATCH 00/16] media: s2255: Coding style cleanup

Lei Huang posted 16 patches 1 week ago
Only 10 patches received!
There is a newer version of this series
drivers/media/usb/s2255/s2255drv.c | 375 ++++++++++++++---------------
1 file changed, 186 insertions(+), 189 deletions(-)
[PATCH 00/16] media: s2255: Coding style cleanup
Posted by Lei Huang 1 week ago
This series is a follow-up to the "media: s2255: Fix firmware parsing
OOB and cleanup coding style" series (0002-media-s2255-Rename-CamelCase
-goto-labels-to-snake_ca.patch). It builds on the changes made there
and should be merged only after that series is integrated.

This series applies a series of coding style cleanups to the
s2255 USB video driver (drivers/media/usb/s2255/s2255drv.c) to bring
it closer to the Linux kernel coding style. The changes address issues
flagged by checkpatch.pl and general style review:

  - Rename remaining CamelCase identifiers to snake_case
  - Remove unnecessary spaces after casts and parentheses around
    conditions
  - Use consistent braces on all arms of if/else statements, and remove
    unnecessary braces from single-statement blocks
  - Replace NULL comparisons with ! notation and bare 'unsigned' with
    'unsigned int'
  - Fix alignment to match open parenthesis and add spaces around
    multiplication operators
  - Parenthesize macro arguments to avoid precedence issues
  - Fix block comment style to kernel format
  - Add blank lines after function/struct/enum declarations and after
    variable declarations, and remove extra blank lines
  - Remove unnecessary return statements from void functions
  - Remove unnecessary debug logging and out-of-memory messages

No functional changes; the driver behavior is unchanged.

Lei Huang (16):
  media: s2255: Rename remaining CamelCase identifiers to snake_case
  media: s2255: Remove unnecessary spaces after casts
  media: s2255: Use consistent braces on all arms of if/else statements
  media: s2255: Replace NULL comparisons with ! notation
  media: s2255: Fix alignment to match open parenthesis
  media: s2255: Remove multiple blank lines and before close braces
  media: s2255: Add blank lines after function/struct/enum declarations
  media: s2255: Add spaces around multiplication operators
  media: s2255: Remove unnecessary parentheses around conditions
  media: s2255: Parenthesize macro argument to avoid precedence issues
  media: s2255: Add blank lines after variable declarations
  media: s2255: Fix block comment style to kernel format
  media: s2255: Remove unnecessary return statements from void functions
  media: s2255: Remove unnecessary debug logging and out-of-memory
    messages
  media: s2255: Remove unnecessary braces from single-statement blocks
  media: s2255: Replace bare 'unsigned' with 'unsigned int'

 drivers/media/usb/s2255/s2255drv.c | 375 ++++++++++++++---------------
 1 file changed, 186 insertions(+), 189 deletions(-)

-- 
2.25.1