Re: [PATCH v2 0/2] Staging: greybus: Fixed whitespace/syntax style issues

[Alison Schofield <alison.schofield@intel.com>]

On Mon, Oct 10, 2022 at 03:51:57PM -0400, Hunter Chasens wrote:
> These two patches address some Errors, Warnings, and Checks reported by `./scripts/checkpatch.pl`
> 
> v1 -> v2
> Separates the patch into a small patchset with the delimitation of syntactical styling vs whitespace styling. 
> (I plan on sending to the maintainers if Outreach thinks this is an appropriate patch.)

Hi Hunter,
Please see the directions in the First Patch Tutorial and also repeated
in the mailing list message: 

>> You MUST send all patches to this mailing list AND to the appropriate
>> mailing lists and maintainers in get_maintainer.pl. The tutorial walks
>> you through how to use get_maintainer.pl.

We don't want to preview patches on the list. It saves you time and
reviewers time, as there on reviewers who will jump on your staging
patches that might not be on the Outreachy list. It's also the way
we develop - in the open.

So - in this case, you'll send out a v3 and in the changelog one of
the changed items needs to say - Expanded the 'CC list to include
maintainers.

Please do ask questions on the list - just do not post patches for
review that do not have the maintainers included.

Thanks!
Alison

> 
> Hunter Chasens (2):
>   Staging: greybus: Fixed whitespace style issues
>   Staging: greybus: Removed redundent 'int'
> 
>  .../Documentation/firmware/authenticate.c     |  4 +--
>  .../greybus/Documentation/firmware/firmware.c | 34 +++++++++----------
>  drivers/staging/greybus/arche-platform.c      |  2 +-
>  .../staging/greybus/greybus_authentication.h  |  1 -
>  drivers/staging/greybus/loopback.c            |  3 +-
>  drivers/staging/greybus/tools/loopback_test.c | 16 +--------
>  6 files changed, 22 insertions(+), 38 deletions(-)
> 
> -- 
> 2.37.3
> 
>