Re: [PATCHv2 2/2] iio: accel: kxcjk-1013: add some blank lines for readability

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

 



On Mon, May 4, 2015 at 12:13 PM, Antonio Ospite <ao2@xxxxxx> wrote:
> Some extra blank lines between if checks don't hurt and improve
> readability.
>
> Signed-off-by: Antonio Ospite <ao2@xxxxxx>
> Cc: Jonathan Cameron <jic23@xxxxxxxxxx>
> Cc: Bastien Nocera <hadess@xxxxxxxxxx>
> Cc: linux-iio@xxxxxxxxxxxxxxx
> ---

Looks nice :).

Reviewed-by: Daniel Baluta <daniel.baluta@xxxxxxxxx>

>  drivers/iio/accel/kxcjk-1013.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/drivers/iio/accel/kxcjk-1013.c b/drivers/iio/accel/kxcjk-1013.c
> index 5b7a860..937bfc2 100644
> --- a/drivers/iio/accel/kxcjk-1013.c
> +++ b/drivers/iio/accel/kxcjk-1013.c
> @@ -1147,8 +1147,10 @@ static const char *kxcjk1013_match_acpi_device(struct device *dev,
>         id = acpi_match_device(dev->driver->acpi_match_table, dev);
>         if (!id)
>                 return NULL;
> +
>         if (strcmp(id->id, "SMO8500") == 0)
>                 *is_smo8500_device = true;
> +
>         *chipset = (enum kx_chipset)id->driver_data;
>
>         return dev_name(dev);
> @@ -1163,6 +1165,7 @@ static int kxcjk1013_gpio_probe(struct i2c_client *client,
>
>         if (!client)
>                 return -EINVAL;
> +
>         if (data->is_smo8500_device)
>                 return -ENOTSUPP;
>
> --
> 2.1.4
>
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[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