diff options
| author | Koen Kooi <koen@openembedded.org> | 2008-08-04 16:38:59 +0000 |
|---|---|---|
| committer | Koen Kooi <koen@openembedded.org> | 2008-08-04 16:38:59 +0000 |
| commit | 2d40567554c5bed84dec725cf1f24cc4deb84ce7 (patch) | |
| tree | 6452b673b291068bb56030d9d2432b328306e518 /packages/linux/linux-omap2-git/beagleboard | |
| parent | 814bc470c250b650dbb145d44cc21a1511bad39f (diff) | |
linux omap2 git: sync with upstream git
Diffstat (limited to 'packages/linux/linux-omap2-git/beagleboard')
11 files changed, 118 insertions, 1921 deletions
diff --git a/packages/linux/linux-omap2-git/beagleboard/TWL4030-01.patch b/packages/linux/linux-omap2-git/beagleboard/TWL4030-01.patch deleted file mode 100644 index c361c33d61..0000000000 --- a/packages/linux/linux-omap2-git/beagleboard/TWL4030-01.patch +++ /dev/null @@ -1,43 +0,0 @@ -TWL4030: remove superfluous PWR interrupt status clear before masking - -From: Paul Walmsley <paul@pwsan.com> - -twl_irq_init() clears PWR interrupt status bits, then masks the interrupts -off, then clears the PWR interrupt status bits again. The first clear -seems unnecessary, so, remove it. - -Signed-off-by: Paul Walmsley <paul@pwsan.com> ---- - - drivers/i2c/chips/twl4030-core.c | 18 ------------------ - 1 files changed, 0 insertions(+), 18 deletions(-) - -diff --git a/drivers/i2c/chips/twl4030-core.c b/drivers/i2c/chips/twl4030-core.c -index adc45d4..ff662bc 100644 ---- a/drivers/i2c/chips/twl4030-core.c -+++ b/drivers/i2c/chips/twl4030-core.c -@@ -719,24 +719,6 @@ static void twl_init_irq(void) - char *msg = "Unable to register interrupt subsystem"; - unsigned int irq_num; - -- /* -- * We end up with interrupts from other modules before -- * they get a chance to handle them... -- */ -- /* PWR_ISR1 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xFF, 0x00); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -- -- /* PWR_ISR2 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xFF, 0x02); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -- - /* PWR_IMR1 */ - res = twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xFF, 0x1); - if (res < 0) { diff --git a/packages/linux/linux-omap2-git/beagleboard/TWL4030-02.patch b/packages/linux/linux-omap2-git/beagleboard/TWL4030-02.patch deleted file mode 100644 index 48a59b945b..0000000000 --- a/packages/linux/linux-omap2-git/beagleboard/TWL4030-02.patch +++ /dev/null @@ -1,71 +0,0 @@ -TWL4030: clear TWL GPIO interrupt status registers - -From: Paul Walmsley <paul@pwsan.com> - -twl_init_irq() does not clear the TWL GPIO ISR registers, but the PIH -ISR thinks that it has. This causes any previously-latched GPIO interrupts -to be stuck on until twl4030-gpio.c initializes, often drowning the console in - - TWL4030 module irq 368 is disabled but can't be masked! - -messages. This seems to be a particular problem when booting on Beagle. - -Signed-off-by: Paul Walmsley <paul@pwsan.com> ---- - - drivers/i2c/chips/twl4030-core.c | 42 ++++++++++++++++++++++++++++++++++++++ - 1 files changed, 42 insertions(+), 0 deletions(-) - -diff --git a/drivers/i2c/chips/twl4030-core.c b/drivers/i2c/chips/twl4030-core.c -index ff662bc..dfc3805 100644 ---- a/drivers/i2c/chips/twl4030-core.c -+++ b/drivers/i2c/chips/twl4030-core.c -@@ -857,6 +857,48 @@ static void twl_init_irq(void) - return; - } - -+ /* GPIO_ISR1A */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x19); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ -+ /* GPIO_ISR2A */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1a); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ -+ /* GPIO_ISR3A */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1b); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ -+ /* GPIO_ISR1B */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1f); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ -+ /* GPIO_ISR2B */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x20); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ -+ /* GPIO_ISR3B */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x21); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ - /* install an irq handler for each of the PIH modules */ - for (i = TWL4030_IRQ_BASE; i < TWL4030_IRQ_END; i++) { - set_irq_chip(i, &twl4030_irq_chip); diff --git a/packages/linux/linux-omap2-git/beagleboard/TWL4030-03.patch b/packages/linux/linux-omap2-git/beagleboard/TWL4030-03.patch deleted file mode 100644 index fe1bea5398..0000000000 --- a/packages/linux/linux-omap2-git/beagleboard/TWL4030-03.patch +++ /dev/null @@ -1,82 +0,0 @@ -TWL4030: use correct register addresses for BCI IMR registers - -From: Paul Walmsley <paul@pwsan.com> - -The existing code to mask and clear BCI interrupts in twl_init_irq() is -wrong. It uses the wrong register offsets, it does not mask all of the -BCI IMR registers, and it does not clear all of the BCI ISR registers. - -Signed-off-by: Paul Walmsley <paul@pwsan.com> ---- - - drivers/i2c/chips/twl4030-core.c | 40 ++++++++++++++++++++++++++++++++------ - 1 files changed, 34 insertions(+), 6 deletions(-) - -diff --git a/drivers/i2c/chips/twl4030-core.c b/drivers/i2c/chips/twl4030-core.c -index dfc3805..bb0732c 100644 ---- a/drivers/i2c/chips/twl4030-core.c -+++ b/drivers/i2c/chips/twl4030-core.c -@@ -750,29 +750,57 @@ static void twl_init_irq(void) - /* POWER HACK (END) */ - /* Slave address 0x4A */ - -- /* BCIIMR1_1 */ -+ /* BCIIMR1A */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x2); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ -+ /* BCIIMR2A */ - res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x3); - if (res < 0) { - pr_err("%s[%d][%d]\n", msg, res, __LINE__); - return; - } - -- /* BCIIMR1_2 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x4); -+ /* BCIIMR1B */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x6); - if (res < 0) { - pr_err("%s[%d][%d]\n", msg, res, __LINE__); - return; - } - -- /* BCIIMR2_1 */ -+ /* BCIIMR2B */ - res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x7); - if (res < 0) { - pr_err("%s[%d][%d]\n", msg, res, __LINE__); - return; - } - -- /* BCIIMR2_2 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x8); -+ /* BCIISR1A */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x0); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ -+ /* BCIISR2A */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x1); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ -+ /* BCIISR1B */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x4); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ -+ /* BCIISR2B */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x5); - if (res < 0) { - pr_err("%s[%d][%d]\n", msg, res, __LINE__); - return; diff --git a/packages/linux/linux-omap2-git/beagleboard/TWL4030-04.patch b/packages/linux/linux-omap2-git/beagleboard/TWL4030-04.patch deleted file mode 100644 index fb65ac98bb..0000000000 --- a/packages/linux/linux-omap2-git/beagleboard/TWL4030-04.patch +++ /dev/null @@ -1,38 +0,0 @@ -TWL4030: clear MADC interrupt status registers upon init - -From: Paul Walmsley <paul@pwsan.com> - -twl_init_irq() does not clear MADC interrupt status registers upon init - -fix. - -Signed-off-by: Paul Walmsley <paul@pwsan.com> ---- - - drivers/i2c/chips/twl4030-core.c | 14 ++++++++++++++ - 1 files changed, 14 insertions(+), 0 deletions(-) - -diff --git a/drivers/i2c/chips/twl4030-core.c b/drivers/i2c/chips/twl4030-core.c -index bb0732c..9d93524 100644 ---- a/drivers/i2c/chips/twl4030-core.c -+++ b/drivers/i2c/chips/twl4030-core.c -@@ -821,6 +821,20 @@ static void twl_init_irq(void) - return; - } - -+ /* MADC_ISR1 */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xFF, 0x61); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ -+ /* MADC_ISR2 */ -+ res = twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xFF, 0x63); -+ if (res < 0) { -+ pr_err("%s[%d][%d]\n", msg, res, __LINE__); -+ return; -+ } -+ - /* key Pad */ - /* KEYPAD - IMR1 */ - res = twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xFF, (0x12)); diff --git a/packages/linux/linux-omap2-git/beagleboard/TWL4030-05.patch b/packages/linux/linux-omap2-git/beagleboard/TWL4030-05.patch deleted file mode 100644 index 02a72ed9df..0000000000 --- a/packages/linux/linux-omap2-git/beagleboard/TWL4030-05.patch +++ /dev/null @@ -1,303 +0,0 @@ -TWL4030: use *_SIH_CTRL.COR bit to determine whether to read or write ISR to clear - -From: Paul Walmsley <paul@pwsan.com> - -TWL4030 interrupt status register bits can be cleared in one of two ways: -either by reading from the register, or by writing a 1 to the -appropriate bit(s) in the register. This behavior can be altered at any -time by the <twlmodule>_SIH_CTRL.COR register bit ("clear-on-read"). - -The TWL4030 TRM is deeply confused as to whether COR=1 means that the -registers are cleared on reads, or cleared on writes. Peter De -Schrijver <peter.de-schrijver> confirms that COR=1 means that the registers -are cleared on read. - -So, for each TWL4030 SIH, check the value of the *_SIH_CTRL.COR bit, and if -it is 1, use reads to clear the ISRs; if it is 0, use writes. - -Also, use WARN_ON() to warn if the read/write failed, and don't skip -the rest of the initialization on failure either. - -Thanks to Peter for his help with this patch. - - -Signed-off-by: Paul Walmsley <paul@pwsan.com> ---- - - drivers/i2c/chips/twl4030-core.c | 183 ++++++++++++++++++++++---------------- - 1 files changed, 106 insertions(+), 77 deletions(-) - -diff --git a/drivers/i2c/chips/twl4030-core.c b/drivers/i2c/chips/twl4030-core.c -index 9d93524..eae0634 100644 ---- a/drivers/i2c/chips/twl4030-core.c -+++ b/drivers/i2c/chips/twl4030-core.c -@@ -133,6 +133,16 @@ - /* on I2C-1 for 2430SDP */ - #define CONFIG_I2C_TWL4030_ID 1 - -+/* SIH_CTRL registers */ -+#define TWL4030_INT_PWR_SIH_CTRL 0x07 -+#define TWL4030_INTERRUPTS_BCISIHCTRL 0x0d -+#define TWL4030_MADC_MADC_SIH_CTRL 0x67 -+#define TWL4030_KEYPAD_KEYP_SIH_CTRL 0x17 -+#define TWL4030_GPIO_GPIO_SIH_CTRL 0x2d -+ -+#define TWL4030_SIH_CTRL_COR_MASK (1 << 2) -+ -+ - /* Helper functions */ - static int - twl4030_detect_client(struct i2c_adapter *adapter, unsigned char sid); -@@ -712,13 +722,61 @@ static int power_companion_init(void) - return e; - } - -+/** -+ * twl4030_i2c_clear_isr - clear TWL4030 SIH ISR regs via read + write -+ * @mod_no: TWL4030 module number -+ * @reg: register index to clear -+ * @cor: value of the <module>_SIH_CTRL.COR bit (1 or 0) -+ * -+ * Either reads (cor == 1) or writes (cor == 0) to a TWL4030 interrupt -+ * status register to ensure that any prior interrupts are cleared. -+ * Returns the status from the I2C read operation. -+ */ -+static int twl4030_i2c_clear_isr(u8 mod_no, u8 reg, u8 cor) -+{ -+ u8 tmp; -+ -+ return (cor) ? twl4030_i2c_read_u8(mod_no, &tmp, reg) : -+ twl4030_i2c_write_u8(mod_no, 0xff, reg); -+} -+ -+/** -+ * twl4030_read_cor_bit - are TWL module ISRs cleared by reads or writes? -+ * @mod_no: TWL4030 module number -+ * @reg: register index to clear -+ * -+ * Returns 1 if the TWL4030 SIH interrupt status registers (ISRs) for -+ * the specified TWL module are cleared by reads, or 0 if cleared by -+ * writes. -+ */ -+static int twl4030_read_cor_bit(u8 mod_no, u8 reg) -+{ -+ u8 tmp = 0; -+ -+ WARN_ON(twl4030_i2c_read_u8(mod_no, &tmp, reg) < 0); -+ -+ tmp &= TWL4030_SIH_CTRL_COR_MASK; -+ tmp >>= __ffs(TWL4030_SIH_CTRL_COR_MASK); -+ -+ return tmp; -+} -+ - static void twl_init_irq(void) - { - int i = 0; - int res = 0; -+ int cor; - char *msg = "Unable to register interrupt subsystem"; - unsigned int irq_num; - -+ /* -+ * For each TWL4030 module with ISR/IMR registers, mask all -+ * interrupts and then clear any existing interrupt status bits, -+ * since we initially do not have any TWL4030 module interrupt -+ * handlers present. -+ */ -+ -+ - /* PWR_IMR1 */ - res = twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xFF, 0x1); - if (res < 0) { -@@ -734,20 +792,18 @@ static void twl_init_irq(void) - } - - /* Clear off any other pending interrupts on power */ -+ -+ /* Are PWR interrupt status bits cleared by reads or writes? */ -+ cor = twl4030_read_cor_bit(TWL4030_MODULE_INT, -+ TWL4030_INT_PWR_SIH_CTRL); -+ WARN_ON(cor < 0); -+ - /* PWR_ISR1 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xFF, 0x00); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INT, 0x00, cor) < 0); - - /* PWR_ISR2 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xFF, 0x02); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -- /* POWER HACK (END) */ -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INT, 0x02, cor) < 0); -+ - /* Slave address 0x4A */ - - /* BCIIMR1A */ -@@ -778,33 +834,22 @@ static void twl_init_irq(void) - return; - } - -+ /* Are BCI interrupt status bits cleared by reads or writes? */ -+ cor = twl4030_read_cor_bit(TWL4030_MODULE_INTERRUPTS, -+ TWL4030_INTERRUPTS_BCISIHCTRL); -+ WARN_ON(cor < 0); -+ - /* BCIISR1A */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x0); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, 0x0, cor) < 0); - - /* BCIISR2A */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x1); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, 0x1, cor) < 0); - - /* BCIISR1B */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x4); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, 0x4, cor) < 0); - - /* BCIISR2B */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x5); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, 0x5, cor) < 0); - - /* MAD C */ - /* MADC_IMR1 */ -@@ -821,19 +866,16 @@ static void twl_init_irq(void) - return; - } - -+ /* Are MADC interrupt status bits cleared by reads or writes? */ -+ cor = twl4030_read_cor_bit(TWL4030_MODULE_MADC, -+ TWL4030_MADC_MADC_SIH_CTRL); -+ WARN_ON(cor < 0); -+ - /* MADC_ISR1 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xFF, 0x61); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_MADC, 0x61, cor) < 0); - - /* MADC_ISR2 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xFF, 0x63); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_MADC, 0x63, cor) < 0); - - /* key Pad */ - /* KEYPAD - IMR1 */ -@@ -842,12 +884,15 @@ static void twl_init_irq(void) - pr_err("%s[%d][%d]\n", msg, res, __LINE__); - return; - } -- { -- u8 clear; -- /* Clear ISR */ -- twl4030_i2c_read_u8(TWL4030_MODULE_KEYPAD, &clear, 0x11); -- twl4030_i2c_read_u8(TWL4030_MODULE_KEYPAD, &clear, 0x11); -- } -+ -+ /* Are keypad interrupt status bits cleared by reads or writes? */ -+ cor = twl4030_read_cor_bit(TWL4030_MODULE_KEYPAD, -+ TWL4030_KEYPAD_KEYP_SIH_CTRL); -+ WARN_ON(cor < 0); -+ -+ /* KEYPAD - ISR1 */ -+ /* XXX does this still need to be done twice for some reason? */ -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_KEYPAD, 0x11, cor) < 0); - - /* KEYPAD - IMR2 */ - res = twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xFF, (0x14)); -@@ -856,6 +901,9 @@ static void twl_init_irq(void) - return; - } - -+ /* KEYPAD - ISR2 */ -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_KEYPAD, 0x13, cor) < 0); -+ - /* Slave address 0x49 */ - /* GPIO_IMR1A */ - res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xFF, (0x1C)); -@@ -899,47 +947,28 @@ static void twl_init_irq(void) - return; - } - -+ /* Are GPIO interrupt status bits cleared by reads or writes? */ -+ cor = twl4030_read_cor_bit(TWL4030_MODULE_GPIO, -+ TWL4030_GPIO_GPIO_SIH_CTRL); -+ WARN_ON(cor < 0); -+ - /* GPIO_ISR1A */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x19); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x19, cor) < 0); - - /* GPIO_ISR2A */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1a); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x1a, cor) < 0); - - /* GPIO_ISR3A */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1b); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x1b, cor) < 0); - - /* GPIO_ISR1B */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1f); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x1f, cor) < 0); - - /* GPIO_ISR2B */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x20); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x20, cor) < 0); - - /* GPIO_ISR3B */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x21); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x21, cor) < 0); - - /* install an irq handler for each of the PIH modules */ - for (i = TWL4030_IRQ_BASE; i < TWL4030_IRQ_END; i++) { diff --git a/packages/linux/linux-omap2-git/beagleboard/TWL4030-06.patch b/packages/linux/linux-omap2-git/beagleboard/TWL4030-06.patch deleted file mode 100644 index 67b837c454..0000000000 --- a/packages/linux/linux-omap2-git/beagleboard/TWL4030-06.patch +++ /dev/null @@ -1,179 +0,0 @@ -TWL4030: change init-time IMR mask code to WARN if error - -From: Paul Walmsley <paul@pwsan.com> - -twl_init_irq() prints error messages and returns if any interrupt mask -register writes fail. Change this to generate a warning traceback and -to continue execution rather than skipping TWL init. (These mask -writes should not fail at all unless either the I2C bus or the TWL4030 -is somehow wedged.) - -Signed-off-by: Paul Walmsley <paul@pwsan.com> ---- - - drivers/i2c/chips/twl4030-core.c | 100 +++++++------------------------------- - 1 files changed, 18 insertions(+), 82 deletions(-) - -diff --git a/drivers/i2c/chips/twl4030-core.c b/drivers/i2c/chips/twl4030-core.c -index eae0634..99cc143 100644 ---- a/drivers/i2c/chips/twl4030-core.c -+++ b/drivers/i2c/chips/twl4030-core.c -@@ -778,18 +778,10 @@ static void twl_init_irq(void) - - - /* PWR_IMR1 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xFF, 0x1); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xff, 0x1) < 0); - - /* PWR_IMR2 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xFF, 0x3); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xff, 0x3) < 0); - - /* Clear off any other pending interrupts on power */ - -@@ -807,32 +799,16 @@ static void twl_init_irq(void) - /* Slave address 0x4A */ - - /* BCIIMR1A */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x2); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, 0x2) < 0); - -- /* BCIIMR2A */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x3); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ /* BCIIMR2A */ -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, 0x3) < 0); - -- /* BCIIMR1B */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x6); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ /* BCIIMR2A */ -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, 0x6) < 0); - - /* BCIIMR2B */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xFF, 0x7); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, 0x7) < 0); - - /* Are BCI interrupt status bits cleared by reads or writes? */ - cor = twl4030_read_cor_bit(TWL4030_MODULE_INTERRUPTS, -@@ -853,18 +829,10 @@ static void twl_init_irq(void) - - /* MAD C */ - /* MADC_IMR1 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xFF, 0x62); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xff, 0x62) < 0); - - /* MADC_IMR2 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xFF, 0x64); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xff, 0x64) < 0); - - /* Are MADC interrupt status bits cleared by reads or writes? */ - cor = twl4030_read_cor_bit(TWL4030_MODULE_MADC, -@@ -879,11 +847,7 @@ static void twl_init_irq(void) - - /* key Pad */ - /* KEYPAD - IMR1 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xFF, (0x12)); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xff, 0x12) < 0); - - /* Are keypad interrupt status bits cleared by reads or writes? */ - cor = twl4030_read_cor_bit(TWL4030_MODULE_KEYPAD, -@@ -895,57 +859,29 @@ static void twl_init_irq(void) - WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_KEYPAD, 0x11, cor) < 0); - - /* KEYPAD - IMR2 */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xFF, (0x14)); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xff, 0x14) < 0); - - /* KEYPAD - ISR2 */ - WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_KEYPAD, 0x13, cor) < 0); - - /* Slave address 0x49 */ - /* GPIO_IMR1A */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xFF, (0x1C)); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1c) < 0); - - /* GPIO_IMR2A */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xFF, (0x1D)); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1d) < 0); - - /* GPIO_IMR3A */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xFF, (0x1E)); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1e) < 0); - - /* GPIO_IMR1B */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xFF, (0x22)); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x22) < 0); - - /* GPIO_IMR2B */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xFF, (0x23)); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x23) < 0); - - /* GPIO_IMR3B */ -- res = twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xFF, (0x24)); -- if (res < 0) { -- pr_err("%s[%d][%d]\n", msg, res, __LINE__); -- return; -- } -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x24) < 0); - - /* Are GPIO interrupt status bits cleared by reads or writes? */ - cor = twl4030_read_cor_bit(TWL4030_MODULE_GPIO, diff --git a/packages/linux/linux-omap2-git/beagleboard/TWL4030-07.patch b/packages/linux/linux-omap2-git/beagleboard/TWL4030-07.patch deleted file mode 100644 index 8e4c4d6be4..0000000000 --- a/packages/linux/linux-omap2-git/beagleboard/TWL4030-07.patch +++ /dev/null @@ -1,274 +0,0 @@ -TWL4030: move TWL module register defs into separate include files - -From: Paul Walmsley <paul@pwsan.com> - -twl_init_irq() uses "magic numbers" to access TWL module IMR and ISR -registers. Symbolic constants are definitely preferred. - -Rather than duplicating already existing symbolic constants in -twl4030-gpio.c and twl4030-pwrirq.c, move the existing constants out -into include files. This patch should not change kernel behavior. - -Signed-off-by: Paul Walmsley <paul@pwsan.com> ---- - - drivers/i2c/chips/twl4030-gpio.c | 48 ----------------------- - drivers/i2c/chips/twl4030-pwrirq.c | 15 +++---- - include/linux/i2c/twl4030-gpio.h | 76 ++++++++++++++++++++++++++++++++++++ - include/linux/i2c/twl4030-pwrirq.h | 37 ++++++++++++++++++ - 4 files changed, 121 insertions(+), 55 deletions(-) - create mode 100644 include/linux/i2c/twl4030-gpio.h - create mode 100644 include/linux/i2c/twl4030-pwrirq.h - -diff --git a/drivers/i2c/chips/twl4030-gpio.c b/drivers/i2c/chips/twl4030-gpio.c -index f16a48b..9d17f45 100644 ---- a/drivers/i2c/chips/twl4030-gpio.c -+++ b/drivers/i2c/chips/twl4030-gpio.c -@@ -38,6 +38,7 @@ - - #include <linux/i2c.h> - #include <linux/i2c/twl4030.h> -+#include <linux/i2c/twl4030-gpio.h> - #include <linux/slab.h> - - #include <asm/arch/irqs.h> -@@ -47,53 +48,6 @@ - - #include <linux/device.h> - --/* -- * GPIO Block Register definitions -- */ -- --#define REG_GPIODATAIN1 0x0 --#define REG_GPIODATAIN2 0x1 --#define REG_GPIODATAIN3 0x2 --#define REG_GPIODATADIR1 0x3 --#define REG_GPIODATADIR2 0x4 --#define REG_GPIODATADIR3 0x5 --#define REG_GPIODATAOUT1 0x6 --#define REG_GPIODATAOUT2 0x7 --#define REG_GPIODATAOUT3 0x8 --#define REG_CLEARGPIODATAOUT1 0x9 --#define REG_CLEARGPIODATAOUT2 0xA --#define REG_CLEARGPIODATAOUT3 0xB --#define REG_SETGPIODATAOUT1 0xC --#define REG_SETGPIODATAOUT2 0xD --#define REG_SETGPIODATAOUT3 0xE --#define REG_GPIO_DEBEN1 0xF --#define REG_GPIO_DEBEN2 0x10 --#define REG_GPIO_DEBEN3 0x11 --#define REG_GPIO_CTRL 0x12 --#define REG_GPIOPUPDCTR1 0x13 --#define REG_GPIOPUPDCTR2 0x14 --#define REG_GPIOPUPDCTR3 0x15 --#define REG_GPIOPUPDCTR4 0x16 --#define REG_GPIOPUPDCTR5 0x17 --#define REG_GPIO_ISR1A 0x19 --#define REG_GPIO_ISR2A 0x1A --#define REG_GPIO_ISR3A 0x1B --#define REG_GPIO_IMR1A 0x1C --#define REG_GPIO_IMR2A 0x1D --#define REG_GPIO_IMR3A 0x1E --#define REG_GPIO_ISR1B 0x1F --#define REG_GPIO_ISR2B 0x20 --#define REG_GPIO_ISR3B 0x21 --#define REG_GPIO_IMR1B 0x22 --#define REG_GPIO_IMR2B 0x23 --#define REG_GPIO_IMR3B 0x24 --#define REG_GPIO_EDR1 0x28 --#define REG_GPIO_EDR2 0x29 --#define REG_GPIO_EDR3 0x2A --#define REG_GPIO_EDR4 0x2B --#define REG_GPIO_EDR5 0x2C --#define REG_GPIO_SIH_CTRL 0x2D -- - /* BitField Definitions */ - - /* Data banks : 3 banks for 8 gpios each */ -diff --git a/drivers/i2c/chips/twl4030-pwrirq.c b/drivers/i2c/chips/twl4030-pwrirq.c -index a4d2e92..1afdb65 100644 ---- a/drivers/i2c/chips/twl4030-pwrirq.c -+++ b/drivers/i2c/chips/twl4030-pwrirq.c -@@ -27,10 +27,8 @@ - #include <linux/random.h> - #include <linux/kthread.h> - #include <linux/i2c/twl4030.h> -+#include <linux/i2c/twl4030-pwrirq.h> - --#define PWR_ISR1 0 --#define PWR_IMR1 1 --#define PWR_SIH_CTRL 7 - #define PWR_SIH_CTRL_COR (1<<2) - - static u8 twl4030_pwrirq_mask; -@@ -93,7 +91,8 @@ static void do_twl4030_pwrmodule_irq(unsigned int irq, irq_desc_t *desc) - twl4030_pwrirq_mask |= 1 << (irq - TWL4030_PWR_IRQ_BASE); - local_irq_enable(); - twl4030_i2c_write_u8(TWL4030_MODULE_INT, -- twl4030_pwrirq_mask, PWR_IMR1); -+ twl4030_pwrirq_mask, -+ TWL4030_INT_PWR_IMR1); - } - } - } -@@ -115,7 +114,7 @@ static void do_twl4030_pwrirq(unsigned int irq, irq_desc_t *desc) - - local_irq_enable(); - ret = twl4030_i2c_read_u8(TWL4030_MODULE_INT, &pwr_isr, -- PWR_ISR1); -+ TWL4030_INT_PWR_ISR1); - if (ret) { - printk(KERN_WARNING - "I2C error %d while reading TWL4030" -@@ -151,7 +150,7 @@ static int twl4030_pwrirq_thread(void *data) - twl4030_pwrirq_mask &= ~local_unmask; - - twl4030_i2c_write_u8(TWL4030_MODULE_INT, twl4030_pwrirq_mask, -- PWR_IMR1); -+ TWL4030_INT_PWR_IMR1); - - local_irq_disable(); - if (!twl4030_pwrirq_pending_unmask) -@@ -172,14 +171,14 @@ static int __init twl4030_pwrirq_init(void) - twl4030_pwrirq_pending_unmask = 0; - - err = twl4030_i2c_write_u8(TWL4030_MODULE_INT, twl4030_pwrirq_mask, -- PWR_IMR1); -+ TWL4030_INT_PWR_IMR1); - if (err) - return err; - - /* Enable clear on read */ - - err = twl4030_i2c_write_u8(TWL4030_MODULE_INT, PWR_SIH_CTRL_COR, -- PWR_SIH_CTRL); -+ TWL4030_INT_PWR_SIH_CTRL); - if (err) - return err; - -diff --git a/include/linux/i2c/twl4030-gpio.h b/include/linux/i2c/twl4030-gpio.h -new file mode 100644 -index 0000000..7cbf610 ---- /dev/null -+++ b/include/linux/i2c/twl4030-gpio.h -@@ -0,0 +1,76 @@ -+/* -+ * twl4030-gpio.h - header for TWL4030 GPIO module -+ * -+ * Copyright (C) 2005-2006, 2008 Texas Instruments, Inc. -+ * Copyright (C) 2008 Nokia Corporation -+ * -+ * Based on tlv320aic23.c: -+ * Copyright (c) by Kai Svahn <kai.svahn@nokia.com> -+ * -+ * This program is free software; you can redistribute it and/or modify -+ * it under the terms of the GNU General Public License as published by -+ * the Free Software Foundation; either version 2 of the License, or -+ * (at your option) any later version. -+ * -+ * This program is distributed in the hope that it will be useful, -+ * but WITHOUT ANY WARRANTY; without even the implied warranty of -+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -+ * GNU General Public License for more details. -+ * -+ * You should have received a copy of the GNU General Public License -+ * along with this program; if not, write to the Free Software -+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA -+ * -+ */ -+ -+#ifndef __TWL4030_GPIO_H_ -+#define __TWL4030_GPIO_H_ -+ -+/* -+ * GPIO Block Register definitions -+ */ -+ -+#define REG_GPIODATAIN1 0x0 -+#define REG_GPIODATAIN2 0x1 -+#define REG_GPIODATAIN3 0x2 -+#define REG_GPIODATADIR1 0x3 -+#define REG_GPIODATADIR2 0x4 -+#define REG_GPIODATADIR3 0x5 -+#define REG_GPIODATAOUT1 0x6 -+#define REG_GPIODATAOUT2 0x7 -+#define REG_GPIODATAOUT3 0x8 -+#define REG_CLEARGPIODATAOUT1 0x9 -+#define REG_CLEARGPIODATAOUT2 0xA -+#define REG_CLEARGPIODATAOUT3 0xB -+#define REG_SETGPIODATAOUT1 0xC -+#define REG_SETGPIODATAOUT2 0xD -+#define REG_SETGPIODATAOUT3 0xE -+#define REG_GPIO_DEBEN1 0xF -+#define REG_GPIO_DEBEN2 0x10 -+#define REG_GPIO_DEBEN3 0x11 -+#define REG_GPIO_CTRL 0x12 -+#define REG_GPIOPUPDCTR1 0x13 -+#define REG_GPIOPUPDCTR2 0x14 -+#define REG_GPIOPUPDCTR3 0x15 -+#define REG_GPIOPUPDCTR4 0x16 -+#define REG_GPIOPUPDCTR5 0x17 -+#define REG_GPIO_ISR1A 0x19 -+#define REG_GPIO_ISR2A 0x1A -+#define REG_GPIO_ISR3A 0x1B -+#define REG_GPIO_IMR1A 0x1C -+#define REG_GPIO_IMR2A 0x1D -+#define REG_GPIO_IMR3A 0x1E -+#define REG_GPIO_ISR1B 0x1F -+#define REG_GPIO_ISR2B 0x20 -+#define REG_GPIO_ISR3B 0x21 -+#define REG_GPIO_IMR1B 0x22 -+#define REG_GPIO_IMR2B 0x23 -+#define REG_GPIO_IMR3B 0x24 -+#define REG_GPIO_EDR1 0x28 -+#define REG_GPIO_EDR2 0x29 -+#define REG_GPIO_EDR3 0x2A -+#define REG_GPIO_EDR4 0x2B -+#define REG_GPIO_EDR5 0x2C -+#define REG_GPIO_SIH_CTRL 0x2D -+ -+#endif /* End of __TWL4030_GPIO_H */ -diff --git a/include/linux/i2c/twl4030-pwrirq.h b/include/linux/i2c/twl4030-pwrirq.h -new file mode 100644 -index 0000000..7a13368 ---- /dev/null -+++ b/include/linux/i2c/twl4030-pwrirq.h -@@ -0,0 +1,37 @@ -+/* -+ * twl4030-gpio.h - header for TWL4030 GPIO module -+ * -+ * Copyright (C) 2008 Texas Instruments, Inc. -+ * Copyright (C) 2008 Nokia Corporation -+ * -+ * This program is free software; you can redistribute it and/or modify -+ * it under the terms of the GNU General Public License as published by -+ * the Free Software Foundation; either version 2 of the License, or -+ * (at your option) any later version. -+ * -+ * This program is distributed in the hope that it will be useful, -+ * but WITHOUT ANY WARRANTY; without even the implied warranty of -+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -+ * GNU General Public License for more details. -+ * -+ * You should have received a copy of the GNU General Public License -+ * along with this program; if not, write to the Free Software -+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA -+ * -+ */ -+ -+#ifndef __TWL4030_PWRIRQ_H_ -+#define __TWL4030_PWRIRQ_H_ -+ -+/* -+ * INT Module Register definitions -+ * (not all registers are defined below) -+ */ -+ -+#define TWL4030_INT_PWR_ISR1 0x0 -+#define TWL4030_INT_PWR_IMR1 0x1 -+#define TWL4030_INT_PWR_ISR2 0x2 -+#define TWL4030_INT_PWR_IMR2 0x3 -+#define TWL4030_INT_PWR_SIH_CTRL 0x7 -+ -+#endif /* End of __TWL4030_PWRIRQ_H */ diff --git a/packages/linux/linux-omap2-git/beagleboard/TWL4030-08.patch b/packages/linux/linux-omap2-git/beagleboard/TWL4030-08.patch deleted file mode 100644 index 9af25a762d..0000000000 --- a/packages/linux/linux-omap2-git/beagleboard/TWL4030-08.patch +++ /dev/null @@ -1,278 +0,0 @@ -TWL4030: use symbolic ISR/IMR register names during twl_init_irq() - -From: Paul Walmsley <paul@pwsan.com> - -twl_init_irq() uses a bunch of magic numbers as register indices; this -has already led to several errors, fixed earlier in this patch series. -Now use descriptive macros instead of magic numbers. This patch should -not change kernel behavior. - -Signed-off-by: Paul Walmsley <paul@pwsan.com> ---- - - drivers/i2c/chips/twl4030-core.c | 188 +++++++++++++++++++------------------- - 1 files changed, 96 insertions(+), 92 deletions(-) - -diff --git a/drivers/i2c/chips/twl4030-core.c b/drivers/i2c/chips/twl4030-core.c -index 99cc143..38c227a 100644 ---- a/drivers/i2c/chips/twl4030-core.c -+++ b/drivers/i2c/chips/twl4030-core.c -@@ -40,6 +40,9 @@ - - #include <linux/i2c.h> - #include <linux/i2c/twl4030.h> -+#include <linux/i2c/twl4030-gpio.h> -+#include <linux/i2c/twl4030-madc.h> -+#include <linux/i2c/twl4030-pwrirq.h> - #include <linux/slab.h> - #include <linux/clk.h> - #include <linux/device.h> -@@ -114,6 +117,23 @@ - #define TWL4030_BASEADD_RTC 0x001C - #define TWL4030_BASEADD_SECURED_REG 0x0000 - -+/* TWL4030 BCI registers */ -+#define TWL4030_INTERRUPTS_BCIIMR1A 0x2 -+#define TWL4030_INTERRUPTS_BCIIMR2A 0x3 -+#define TWL4030_INTERRUPTS_BCIIMR1B 0x6 -+#define TWL4030_INTERRUPTS_BCIIMR2B 0x7 -+#define TWL4030_INTERRUPTS_BCIISR1A 0x0 -+#define TWL4030_INTERRUPTS_BCIISR2A 0x1 -+#define TWL4030_INTERRUPTS_BCIISR1B 0x4 -+#define TWL4030_INTERRUPTS_BCIISR2B 0x5 -+ -+/* TWL4030 keypad registers */ -+#define TWL4030_KEYPAD_KEYP_IMR1 0x12 -+#define TWL4030_KEYPAD_KEYP_IMR2 0x14 -+#define TWL4030_KEYPAD_KEYP_ISR1 0x11 -+#define TWL4030_KEYPAD_KEYP_ISR2 0x13 -+ -+ - /* Triton Core internal information (END) */ - - /* Few power values */ -@@ -133,12 +153,10 @@ - /* on I2C-1 for 2430SDP */ - #define CONFIG_I2C_TWL4030_ID 1 - --/* SIH_CTRL registers */ --#define TWL4030_INT_PWR_SIH_CTRL 0x07 -+/* SIH_CTRL registers that aren't defined elsewhere */ - #define TWL4030_INTERRUPTS_BCISIHCTRL 0x0d - #define TWL4030_MADC_MADC_SIH_CTRL 0x67 - #define TWL4030_KEYPAD_KEYP_SIH_CTRL 0x17 --#define TWL4030_GPIO_GPIO_SIH_CTRL 0x2d - - #define TWL4030_SIH_CTRL_COR_MASK (1 << 2) - -@@ -776,135 +794,121 @@ static void twl_init_irq(void) - * handlers present. - */ - -- -- /* PWR_IMR1 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xff, 0x1) < 0); -- -- /* PWR_IMR2 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xff, 0x3) < 0); -- -- /* Clear off any other pending interrupts on power */ -+ /* Mask INT (PWR) interrupts at TWL4030 */ -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xff, -+ TWL4030_INT_PWR_IMR1) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xff, -+ TWL4030_INT_PWR_IMR2) < 0); - - /* Are PWR interrupt status bits cleared by reads or writes? */ - cor = twl4030_read_cor_bit(TWL4030_MODULE_INT, - TWL4030_INT_PWR_SIH_CTRL); - WARN_ON(cor < 0); - -- /* PWR_ISR1 */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INT, 0x00, cor) < 0); -- -- /* PWR_ISR2 */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INT, 0x02, cor) < 0); -+ /* Clear TWL4030 INT (PWR) ISRs */ -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INT, -+ TWL4030_INT_PWR_ISR1, cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INT, -+ TWL4030_INT_PWR_ISR2, cor) < 0); - - /* Slave address 0x4A */ - -- /* BCIIMR1A */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, 0x2) < 0); -- -- /* BCIIMR2A */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, 0x3) < 0); -- -- /* BCIIMR2A */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, 0x6) < 0); -- -- /* BCIIMR2B */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, 0x7) < 0); -+ /* Mask BCI interrupts at TWL4030 */ -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, -+ TWL4030_INTERRUPTS_BCIIMR1A) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, -+ TWL4030_INTERRUPTS_BCIIMR2A) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, -+ TWL4030_INTERRUPTS_BCIIMR1B) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, -+ TWL4030_INTERRUPTS_BCIIMR2B) < 0); - - /* Are BCI interrupt status bits cleared by reads or writes? */ - cor = twl4030_read_cor_bit(TWL4030_MODULE_INTERRUPTS, - TWL4030_INTERRUPTS_BCISIHCTRL); - WARN_ON(cor < 0); - -- /* BCIISR1A */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, 0x0, cor) < 0); -- -- /* BCIISR2A */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, 0x1, cor) < 0); -- -- /* BCIISR1B */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, 0x4, cor) < 0); -- -- /* BCIISR2B */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, 0x5, cor) < 0); -+ /* Clear TWL4030 BCI ISRs */ -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, -+ TWL4030_INTERRUPTS_BCIISR1A, cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, -+ TWL4030_INTERRUPTS_BCIISR2A, cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, -+ TWL4030_INTERRUPTS_BCIISR1B, cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, -+ TWL4030_INTERRUPTS_BCIISR2B, cor) < 0); - - /* MAD C */ -- /* MADC_IMR1 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xff, 0x62) < 0); -- -- /* MADC_IMR2 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xff, 0x64) < 0); -+ /* Mask MADC interrupts at TWL4030 */ -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xff, -+ TWL4030_MADC_IMR1) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xff, -+ TWL4030_MADC_IMR2) < 0); - - /* Are MADC interrupt status bits cleared by reads or writes? */ - cor = twl4030_read_cor_bit(TWL4030_MODULE_MADC, - TWL4030_MADC_MADC_SIH_CTRL); - WARN_ON(cor < 0); - -- /* MADC_ISR1 */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_MADC, 0x61, cor) < 0); -- -- /* MADC_ISR2 */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_MADC, 0x63, cor) < 0); -+ /* Clear TWL4030 MADC ISRs */ -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_MADC, -+ TWL4030_MADC_ISR1, cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_MADC, -+ TWL4030_MADC_ISR2, cor) < 0); - - /* key Pad */ -- /* KEYPAD - IMR1 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xff, 0x12) < 0); -+ /* Mask keypad interrupts at TWL4030 */ -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xff, -+ TWL4030_KEYPAD_KEYP_IMR1) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xff, -+ TWL4030_KEYPAD_KEYP_IMR2) < 0); - - /* Are keypad interrupt status bits cleared by reads or writes? */ - cor = twl4030_read_cor_bit(TWL4030_MODULE_KEYPAD, - TWL4030_KEYPAD_KEYP_SIH_CTRL); - WARN_ON(cor < 0); - -- /* KEYPAD - ISR1 */ -+ /* Clear TWL4030 keypad ISRs */ - /* XXX does this still need to be done twice for some reason? */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_KEYPAD, 0x11, cor) < 0); -- -- /* KEYPAD - IMR2 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xff, 0x14) < 0); -- -- /* KEYPAD - ISR2 */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_KEYPAD, 0x13, cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_KEYPAD, -+ TWL4030_KEYPAD_KEYP_ISR1, cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_KEYPAD, -+ TWL4030_KEYPAD_KEYP_ISR2, cor) < 0); - - /* Slave address 0x49 */ -- /* GPIO_IMR1A */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1c) < 0); -- -- /* GPIO_IMR2A */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1d) < 0); -- -- /* GPIO_IMR3A */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x1e) < 0); -- -- /* GPIO_IMR1B */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x22) < 0); - -- /* GPIO_IMR2B */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x23) < 0); -- -- /* GPIO_IMR3B */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, 0x24) < 0); -+ /* Mask GPIO interrupts at TWL4030 */ -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -+ REG_GPIO_IMR1A) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -+ REG_GPIO_IMR2A) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -+ REG_GPIO_IMR3A) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -+ REG_GPIO_IMR1B) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -+ REG_GPIO_IMR2B) < 0); -+ WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -+ REG_GPIO_IMR3B) < 0); - - /* Are GPIO interrupt status bits cleared by reads or writes? */ - cor = twl4030_read_cor_bit(TWL4030_MODULE_GPIO, -- TWL4030_GPIO_GPIO_SIH_CTRL); -+ REG_GPIO_SIH_CTRL); - WARN_ON(cor < 0); - -- /* GPIO_ISR1A */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x19, cor) < 0); -- -- /* GPIO_ISR2A */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x1a, cor) < 0); -- -- /* GPIO_ISR3A */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x1b, cor) < 0); -- -- /* GPIO_ISR1B */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x1f, cor) < 0); -- -- /* GPIO_ISR2B */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x20, cor) < 0); -- -- /* GPIO_ISR3B */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, 0x21, cor) < 0); -+ /* Clear TWL4030 GPIO ISRs */ -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR1A, -+ cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR2A, -+ cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR3A, -+ cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR1B, -+ cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR2B, -+ cor) < 0); -+ WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR3B, -+ cor) < 0); - - /* install an irq handler for each of the PIH modules */ - for (i = TWL4030_IRQ_BASE; i < TWL4030_IRQ_END; i++) { diff --git a/packages/linux/linux-omap2-git/beagleboard/TWL4030-09.patch b/packages/linux/linux-omap2-git/beagleboard/TWL4030-09.patch deleted file mode 100644 index ab6cc6d87b..0000000000 --- a/packages/linux/linux-omap2-git/beagleboard/TWL4030-09.patch +++ /dev/null @@ -1,341 +0,0 @@ -TWL4030: convert early interrupt mask/clear funcs to use array - -From: Paul Walmsley <paul@pwsan.com> - -Mask/clear TWL module IMRs/ISRs by iterating through arrays rather than -using a block of cut-and-pasted commands. Removes 1056 bytes of bloat. - -Signed-off-by: Paul Walmsley <paul@pwsan.com> ---- - - drivers/i2c/chips/twl4030-core.c | 302 +++++++++++++++++++++++--------------- - 1 files changed, 180 insertions(+), 122 deletions(-) - -diff --git a/drivers/i2c/chips/twl4030-core.c b/drivers/i2c/chips/twl4030-core.c -index 38c227a..776b1dd 100644 ---- a/drivers/i2c/chips/twl4030-core.c -+++ b/drivers/i2c/chips/twl4030-core.c -@@ -160,6 +160,136 @@ - - #define TWL4030_SIH_CTRL_COR_MASK (1 << 2) - -+/** -+ * struct twl4030_mod_iregs - TWL module IMR/ISR regs to mask/clear at init -+ * @mod_no: TWL4030 module number (e.g., TWL4030_MODULE_GPIO) -+ * @sih_ctrl: address of module SIH_CTRL register -+ * @reg_cnt: number of IMR/ISR regs -+ * @imrs: pointer to array of TWL module interrupt mask register indices -+ * @isrs: pointer to array of TWL module interrupt status register indices -+ * -+ * Ties together TWL4030 modules and lists of IMR/ISR registers to mask/clear -+ * during twl_init_irq(). -+ */ -+struct twl4030_mod_iregs { -+ const u8 mod_no; -+ const u8 sih_ctrl; -+ const u8 reg_cnt; -+ const u8 *imrs; -+ const u8 *isrs; -+}; -+ -+/* TWL4030 INT module interrupt mask registers */ -+static const u8 __initconst twl4030_int_imr_regs[] = { -+ TWL4030_INT_PWR_IMR1, -+ TWL4030_INT_PWR_IMR2, -+}; -+ -+/* TWL4030 INT module interrupt status registers */ -+static const u8 __initconst twl4030_int_isr_regs[] = { -+ TWL4030_INT_PWR_ISR1, -+ TWL4030_INT_PWR_ISR2, -+}; -+ -+/* TWL4030 INTERRUPTS module interrupt mask registers */ -+static const u8 __initconst twl4030_interrupts_imr_regs[] = { -+ TWL4030_INTERRUPTS_BCIIMR1A, -+ TWL4030_INTERRUPTS_BCIIMR1B, -+ TWL4030_INTERRUPTS_BCIIMR2A, -+ TWL4030_INTERRUPTS_BCIIMR2B, -+}; -+ -+/* TWL4030 INTERRUPTS module interrupt status registers */ -+static const u8 __initconst twl4030_interrupts_isr_regs[] = { -+ TWL4030_INTERRUPTS_BCIISR1A, -+ TWL4030_INTERRUPTS_BCIISR1B, -+ TWL4030_INTERRUPTS_BCIISR2A, -+ TWL4030_INTERRUPTS_BCIISR2B, -+}; -+ -+/* TWL4030 MADC module interrupt mask registers */ -+static const u8 __initconst twl4030_madc_imr_regs[] = { -+ TWL4030_MADC_IMR1, -+ TWL4030_MADC_IMR2, -+}; -+ -+/* TWL4030 MADC module interrupt status registers */ -+static const u8 __initconst twl4030_madc_isr_regs[] = { -+ TWL4030_MADC_ISR1, -+ TWL4030_MADC_ISR2, -+}; -+ -+/* TWL4030 keypad module interrupt mask registers */ -+static const u8 __initconst twl4030_keypad_imr_regs[] = { -+ TWL4030_KEYPAD_KEYP_IMR1, -+ TWL4030_KEYPAD_KEYP_IMR2, -+}; -+ -+/* TWL4030 keypad module interrupt status registers */ -+static const u8 __initconst twl4030_keypad_isr_regs[] = { -+ TWL4030_KEYPAD_KEYP_ISR1, -+ TWL4030_KEYPAD_KEYP_ISR2, -+}; -+ -+/* TWL4030 GPIO module interrupt mask registers */ -+static const u8 __initconst twl4030_gpio_imr_regs[] = { -+ REG_GPIO_IMR1A, -+ REG_GPIO_IMR1B, -+ REG_GPIO_IMR2A, -+ REG_GPIO_IMR2B, -+ REG_GPIO_IMR3A, -+ REG_GPIO_IMR3B, -+}; -+ -+/* TWL4030 GPIO module interrupt status registers */ -+static const u8 __initconst twl4030_gpio_isr_regs[] = { -+ REG_GPIO_ISR1A, -+ REG_GPIO_ISR1B, -+ REG_GPIO_ISR2A, -+ REG_GPIO_ISR2B, -+ REG_GPIO_ISR3A, -+ REG_GPIO_ISR3B, -+}; -+ -+/* TWL4030 modules that have IMR/ISR registers that must be masked/cleared */ -+static const struct twl4030_mod_iregs __initconst twl4030_mod_regs[] = { -+ { -+ .mod_no = TWL4030_MODULE_INT, -+ .sih_ctrl = TWL4030_INT_PWR_SIH_CTRL, -+ .reg_cnt = ARRAY_SIZE(twl4030_int_imr_regs), -+ .imrs = twl4030_int_imr_regs, -+ .isrs = twl4030_int_isr_regs, -+ }, -+ { -+ .mod_no = TWL4030_MODULE_INTERRUPTS, -+ .sih_ctrl = TWL4030_INTERRUPTS_BCISIHCTRL, -+ .reg_cnt = ARRAY_SIZE(twl4030_interrupts_imr_regs), -+ .imrs = twl4030_interrupts_imr_regs, -+ .isrs = twl4030_interrupts_isr_regs, -+ }, -+ { -+ .mod_no = TWL4030_MODULE_MADC, -+ .sih_ctrl = TWL4030_MADC_MADC_SIH_CTRL, -+ .reg_cnt = ARRAY_SIZE(twl4030_madc_imr_regs), -+ .imrs = twl4030_madc_imr_regs, -+ .isrs = twl4030_madc_isr_regs, -+ }, -+ { -+ .mod_no = TWL4030_MODULE_KEYPAD, -+ .sih_ctrl = TWL4030_KEYPAD_KEYP_SIH_CTRL, -+ .reg_cnt = ARRAY_SIZE(twl4030_keypad_imr_regs), -+ .imrs = twl4030_keypad_imr_regs, -+ .isrs = twl4030_keypad_isr_regs, -+ }, -+ { -+ .mod_no = TWL4030_MODULE_GPIO, -+ .sih_ctrl = REG_GPIO_SIH_CTRL, -+ .reg_cnt = ARRAY_SIZE(twl4030_gpio_imr_regs), -+ .imrs = twl4030_gpio_imr_regs, -+ .isrs = twl4030_gpio_isr_regs, -+ }, -+}; -+ - - /* Helper functions */ - static int -@@ -779,136 +909,64 @@ static int twl4030_read_cor_bit(u8 mod_no, u8 reg) - return tmp; - } - -+/** -+ * twl4030_mask_clear_intrs - mask and clear all TWL4030 interrupts -+ * @t: pointer to twl4030_mod_iregs array -+ * @t_sz: ARRAY_SIZE(t) (starting at 1) -+ * -+ * Mask all TWL4030 interrupt mask registers (IMRs) and clear all -+ * interrupt status registers (ISRs). No return value, but will WARN if -+ * any I2C operations fail. -+ */ -+static void __init twl4030_mask_clear_intrs(const struct twl4030_mod_iregs *t, -+ const u8 t_sz) -+{ -+ int i, j; -+ -+ /* -+ * N.B. - further efficiency is possible here. Eight I2C -+ * operations on BCI and GPIO modules are avoidable if I2C -+ * burst read/write transactions were implemented. Would -+ * probably save about 1ms of boot time and a small amount of -+ * power. -+ */ -+ for (i = 0; i < t_sz; i++) { -+ const struct twl4030_mod_iregs tmr = t[i]; -+ -+ for (j = 0; j < tmr.reg_cnt; j++) { -+ int cor; -+ -+ /* Mask interrupts at the TWL4030 */ -+ WARN_ON(twl4030_i2c_write_u8(tmr.mod_no, 0xff, -+ tmr.imrs[j]) < 0); -+ -+ /* Are ISRs cleared by reads or writes? */ -+ cor = twl4030_read_cor_bit(tmr.mod_no, tmr.sih_ctrl); -+ WARN_ON(cor < 0); -+ -+ /* Clear TWL4030 ISRs */ -+ WARN_ON(twl4030_i2c_clear_isr(tmr.mod_no, -+ tmr.isrs[j], cor) < 0); -+ } -+ } -+ -+ return; -+} -+ -+ - static void twl_init_irq(void) - { -- int i = 0; -+ int i; - int res = 0; -- int cor; - char *msg = "Unable to register interrupt subsystem"; - unsigned int irq_num; - - /* -- * For each TWL4030 module with ISR/IMR registers, mask all -- * interrupts and then clear any existing interrupt status bits, -- * since we initially do not have any TWL4030 module interrupt -- * handlers present. -+ * Mask and clear all TWL4030 interrupts since initially we do -+ * not have any TWL4030 module interrupt handlers present - */ -- -- /* Mask INT (PWR) interrupts at TWL4030 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xff, -- TWL4030_INT_PWR_IMR1) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INT, 0xff, -- TWL4030_INT_PWR_IMR2) < 0); -- -- /* Are PWR interrupt status bits cleared by reads or writes? */ -- cor = twl4030_read_cor_bit(TWL4030_MODULE_INT, -- TWL4030_INT_PWR_SIH_CTRL); -- WARN_ON(cor < 0); -- -- /* Clear TWL4030 INT (PWR) ISRs */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INT, -- TWL4030_INT_PWR_ISR1, cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INT, -- TWL4030_INT_PWR_ISR2, cor) < 0); -- -- /* Slave address 0x4A */ -- -- /* Mask BCI interrupts at TWL4030 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, -- TWL4030_INTERRUPTS_BCIIMR1A) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, -- TWL4030_INTERRUPTS_BCIIMR2A) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, -- TWL4030_INTERRUPTS_BCIIMR1B) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, 0xff, -- TWL4030_INTERRUPTS_BCIIMR2B) < 0); -- -- /* Are BCI interrupt status bits cleared by reads or writes? */ -- cor = twl4030_read_cor_bit(TWL4030_MODULE_INTERRUPTS, -- TWL4030_INTERRUPTS_BCISIHCTRL); -- WARN_ON(cor < 0); -- -- /* Clear TWL4030 BCI ISRs */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, -- TWL4030_INTERRUPTS_BCIISR1A, cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, -- TWL4030_INTERRUPTS_BCIISR2A, cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, -- TWL4030_INTERRUPTS_BCIISR1B, cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_INTERRUPTS, -- TWL4030_INTERRUPTS_BCIISR2B, cor) < 0); -- -- /* MAD C */ -- /* Mask MADC interrupts at TWL4030 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xff, -- TWL4030_MADC_IMR1) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_MADC, 0xff, -- TWL4030_MADC_IMR2) < 0); -- -- /* Are MADC interrupt status bits cleared by reads or writes? */ -- cor = twl4030_read_cor_bit(TWL4030_MODULE_MADC, -- TWL4030_MADC_MADC_SIH_CTRL); -- WARN_ON(cor < 0); -- -- /* Clear TWL4030 MADC ISRs */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_MADC, -- TWL4030_MADC_ISR1, cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_MADC, -- TWL4030_MADC_ISR2, cor) < 0); -- -- /* key Pad */ -- /* Mask keypad interrupts at TWL4030 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xff, -- TWL4030_KEYPAD_KEYP_IMR1) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_KEYPAD, 0xff, -- TWL4030_KEYPAD_KEYP_IMR2) < 0); -- -- /* Are keypad interrupt status bits cleared by reads or writes? */ -- cor = twl4030_read_cor_bit(TWL4030_MODULE_KEYPAD, -- TWL4030_KEYPAD_KEYP_SIH_CTRL); -- WARN_ON(cor < 0); -- -- /* Clear TWL4030 keypad ISRs */ -- /* XXX does this still need to be done twice for some reason? */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_KEYPAD, -- TWL4030_KEYPAD_KEYP_ISR1, cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_KEYPAD, -- TWL4030_KEYPAD_KEYP_ISR2, cor) < 0); -- -- /* Slave address 0x49 */ -- -- /* Mask GPIO interrupts at TWL4030 */ -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -- REG_GPIO_IMR1A) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -- REG_GPIO_IMR2A) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -- REG_GPIO_IMR3A) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -- REG_GPIO_IMR1B) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -- REG_GPIO_IMR2B) < 0); -- WARN_ON(twl4030_i2c_write_u8(TWL4030_MODULE_GPIO, 0xff, -- REG_GPIO_IMR3B) < 0); -- -- /* Are GPIO interrupt status bits cleared by reads or writes? */ -- cor = twl4030_read_cor_bit(TWL4030_MODULE_GPIO, -- REG_GPIO_SIH_CTRL); -- WARN_ON(cor < 0); -- -- /* Clear TWL4030 GPIO ISRs */ -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR1A, -- cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR2A, -- cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR3A, -- cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR1B, -- cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR2B, -- cor) < 0); -- WARN_ON(twl4030_i2c_clear_isr(TWL4030_MODULE_GPIO, REG_GPIO_ISR3B, -- cor) < 0); -+ twl4030_mask_clear_intrs(twl4030_mod_regs, -+ ARRAY_SIZE(twl4030_mod_regs)); - - /* install an irq handler for each of the PIH modules */ - for (i = TWL4030_IRQ_BASE; i < TWL4030_IRQ_END; i++) { diff --git a/packages/linux/linux-omap2-git/beagleboard/flash.patch b/packages/linux/linux-omap2-git/beagleboard/flash.patch index 4c76cd97bd..639d2d0e55 100644 --- a/packages/linux/linux-omap2-git/beagleboard/flash.patch +++ b/packages/linux/linux-omap2-git/beagleboard/flash.patch @@ -1,171 +1,108 @@ -diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile -index 13d0043..d582b8f 100644 ---- a/arch/arm/mach-omap2/Makefile -+++ b/arch/arm/mach-omap2/Makefile -@@ -44,7 +44,8 @@ obj-$(CONFIG_MACH_OMAP3EVM) += board-omap3evm.o \ - board-omap3evm-flash.o - obj-$(CONFIG_MACH_OMAP3_BEAGLE) += board-omap3beagle.o \ - usb-musb.o usb-ehci.o \ -- hsmmc.o -+ hsmmc.o \ -+ board-omap3beagle-flash.o - obj-$(CONFIG_MACH_OMAP_LDP) += board-ldp.o \ - hsmmc.o \ - usb-musb.o -diff --git a/arch/arm/mach-omap2/board-omap3beagle-flash.c b/arch/arm/mach-omap2/board-omap3beagle-flash.c -new file mode 100644 -index 0000000..5346df0 ---- /dev/null -+++ b/arch/arm/mach-omap2/board-omap3beagle-flash.c -@@ -0,0 +1,119 @@ -+/* -+ * board-omap3beagle-flash.c -+ * -+ * Copyright (c) 2008 Texas Instruments -+ * -+ * Modified from board-omap3evm-flash.c -+ * -+ * This program is free software; you can redistribute it and/or modify -+ * it under the terms of the GNU General Public License version 2 as -+ * published by the Free Software Foundation. -+ */ -+ -+#include <linux/kernel.h> -+#include <linux/platform_device.h> -+#include <linux/mtd/mtd.h> -+#include <linux/mtd/partitions.h> -+#include <linux/mtd/nand.h> -+#include <linux/types.h> -+#include <linux/io.h> -+ -+#include <asm/mach/flash.h> -+#include <asm/arch/board.h> -+#include <asm/arch/gpmc.h> -+#include <asm/arch/nand.h> -+ -+#define GPMC_CS0_BASE 0x60 -+#define GPMC_CS_SIZE 0x30 -+ -+static struct mtd_partition omap3beagle_nand_partitions[] = { -+ /* All the partition sizes are listed in terms of NAND block size */ -+ { -+ .name = "X-Loader", -+ .offset = 0, -+ .size = 4*(64 * 2048), -+ .mask_flags = MTD_WRITEABLE, /* force read-only */ -+ }, -+ { -+ .name = "U-Boot", -+ .offset = MTDPART_OFS_APPEND, /* Offset = 0x80000 */ -+ .size = 15*(64 * 2048), -+ .mask_flags = MTD_WRITEABLE, /* force read-only */ -+ }, -+ { -+ .name = "U-Boot Env", -+ .offset = MTDPART_OFS_APPEND, /* Offset = 0x260000 */ -+ .size = 1*(64 * 2048), -+ }, -+ { -+ .name = "Kernel", -+ .offset = MTDPART_OFS_APPEND, /* Offset = 0x280000 */ -+ .size = 32*(64 * 2048), -+ }, -+ { -+ .name = "File System", -+ .offset = MTDPART_OFS_APPEND, /* Offset = 0x680000 */ -+ .size = MTDPART_SIZ_FULL, -+ }, -+}; -+ -+static struct omap_nand_platform_data omap3beagle_nand_data = { -+ .parts = omap3beagle_nand_partitions, -+ .nr_parts = ARRAY_SIZE(omap3beagle_nand_partitions), -+ .dma_channel = -1, /* disable DMA in OMAP NAND driver */ -+ .nand_setup = NULL, -+ .dev_ready = NULL, -+}; -+ -+static struct resource omap3beagle_nand_resource = { -+ .flags = IORESOURCE_MEM, -+}; -+ -+static struct platform_device omap3beagle_nand_device = { -+ .name = "omap2-nand", -+ .id = -1, -+ .dev = { -+ .platform_data = &omap3beagle_nand_data, -+ }, -+ .num_resources = 1, -+ .resource = &omap3beagle_nand_resource, -+}; -+ -+ -+void __init omap3beagle_flash_init(void) -+{ -+ u8 cs = 0; -+ u8 nandcs = GPMC_CS_NUM + 1; -+ -+ u32 gpmc_base_add = OMAP34XX_GPMC_VIRT; -+ -+ /* find out the chip-select on which NAND exists */ -+ while (cs < GPMC_CS_NUM) { -+ u32 ret = 0; -+ ret = gpmc_cs_read_reg(cs, GPMC_CS_CONFIG1); -+ -+ if ((ret & 0xC00) == 0x800) { -+ printk(KERN_INFO "Found NAND on CS%d\n", cs); -+ if (nandcs > GPMC_CS_NUM) -+ nandcs = cs; -+ } -+ cs++; -+ } -+ -+ if (nandcs > GPMC_CS_NUM) { -+ printk(KERN_INFO "NAND: Unable to find configuration " -+ "in GPMC\n "); -+ return; -+ } -+ -+ if (nandcs < GPMC_CS_NUM) { -+ omap3beagle_nand_data.cs = nandcs; -+ omap3beagle_nand_data.gpmc_cs_baseaddr = (void *)(gpmc_base_add + -+ GPMC_CS0_BASE + nandcs * GPMC_CS_SIZE); -+ omap3beagle_nand_data.gpmc_baseaddr = (void *) (gpmc_base_add); -+ -+ printk(KERN_INFO "Registering NAND on CS%d\n", nandcs); -+ if (platform_device_register(&omap3beagle_nand_device) < 0) -+ printk(KERN_ERR "Unable to register NAND device\n"); -+ } -+} -diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c -index c992cc7..99e042e 100644 ---- a/arch/arm/mach-omap2/board-omap3beagle.c -+++ b/arch/arm/mach-omap2/board-omap3beagle.c -@@ -94,6 +94,7 @@ static void __init omap3_beagle_init(void) - hsmmc_init(); - usb_musb_init(); - usb_ehci_init(); -+ omap3beagle_flash_init(); - } - - arch_initcall(omap3_beagle_i2c_init); -diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig -index 3d5e432..02b9ced 100644 ---- a/drivers/mtd/nand/Kconfig -+++ b/drivers/mtd/nand/Kconfig -@@ -71,7 +71,7 @@ config MTD_NAND_AMS_DELTA +Received: from mail.service.utwente.nl ([130.89.5.253]) by exchange.service.utwente.nl with Microsoft SMTPSVC(6.0.3790.3959); + Mon, 4 Aug 2008 17:59:28 +0200 +Received: from mx.utwente.nl ([130.89.2.12]) by mail.service.utwente.nl with Microsoft SMTPSVC(6.0.3790.3959); + Mon, 4 Aug 2008 17:59:27 +0200 +Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) + by mx.utwente.nl (8.12.10/SuSE Linux 0.7) with ESMTP id m74FwiDg028981 + for <k.kooi@student.utwente.nl>; Mon, 4 Aug 2008 17:58:45 +0200 +Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand + id S1754560AbYHDP6O (ORCPT <rfc822;k.kooi@student.utwente.nl>); + Mon, 4 Aug 2008 11:58:14 -0400 +Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756033AbYHDP6N + (ORCPT <rfc822;linux-omap-outgoing>); Mon, 4 Aug 2008 11:58:13 -0400 +Received: from fg-out-1718.google.com ([72.14.220.154]:19323 "EHLO + fg-out-1718.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org + with ESMTP id S1757716AbYHDP6L (ORCPT + <rfc822;linux-omap@vger.kernel.org>); Mon, 4 Aug 2008 11:58:11 -0400 +Received: by fg-out-1718.google.com with SMTP id 19so1022490fgg.17 + for <linux-omap@vger.kernel.org>; Mon, 04 Aug 2008 08:58:10 -0700 (PDT) +DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; + d=googlemail.com; s=gamma; + h=domainkey-signature:received:received:message-id:date:from + :user-agent:x-accept-language:mime-version:to:cc:subject + :content-type; + bh=9UH32BpZIt6pkxG3XXDaUMdlsAxT/6HeJgy/cl4UMoY=; + b=f81vPPHp6KYejbQyr/s6gwGwGNYUr4art1U829ybVG5aVezJQUy8iuNQqgwPGNLhz+ + sCPYTp/d7RKpnui9FFxQcy5HOMggvJAgdxr49Y61RZMM+OJS2BugTaUUXWS3/RoEBR+h + o/UXxMqR45NI684Nvvcx56Qc4NZQOIyiW3U1I= +DomainKey-Signature: a=rsa-sha1; c=nofws; + d=googlemail.com; s=gamma; + h=message-id:date:from:user-agent:x-accept-language:mime-version:to + :cc:subject:content-type; + b=lVauU4WHsnkoGN1/uhQ8jzixuaX2VXizHsSNc8Tq8RZPvOw+yZfIGd1ix6QWVBjttR + 9TgcojtGA0jx2hYbplibTmFPnoDcKym2qyF5DrX+G0ZLD/nVA+K9bd16sHErFeNz5eId + tx0WvpQ2hkVKKG4NgJqINAGB3iNcP80PlM/vk= +Received: by 10.86.70.11 with SMTP id s11mr3676282fga.15.1217865490017; + Mon, 04 Aug 2008 08:58:10 -0700 (PDT) +Received: from ?192.168.178.25? ( [195.4.48.71]) + by mx.google.com with ESMTPS id d4sm55517fga.8.2008.08.04.08.58.08 + (version=TLSv1/SSLv3 cipher=RC4-MD5); + Mon, 04 Aug 2008 08:58:09 -0700 (PDT) +Message-ID: <4897270F.7000205@googlemail.com> +Date: Mon, 04 Aug 2008 17:58:07 +0200 +From: Dirk Behme <dirk.behme@googlemail.com> +User-Agent: Mozilla Thunderbird 1.0.7 (X11/20050923) +X-Accept-Language: en-us, en +MIME-Version: 1.0 +To: "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org> +CC: Steve Sakoman <sakoman@gmail.com> +Subject: [PATCH 1/3 v2] mtd omap2 nand driver: extend to work with omap3 boards +Content-Type: multipart/mixed; + boundary="------------070804080507040009060104" +Sender: linux-omap-owner@vger.kernel.org +Precedence: bulk +List-ID: <linux-omap.vger.kernel.org> +X-Mailing-List: linux-omap@vger.kernel.org +X-UTwente-MailScanner-Information: Scanned by MailScanner. Contact servicedesk@icts.utwente.nl for more information. +X-UTwente-MailScanner: Found to be clean +X-UTwente-MailScanner-From: linux-omap-owner@vger.kernel.org +X-Spam-Status: No +Return-Path: linux-omap-owner@vger.kernel.org +X-OriginalArrivalTime: 04 Aug 2008 15:59:28.0133 (UTC) FILETIME=[12F8BB50:01C8F64B] + +This is a multi-part message in MIME format. +--------------070804080507040009060104 +Content-Type: text/plain; charset=ISO-8859-1; format=flowed +Content-Transfer-Encoding: 7bit + +From: Teerth Reddy <teerth@ti.com>, Steve Sakoman <steve@sakoman.com>, +Dirk Behme <dirk.behme@gmail.com> + +Extend omap2 mtd nand driver to work with ARCH_OMAP3 boards + +Signed-off-by: Steve Sakoman <steve@sakoman.com> +Signed-off-by: Dirk Behme <dirk.behme@gmail.com> + + +--------------070804080507040009060104 +Content-Type: text/plain; + name="1_nand.txt" +Content-Transfer-Encoding: 7bit +Content-Disposition: inline; + filename="1_nand.txt" + +Index: linux-beagle/drivers/mtd/nand/Kconfig +=================================================================== +--- linux-beagle.orig/drivers/mtd/nand/Kconfig ++++ linux-beagle/drivers/mtd/nand/Kconfig +@@ -70,10 +70,10 @@ config MTD_NAND_AMS_DELTA + Support for NAND flash on Amstrad E3 (Delta). config MTD_NAND_OMAP2 - tristate "NAND Flash device on OMAP 2420H4/2430SDP boards" +- tristate "NAND Flash device on OMAP 2420H4/2430SDP boards" - depends on (ARM && ARCH_OMAP2 && MTD_NAND) ++ tristate "NAND Flash device on OMAP2 and OMAP3" + depends on ARM && MTD_NAND && (ARCH_OMAP2 || ARCH_OMAP3) help - Support for NAND flash on Texas Instruments 2430SDP/2420H4 platforms. +- Support for NAND flash on Texas Instruments 2430SDP/2420H4 platforms. ++ Support for NAND flash on Texas Instruments OMAP2 and OMAP3 platforms. -diff --git a/drivers/mtd/nand/omap2.c b/drivers/mtd/nand/omap2.c -index 3b7307c..3aac1d2 100644 ---- a/drivers/mtd/nand/omap2.c -+++ b/drivers/mtd/nand/omap2.c + config MTD_NAND_OMAP + tristate "NAND Flash device on OMAP H3/H2/P2 boards" +Index: linux-beagle/drivers/mtd/nand/omap2.c +=================================================================== +--- linux-beagle.orig/drivers/mtd/nand/omap2.c ++++ linux-beagle/drivers/mtd/nand/omap2.c @@ -111,15 +111,6 @@ static const char *part_probes[] = { "cmdlinepart", NULL }; #endif @@ -196,7 +133,7 @@ index 3b7307c..3aac1d2 100644 static void omap_nand_wp(struct mtd_info *mtd, int mode) { struct omap_nand_info *info = container_of(mtd, -@@ -189,11 +187,11 @@ static void omap_hwcontrol(struct mtd_info *mtd, int cmd, unsigned int ctrl) +@@ -189,11 +187,11 @@ static void omap_hwcontrol(struct mtd_in } /* @@ -213,7 +150,7 @@ index 3b7307c..3aac1d2 100644 static void omap_read_buf(struct mtd_info *mtd, u_char *buf, int len) { struct omap_nand_info *info = container_of(mtd, -@@ -207,11 +205,11 @@ static void omap_read_buf(struct mtd_info *mtd, u_char *buf, int len) +@@ -207,11 +205,11 @@ static void omap_read_buf(struct mtd_inf } /* @@ -230,7 +167,7 @@ index 3b7307c..3aac1d2 100644 static void omap_write_buf(struct mtd_info *mtd, const u_char * buf, int len) { struct omap_nand_info *info = container_of(mtd, -@@ -250,10 +248,16 @@ static int omap_verify_buf(struct mtd_info *mtd, const u_char * buf, int len) +@@ -250,10 +248,16 @@ static int omap_verify_buf(struct mtd_in return 0; } @@ -247,7 +184,7 @@ index 3b7307c..3aac1d2 100644 unsigned long val = 0x0; /* Read from ECC Control Register */ -@@ -264,16 +268,15 @@ static void omap_hwecc_init(struct mtd_info *mtd) +@@ -264,16 +268,15 @@ static void omap_hwecc_init(struct mtd_i /* Read from ECC Size Config Register */ val = __raw_readl(info->gpmc_baseaddr + GPMC_ECC_SIZE_CONFIG); @@ -283,7 +220,7 @@ index 3b7307c..3aac1d2 100644 */ static int omap_compare_ecc(u8 *ecc_data1, /* read from NAND memory */ u8 *ecc_data2, /* read from register */ -@@ -409,6 +416,14 @@ static int omap_compare_ecc(u8 *ecc_data1, /* read from NAND memory */ +@@ -409,6 +416,14 @@ static int omap_compare_ecc(u8 *ecc_data } } @@ -298,7 +235,7 @@ index 3b7307c..3aac1d2 100644 static int omap_correct_data(struct mtd_info *mtd, u_char * dat, u_char * read_ecc, u_char * calc_ecc) { -@@ -436,65 +451,64 @@ static int omap_correct_data(struct mtd_info *mtd, u_char * dat, +@@ -436,65 +451,64 @@ static int omap_correct_data(struct mtd_ } /* @@ -396,12 +333,12 @@ index 3b7307c..3aac1d2 100644 break; default: DEBUG(MTD_DEBUG_LEVEL0, "Error: Unrecognized Mode[%d]!\n", -@@ -504,7 +518,38 @@ static void omap_enable_hwecc(struct mtd_info *mtd, int mode) +@@ -504,7 +518,38 @@ static void omap_enable_hwecc(struct mtd __raw_writel(val, info->gpmc_baseaddr + GPMC_ECC_CONFIG); } +#endif - ++ +/* + * omap_wait - Wait function is called during Program and erase + * operations and the way it is called from MTD layer, we should wait @@ -427,7 +364,7 @@ index 3b7307c..3aac1d2 100644 + } + return status; +} -+ + +/* + * omap_dev_ready - calls the platform specific dev_ready function + * @mtd: MTD device structure @@ -435,7 +372,7 @@ index 3b7307c..3aac1d2 100644 static int omap_dev_ready(struct mtd_info *mtd) { struct omap_nand_info *info = container_of(mtd, struct omap_nand_info, -@@ -534,7 +579,7 @@ static int __devinit omap_nand_probe(struct platform_device *pdev) +@@ -534,7 +579,7 @@ static int __devinit omap_nand_probe(str struct omap_nand_info *info; struct omap_nand_platform_data *pdata; int err; @@ -444,7 +381,7 @@ index 3b7307c..3aac1d2 100644 pdata = pdev->dev.platform_data; -@@ -568,15 +613,20 @@ static int __devinit omap_nand_probe(struct platform_device *pdev) +@@ -568,15 +613,20 @@ static int __devinit omap_nand_probe(str } /* Enable RD PIN Monitoring Reg */ @@ -468,7 +405,7 @@ index 3b7307c..3aac1d2 100644 if (!request_mem_region(info->phys_base, NAND_IO_SIZE, pdev->dev.driver->name)) { err = -EBUSY; -@@ -597,29 +647,39 @@ static int __devinit omap_nand_probe(struct platform_device *pdev) +@@ -597,29 +647,39 @@ static int __devinit omap_nand_probe(str info->nand.write_buf = omap_write_buf; info->nand.verify_buf = omap_verify_buf; @@ -528,14 +465,7 @@ index 3b7307c..3aac1d2 100644 /* DIP switches on some boards change between 8 and 16 bit * bus widths for flash. Try the other width if the first try fails. -@@ -636,14 +696,12 @@ static int __devinit omap_nand_probe(struct platform_device *pdev) - err = parse_mtd_partitions(&info->mtd, part_probes, &info->parts, 0); - if (err > 0) - add_mtd_partitions(&info->mtd, info->parts, err); -- else if (err < 0 && pdata->parts) -+ else if (err <= 0 && pdata->parts) - add_mtd_partitions(&info->mtd, pdata->parts, pdata->nr_parts); - else +@@ -642,8 +702,6 @@ static int __devinit omap_nand_probe(str #endif add_mtd_device(&info->mtd); @@ -544,15 +474,9 @@ index 3b7307c..3aac1d2 100644 platform_set_drvdata(pdev, &info->mtd); return 0; -diff --git a/include/asm-arm/arch-omap/board-omap3beagle.h b/include/asm-arm/arch-omap/board-omap3beagle.h -index 46dff31..26ecfb8 100644 ---- a/include/asm-arm/arch-omap/board-omap3beagle.h -+++ b/include/asm-arm/arch-omap/board-omap3beagle.h -@@ -29,5 +29,7 @@ - #ifndef __ASM_ARCH_OMAP3_BEAGLE_H - #define __ASM_ARCH_OMAP3_BEAGLE_H - -+extern void omap3beagle_flash_init(void); -+ - #endif /* __ASM_ARCH_OMAP3_BEAGLE_H */ - + +--------------070804080507040009060104-- +-- +To unsubscribe from this list: send the line "unsubscribe linux-omap" in +the body of a message to majordomo@vger.kernel.org +More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/packages/linux/linux-omap2-git/beagleboard/i2c-omap-race-fix.diff b/packages/linux/linux-omap2-git/beagleboard/i2c-omap-race-fix.diff deleted file mode 100644 index 6eb33f76b7..0000000000 --- a/packages/linux/linux-omap2-git/beagleboard/i2c-omap-race-fix.diff +++ /dev/null @@ -1,118 +0,0 @@ -From linux-omap-owner@vger.kernel.org Tue Jul 15 21:23:13 2008 -Received: from localhost - ([127.0.0.1] helo=dominion ident=koen) - by dominion.dominion.void with esmtp (Exim 4.69) - (envelope-from <linux-omap-owner@vger.kernel.org>) - id 1KIq7E-0004FX-VS - for koen@localhost; Tue, 15 Jul 2008 21:23:13 +0200 -Received: from xs.service.utwente.nl [130.89.5.250] - by dominion with POP3 (fetchmail-6.3.6) - for <koen@localhost> (single-drop); Tue, 15 Jul 2008 21:23:12 +0200 (CEST) -Received: from mail.service.utwente.nl ([130.89.5.254]) by exchange.service.utwente.nl with Microsoft SMTPSVC(6.0.3790.3959); - Tue, 15 Jul 2008 21:01:02 +0200 -Received: from mx.utwente.nl ([130.89.2.12]) by mail.service.utwente.nl with Microsoft SMTPSVC(6.0.3790.3959); - Tue, 15 Jul 2008 21:01:01 +0200 -Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) - by mx.utwente.nl (8.12.10/SuSE Linux 0.7) with ESMTP id m6FJ0qDf031889 - for <k.kooi@student.utwente.nl>; Tue, 15 Jul 2008 21:00:52 +0200 -Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand - id S1756776AbYGOTAV (ORCPT <rfc822;k.kooi@student.utwente.nl>); - Tue, 15 Jul 2008 15:00:21 -0400 -Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755065AbYGOTAV - (ORCPT <rfc822;linux-omap-outgoing>); - Tue, 15 Jul 2008 15:00:21 -0400 -Received: from utopia.booyaka.com ([72.9.107.138]:35569 "EHLO - utopia.booyaka.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org - with ESMTP id S1756776AbYGOTAU (ORCPT - <rfc822;linux-omap@vger.kernel.org>); Tue, 15 Jul 2008 15:00:20 -0400 -Received: (qmail 2982 invoked by uid 526); 15 Jul 2008 19:00:18 -0000 -Date: Tue, 15 Jul 2008 13:00:18 -0600 (MDT) -From: Paul Walmsley <paul@pwsan.com> -To: linux-omap@vger.kernel.org -Subject: [PATCH] i2c-omap: close suspected race between omap_i2c_idle() and - omap_i2c_isr() -Message-ID: <alpine.DEB.1.00.0807151259180.467@utopia.booyaka.com> -User-Agent: Alpine 1.00 (DEB 882 2007-12-20) -MIME-Version: 1.0 -Content-Type: TEXT/PLAIN; charset=US-ASCII -Sender: linux-omap-owner@vger.kernel.org -Precedence: bulk -List-ID: <linux-omap.vger.kernel.org> -X-Mailing-List: linux-omap@vger.kernel.org -X-UTwente-MailScanner-Information: Scanned by MailScanner. Contact servicedesk@icts.utwente.nl for more information. -X-UTwente-MailScanner: Found to be clean -X-UTwente-MailScanner-From: linux-omap-owner@vger.kernel.org -X-Spam-Status: No -X-OriginalArrivalTime: 15 Jul 2008 19:01:01.0610 (UTC) FILETIME=[1FBA68A0:01C8E6AD] - - -omap_i2c_idle() sets an internal flag, "dev->idle", instructing its -ISR to decline interrupts. It sets this flag before it actually masks -the interrupts on the I2C controller. This is problematic, since an -I2C interrupt could arrive after dev->idle is set, but before the -interrupt source is masked. When this happens, Linux disables the I2C -controller's IRQ, causing all future transactions on the bus to fail. - -Symptoms, happening on about 7% of boots: - - irq 56: nobody cared (try booting with the "irqpoll" option) - <warning traceback here> - Disabling IRQ #56 - i2c_omap i2c_omap.1: controller timed out - -In omap_i2c_idle(), this patch sets dev->idle only after the interrupt -mask write to the I2C controller has left the ARM write buffer. -That's probably the major offender. For additional prophylaxis, in -omap_i2c_unidle(), the patch clears the dev->idle flag before -interrupts are enabled, rather than afterwards. - -The patch has survived twenty-two reboots on the 3430SDP here without -wedging I2C1. Not absolutely dispositive, but promising! - - -Signed-off-by: Paul Walmsley <paul@pwsan.com> ---- - - drivers/i2c/busses/i2c-omap.c | 10 ++++++++-- - 1 files changed, 8 insertions(+), 2 deletions(-) - -diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c -index 55779f5..ed7e9ad 100644 ---- a/drivers/i2c/busses/i2c-omap.c -+++ b/drivers/i2c/busses/i2c-omap.c -@@ -209,22 +209,28 @@ static void omap_i2c_unidle(struct omap_i2c_dev *dev) - if (dev->iclk != NULL) - clk_enable(dev->iclk); - clk_enable(dev->fclk); -+ dev->idle = 0; - if (dev->iestate) - omap_i2c_write_reg(dev, OMAP_I2C_IE_REG, dev->iestate); -- dev->idle = 0; - } - - static void omap_i2c_idle(struct omap_i2c_dev *dev) - { - u16 iv; - -- dev->idle = 1; - dev->iestate = omap_i2c_read_reg(dev, OMAP_I2C_IE_REG); - omap_i2c_write_reg(dev, OMAP_I2C_IE_REG, 0); - if (dev->rev1) - iv = omap_i2c_read_reg(dev, OMAP_I2C_IV_REG); - else - omap_i2c_write_reg(dev, OMAP_I2C_STAT_REG, dev->iestate); -+ /* -+ * The wmb() is to ensure that the I2C interrupt mask write -+ * reaches the I2C controller before the dev->idle store -+ * occurs. -+ */ -+ wmb(); -+ dev->idle = 1; - clk_disable(dev->fclk); - if (dev->iclk != NULL) - clk_disable(dev->iclk); --- -To unsubscribe from this list: send the line "unsubscribe linux-omap" in -the body of a message to majordomo@vger.kernel.org -More majordomo info at http://vger.kernel.org/majordomo-info.html - |