On 04/05/15 11:51, Daniel Baluta wrote: > 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> 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@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- 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
