linux-kernel - Re: [PATCH 1/4] Staging: comedi: Fix WARNING issue in s626.c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 16 May 2016 11:02:02 +0100
From:	Ian Abbott <abbotti@....co.uk>
To:	Ravishankar Karkala Mallikarjunayya <ravishankarkm32@...il.com>,
	hsweeten@...ionengravers.com, gregkh@...uxfoundation.org
Cc:	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/4] Staging: comedi: Fix WARNING issue in s626.c

On 12/05/16 07:59, Ravishankar Karkala Mallikarjunayya wrote:
> This is a patch to the s626.c file that fixes up a Block comments
> issues found by the checkpatch.pl tool.
>
> i.e. Block comments use a trailing */ on a separate line
>
> Signed-off-by: Ravishankar Karkala Mallikarjunayya <ravishankarkm32@...il.com>
> ---
>   drivers/staging/comedi/drivers/s626.c | 30 ++++++++++++++++++++----------
>   1 file changed, 20 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/staging/comedi/drivers/s626.c b/drivers/staging/comedi/drivers/s626.c
> index c5e0863..43ab2e4 100644
> --- a/drivers/staging/comedi/drivers/s626.c
> +++ b/drivers/staging/comedi/drivers/s626.c
> @@ -77,23 +77,30 @@ struct s626_buffer_dma {
>   struct s626_private {
>   	uint8_t ai_cmd_running;		/* ai_cmd is running */
>   	unsigned int ai_sample_timer;	/* time between samples in
> -					 * units of the timer */
> +					 * units of the timer
> +					 */
>   	int ai_convert_count;		/* conversion counter */
>   	unsigned int ai_convert_timer;	/* time between conversion in
> -					 * units of the timer */
> +					 * units of the timer
> +					 */
>   	uint16_t counter_int_enabs;	/* counter interrupt enable mask
> -					 * for MISC2 register */
> +					 * for MISC2 register
> +					 */
>   	uint8_t adc_items;		/* number of items in ADC poll list */
>   	struct s626_buffer_dma rps_buf;	/* DMA buffer used to hold ADC (RPS1)
> -					 * program */
> +					 * program
> +					 */
>   	struct s626_buffer_dma ana_buf;	/* DMA buffer used to receive ADC data
> -					 * and hold DAC data */
> +					 * and hold DAC data
> +					 */
>   	uint32_t *dac_wbuf;		/* pointer to logical adrs of DMA buffer
> -					 * used to hold DAC data */
> +					 * used to hold DAC data
> +					 */
>   	uint16_t dacpol;		/* image of DAC polarity register */
>   	uint8_t trim_setpoint[12];	/* images of TrimDAC setpoints */
>   	uint32_t i2c_adrs;		/* I2C device address for onboard EEPROM
> -					 * (board rev dependent) */
> +					 * (board rev dependent)
> +					 */
>   };
>
>   /* Counter overflow/index event flag masks for RDMISC2. */
> @@ -572,11 +579,14 @@ static int s626_set_dac(struct comedi_device *dev,
>   	 * running after the packet has been sent to the target DAC.
>   	 */
>   	val = 0x0F000000;	/* Continue clock after target DAC data
> -				 * (write to non-existent trimdac). */
> +				 * (write to non-existent trimdac).
> +				 */
>   	val |= 0x00004000;	/* Address the two main dual-DAC devices
> -				 * (TSL's chip select enables target device). */
> +				 * (TSL's chip select enables target device).
> +				 */
>   	val |= ((uint32_t)(chan & 1) << 15);	/* Address the DAC channel
> -						 * within the device. */
> +						 * within the device.
> +						 */
>   	val |= (uint32_t)dacdata;	/* Include DAC setpoint data. */
>   	return s626_send_dac(dev, val);
>   }
>

Thanks!

That's not our usual commenting style, but it will do for now.  (Our 
usual style for "end of line" comments that don't fit on the line is to 
convert them to comments on the preceding lines.)

Reviewed-by: Ian Abbott <abbotti@....co.uk>

-- 
-=( Ian Abbott @ MEV Ltd.    E-mail: <abbotti@....co.uk> )=-
-=(                          Web: http://www.mev.co.uk/  )=-