On Sat, 2010-06-19 at 09:29 -0700, Gregory Bean wrote: > +#if defined(CONFIG_ARCH_MSM7X30) > +#define GPIO1_REG(off) (MSM_GPIO1_BASE + (off)) > +#define GPIO2_REG(off) (MSM_GPIO2_BASE + 0x400 + (off)) > +#else > +#define GPIO1_REG(off) (MSM_GPIO1_BASE + 0x800 + (off)) > +#define GPIO2_REG(off) (MSM_GPIO2_BASE + 0xC00 + (off)) > +#endif > + > +#if defined(CONFIG_ARCH_QSD8X50) > +#include "gpio_hw-8x50.h" > +#elif defined(CONFIG_ARCH_MSM7X30) > +#include "gpio_hw-7x30.h" > +#else > +#include "gpio_hw-7xxx.h" > +#endif I don't think this compiles for 7xxx , since your adding the ifdef block but not the header.. It would be nice to also include that. don't respin till the license thing is settled .. Daniel -- Sent by a consultant of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html