Re: [PATCH v2 7/7] iio:light:cm3323: add empty lines for code structure

[Jonathan Cameron <jic23@kernel.org>]

On 17/06/15 23:32, Hartmut Knaack wrote:
> Add some empty lines to visually separate logical structure blocks, as
> after if-blocks or before regular returns.
> 
> Signed-off-by: Hartmut Knaack <knaack.h@gmx.de>
> Reviewed-by: Daniel Baluta <daniel.baluta@intel.com>
Applied.
> ---
>  drivers/iio/light/cm3323.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/iio/light/cm3323.c b/drivers/iio/light/cm3323.c
> index 1fc9f78..0831f06 100644
> --- a/drivers/iio/light/cm3323.c
> +++ b/drivers/iio/light/cm3323.c
> @@ -131,9 +131,11 @@ static int cm3323_set_it_bits(struct cm3323_data *data, int val, int val2)
>  				return ret;
>  
>  			data->reg_conf = reg_conf;
> +
>  			return 0;
>  		}
>  	}
> +
>  	return -EINVAL;
>  }
>  
> @@ -146,6 +148,7 @@ static int cm3323_get_it_bits(struct cm3323_data *data)
>  
>  	if (bits >= ARRAY_SIZE(cm3323_int_time))
>  		return -EINVAL;
> +
>  	return bits;
>  }
>  
> @@ -241,11 +244,13 @@ static int cm3323_probe(struct i2c_client *client,
>  		dev_err(&client->dev, "cm3323 chip init failed\n");
>  		return ret;
>  	}
> +
>  	ret = iio_device_register(indio_dev);
>  	if (ret < 0) {
>  		dev_err(&client->dev, "failed to register iio dev\n");
>  		goto err_init;
>  	}
> +
>  	return 0;
>  err_init:
>  	cm3323_disable(indio_dev);
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in