linux-kernel - [PATCH 1/1] extcon: gpio: queue work only if debounce is NZ

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 8 Mar 2016 17:15:53 +0530
From:	Venkat Reddy Talla <vreddytalla@...dia.com>
To:	MyungJoo Ham <myungjoo.ham@...sung.com>,
	Chanwoo Choi <cw00.choi@...sung.com>
CC:	<linux-kernel@...r.kernel.org>,
	Laxman Dewangan <ldewangan@...dia.com>,
	Venkat Reddy Talla <vreddytalla@...dia.com>
Subject: [PATCH 1/1] extcon: gpio: queue work only if debounce is NZ

Use queue_delayed_work only when debounce time is
required to read gpio state properly, read the gpio
state and set the extcon cable state in IRQ handler
context if gpio settling time is not needed.

Signed-off-by: Venkat Reddy Talla <vreddytalla@...dia.com>
---
 drivers/extcon/extcon-gpio.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/extcon/extcon-gpio.c b/drivers/extcon/extcon-gpio.c
index 279ff8f..1beb086 100644
--- a/drivers/extcon/extcon-gpio.c
+++ b/drivers/extcon/extcon-gpio.c
@@ -56,8 +56,12 @@ static irqreturn_t gpio_irq_handler(int irq, void *dev_id)
 {
 	struct gpio_extcon_data *data = dev_id;
 
-	queue_delayed_work(system_power_efficient_wq, &data->work,
+	if (data->debounce_jiffies)
+		queue_delayed_work(system_power_efficient_wq, &data->work,
 			      data->debounce_jiffies);
+	else
+		gpio_extcon_work(&data->work.work);
+
 	return IRQ_HANDLED;
 }
 
-- 
2.1.4