On Sat, Apr 19, 2008 at 09:24:50PM +0530, Syed Mohammed, Khasim wrote: > > > > -----Original Message----- > > From: Felipe Balbi [mailto:me@xxxxxxxxxxxxxxx] > > Sent: Saturday, April 19, 2008 6:22 PM > > To: Tony Lindgren > > Cc: Felipe Balbi; Syed Mohammed, Khasim; linux-omap@xxxxxxxxxxxxxxx; > > discussion@xxxxxxxxxxxxxxx > > Subject: Re: [PATCH 3/4] Adding support for OMAP3 Beagle Board > > > > On Fri, Apr 18, 2008 at 12:05:54PM -0700, Tony Lindgren wrote: > > > > > +#ifndef __ASM_ARCH_OMAP3_BEAGLE_H > > > > > +#define __ASM_ARCH_OMAP3_BEAGLE_H > > > > > + > > > > > +#ifdef CONFIG_TWL4030_CORE > > > > > + > > > > > +#define TWL4030_IRQNUM INT_34XX_SYS_NIRQ > > > > > + > > > > > +/* TWL4030 Primary Interrupt Handler (PIH) interrupts */ > > > > > +#define IH_TWL4030_BASE IH_BOARD_BASE > > > > > +#define IH_TWL4030_END (IH_TWL4030_BASE+8) > > > > > +#define IH_TWL4030_PWRBASE (IH_TWL4030_END) > > > > > +#define IH_TWL4030_PWRBASE_END (IH_TWL4030_PWRBASE+8) > > > > > + > > > > > +#ifdef CONFIG_TWL4030_GPIO > > > > > +/* TWL4030 GPIO Interrupts */ > > > > > +#define IH_TWL4030_GPIO_BASE (IH_TWL4030_PWRBASE_END) > > > > > +#define IH_TWL4030_GPIO_END (IH_TWL4030_GPIO_BASE+18) > > > > > +#define NR_IRQS (IH_TWL4030_GPIO_END) > > > > > +#else > > > > > +#define NR_IRQS (IH_TWL4030_PWRBASE_END) > > > > > +#endif /* CONFIG_I2C_TWL4030_GPIO */ > > > > > > > > shouldn't you consider the power base as the other boards ? > > > > > > Felipe, can you clarify what you mean here? > > > > Just check board-3430sdp.h > > > > #ifdef CONFIG_TWL4030_CORE > > > > #define TWL4030_IRQNUM INT_34XX_SYS_NIRQ > > > > /* TWL4030 Primary Interrupt Handler (PIH) interrupts */ > > #define IH_TWL4030_BASE IH_BOARD_BASE > > #define IH_TWL4030_END (IH_TWL4030_BASE+8) > > > > #define IH_TWL4030_PWRBASE (IH_TWL4030_END) > > #define IH_TWL4030_PWRBASE_END (IH_TWL4030_PWRBASE+8) > > > > #ifdef CONFIG_TWL4030_GPIO > > > > /* TWL4030 GPIO Interrupts */ > > #define IH_TWL4030_GPIO_BASE (IH_TWL4030_PWRBASE_END) > > #define IH_TWL4030_GPIO_END (IH_TWL4030_GPIO_BASE+18) > > #define NR_IRQS (IH_TWL4030_GPIO_END) > > #else > > #define NR_IRQS (IH_TWL4030_PWRBASE_END) > > #endif /* CONFIG_I2C_TWL4030_GPIO */ > > #endif /* End of support for TWL4030 */ > > #endif /* __ASM_ARCH_OMAP_3430SDP_H */ > > > > if there are no such defines for power base, it will break compilation > > of twl4030-pwrirq.c which is always built whenever we enable twl4030 > > support. > > > > At present I have enabled TWL4030 support in both EVM and beagle patches and it is tested booting on both beagle and evm boards, "with out these defines". > > So, these defines are not required now. They will be added when necessary along with relevant functionality. I could have added but then your comment might have been, why to add these defines when they are not used :) I can't see how can you possibly build twl4030 support without those defines. Looking at drivers/i2c/chips/twl4030-pwrirq.c:twl4030_pwrirq_enableint(): static void twl4030_pwrirq_enableint(unsigned int irq) { twl4030_pwrirq_pending_unmask |= 1 << (irq - IH_TWL4030_PWRBASE); if (twl4030_pwrirq_unmask_thread && twl4030_pwrirq_unmask_thread->state != TASK_RUNNING) wake_up_process(twl4030_pwrirq_unmask_thread); } I clearly see the use of IH_TWL4030_PWRBASE and looking at drivers/i2c/chips/Makefile: obj-$(CONFIG_TWL4030_CORE) += twl4030-core.o twl4030-pwrirq.o I also clearly see that compilation it will always fail whenever you enable CONFIG_TWL4030_CORE. But you already put such defines anyway :-p -- Best Regards, Felipe Balbi me@xxxxxxxxxxxxxxx http://blog.felipebalbi.com -- 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