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.

> > #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 /* End of support for TWL4030 */
> > > +#endif /*  __ASM_ARCH_OMAP3_BEAGLE_H */
> > > +
> > > 
> > > Index: my_branch/include/asm-arm/arch-omap/hardware.h
> > > ===================================================================
> > > --- my_branch.orig/include/asm-arm/arch-omap/hardware.h   2008-04-16
> > > 14:44:09.000000000 +0530
> > > +++ my_branch/include/asm-arm/arch-omap/hardware.h        2008-04-16
> > > 14:47:14.000000000 +0530
> > > @@ -339,6 +339,10 @@
> > >  #include "board-omap3evm.h"
> > >  #endif
> > > 
> > > +#ifdef CONFIG_MACH_OMAP3_BEAGLE
> > > +#include "board-omap3beagle.h"
> > > +#endif
> > > +
> > >  #ifdef CONFIG_MACH_OMAP_APOLLON
> > >  #include "board-apollon.h"
> > >  #endif
> > > --
> > > To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> > > the body of a message to [EMAIL PROTECTED]
> > > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > -- 
> > Best Regards,
> > 
> > Felipe Balbi
> > http://felipebalbi.com
> > [EMAIL PROTECTED]
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> > the body of a message to [EMAIL PROTECTED]
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to [EMAIL PROTECTED]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 
Best Regards,

Felipe Balbi
[EMAIL PROTECTED]
http://blog.felipebalbi.com
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to