Re: [PATCH] Staging: comedi: fix coding style issues in poc.c

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

 



You have not understood what Joe said.

On Mon, Feb 24, 2014 at 08:48:47PM +1100, David Roddick wrote:
> Removed unnecessary spaces after function pointer names and corrected multi-line wrap alignment
> 

This should be line wrapped at 72 characters (same rule as email).  In
other words it should take two lines like this:

Removed unnecessary spaces after function pointer names and corrected
multi-line wrap alignment.

> Signed-off-by: David Roddick <dgroddick@xxxxxxxxx>
> ---
>  drivers/staging/comedi/drivers/poc.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/staging/comedi/drivers/poc.c b/drivers/staging/comedi/drivers/poc.c
> index 2ae4ee1..514ccd8 100644
> --- a/drivers/staging/comedi/drivers/poc.c
> +++ b/drivers/staging/comedi/drivers/poc.c
> @@ -36,16 +36,16 @@ Configuration options:
>  struct boarddef_struct {
>  	const char *name;
>  	unsigned int iosize;
> -	int (*setup) (struct comedi_device *);
> +	int (*setup)(struct comedi_device *);
>  	int type;
>  	int n_chan;
>  	int n_bits;
> -	int (*winsn) (struct comedi_device *, struct comedi_subdevice *,
> -		      struct comedi_insn *, unsigned int *);
> -	int (*rinsn) (struct comedi_device *, struct comedi_subdevice *,
> -		      struct comedi_insn *, unsigned int *);
> -	int (*insnbits) (struct comedi_device *, struct comedi_subdevice *,
> -			 struct comedi_insn *, unsigned int *);
> +	int (*winsn)(struct comedi_device *, struct comedi_subdevice *,
> +		struct comedi_insn *, unsigned int *);

This is wrong.  It should be exactly like Joe said:

	int (*winsn)(struct comedi_device *, struct comedi_subdevice *,
		     struct comedi_insn *, unsigned int *);

[tab][tab][space][space][space][space][space]struct comedi_insn *,

Ideally it would have a variable name here, but since that wasn't there
in the original, we would accept the patch without it.  All the data
types are pretty clear what they are already, the only thing which is
not clear is what the last parameter does.

regards,
dan carpenter

_______________________________________________
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