On Sun, Sep 19, 2010 at 07:12:21PM +0100, Maurice Dawson wrote: > This is a patch to the ni_labpc.c file that fixes up multiple please, > no space for starting a line warnings, found by the checkpatch.pl tool > > Signed-off-by: Maurice Dawson <mauricedawson2699@xxxxxxxxx> > --- > drivers/staging/comedi/drivers/ni_labpc.c | 40 ++++++++++++++-------------- > 1 files changed, 20 insertions(+), 20 deletions(-) > > diff --git a/drivers/staging/comedi/drivers/ni_labpc.c b/drivers/staging/comedi/drivers/ni_labpc.c > index 3acf7e6..8d120cd 100644 > --- a/drivers/staging/comedi/drivers/ni_labpc.c > +++ b/drivers/staging/comedi/drivers/ni_labpc.c > @@ -1,21 +1,21 @@ > /* > - comedi/drivers/ni_labpc.c > - Driver for National Instruments Lab-PC series boards and compatibles > - Copyright (C) 2001, 2002, 2003 Frank Mori Hess <fmhess@xxxxxxxxxxxxxxxxxxxxx> > +comedi/drivers/ni_labpc.c > +Driver for National Instruments Lab-PC series boards and compatibles > +Copyright (C) 2001, 2002, 2003 Frank Mori Hess <fmhess@xxxxxxxxxxxxxxxxxxxxx> > > - This program is free software; you can redistribute it and/or modify > - it under the terms of the GNU General Public License as published by > - the Free Software Foundation; either version 2 of the License, or > - (at your option) any later version. > +This program is free software; you can redistribute it and/or modify > +it under the terms of the GNU General Public License as published by > +the Free Software Foundation; either version 2 of the License, or > +(at your option) any later version. > > - This program is distributed in the hope that it will be useful, > - but WITHOUT ANY WARRANTY; without even the implied warranty of > - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > - GNU General Public License for more details. > +This program is distributed in the hope that it will be useful, > +but WITHOUT ANY WARRANTY; without even the implied warranty of > +MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > +GNU General Public License for more details. > > - You should have received a copy of the GNU General Public License > - along with this program; if not, write to the Free Software > - Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. > +You should have received a copy of the GNU General Public License > +along with this program; if not, write to the Free Software > +Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. > > ************************************************************************ > */ > @@ -24,7 +24,7 @@ Driver: ni_labpc > Description: National Instruments Lab-PC (& compatibles) > Author: Frank Mori Hess <fmhess@xxxxxxxxxxxxxxxxxxxxx> > Devices: [National Instruments] Lab-PC-1200 (labpc-1200), > - Lab-PC-1200AI (labpc-1200ai), Lab-PC+ (lab-pc+), PCI-1200 (ni_labpc) > + Lab-PC-1200AI(labpc-1200ai), Lab-PC + (lab-pc+), PCI-1200 (ni_labpc) Why drop the ' 's ? > Status: works > > Tested with lab-pc-1200. For the older Lab-PC+, not all input ranges > @@ -47,13 +47,13 @@ from the eeprom and store them into a file that can be then be used by > comedilib, use the comedi_calibrate program. > > Configuration options - ISA boards: > - [0] - I/O port base address > - [1] - IRQ (optional, required for timed or externally triggered conversions) > - [2] - DMA channel (optional) > +[0] - I/O port base address > +[1] - IRQ(optional, required for timed or externally triggered conversions) > +[2] - DMA channel(optional) same here? Why make that change? Heck, these lines shouldn't be changed at all anyway, right? thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
