Re: [PATCH v2 09/17] staging:iio:adc:ad7280a: Trivial comment formatting cleanup

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

 



On 12/05, Jonathan Cameron wrote:
> From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
> 
> IIO uses the
> /*
>  * stuff
>  * more stuff
>  */
> 
> multi-line style, so use that here as well.
> 
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
Reviewed-by: Marcelo Schmitt <marcelo.schmitt1@xxxxxxxxx>

> ---
>  drivers/staging/iio/adc/ad7280a.c | 21 +++++++++++----------
>  1 file changed, 11 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/staging/iio/adc/ad7280a.c b/drivers/staging/iio/adc/ad7280a.c
> index 58bddd07df0c..b186dda03432 100644
> --- a/drivers/staging/iio/adc/ad7280a.c
> +++ b/drivers/staging/iio/adc/ad7280a.c
> @@ -139,9 +139,10 @@ static unsigned int ad7280a_devaddr(unsigned int addr)
>  	       ((addr & 0x10) >> 4);
>  }
>  
> -/* During a read a valid write is mandatory.
> - * So writing to the highest available address (Address 0x1F)
> - * and setting the address all parts bit to 0 is recommended
> +/*
> + * During a read a valid write is mandatory.
> + * So writing to the highest available address (Address 0x1F) and setting the
> + * address all parts bit to 0 is recommended.
>   * So the TXVAL is AD7280A_DEVADDR_ALL + CRC
>   */
>  #define AD7280A_READ_TXVAL	0xF800030A
> @@ -180,7 +181,7 @@ static unsigned char ad7280_calc_crc8(unsigned char *crc_tab, unsigned int val)
>  	crc = crc_tab[val >> 16 & 0xFF];
>  	crc = crc_tab[crc ^ (val >> 8 & 0xFF)];
>  
> -	return  crc ^ (val & 0xFF);
> +	return crc ^ (val & 0xFF);
>  }
>  
>  static int ad7280_check_crc(struct ad7280_state *st, unsigned int val)
> @@ -193,12 +194,12 @@ static int ad7280_check_crc(struct ad7280_state *st, unsigned int val)
>  	return 0;
>  }
>  
> -/* After initiating a conversion sequence we need to wait until the
> - * conversion is done. The delay is typically in the range of 15..30 us
> - * however depending an the number of devices in the daisy chain and the
> - * number of averages taken, conversion delays and acquisition time options
> - * it may take up to 250us, in this case we better sleep instead of busy
> - * wait.
> +/*
> + * After initiating a conversion sequence we need to wait until the conversion
> + * is done. The delay is typically in the range of 15..30us however depending on
> + * the number of devices in the daisy chain, the number of averages taken,
> + * conversion delays and acquisition time options it may take up to 250us, in
> + * this case we better sleep instead of busy wait.
>   */
>  
>  static void ad7280_delay(struct ad7280_state *st)
> -- 
> 2.34.1
> 



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux