On Thu, May 13, 2010 at 03:40:32AM -0700, Joe Perches wrote:
> On Thu, 2010-05-13 at 18:28 +0800, Mark wrote:
> > I'd like to get some feedback on this patch, since I'm not sure this is the best
> > way to fix the long line lengths.
> > ---
> > drivers/staging/comedi/comedi.h | 15 ++++++++++-----
> > 1 files changed, 10 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/staging/comedi/comedi.h b/drivers/staging/comedi/comedi.h
> > index 3e3663f..ea96a83 100644
> > --- a/drivers/staging/comedi/comedi.h
> > +++ b/drivers/staging/comedi/comedi.h
> > @@ -267,7 +267,8 @@
> > * streaming input/output */
> > INSN_CONFIG_GET_HARDWARE_BUFFER_SIZE = 2006,
> > INSN_CONFIG_SET_COUNTER_MODE = 4097,
> > - INSN_CONFIG_8254_SET_MODE = INSN_CONFIG_SET_COUNTER_MODE, /* deprecated */
> > + /* deprecated */
>
> /* INSN_CONFIG_8254_SET_MODE is deprecated */
Yes, please change this to be like this.
> > - static inline unsigned NI_CDIO_SCAN_BEGIN_SRC_RTSI(unsigned rtsi_channel)
> > + static inline unsigned
> > + NI_CDIO_SCAN_BEGIN_SRC_RTSI(unsigned rtsi_channel)
> > {
> > return NI_USUAL_RTSI_SELECT(rtsi_channel);
> > }
>
> It'd be better to start the functions in column 1
>
> static inline unsigned int NI_CDIO_SCAN_BEGIN_RTSI(unsigned int rtsi_channel)
> {
> return NI_USUAL_RTSI_SELECT(rtsi_channel);
> }
Hm, yes, that would fix most of the problems in this file. Mark, care
to do this instead?
thanks,
greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
