Skip to content

staging: greybus: Correct code style issues

Frank A. Cancio Bello outreachy 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest6 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6staging: greybus: Remove unnecessary blank linesdrivers/staging/greybus/arche-apb-ctrl.c drivers/staging/greybus/audio_codec.c drivers/staging/greybus/audio_topology.c +2 more+0 -101 reply
2/6staging: greybus: Fix alignment issuesdrivers/staging/greybus/arche-apb-ctrl.c drivers/staging/greybus/arche-platform.c drivers/staging/greybus/audio_codec.c +5 more+67 -670 replies
3/6staging: greybus: Use u32 instead of uint32_tdrivers/staging/greybus/audio_codec.c+3 -30 replies
4/6staging: greybus: Add space around ternary operatordrivers/staging/greybus/audio_codec.c+2 -20 replies
5/6staging: greybus: Move logical operator in previous linedrivers/staging/greybus/audio_codec.c+2 -20 replies
6/6staging: greybus: Rewrite NULL comparisonsdrivers/staging/greybus/bundle.c drivers/staging/greybus/camera.c drivers/staging/greybus/core.c+10 -100 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/6] staging: greybus: Correct code style issues

This patchset address code style issues with blank lines, aligment, spacing. Beside it change 'var_name == NULL' by '!var_name' and use kernel type instead of C99 types.

Frank A. Cancio Bello (6):
staging: greybus: Remove unnecessary blank lines
staging: greybus: Fix alignment issues
staging: greybus: Use u32 instead of uint32_t
staging: greybus: Add space around ternary operator
staging: greybus: Move logical operator in previous line
staging: greybus: Rewrite NULL comparisons

 drivers/staging/greybus/arche-apb-ctrl.c | 37 +++++++++++----------
 drivers/staging/greybus/arche-platform.c | 10 +++---
 drivers/staging/greybus/audio_codec.c    | 23 ++++++-------
 drivers/staging/greybus/audio_codec.h    |  2 +-
 drivers/staging/greybus/audio_module.c   |  4 +--
 drivers/staging/greybus/audio_topology.c | 55 ++++++++++++++++----------------
 drivers/staging/greybus/authentication.c |  2 +-
 drivers/staging/greybus/bootrom.c        | 20 ++++++------
 drivers/staging/greybus/bundle.c         |  2 +-
 drivers/staging/greybus/camera.c         | 16 +++++-----
 drivers/staging/greybus/connection.c     |  4 ---
 drivers/staging/greybus/control.c        |  1 -
 drivers/staging/greybus/core.c           |  2 +-
 13 files changed, 84 insertions(+), 94 deletions(-)
Signature · 1 line
2.7.4