On Tue, May 10, 2016 at 12:37 PM, Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx> wrote: > @@ -92,7 +93,7 @@ soc_button_device_create(struct platform_device *pdev, > continue; > > gpio = soc_button_lookup_gpio(&pdev->dev, info->acpi_index); > - if (gpio < 0) > + if (gpio_is_valid(gpio)) Shouldn't it be "if (!gpio_is_valid(gpio))" ? -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
