Home » Mailing list archives » linux-kernel » 2009 » April » 14

[PATCH] Input: bf54x-keys: Remove depreciated IRQF_SAMPLE_RANDOM flag

Previous thread: [PATCH] mtd: CFI: fix bug - bonnie++ error on BF548 NOR flash by Bryan Wu on Thursday, March 5, 2009 - 8:32 pm. (1 message)

Next thread: linux-next: state of the trees by Stephen Rothwell on Thursday, March 5, 2009 - 9:56 pm. (3 messages)
[view original message]
From: Bryan Wu
Subject: [PATCH] Input: bf54x-keys: Fix typo
Date: Thursday, March 5, 2009 - 8:34 pm

From: Michael Hennerich <michael.hennerich@analog.com>

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
Signed-off-by: Bryan Wu <cooloney@kernel.org>
---
 drivers/input/keyboard/bf54x-keys.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/input/keyboard/bf54x-keys.c b/drivers/input/keyboard/bf54x-keys.c
index ee855c5..c6f4e16 100644
--- a/drivers/input/keyboard/bf54x-keys.c
+++ b/drivers/input/keyboard/bf54x-keys.c
@@ -212,7 +212,7 @@ static int __devinit bfin_kpad_probe(struct platform_device *pdev)
 	if (!pdata->debounce_time || pdata->debounce_time > MAX_MULT ||
 	    !pdata->coldrive_time || pdata->coldrive_time > MAX_MULT) {
 		printk(KERN_ERR DRV_NAME
-			": Invalid Debounce/Columdrive Time from pdata\n");
+			": Invalid Debounce/Columndrive Time from platform data\n");
 		bfin_write_KPAD_MSEL(0xFF0);	/* Default MSEL	*/
 	} else {
 		bfin_write_KPAD_MSEL(
-- 
1.5.6.3
--


[ message continues ]
[view original message]
From: Dmitry Torokhov
Subject: Re: [PATCH] Input: bf54x-keys: Fix typo
Date: Friday, March 6, 2009 - 2:38 am

Hi Bryan,


Applied, but lowered severity form error to warning since the driver
is still functional even with bad timing values in platform data.

-- 
Dmitry
--


[ message continues ]
[view original message]
From: Hennerich, Michael
Subject: [PATCH] Input: bf54x-keys: Remove depreciated IRQF_SAMPLE_RANDOM flag
Date: Tuesday, April 14, 2009 - 2:40 am

From: Michael Hennerich <michael.hennerich@analog.com>

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>

diff --git a/drivers/input/keyboard/bf54x-keys.c
b/drivers/input/keyboard/bf54x-keys.c
index c6f4e16..73c4481 100644
--- a/drivers/input/keyboard/bf54x-keys.c
+++ b/drivers/input/keyboard/bf54x-keys.c
@@ -252,7 +252,7 @@ static int __devinit bfin_kpad_probe(struct
platform_device *pdev)
        }

        error = request_irq(bf54x_kpad->irq, bfin_kpad_isr,
-                                IRQF_SAMPLE_RANDOM, DRV_NAME, pdev);
+                                0 , DRV_NAME, pdev);
        if (error) {
                printk(KERN_ERR DRV_NAME
                        ": unable to claim irq %d; error %d\n",

--


[ message continues ]
[view original message]
From: Hennerich, Michael
Subject: [PATCH] Input: ad7877.c and ad7879.c: Remove depreciated IRQF_SAMPLE_RANDOM flag
Date: Tuesday, April 14, 2009 - 3:44 am

From: Michael Hennerich <michael.hennerich@analog.com>

This patch removes depreciated IRQF_SAMPLE_RANDOM flags
from drivers currently queued in Dmitry's input subsystem tree. 

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>

---
 drivers/input/touchscreen/ad7877.c |    4 ++--                       
 drivers/input/touchscreen/ad7879.c |    3 +--                        
 2 files changed, 3 insertions(+), 4 deletions(-)
                     
diff --git a/drivers/input/joystick/bfin_rotary.c
b/drivers/input/joystick/bfin_rotary.c
index c318849..7e91bff 100644

--- a/drivers/input/joystick/bfin_rotary.c

+++ b/drivers/input/joystick/bfin_rotary.c

@@ -138,7 +138,7 @@ static int __devinit bfin_rotary_probe(struct
platform_device *pdev)
                goto out2;

 

        ret = request_irq(rotary->irq, bfin_rotary_isr,

-                                IRQF_SAMPLE_RANDOM,
dev_name(&pdev->dev), pdev);       
+                                0 , dev_name(&pdev->dev), pdev);

        if (ret) {

                dev_err(&pdev->dev,

                        "unable to claim irq %d; error %d\n",

diff --git a/drivers/input/keyboard/adp5588_keys.c
b/drivers/input/keyboard/adp5588_keys.c
index b424ce4..1da7000 100644

--- a/drivers/input/keyboard/adp5588_keys.c

+++ b/drivers/input/keyboard/adp5588_keys.c

@@ -308,7 +308,7 @@ static int __devinit adp5588_probe(struct i2c_client
*client,         
        }

 

        ret = request_irq(client->irq, adp5588_irq, IRQF_TRIGGER_FALLING
|                
-               IRQF_SAMPLE_RANDOM | IRQF_DISABLED,

+               IRQF_DISABLED,

                client->dev.driver->name, kpad);

        if (ret) {

                dev_err(&client->dev, "irq %d busy?\n", client->irq);
--


[ message continues ]
[view original message]
From: Hennerich, Michael
Subject: RE: [PATCH] Input: ad7877.c and ad7879.c: Remove depreciated IRQF_SAMPLE_RANDOM flag
Date: Tuesday, April 14, 2009 - 3:57 am

From: Michael Hennerich <michael.hennerich@analog.com>

Dmitry,

Please ignore my last email.
 
This patch removes depreciated IRQF_SAMPLE_RANDOM flags from drivers
currently queued in Dmitry's input subsystem tree. 

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>

---
 drivers/input/touchscreen/ad7877.c |    4 ++--                       
 drivers/input/touchscreen/ad7879.c |    3 +--                        
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/input/touchscreen/ad7877.c
b/drivers/input/touchscreen/ad7877.c
index 047bb57..dae7357 100644
--- a/drivers/input/touchscreen/ad7877.c
+++ b/drivers/input/touchscreen/ad7877.c
@@ -739,8 +739,8 @@ static int __devinit ad7877_probe(struct spi_device
*spi)
 
 	/* Request AD7877 /DAV GPIO interrupt */
 
-	err = request_irq(spi->irq, ad7877_irq, IRQF_TRIGGER_FALLING |
-			IRQF_SAMPLE_RANDOM, spi->dev.driver->name, ts);
+	err = request_irq(spi->irq, ad7877_irq, IRQF_TRIGGER_FALLING,
+			spi->dev.driver->name, ts);
 	if (err) {
 		dev_dbg(&spi->dev, "irq %d busy?\n", spi->irq);
 		goto err_free_mem;
diff --git a/drivers/input/touchscreen/ad7879.c
b/drivers/input/touchscreen/ad7879.c
index 59fad2f..2eb40ee 100644
--- a/drivers/input/touchscreen/ad7879.c
+++ b/drivers/input/touchscreen/ad7879.c
@@ -448,8 +448,7 @@ static int __devinit ad7879_construct(bus_device
*bus, struct ad7879 *ts)
 	ad7879_setup(ts);
 
 	err = request_irq(bus->irq, ad7879_irq,
-			  IRQF_TRIGGER_FALLING | IRQF_SAMPLE_RANDOM,
-			  bus->dev.driver->name, ts);
+			  IRQF_TRIGGER_FALLING, bus->dev.driver->name,
ts);
 
 	if (err) {
--


[ message continues ]
Previous thread: [PATCH] mtd: CFI: fix bug - bonnie++ error on BF548 NOR flash by Bryan Wu on Thursday, March 5, 2009 - 8:32 pm. (1 message)

Next thread: linux-next: state of the trees by Stephen Rothwell on Thursday, March 5, 2009 - 9:56 pm. (3 messages)