Re: [PATCH v2] Staging: bcm2048: Fix function argument alignment in radio-bcm2048.c.

[Hans Verkuil <hverkuil@xs4all.nl>]

On 02/20/2018 07:53 AM, Quytelda Kahja wrote:
> Fix a coding style problem.

What coding style problem? You should give a short description of
what you are fixing.

> 
> Signed-off-by: Quytelda Kahja <quytelda@tamalin.org>
> ---
> This is the patch without the unnecessary fixes for line length.
> 
>  drivers/staging/media/bcm2048/radio-bcm2048.c | 22 +++++++++++-----------
>  1 file changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c
> index 06d1920150da..f38a4f2acdde 100644
> --- a/drivers/staging/media/bcm2048/radio-bcm2048.c
> +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c
> @@ -1864,7 +1864,7 @@ static int bcm2048_probe(struct bcm2048_device *bdev)
>  		goto unlock;
>  
>  	err = bcm2048_set_fm_search_rssi_threshold(bdev,
> -					BCM2048_DEFAULT_RSSI_THRESHOLD);
> +						   BCM2048_DEFAULT_RSSI_THRESHOLD);
>  	if (err < 0)
>  		goto unlock;
>  

Just drop this change: it will replace one warning (non-aligned) with
another (> 80 cols).

This code is fine as it is.

Regards,

	Hans

> @@ -1942,9 +1942,9 @@ static irqreturn_t bcm2048_handler(int irq, void *dev)
>   */
>  #define property_write(prop, type, mask, check)				\
>  static ssize_t bcm2048_##prop##_write(struct device *dev,		\
> -					struct device_attribute *attr,	\
> -					const char *buf,		\
> -					size_t count)			\
> +				      struct device_attribute *attr,	\
> +				      const char *buf,			\
> +				      size_t count)			\
>  {									\
>  	struct bcm2048_device *bdev = dev_get_drvdata(dev);		\
>  	type value;							\
> @@ -1966,8 +1966,8 @@ static ssize_t bcm2048_##prop##_write(struct device *dev,		\
>  
>  #define property_read(prop, mask)					\
>  static ssize_t bcm2048_##prop##_read(struct device *dev,		\
> -					struct device_attribute *attr,	\
> -					char *buf)			\
> +				     struct device_attribute *attr,	\
> +				     char *buf)				\
>  {									\
>  	struct bcm2048_device *bdev = dev_get_drvdata(dev);		\
>  	int value;							\
> @@ -1985,8 +1985,8 @@ static ssize_t bcm2048_##prop##_read(struct device *dev,		\
>  
>  #define property_signed_read(prop, size, mask)				\
>  static ssize_t bcm2048_##prop##_read(struct device *dev,		\
> -					struct device_attribute *attr,	\
> -					char *buf)			\
> +				     struct device_attribute *attr,	\
> +				     char *buf)				\
>  {									\
>  	struct bcm2048_device *bdev = dev_get_drvdata(dev);		\
>  	size value;							\
> @@ -2005,8 +2005,8 @@ property_read(prop, mask)						\
>  
>  #define property_str_read(prop, size)					\
>  static ssize_t bcm2048_##prop##_read(struct device *dev,		\
> -					struct device_attribute *attr,	\
> -					char *buf)			\
> +				     struct device_attribute *attr,	\
> +				     char *buf)				\
>  {									\
>  	struct bcm2048_device *bdev = dev_get_drvdata(dev);		\
>  	int count;							\
> @@ -2175,7 +2175,7 @@ static int bcm2048_fops_release(struct file *file)
>  }
>  
>  static __poll_t bcm2048_fops_poll(struct file *file,
> -				      struct poll_table_struct *pts)
> +				  struct poll_table_struct *pts)
>  {
>  	struct bcm2048_device *bdev = video_drvdata(file);
>  	__poll_t retval = 0;
>