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

[Jonathan Cameron <jic23@kernel.org>]

On 04/05/15 11:51, Daniel Baluta wrote:
> On Mon, May 4, 2015 at 12:13 PM, Antonio Ospite <ao2@ao2.it> wrote:
>> Some extra blank lines between if checks don't hurt and improve
>> readability.
>>
>> Signed-off-by: Antonio Ospite <ao2@ao2.it>
>> Cc: Jonathan Cameron <jic23@kernel.org>
>> Cc: Bastien Nocera <hadess@hadess.net>
>> Cc: linux-iio@vger.kernel.org
>> ---
> 
> Looks nice :).
> 
> Reviewed-by: Daniel Baluta <daniel.baluta@intel.com>
added to the patch.


> 
>>  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@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>