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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>

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@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@t430:~/work/kernel1$ git reset --hard 97bf6af1f928
HEAD is now at 97bf6af1f928 Linux 3.19-rc1
mpoirier@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@t430:~/work/kernel1$ ./scripts/objdiff record drivers/staging/skein/skein_block.o
mpoirier@t430:~/work/kernel1$ git reset --hard 146320048532
HEAD is now at 146320048532 staging: skein: Fixing single statement macro checkpatch warning
mpoirier@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@t430:~/work/kernel1$ ./scripts/objdiff record drivers/staging/skein/skein_block.o
mpoirier@t430:~/work/kernel1$ ./scripts/objdiff diff 97bf6af1f928 146320048532
mpoirier@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(-)

-- 
1.9.1

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux