Re: [PATCH v3 0/5] staging: mt7621-spi: Fix C Coding style issues.

From: NeilBrown
Date: Sun Jun 03 2018 - 20:18:46 EST


On Sun, Jun 03 2018, Sankalp Negi wrote:

> This patch series lists different changes for coding style fixes
> as reported by checkpatch.pl, changes can be broken down as :
>
> 1. Indent switch and case labels at the same level.
> 2. Fix a line over 80 columns by refactoring.
> 3. Use tabs for indentation instead of spaces.
> 4. Add a space before open paranthesis in switch.
> 5. Remove unnecessary braces {} from single statement if block.
>
> version 2 changes
> - segregated changes in individual commits as per fix category.
>
> version 3 changes
> - added unique subjects for each patch in the series.
>
>
> Sankalp Negi (5):
> staging: mt7621-spi: Indent case labels and switch at the same level.
> staging: mt7621-spi: Fix line over 80 characters by refactoring.
> staging: mt7621-spi: Use tabs for indentation instead of spaces.
> staging: mt7621-spi: Add a space before open paranthesis.
> staging: mt7621-spi: Remove unnecessary braces {} from single
> statement if block.
>

Thanks,
All
Reviewed-by: NeilBrown <neil@xxxxxxxxxx>

NeilBrown

Attachment: signature.asc
Description: PGP signature