From 5b2c8da1b040c4608c7cfdc0dd96bcaa190af268 Mon Sep 17 00:00:00 2001 From: Hans Verkuil Date: Mon, 25 Sep 2017 05:02:57 -0300 Subject: [PATCH] [media] cec-gpio: don't generate spurious HPD events Only send HPD_LOW/HIGH event if the gpio actually changed value. Signed-off-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab --- drivers/media/platform/cec-gpio/cec-gpio.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/media/platform/cec-gpio/cec-gpio.c b/drivers/media/platform/cec-gpio/cec-gpio.c index eb982bce99fc..5debdf08fbe7 100644 --- a/drivers/media/platform/cec-gpio/cec-gpio.c +++ b/drivers/media/platform/cec-gpio/cec-gpio.c @@ -80,9 +80,12 @@ static irqreturn_t cec_hpd_gpio_irq_handler_thread(int irq, void *priv) static irqreturn_t cec_hpd_gpio_irq_handler(int irq, void *priv) { struct cec_gpio *cec = priv; + bool is_high = gpiod_get_value(cec->hpd_gpio); + if (is_high == cec->hpd_is_high) + return IRQ_HANDLED; cec->hpd_ts = ktime_get(); - cec->hpd_is_high = gpiod_get_value(cec->hpd_gpio); + cec->hpd_is_high = is_high; return IRQ_WAKE_THREAD; } -- 2.11.0