iio: imu: st_lsm6dsx: fix checkpatch warning
diff mbox series

Message ID 4a1fefa9f941bf2c350b4a886fd05909e5ba8ea3.1575017118.git.lorenzo@kernel.org
State New
Headers show
Series
  • iio: imu: st_lsm6dsx: fix checkpatch warning
Related show

Commit Message

Lorenzo Bianconi Nov. 29, 2019, 8:46 a.m. UTC
Fix following checkpatch warning:

CHECK: Alignment should match open parenthesis
+static int st_lsm6dsx_read_event(struct iio_dev *iio_dev,
+                                  const struct iio_chan_spec *chan,

Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
---
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

Comments

Jonathan Cameron Dec. 1, 2019, 9 p.m. UTC | #1
On Fri, 29 Nov 2019 10:46:44 +0200
Lorenzo Bianconi <lorenzo@kernel.org> wrote:

> Fix following checkpatch warning:
> 
> CHECK: Alignment should match open parenthesis
> +static int st_lsm6dsx_read_event(struct iio_dev *iio_dev,
> +                                  const struct iio_chan_spec *chan,
Fair enough on this tidy up, but the other bit of reorganization?
Why and why not mention it in the patch description?

Also, why the extra line break?  As far as I can see it can
be just fixed without that and still be under 80 chars.
Right now the only issue is that it's indented 2 spaces too many.

Jonathan


> 
> Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
> ---
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 15 ++++++++-------
>  1 file changed, 8 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
> index a7d40c02ce6b..990643da9f4e 100644
> --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
> +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
> @@ -1607,12 +1607,13 @@ static int st_lsm6dsx_event_setup(struct st_lsm6dsx_hw *hw, int state)
>  					     hw->irq_routing->mask, data);
>  }
>  
> -static int st_lsm6dsx_read_event(struct iio_dev *iio_dev,
> -				   const struct iio_chan_spec *chan,
> -				   enum iio_event_type type,
> -				   enum iio_event_direction dir,
> -				   enum iio_event_info info,
> -				   int *val, int *val2)
> +static int
> +st_lsm6dsx_read_event(struct iio_dev *iio_dev,
> +		      const struct iio_chan_spec *chan,
> +		      enum iio_event_type type,
> +		      enum iio_event_direction dir,
> +		      enum iio_event_info info,
> +		      int *val, int *val2)
>  {
>  	struct st_lsm6dsx_sensor *sensor = iio_priv(iio_dev);
>  	struct st_lsm6dsx_hw *hw = sensor->hw;
> @@ -1620,8 +1621,8 @@ static int st_lsm6dsx_read_event(struct iio_dev *iio_dev,
>  	if (type != IIO_EV_TYPE_THRESH)
>  		return -EINVAL;
>  
> -	*val2 = 0;
>  	*val = hw->event_threshold;
> +	*val2 = 0;
>  
>  	return IIO_VAL_INT;
>  }

Patch
diff mbox series

diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
index a7d40c02ce6b..990643da9f4e 100644
--- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
+++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
@@ -1607,12 +1607,13 @@  static int st_lsm6dsx_event_setup(struct st_lsm6dsx_hw *hw, int state)
 					     hw->irq_routing->mask, data);
 }
 
-static int st_lsm6dsx_read_event(struct iio_dev *iio_dev,
-				   const struct iio_chan_spec *chan,
-				   enum iio_event_type type,
-				   enum iio_event_direction dir,
-				   enum iio_event_info info,
-				   int *val, int *val2)
+static int
+st_lsm6dsx_read_event(struct iio_dev *iio_dev,
+		      const struct iio_chan_spec *chan,
+		      enum iio_event_type type,
+		      enum iio_event_direction dir,
+		      enum iio_event_info info,
+		      int *val, int *val2)
 {
 	struct st_lsm6dsx_sensor *sensor = iio_priv(iio_dev);
 	struct st_lsm6dsx_hw *hw = sensor->hw;
@@ -1620,8 +1621,8 @@  static int st_lsm6dsx_read_event(struct iio_dev *iio_dev,
 	if (type != IIO_EV_TYPE_THRESH)
 		return -EINVAL;
 
-	*val2 = 0;
 	*val = hw->event_threshold;
+	*val2 = 0;
 
 	return IIO_VAL_INT;
 }