On 3/14/2012 9:59 PM, Kevin Hilman wrote:
Benoit Cousson<b-cousson@xxxxxx> writes:
From: Felipe Balbi<balbi@xxxxxx>
With sparse IRQs the driver shouldn't depend at all on
any IRQ values coming from board-file.
Remove every occurences of pdata->irq_base/end.
Well, not quite *every*. :)
Hehe, good catch, thanks.
If the driver isn't going to use those fields anymore, they should also
be removed from the pdata struct too[1]. Also, the remaining board/init
code that is initializing those fields should be removed as well[2], since
it obviously has no effect.
The first patch below should probably be folded into this one, and then
the 2nd patch removes all the users.
We should probably remove all the users before removing the attribute?
It looks like your patches are ordered in the opposite way.
I'll add that in the series and repost the pull request.
Regards,
Benoit
Kevin
[1]
From 70a1833ca20142ef5b6b96dbc92a63c7f8ef9a04 Mon Sep 17 00:00:00 2001
From: Kevin Hilman<khilman@xxxxxx>
Date: Wed, 14 Mar 2012 13:55:31 -0700
Subject: [PATCH 1/2] mfd: twl-core: remove pdata->irq_base/end
Signed-off-by: Kevin Hilman<khilman@xxxxxx>
---
include/linux/i2c/twl.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/linux/i2c/twl.h b/include/linux/i2c/twl.h
index 2463b61..fa76006 100644
--- a/include/linux/i2c/twl.h
+++ b/include/linux/i2c/twl.h
@@ -697,7 +697,6 @@ struct twl4030_audio_data {
};
struct twl4030_platform_data {
- unsigned irq_base, irq_end;
struct twl4030_clock_init_data *clock;
struct twl4030_bci_platform_data *bci;
struct twl4030_gpio_platform_data *gpio;
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
