From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:43899 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753110AbbFUN2r (ORCPT ); Sun, 21 Jun 2015 09:28:47 -0400 Message-ID: <5586BC0C.1040005@kernel.org> Date: Sun, 21 Jun 2015 14:28:44 +0100 From: Jonathan Cameron MIME-Version: 1.0 To: Hartmut Knaack , linux-iio@vger.kernel.org CC: Lars-Peter Clausen , Peter Meerwald , Kevin Tsai , Daniel Baluta Subject: Re: [PATCH v2 7/7] iio:light:cm3323: add empty lines for code structure References: In-Reply-To: Content-Type: text/plain; charset=windows-1252 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.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 > Reviewed-by: Daniel Baluta 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