* [PATCH 1/3] OMAP3 EVM: Add twl4030 keypad driver support
@ 2008-06-23 21:51 Miguel Angel Aguilar Ulloa
2008-06-23 22:44 ` Felipe Balbi
2008-06-25 14:04 ` arun c
0 siblings, 2 replies; 7+ messages in thread
From: Miguel Angel Aguilar Ulloa @ 2008-06-23 21:51 UTC (permalink / raw)
To: linux-omap
Add twl4030 keypad driver support to OMAP3 EVM
Signed-off-by: Miguel Angel Aguilar <miguel.aguilar@ridgerun.com>
--- a/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23 15:07:13.000000000 -0600
+++ b/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23 14:32:29.000000000 -0600
@@ -19,6 +19,7 @@
#include <linux/err.h>
#include <linux/clk.h>
#include <linux/io.h>
+#include <linux/input.h>
#include <linux/spi/spi.h>
#include <linux/spi/ads7846.h>
@@ -28,6 +29,7 @@
#include <asm/mach/map.h>
#include <asm/arch/gpio.h>
+#include <asm/arch/keypad.h>
#include <asm/arch/board.h>
#include <asm/arch/hsmmc.h>
#include <asm/arch/usb-musb.h>
@@ -149,6 +151,56 @@ struct spi_board_info omap3evm_spi_board
},
};
+static int omap3evm_keymap[] = {
+ KEY(0, 0, KEY_LEFT),
+ KEY(0, 1, KEY_RIGHT),
+ KEY(0, 2, KEY_A),
+ KEY(0, 3, KEY_B),
+ KEY(0, 4, KEY_C),
+ KEY(1, 0, KEY_DOWN),
+ KEY(1, 1, KEY_UP),
+ KEY(1, 2, KEY_E),
+ KEY(1, 3, KEY_F),
+ KEY(1, 4, KEY_G),
+ KEY(2, 0, KEY_ENTER),
+ KEY(2, 1, KEY_I),
+ KEY(2, 2, KEY_J),
+ KEY(2, 3, KEY_K),
+ KEY(2, 4, KEY_3),
+ KEY(3, 0, KEY_M),
+ KEY(3, 1, KEY_N),
+ KEY(3, 2, KEY_O),
+ KEY(3, 3, KEY_P),
+ KEY(3, 4, KEY_Q),
+ KEY(4, 0, KEY_R),
+ KEY(4, 1, KEY_4),
+ KEY(4, 2, KEY_T),
+ KEY(4, 3, KEY_U),
+ KEY(4, 4, KEY_D),
+ KEY(5, 0, KEY_V),
+ KEY(5, 1, KEY_W),
+ KEY(5, 2, KEY_L),
+ KEY(5, 3, KEY_S),
+ KEY(5, 4, KEY_H),
+ 0
+};
+
+static struct omap_kp_platform_data omap3evm_kp_data = {
+ .rows = 4,
+ .cols = 4,
+ .keymap = omap3evm_keymap,
+ .keymapsize = ARRAY_SIZE(omap3evm_keymap),
+ .rep = 1,
+};
+
+static struct platform_device omap3evm_kp_device = {
+ .name = "omap_twl4030keypad",
+ .id = -1,
+ .dev = {
+ .platform_data = &omap3evm_kp_data,
+ },
+};
+
static void __init omap3_evm_init_irq(void)
{
omap2_init_common_hw();
@@ -165,6 +217,7 @@ static struct omap_board_config_kernel o
static struct platform_device *omap3_evm_devices[] __initdata = {
&omap3_evm_lcd_device,
+ &omap3evm_kp_device,
#ifdef CONFIG_RTC_DRV_TWL4030
&omap3_evm_twl4030rtc_device,
#endif
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] OMAP3 EVM: Add twl4030 keypad driver support
2008-06-23 21:51 [PATCH 1/3] OMAP3 EVM: Add twl4030 keypad driver support Miguel Angel Aguilar Ulloa
@ 2008-06-23 22:44 ` Felipe Balbi
2008-06-25 14:04 ` arun c
1 sibling, 0 replies; 7+ messages in thread
From: Felipe Balbi @ 2008-06-23 22:44 UTC (permalink / raw)
To: Miguel Angel Aguilar Ulloa; +Cc: linux-omap
On Mon, Jun 23, 2008 at 03:51:09PM -0600, Miguel Angel Aguilar Ulloa wrote:
> Add twl4030 keypad driver support to OMAP3 EVM
Generally all your patches look really good but the subject should be
something more self-explanatory.
For this first is ok, but second and third should be something like:
OMAP3: EVM: KEYPAD: Fix Kconfig dependency
OMAP3: EVM: KEYPAD: Add twl4030 keypad to defconfig
or something similar ;-)
--
Best Regards,
Felipe Balbi
me@felipebalbi.com
http://blog.felipebalbi.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/3] OMAP3: EVM: Add twl4030 keypad driver support
@ 2008-06-24 13:59 Miguel Angel Aguilar Ulloa
0 siblings, 0 replies; 7+ messages in thread
From: Miguel Angel Aguilar Ulloa @ 2008-06-24 13:59 UTC (permalink / raw)
To: linux-omap
Add twl4030 keypad driver support to OMAP3 EVM
Signed-off-by: Miguel Angel Aguilar <miguel.aguilar@ridgerun.com>
--- a/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23 15:07:13.000000000 -0600
+++ b/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23 14:32:29.000000000 -0600
@@ -19,6 +19,7 @@
#include <linux/err.h>
#include <linux/clk.h>
#include <linux/io.h>
+#include <linux/input.h>
#include <linux/spi/spi.h>
#include <linux/spi/ads7846.h>
@@ -28,6 +29,7 @@
#include <asm/mach/map.h>
#include <asm/arch/gpio.h>
+#include <asm/arch/keypad.h>
#include <asm/arch/board.h>
#include <asm/arch/hsmmc.h>
#include <asm/arch/usb-musb.h>
@@ -149,6 +151,56 @@ struct spi_board_info omap3evm_spi_board
},
};
+static int omap3evm_keymap[] = {
+ KEY(0, 0, KEY_LEFT),
+ KEY(0, 1, KEY_RIGHT),
+ KEY(0, 2, KEY_A),
+ KEY(0, 3, KEY_B),
+ KEY(0, 4, KEY_C),
+ KEY(1, 0, KEY_DOWN),
+ KEY(1, 1, KEY_UP),
+ KEY(1, 2, KEY_E),
+ KEY(1, 3, KEY_F),
+ KEY(1, 4, KEY_G),
+ KEY(2, 0, KEY_ENTER),
+ KEY(2, 1, KEY_I),
+ KEY(2, 2, KEY_J),
+ KEY(2, 3, KEY_K),
+ KEY(2, 4, KEY_3),
+ KEY(3, 0, KEY_M),
+ KEY(3, 1, KEY_N),
+ KEY(3, 2, KEY_O),
+ KEY(3, 3, KEY_P),
+ KEY(3, 4, KEY_Q),
+ KEY(4, 0, KEY_R),
+ KEY(4, 1, KEY_4),
+ KEY(4, 2, KEY_T),
+ KEY(4, 3, KEY_U),
+ KEY(4, 4, KEY_D),
+ KEY(5, 0, KEY_V),
+ KEY(5, 1, KEY_W),
+ KEY(5, 2, KEY_L),
+ KEY(5, 3, KEY_S),
+ KEY(5, 4, KEY_H),
+ 0
+};
+
+static struct omap_kp_platform_data omap3evm_kp_data = {
+ .rows = 4,
+ .cols = 4,
+ .keymap = omap3evm_keymap,
+ .keymapsize = ARRAY_SIZE(omap3evm_keymap),
+ .rep = 1,
+};
+
+static struct platform_device omap3evm_kp_device = {
+ .name = "omap_twl4030keypad",
+ .id = -1,
+ .dev = {
+ .platform_data = &omap3evm_kp_data,
+ },
+};
+
static void __init omap3_evm_init_irq(void)
{
omap2_init_common_hw();
@@ -165,6 +217,7 @@ static struct omap_board_config_kernel o
static struct platform_device *omap3_evm_devices[] __initdata = {
&omap3_evm_lcd_device,
+ &omap3evm_kp_device,
#ifdef CONFIG_RTC_DRV_TWL4030
&omap3_evm_twl4030rtc_device,
#endif
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] OMAP3 EVM: Add twl4030 keypad driver support
2008-06-23 21:51 [PATCH 1/3] OMAP3 EVM: Add twl4030 keypad driver support Miguel Angel Aguilar Ulloa
2008-06-23 22:44 ` Felipe Balbi
@ 2008-06-25 14:04 ` arun c
2008-06-25 15:13 ` Miguel Angel Aguilar Ulloa
1 sibling, 1 reply; 7+ messages in thread
From: arun c @ 2008-06-25 14:04 UTC (permalink / raw)
To: Miguel Angel Aguilar Ulloa; +Cc: linux-omap
On Tue, Jun 24, 2008 at 3:21 AM, Miguel Angel Aguilar Ulloa
<miguel.aguilar@ridgerun.com> wrote:
> Add twl4030 keypad driver support to OMAP3 EVM
>
> Signed-off-by: Miguel Angel Aguilar <miguel.aguilar@ridgerun.com>
>
> --- a/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23
> 15:07:13.000000000 -0600
> +++ b/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23
> 14:32:29.000000000 -0600
> @@ -19,6 +19,7 @@
> #include <linux/err.h>
> #include <linux/clk.h>
> #include <linux/io.h>
> +#include <linux/input.h>
> #include <linux/spi/spi.h>
> #include <linux/spi/ads7846.h>
>
> @@ -28,6 +29,7 @@
> #include <asm/mach/map.h>
>
> #include <asm/arch/gpio.h>
> +#include <asm/arch/keypad.h>
> #include <asm/arch/board.h>
> #include <asm/arch/hsmmc.h>
> #include <asm/arch/usb-musb.h>
> @@ -149,6 +151,56 @@ struct spi_board_info omap3evm_spi_board
> },
> };
>
> +static int omap3evm_keymap[] = {
> + KEY(0, 0, KEY_LEFT),
> + KEY(0, 1, KEY_RIGHT),
> + KEY(0, 2, KEY_A),
> + KEY(0, 3, KEY_B),
> + KEY(0, 4, KEY_C),
> + KEY(1, 0, KEY_DOWN),
> + KEY(1, 1, KEY_UP),
> + KEY(1, 2, KEY_E),
> + KEY(1, 3, KEY_F),
> + KEY(1, 4, KEY_G),
> + KEY(2, 0, KEY_ENTER),
> + KEY(2, 1, KEY_I),
> + KEY(2, 2, KEY_J),
> + KEY(2, 3, KEY_K),
> + KEY(2, 4, KEY_3),
> + KEY(3, 0, KEY_M),
> + KEY(3, 1, KEY_N),
> + KEY(3, 2, KEY_O),
> + KEY(3, 3, KEY_P),
> + KEY(3, 4, KEY_Q),
> + KEY(4, 0, KEY_R),
> + KEY(4, 1, KEY_4),
> + KEY(4, 2, KEY_T),
> + KEY(4, 3, KEY_U),
> + KEY(4, 4, KEY_D),
> + KEY(5, 0, KEY_V),
> + KEY(5, 1, KEY_W),
> + KEY(5, 2, KEY_L),
> + KEY(5, 3, KEY_S),
> + KEY(5, 4, KEY_H),
> + 0
> +};
> +
Omap3evm has only 15 keys right?(Some body correct me if I am wrong).
Then how the
mapping and dividing into rows and columns is done?
> +static struct omap_kp_platform_data omap3evm_kp_data = {
> + .rows = 4,
> + .cols = 4,
> + .keymap = omap3evm_keymap,
> + .keymapsize = ARRAY_SIZE(omap3evm_keymap),
> + .rep = 1,
> +};
> +
> +static struct platform_device omap3evm_kp_device = {
> + .name = "omap_twl4030keypad",
> + .id = -1,
> + .dev = {
> + .platform_data = &omap3evm_kp_data,
> + },
> +};
> +
> static void __init omap3_evm_init_irq(void)
> {
> omap2_init_common_hw();
> @@ -165,6 +217,7 @@ static struct omap_board_config_kernel o
>
> static struct platform_device *omap3_evm_devices[] __initdata = {
> &omap3_evm_lcd_device,
> + &omap3evm_kp_device,
> #ifdef CONFIG_RTC_DRV_TWL4030
> &omap3_evm_twl4030rtc_device,
> #endif
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] OMAP3 EVM: Add twl4030 keypad driver support
2008-06-25 14:04 ` arun c
@ 2008-06-25 15:13 ` Miguel Angel Aguilar Ulloa
2008-06-26 6:29 ` arun c
0 siblings, 1 reply; 7+ messages in thread
From: Miguel Angel Aguilar Ulloa @ 2008-06-25 15:13 UTC (permalink / raw)
To: arun c; +Cc: linux-omap, Diego Dompe, Todd Fischer
The OMAP3 EVM has 15 keys arranged in a matrix from SW4 to SW18, but it also has
a user key (power on/off) in SW3, this key is included in the keypad mapping.
This is how mapping is done in an 4x4 array.
arun c wrote:
> On Tue, Jun 24, 2008 at 3:21 AM, Miguel Angel Aguilar Ulloa
> <miguel.aguilar@ridgerun.com> wrote:
>> Add twl4030 keypad driver support to OMAP3 EVM
>>
>> Signed-off-by: Miguel Angel Aguilar <miguel.aguilar@ridgerun.com>
>>
>> --- a/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23
>> 15:07:13.000000000 -0600
>> +++ b/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23
>> 14:32:29.000000000 -0600
>> @@ -19,6 +19,7 @@
>> #include <linux/err.h>
>> #include <linux/clk.h>
>> #include <linux/io.h>
>> +#include <linux/input.h>
>> #include <linux/spi/spi.h>
>> #include <linux/spi/ads7846.h>
>>
>> @@ -28,6 +29,7 @@
>> #include <asm/mach/map.h>
>>
>> #include <asm/arch/gpio.h>
>> +#include <asm/arch/keypad.h>
>> #include <asm/arch/board.h>
>> #include <asm/arch/hsmmc.h>
>> #include <asm/arch/usb-musb.h>
>> @@ -149,6 +151,56 @@ struct spi_board_info omap3evm_spi_board
>> },
>> };
>>
>> +static int omap3evm_keymap[] = {
>> + KEY(0, 0, KEY_LEFT),
>> + KEY(0, 1, KEY_RIGHT),
>> + KEY(0, 2, KEY_A),
>> + KEY(0, 3, KEY_B),
>> + KEY(0, 4, KEY_C),
>> + KEY(1, 0, KEY_DOWN),
>> + KEY(1, 1, KEY_UP),
>> + KEY(1, 2, KEY_E),
>> + KEY(1, 3, KEY_F),
>> + KEY(1, 4, KEY_G),
>> + KEY(2, 0, KEY_ENTER),
>> + KEY(2, 1, KEY_I),
>> + KEY(2, 2, KEY_J),
>> + KEY(2, 3, KEY_K),
>> + KEY(2, 4, KEY_3),
>> + KEY(3, 0, KEY_M),
>> + KEY(3, 1, KEY_N),
>> + KEY(3, 2, KEY_O),
>> + KEY(3, 3, KEY_P),
>> + KEY(3, 4, KEY_Q),
>> + KEY(4, 0, KEY_R),
>> + KEY(4, 1, KEY_4),
>> + KEY(4, 2, KEY_T),
>> + KEY(4, 3, KEY_U),
>> + KEY(4, 4, KEY_D),
>> + KEY(5, 0, KEY_V),
>> + KEY(5, 1, KEY_W),
>> + KEY(5, 2, KEY_L),
>> + KEY(5, 3, KEY_S),
>> + KEY(5, 4, KEY_H),
>> + 0
>> +};
>> +
>
> Omap3evm has only 15 keys right?(Some body correct me if I am wrong).
> Then how the
> mapping and dividing into rows and columns is done?
>
>> +static struct omap_kp_platform_data omap3evm_kp_data = {
>> + .rows = 4,
>> + .cols = 4,
>> + .keymap = omap3evm_keymap,
>> + .keymapsize = ARRAY_SIZE(omap3evm_keymap),
>> + .rep = 1,
>> +};
>> +
>> +static struct platform_device omap3evm_kp_device = {
>> + .name = "omap_twl4030keypad",
>> + .id = -1,
>> + .dev = {
>> + .platform_data = &omap3evm_kp_data,
>> + },
>> +};
>> +
>> static void __init omap3_evm_init_irq(void)
>> {
>> omap2_init_common_hw();
>> @@ -165,6 +217,7 @@ static struct omap_board_config_kernel o
>>
>> static struct platform_device *omap3_evm_devices[] __initdata = {
>> &omap3_evm_lcd_device,
>> + &omap3evm_kp_device,
>> #ifdef CONFIG_RTC_DRV_TWL4030
>> &omap3_evm_twl4030rtc_device,
>> #endif
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
--
Miguel Angel Aguilar Ulloa
Embedded Software Engineer
RidgeRun Embedded Solutions
miguel.aguilar@ridgerun.com
Office: +(506) 2225-9596
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] OMAP3 EVM: Add twl4030 keypad driver support
2008-06-25 15:13 ` Miguel Angel Aguilar Ulloa
@ 2008-06-26 6:29 ` arun c
2008-06-26 14:47 ` Miguel Aguilar
0 siblings, 1 reply; 7+ messages in thread
From: arun c @ 2008-06-26 6:29 UTC (permalink / raw)
To: Miguel Angel Aguilar Ulloa; +Cc: linux-omap
On Wed, Jun 25, 2008 at 8:43 PM, Miguel Angel Aguilar Ulloa
<miguel.aguilar@ridgerun.com> wrote:
> The OMAP3 EVM has 15 keys arranged in a matrix from SW4 to SW18, but it also
> has a user key (power on/off) in SW3, this key is included in the keypad
> mapping. This is how mapping is done in an 4x4 array.
>
> arun c wrote:
>>
>> On Tue, Jun 24, 2008 at 3:21 AM, Miguel Angel Aguilar Ulloa
>> <miguel.aguilar@ridgerun.com> wrote:
>>>
>>> Add twl4030 keypad driver support to OMAP3 EVM
>>>
>>> Signed-off-by: Miguel Angel Aguilar <miguel.aguilar@ridgerun.com>
>>>
>>> --- a/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23
>>> 15:07:13.000000000 -0600
>>> +++ b/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23
>>> 14:32:29.000000000 -0600
>>> @@ -19,6 +19,7 @@
>>> #include <linux/err.h>
>>> #include <linux/clk.h>
>>> #include <linux/io.h>
>>> +#include <linux/input.h>
>>> #include <linux/spi/spi.h>
>>> #include <linux/spi/ads7846.h>
>>>
>>> @@ -28,6 +29,7 @@
>>> #include <asm/mach/map.h>
>>>
>>> #include <asm/arch/gpio.h>
>>> +#include <asm/arch/keypad.h>
>>> #include <asm/arch/board.h>
>>> #include <asm/arch/hsmmc.h>
>>> #include <asm/arch/usb-musb.h>
>>> @@ -149,6 +151,56 @@ struct spi_board_info omap3evm_spi_board
>>> },
>>> };
>>>
Yes option is there to add sw3 to key matrix. So we have a total of 16
keys, then
why there are more no of keys(30keys) in omap3evm_keymap[] array?
>>> +static int omap3evm_keymap[] = {
>>> + KEY(0, 0, KEY_LEFT),
>>> + KEY(0, 1, KEY_RIGHT),
>>> + KEY(0, 2, KEY_A),
>>> + KEY(0, 3, KEY_B),
>>> + KEY(0, 4, KEY_C),
>>> + KEY(1, 0, KEY_DOWN),
>>> + KEY(1, 1, KEY_UP),
>>> + KEY(1, 2, KEY_E),
>>> + KEY(1, 3, KEY_F),
>>> + KEY(1, 4, KEY_G),
>>> + KEY(2, 0, KEY_ENTER),
>>> + KEY(2, 1, KEY_I),
>>> + KEY(2, 2, KEY_J),
>>> + KEY(2, 3, KEY_K),
>>> + KEY(2, 4, KEY_3),
>>> + KEY(3, 0, KEY_M),
>>> + KEY(3, 1, KEY_N),
>>> + KEY(3, 2, KEY_O),
>>> + KEY(3, 3, KEY_P),
>>> + KEY(3, 4, KEY_Q),
>>> + KEY(4, 0, KEY_R),
>>> + KEY(4, 1, KEY_4),
>>> + KEY(4, 2, KEY_T),
>>> + KEY(4, 3, KEY_U),
>>> + KEY(4, 4, KEY_D),
>>> + KEY(5, 0, KEY_V),
>>> + KEY(5, 1, KEY_W),
>>> + KEY(5, 2, KEY_L),
>>> + KEY(5, 3, KEY_S),
>>> + KEY(5, 4, KEY_H),
>>> + 0
>>> +};
>>> +
>>
>> Omap3evm has only 15 keys right?(Some body correct me if I am wrong).
>> Then how the
>> mapping and dividing into rows and columns is done?
>>
>>> +static struct omap_kp_platform_data omap3evm_kp_data = {
>>> + .rows = 4,
>>> + .cols = 4,
>>> + .keymap = omap3evm_keymap,
>>> + .keymapsize = ARRAY_SIZE(omap3evm_keymap),
>>> + .rep = 1,
>>> +};
>>> +
>>> +static struct platform_device omap3evm_kp_device = {
>>> + .name = "omap_twl4030keypad",
>>> + .id = -1,
>>> + .dev = {
>>> + .platform_data = &omap3evm_kp_data,
>>> + },
>>> +};
>>> +
>>> static void __init omap3_evm_init_irq(void)
>>> {
>>> omap2_init_common_hw();
>>> @@ -165,6 +217,7 @@ static struct omap_board_config_kernel o
>>>
>>> static struct platform_device *omap3_evm_devices[] __initdata = {
>>> &omap3_evm_lcd_device,
>>> + &omap3evm_kp_device,
>>> #ifdef CONFIG_RTC_DRV_TWL4030
>>> &omap3_evm_twl4030rtc_device,
>>> #endif
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>>
>
>
> --
> Miguel Angel Aguilar Ulloa
> Embedded Software Engineer
> RidgeRun Embedded Solutions
> miguel.aguilar@ridgerun.com
> Office: +(506) 2225-9596
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] OMAP3 EVM: Add twl4030 keypad driver support
2008-06-26 6:29 ` arun c
@ 2008-06-26 14:47 ` Miguel Aguilar
0 siblings, 0 replies; 7+ messages in thread
From: Miguel Aguilar @ 2008-06-26 14:47 UTC (permalink / raw)
To: arun c; +Cc: linux-omap
arun c wrote:
> On Wed, Jun 25, 2008 at 8:43 PM, Miguel Angel Aguilar Ulloa
> <miguel.aguilar@ridgerun.com> wrote:
>> The OMAP3 EVM has 15 keys arranged in a matrix from SW4 to SW18, but it also
>> has a user key (power on/off) in SW3, this key is included in the keypad
>> mapping. This is how mapping is done in an 4x4 array.
>>
>> arun c wrote:
>>> On Tue, Jun 24, 2008 at 3:21 AM, Miguel Angel Aguilar Ulloa
>>> <miguel.aguilar@ridgerun.com> wrote:
>>>> Add twl4030 keypad driver support to OMAP3 EVM
>>>>
>>>> Signed-off-by: Miguel Angel Aguilar <miguel.aguilar@ridgerun.com>
>>>>
>>>> --- a/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23
>>>> 15:07:13.000000000 -0600
>>>> +++ b/arch/arm/mach-omap2/board-omap3evm.c 2008-06-23
>>>> 14:32:29.000000000 -0600
>>>> @@ -19,6 +19,7 @@
>>>> #include <linux/err.h>
>>>> #include <linux/clk.h>
>>>> #include <linux/io.h>
>>>> +#include <linux/input.h>
>>>> #include <linux/spi/spi.h>
>>>> #include <linux/spi/ads7846.h>
>>>>
>>>> @@ -28,6 +29,7 @@
>>>> #include <asm/mach/map.h>
>>>>
>>>> #include <asm/arch/gpio.h>
>>>> +#include <asm/arch/keypad.h>
>>>> #include <asm/arch/board.h>
>>>> #include <asm/arch/hsmmc.h>
>>>> #include <asm/arch/usb-musb.h>
>>>> @@ -149,6 +151,56 @@ struct spi_board_info omap3evm_spi_board
>>>> },
>>>> };
>>>>
>
> Yes option is there to add sw3 to key matrix. So we have a total of 16
> keys, then
> why there are more no of keys(30keys) in omap3evm_keymap[] array?
>
Check new patch version
>
>>>> +static int omap3evm_keymap[] = {
>>>> + KEY(0, 0, KEY_LEFT),
>>>> + KEY(0, 1, KEY_RIGHT),
>>>> + KEY(0, 2, KEY_A),
>>>> + KEY(0, 3, KEY_B),
>>>> + KEY(0, 4, KEY_C),
>>>> + KEY(1, 0, KEY_DOWN),
>>>> + KEY(1, 1, KEY_UP),
>>>> + KEY(1, 2, KEY_E),
>>>> + KEY(1, 3, KEY_F),
>>>> + KEY(1, 4, KEY_G),
>>>> + KEY(2, 0, KEY_ENTER),
>>>> + KEY(2, 1, KEY_I),
>>>> + KEY(2, 2, KEY_J),
>>>> + KEY(2, 3, KEY_K),
>>>> + KEY(2, 4, KEY_3),
>>>> + KEY(3, 0, KEY_M),
>>>> + KEY(3, 1, KEY_N),
>>>> + KEY(3, 2, KEY_O),
>>>> + KEY(3, 3, KEY_P),
>>>> + KEY(3, 4, KEY_Q),
>>>> + KEY(4, 0, KEY_R),
>>>> + KEY(4, 1, KEY_4),
>>>> + KEY(4, 2, KEY_T),
>>>> + KEY(4, 3, KEY_U),
>>>> + KEY(4, 4, KEY_D),
>>>> + KEY(5, 0, KEY_V),
>>>> + KEY(5, 1, KEY_W),
>>>> + KEY(5, 2, KEY_L),
>>>> + KEY(5, 3, KEY_S),
>>>> + KEY(5, 4, KEY_H),
>>>> + 0
>>>> +};
>>>> +
>>> Omap3evm has only 15 keys right?(Some body correct me if I am wrong).
>>> Then how the
>>> mapping and dividing into rows and columns is done?
>>>
>>>> +static struct omap_kp_platform_data omap3evm_kp_data = {
>>>> + .rows = 4,
>>>> + .cols = 4,
>>>> + .keymap = omap3evm_keymap,
>>>> + .keymapsize = ARRAY_SIZE(omap3evm_keymap),
>>>> + .rep = 1,
>>>> +};
>>>> +
>>>> +static struct platform_device omap3evm_kp_device = {
>>>> + .name = "omap_twl4030keypad",
>>>> + .id = -1,
>>>> + .dev = {
>>>> + .platform_data = &omap3evm_kp_data,
>>>> + },
>>>> +};
>>>> +
>>>> static void __init omap3_evm_init_irq(void)
>>>> {
>>>> omap2_init_common_hw();
>>>> @@ -165,6 +217,7 @@ static struct omap_board_config_kernel o
>>>>
>>>> static struct platform_device *omap3_evm_devices[] __initdata = {
>>>> &omap3_evm_lcd_device,
>>>> + &omap3evm_kp_device,
>>>> #ifdef CONFIG_RTC_DRV_TWL4030
>>>> &omap3_evm_twl4030rtc_device,
>>>> #endif
>>>> --
>>>> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
>>>> the body of a message to majordomo@vger.kernel.org
>>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>>>
>>
>> --
>> Miguel Angel Aguilar Ulloa
>> Embedded Software Engineer
>> RidgeRun Embedded Solutions
>> miguel.aguilar@ridgerun.com
>> Office: +(506) 2225-9596
>>
--
Miguel Angel Aguilar Ulloa
Embedded Software Engineer
RidgeRun Embedded Solutions
miguel.aguilar@ridgerun.com
Office: +(506) 2225-9596
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2008-06-26 14:43 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-06-23 21:51 [PATCH 1/3] OMAP3 EVM: Add twl4030 keypad driver support Miguel Angel Aguilar Ulloa
2008-06-23 22:44 ` Felipe Balbi
2008-06-25 14:04 ` arun c
2008-06-25 15:13 ` Miguel Angel Aguilar Ulloa
2008-06-26 6:29 ` arun c
2008-06-26 14:47 ` Miguel Aguilar
-- strict thread matches above, loose matches on Subject: below --
2008-06-24 13:59 [PATCH 1/3] OMAP3: " Miguel Angel Aguilar Ulloa
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox