Messages in this thread | | | From | Hartley Sweeten <> | Subject | RE: [PATCH 00/14] staging: comedi: daqboard2000: checkpatch clean-ups | Date | Tue, 17 May 2016 17:46:22 +0000 |
| |
On Tuesday, May 17, 2016 2:53 AM, Ian Abbott wrote: > This series of patches to the daqboard2000 driver is mostly to fix the > checkpatch.pl warnings. There is one warning remaining about one of the > `udelay` calls with a parameter of 10 microseconds, but I decided to > leave it alone, as converting it to `usleep_range` could increase > firmware loading time. > > Patches 03 and 06 have checkpatch warnings themselves about CamelCase > issues, but they are not "new" issues, and are resolved by the later > patches in the series. > > 01) staging: comedi: daqboard2000: remove commented out code > 02) staging: comedi: daqboard2000: use usual block comment style > 03) staging: comedi: daqboard2000: CHECK: spaces preferred around that > '*' > 04) staging: comedi: daqboard2000: add blank line after struct > declaration > 05) staging: comedi: daqboard2000: rename serial EEPROM register macros > 06) staging: comedi: daqboard2000: rename register offset macros > 07) staging: comedi: daqboard2000: rename acquisition control register > macros > 08) staging: comedi: daqboard2000: rename acq status register macros > 09) staging: comedi: daqboard2000: redo DAC control register macros > 10) staging: comedi: daqboard2000: redo DAC status macros and fix busy > 11) staging: comedi: daqboard2000: rename trigger control register > macros > 12) staging: comedi: daqboard2000: rename reference DACs register macros > 13) staging: comedi: daqboard2000: rename CamelCase functions > 14) staging: comedi: daqboard2000: prefer usleep_range() > > drivers/staging/comedi/drivers/daqboard2000.c | 376 +++++++++++++------------- > 1 file changed, 188 insertions(+), 188 deletions(-)
Ian,
I had a couple comments on patches 1, 5, 6, and 14 but they are really just nitpicks. If you prefer to leave this series as-is:
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Thanks for going through this one, it's annoyed me...
Hartley
| |