[PATCH v2 0/2] staging: skein: Fixing various coding style problems

Jason Cooper jason at lakedaemon.net
Tue Dec 30 15:29:23 UTC 2014


Mathieu,

On Mon, Dec 22, 2014 at 08:20:37AM -0700, mathieu.poirier at linaro.org wrote:
> From: Mathieu Poirier <mathieu.poirier at linaro.org>
> 
> This patchset is fixing miscellaneous coding style problems as reported
> by checkpatch.  More interesting is the removal of 'do {} while (0) loop'
> statements for single line macros.  The following provides a guarantee the
> semantic of the object code hasn't been altered by the rectification:
> 
> mpoirier at t430:~/work/kernel1$
> 146320048532 staging: skein: Fixing single statement macro checkpatch warning
> ef68ffa9fd3e staging: skein: Fixing trailing whitespace error
> 97bf6af1f928 Linux 3.19-rc1
> mpoirier at t430:~/work/kernel1$ git reset --hard 97bf6af1f928
> HEAD is now at 97bf6af1f928 Linux 3.19-rc1
> mpoirier at t430:~/work/kernel1$ make drivers/staging/skein/skein_block.o
>   CHK     include/config/kernel.release
>   UPD     include/config/kernel.release
>   CHK     include/generated/uapi/linux/version.h
>   CHK     include/generated/utsrelease.h
>   UPD     include/generated/utsrelease.h
>   CALL    scripts/checksyscalls.sh
>   CC      drivers/staging/skein/skein_block.o
> mpoirier at t430:~/work/kernel1$ ./scripts/objdiff record drivers/staging/skein/skein_block.o
> mpoirier at t430:~/work/kernel1$ git reset --hard 146320048532
> HEAD is now at 146320048532 staging: skein: Fixing single statement macro checkpatch warning
> mpoirier at t430:~/work/kernel1$ make drivers/staging/skein/skein_block.o
>   CHK     include/config/kernel.release
>   UPD     include/config/kernel.release
>   CHK     include/generated/uapi/linux/version.h
>   CHK     include/generated/utsrelease.h
>   UPD     include/generated/utsrelease.h
>   CALL    scripts/checksyscalls.sh
>   CC      drivers/staging/skein/skein_block.o
> mpoirier at t430:~/work/kernel1$ ./scripts/objdiff record drivers/staging/skein/skein_block.o
> mpoirier at t430:~/work/kernel1$ ./scripts/objdiff diff 97bf6af1f928 146320048532
> mpoirier at t430:~/work/kernel1$
> 
> New to this patchset:
> . Introduction of a cover letter that includes a scripts/objdiff snapshot
> . Splitting of the change set by semantic
> 
> Mathieu Poirier (2):
>   staging: skein: Fixing trailing whitespace error
>   staging: skein: Fixing single statement macro checkpatch warning
> 
>  drivers/staging/skein/skein_block.c   | 17 ++++-------------
>  drivers/staging/skein/skein_generic.c |  1 -
>  2 files changed, 4 insertions(+), 14 deletions(-)

Awesome!  For the whole series,

Acked-by:  Jason Cooper <jason at lakedaemon.net>

thx,

Jason.


More information about the devel mailing list