* Hiremath, Vaibhav <hvaibhav@xxxxxx> [091125 21:13]: > > > -----Original Message----- > > From: Hiremath, Vaibhav > > Sent: Thursday, November 26, 2009 10:41 AM > > To: linux-omap@xxxxxxxxxxxxxxx > > Cc: tony@xxxxxxxxxxx; Hiremath, Vaibhav > > Subject: [PATCH V2] AM3517: Add support for TSC2004 driver > > > > From: Vaibhav Hiremath <hvaibhav@xxxxxx> > > > > Changes: > > - Removed omap_cfg_reg() > > > > Reviewed-by: Tony Lindgren <tony@xxxxxxxxxxx> > > Signed-off-by: Vaibhav Hiremath <hvaibhav@xxxxxx> > > --- > > arch/arm/mach-omap2/board-am3517evm.c | 61 > > +++++++++++++++++++++++++++++++++ > > 1 files changed, 61 insertions(+), 0 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/board-am3517evm.c b/arch/arm/mach- > > omap2/board-am3517evm.c > > index 415a13d..b183d93 100644 > > --- a/arch/arm/mach-omap2/board-am3517evm.c > > +++ b/arch/arm/mach-omap2/board-am3517evm.c > > @@ -20,6 +20,8 @@ > > #include <linux/init.h> > > #include <linux/platform_device.h> > > #include <linux/gpio.h> > > +#include <linux/irq.h> > > +#include <linux/i2c/tsc2004.h> > > > > #include <mach/hardware.h> > > #include <asm/mach-types.h> > > @@ -27,10 +29,64 @@ > > #include <asm/mach/map.h> > > > > #include <plat/board.h> > > +#include <plat/mux.h> > > #include <plat/common.h> > > #include <plat/usb.h> > > > > /* > > + * TSC 2004 Support > > + */ > > +#define GPIO_TSC2004_IRQ 65 > > + > > +static int tsc2004_init_irq(void) > > +{ > > + int ret = 0; > > + > > + ret = gpio_request(GPIO_TSC2004_IRQ, "tsc2004-irq"); > > + if (ret < 0) { > > + printk(KERN_WARNING "failed to request GPIO#%d: %d\n", > > + GPIO_TSC2004_IRQ, ret); > > + return ret; > > + } > > + > > + if (gpio_direction_input(GPIO_TSC2004_IRQ)) { > > + printk(KERN_WARNING "GPIO#%d cannot be configured as " > > + "input\n", GPIO_TSC2004_IRQ); > > + return -ENXIO; > > + } > > + > > + omap_set_gpio_debounce(GPIO_TSC2004_IRQ, 1); > > + omap_set_gpio_debounce_time(GPIO_TSC2004_IRQ, 0xa); > > + return ret; > > +} > > + > > +static void tsc2004_exit_irq(void) > > +{ > > + gpio_free(GPIO_TSC2004_IRQ); > > +} > > + > > +static int tsc2004_get_irq_level(void) > > +{ > > + return gpio_get_value(GPIO_TSC2004_IRQ) ? 0 : 1; > > +} > > + > > +struct tsc2004_platform_data am3517evm_tsc2004data = { > > + .model = 2004, > > + .x_plate_ohms = 180, > > + .get_pendown_state = tsc2004_get_irq_level, > > + .init_platform_hw = tsc2004_init_irq, > > + .exit_platform_hw = tsc2004_exit_irq, > > +}; > > + > > +static struct i2c_board_info __initdata > > am3517evm_tsc_i2c_boardinfo[] = { > > + { > > + I2C_BOARD_INFO("tsc2004", 0x4B), > > + .type = "tsc2004", > > + .platform_data = &am3517evm_tsc2004data, > > + }, > > +}; > > + > > +/* > > * Board initialization > > */ > > static struct omap_board_config_kernel am3517_evm_config[] > > __initdata = { > > @@ -67,6 +123,11 @@ static void __init am3517_evm_init(void) > > > > omap_serial_init(); > > usb_ehci_init(&ehci_pdata); > > + > > + /* TSC 2004 */ > > + am3517evm_tsc_i2c_boardinfo[0].irq = > > gpio_to_irq(GPIO_TSC2004_IRQ); > > + i2c_register_board_info(1, am3517evm_tsc_i2c_boardinfo, > > + ARRAY_SIZE(am3517evm_tsc_i2c_boardinfo)); > > } > > > > static void __init am3517_evm_map_io(void) > [Hiremath, Vaibhav] Hi Tony, > > Since I haven't received any comments on driver code it should get merged. But I am not sure what the procedure is for input subsystem drivers? Is it happens through linux-omap or linux-input? I'd rather see it merged via linux-input as that's where most of the code is. The platform init code looks OK to me to merge via linux-input too. Tony -- 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