* Steve Sakoman <sakoman@xxxxxxxxx> [080506 07:16]: > Tony, > > > > #define TWL4030_IRQNUM INT_34XX_SYS_NIRQ > > > > And this should really be OMAP3_TWL4030_IRQNUM defined in 34xx.h. > > I noticed that in another thread you asked Nishant Kamat to submit > this as a separate general patch for all boards that use this define. > > Would you prefer that I wait for him to do this before resubmitting, > or should I resubmit now? Well it would be nice to get that out of the way, so whoever gets a chance to do that should go for it :) Otherwise I need to re-edit these patches when submitting them upstream, which adds unnecessary steps. Tony > Steve > > On Fri, May 2, 2008 at 4:15 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote: > > Hi, > > > > This patch should be first one in the series as you cannot really > > enable it in the .config before this patch. > > > > Also see some comments below. > > > > * Steve Sakoman <sakoman@xxxxxxxxx> [080430 12:35]: > > > > > > > From: Steve Sakoman <steve@xxxxxxxxxxx> > > > > > > Add touchscreen support for OMAP3 EVM > > > > > > Signed-off-by: Steve Sakoman <steve@xxxxxxxxxxx> > > > --- > > > arch/arm/mach-omap2/board-omap3evm.c | 42 +++++++++++++++++++++++++++++ > > > include/asm-arm/arch-omap/board-omap3evm.h | 1 > > > 2 files changed, 43 insertions(+) > > > diff -uprN -X a/Documentation/dontdiff > > > a/arch/arm/mach-omap2/board-omap3evm.c > > > b/arch/arm/mach-omap2/board-omap3evm.c > > > --- a/arch/arm/mach-omap2/board-omap3evm.c 2008-04-28 21:03:22.000000000 -0700 > > > +++ b/arch/arm/mach-omap2/board-omap3evm.c 2008-04-28 21:16:35.000000000 -0700 > > > @@ -19,6 +19,8 @@ > > > #include <linux/err.h> > > > #include <linux/clk.h> > > > #include <linux/io.h> > > > +#include <linux/spi/spi.h> > > > +#include <linux/spi/ads7846.h> > > > > > > #include <asm/hardware.h> > > > #include <asm/mach-types.h> > > > @@ -31,6 +33,7 @@ > > > #include <asm/arch/usb-musb.h> > > > #include <asm/arch/usb-ehci.h> > > > #include <asm/arch/common.h> > > > +#include <asm/arch/mcspi.h> > > > > > > static struct omap_uart_config omap3_evm_uart_config __initdata = { > > > .enabled_uarts = ((1 << 0) | (1 << 1) | (1 << 2)), > > > @@ -65,6 +68,44 @@ > > > .id = -1, > > > }; > > > > > > +static void ads7846_dev_init(void) > > > +{ > > > + if (omap_request_gpio(TS_GPIO) < 0) > > > + printk(KERN_ERR "can't get ads7846 pen down GPIO\n"); > > > + > > > + omap_set_gpio_direction(TS_GPIO, 1); > > > + > > > + omap_set_gpio_debounce(TS_GPIO, 1); > > > + omap_set_gpio_debounce_time(TS_GPIO, 0xa); > > > +} > > > + > > > +static int ads7846_get_pendown_state(void) > > > +{ > > > + return !omap_get_gpio_datain(TS_GPIO); > > > +} > > > + > > > +struct ads7846_platform_data ads7846_config = { > > > + .get_pendown_state = ads7846_get_pendown_state, > > > + .keep_vref_on = 1, > > > +}; > > > + > > > +static struct omap2_mcspi_device_config ads7846_mcspi_config = { > > > + .turbo_mode = 0, > > > + .single_channel = 1, /* 0: slave, 1: master */ > > > +}; > > > + > > > +struct spi_board_info omap3evm_spi_board_info[] = { > > > + [0] = { > > > + .modalias = "ads7846", > > > + .bus_num = 1, > > > + .chip_select = 0, > > > + .max_speed_hz = 1500000, > > > + .controller_data = &ads7846_mcspi_config, > > > + .irq = OMAP_GPIO_IRQ(TS_GPIO), > > > + .platform_data = &ads7846_config, > > > + }, > > > +}; > > > + > > > static void __init omap3_evm_init_irq(void) > > > { > > > omap2_init_common_hw(); > > > @@ -90,6 +90,10 @@ static void __init omap3_evm_init(void) > > > platform_add_devices(omap3_evm_devices, ARRAY_SIZE(omap3_evm_devices)); > > > omap_board_config = omap3_evm_config; > > > omap_board_config_size = ARRAY_SIZE(omap3_evm_config); > > > + > > > + spi_register_board_info(omap3evm_spi_board_info, > > > + ARRAY_SIZE(omap3evm_spi_board_info)); > > > + > > > omap_serial_init(); > > > hsmmc_init(); > > > usb_musb_init(); > > > @@ -95,6 +136,7 @@ > > > usb_musb_init(); > > > usb_ehci_init(); > > > omap3evm_flash_init(); > > > + ads7846_dev_init(); > > > } > > > > > > arch_initcall(omap3_evm_i2c_init); > > > diff -uprN -X a/Documentation/dontdiff > > > a/include/asm-arm/arch-omap/board-omap3evm.h > > > b/include/asm-arm/arch-omap/board-omap3evm.h > > > --- a/include/asm-arm/arch-omap/board-omap3evm.h 2008-04-28 > > > 21:03:19.000000000 -0700 > > > +++ b/include/asm-arm/arch-omap/board-omap3evm.h 2008-04-28 > > > 21:11:29.000000000 -0700 > > > @@ -31,6 +31,7 @@ > > > > > > extern void omap3evm_flash_init(void); > > > > > > +#define TS_GPIO 175 > > > > This should be OMAP3_EVM_TS_GPIO or similar. Otherwise things will > > get redefined when compiling in support for multiple boards. > > > > > > > #define TWL4030_IRQNUM INT_34XX_SYS_NIRQ > > > > And this should really be OMAP3_TWL4030_IRQNUM defined in 34xx.h. > > > > Tony > > > > > > > > > > #define ONENAND_MAP 0x20000000 > > > -- > > > 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 > > -- 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