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

[Johan Hovold <johan@kernel.org>]

On Thu, Oct 19, 2017 at 08:06:22PM -0400, Frank A. Cancio Bello wrote:
> 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.

Thanks for the clean ups. Some of these are good, some fall in category
of style choices that are really up to the author of the code to decide
on (e.g. using double newlines as a separator in a source file).

Either way, you need to send these also to the public lists.
scripts/get_maintainer.pl will tell you which (you don't need to CC lkml
unless you prefer).

Thanks,
Johan

> 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(-)