media: cec: seco: Drop pointless include
authorLinus Walleij <linus.walleij@linaro.org>
Tue, 22 Feb 2022 22:23:31 +0000 (23:23 +0100)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Wed, 23 Feb 2022 07:53:09 +0000 (08:53 +0100)
This driver uses GPIO descriptors not the old legacy GPIO
API so stop including <linux/gpio.h>.

Fix a bug using a completely unrelated legacy API flag
GPIOF_IN by switching to the actually desired flag
GPIOD_IN.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/cec/platform/seco/seco-cec.c

index ae138cc..4df5609 100644 (file)
@@ -12,7 +12,6 @@
 #include <linux/delay.h>
 #include <linux/dmi.h>
 #include <linux/gpio/consumer.h>
-#include <linux/gpio.h>
 #include <linux/interrupt.h>
 #include <linux/pci.h>
 #include <linux/platform_device.h>
@@ -551,7 +550,7 @@ static int secocec_acpi_probe(struct secocec_data *sdev)
        struct gpio_desc *gpio;
        int irq = 0;
 
-       gpio = devm_gpiod_get(dev, NULL, GPIOF_IN);
+       gpio = devm_gpiod_get(dev, NULL, GPIOD_IN);
        if (IS_ERR(gpio)) {
                dev_err(dev, "Cannot request interrupt gpio");
                return PTR_ERR(gpio);