[PATCH 0/6] keucr: checkpatch fixes

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

 



hi,

we started cleaning up a bit in drivers/staging/keucr, resolved a lot of
checkpatch warnings and errors.


[PATCH 1/6] keucr: remove some unneccesary typedefs

in ../init.c the two unused typedefed structures ADDRESS_T and SSFDCTYPE_T are
removed, in other files typedefs are removed and replaced with actual
structures and pointers.


[PATCH 2/6] keucr: fix some camelCase warnings

still a lot of camelCase to go, but this is a start. checkpatch.pl is mostly
satisfied with the progress.


[PATCH 3/6] migrate printk to dev_dbg/info/warn/err

printk()-calls without facility levels replaced with dev_{dbg,info,warn,err}
accordingly.


[PATCH 4/6] keucr: make error messages less aggressive

in .../keucr/smilsub.c a lot of error messages ended with "!!\n", we changed
that to a less shouting version. also, no punctuation in error messages is
probably a good thing.


[PATCH 5/6] keucr: various alignment- and whitespace-problems

line-lengths, whitespace around operators and placement of braces now as
demanded by checkpatch


[PATCH 6/6] keucr: use more specific max_t(int, ..

one more change, didn't fit in any of the other categories: change max(a, b)
to max_t(int, a, b) in .../keucr/transport.c, as suggested by checkpatch



all in all we're on a good way to get clean here, by far the biggest part that
is left is "externs should be avoided in .c files", those will need some more
careful work.


regards,
   laura lawniczak, johannes schilling



total diffstat:

Johannes Schilling (6):
  keucr: remove some unneccesary typedefs
  keucr: fix some camelCase warnings
  keucr: migrate printk to dev_dbg/info/warn/err
  keucr: make error messages less aggressive
  keucr: fix some alignment- and whitespace-problems
  keucr: use more specific max_t(int, ..

 drivers/staging/keucr/init.c      |   72 +++++++++++-----------
 drivers/staging/keucr/init.h      |    8 +--
 drivers/staging/keucr/scsiglue.c  |    7 ++-
 drivers/staging/keucr/smil.h      |   16 -----
 drivers/staging/keucr/smilmain.c  |  122 ++++++++++++++++++-------------------
 drivers/staging/keucr/smilsub.c   |   14 ++---
 drivers/staging/keucr/smscsi.c    |    6 +-
 drivers/staging/keucr/transport.c |    4 +-
 drivers/staging/keucr/usb.c       |    8 +--
 drivers/staging/keucr/usb.h       |   30 ++++-----
 10 files changed, 132 insertions(+), 155 deletions(-)


Signed-off-by: Laura Lawniczak <laura.lawniczak@xxxxxxxxxxxxxx>
Signed-off-by: Johannes Schilling <of82ecuq@xxxxxxxxxxxxx> 

-- 
1.7.10.4

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/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