* [PATCHv7] arm/dts: omap4-sdp: Add keypad data
@ 2012-08-14 11:15 Sourav Poddar
2012-08-20 11:19 ` Poddar, Sourav
0 siblings, 1 reply; 5+ messages in thread
From: Sourav Poddar @ 2012-08-14 11:15 UTC (permalink / raw)
To: devicetree-discuss, linux-arm-kernel, linux-omap, linux-kernel
Cc: Sourav Poddar, Tony Lindgren, Benoit Cousson, Rob Herring,
Grant Likely, Felipe Balbi
Add keypad data node in omap4 device tree file.
Also fill the device tree binding parameters
with the required value in "omap4-sdp" dts file.
Tested on omap4430 sdp.
Cc: Tony Lindgren <tony@atomide.com>
Cc: Benoit Cousson <b-cousson@ti.com>
Cc: Rob Herring <rob.herring@calxeda.com>
Cc: Grant Likely <grant.likely@secretlab.ca>
Cc: Felipe Balbi <balbi@ti.com>
Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
---
v6->v7
Add key name as a comment.
arch/arm/boot/dts/omap4-sdp.dts | 70 +++++++++++++++++++++++++++++++++++++++
arch/arm/boot/dts/omap4.dtsi | 5 +++
2 files changed, 75 insertions(+), 0 deletions(-)
diff --git a/arch/arm/boot/dts/omap4-sdp.dts b/arch/arm/boot/dts/omap4-sdp.dts
index 72216e9..8e200f4 100644
--- a/arch/arm/boot/dts/omap4-sdp.dts
+++ b/arch/arm/boot/dts/omap4-sdp.dts
@@ -113,6 +113,76 @@
"DMic", "Digital Mic",
"Digital Mic", "Digital Mic1 Bias";
};
+
+ keypad {
+ keypad,num-rows = <8>;
+ keypad,num-columns = <8>;
+ linux,keymap = < 0x00000012 /* KEY_E */
+ 0x00010013 /* KEY_R */
+ 0x00020014 /* KEY_T */
+ 0x00030066 /* KEY_HOME */
+ 0x0004003f /* KEY_F5 */
+ 0x000500f0 /* KEY_UNKNOWN */
+ 0x00060017 /* KEY_I */
+ 0x0007002a /* KEY_LEFTSHIFT */
+ 0x01000020 /* KEY_D*/
+ 0x01010021 /* KEY_F */
+ 0x01020022 /* KEY_G */
+ 0x010300e7 /* KEY_SEND */
+ 0x01040040 /* KEY_F6 */
+ 0x010500f0 /* KEY_UNKNOWN */
+ 0x01060025 /* KEY_K */
+ 0x0107001c /* KEY_ENTER */
+ 0x0200002d /* KEY_X */
+ 0x0201002e /* KEY_C */
+ 0x0202002f /* KEY_V */
+ 0x0203006b /* KEY_END */
+ 0x02040041 /* KEY_F7 */
+ 0x020500f0 /* KEY_UNKNOWN */
+ 0x02060034 /* KEY_DOT */
+ 0x0207003a /* KEY_CAPSLOCK */
+ 0x0300002c /* KEY_Z */
+ 0x0301004e /* KEY_KPLUS */
+ 0x03020030 /* KEY_B */
+ 0x0303003b /* KEY_F1 */
+ 0x03040042 /* KEY_F8 */
+ 0x030500f0 /* KEY_UNKNOWN */
+ 0x03060018 /* KEY_O */
+ 0x03070039 /* KEY_SPACE */
+ 0x04000011 /* KEY_W */
+ 0x04010015 /* KEY_Y */
+ 0x04020016 /* KEY_U */
+ 0x0403003c /* KEY_F2 */
+ 0x04040073 /* KEY_VOLUMEUP */
+ 0x040500f0 /* KEY_UNKNOWN */
+ 0x04060026 /* KEY_L */
+ 0x04070069 /* KEY_LEFT */
+ 0x0500001f /* KEY_S */
+ 0x05010023 /* KEY_H */
+ 0x05020024 /* KEY_J */
+ 0x0503003d /* KEY_F3 */
+ 0x05040043 /* KEY_F9 */
+ 0x05050072 /* KEY_VOLUMEDOWN */
+ 0x05060032 /* KEY_M */
+ 0x0507006a /* KEY_RIGHT */
+ 0x06000010 /* KEY_Q */
+ 0x0601001e /* KEY_A */
+ 0x06020031 /* KEY_N */
+ 0x0603009e /* KEY_BACK */
+ 0x0604000e /* KEY_BACKSPACE */
+ 0x060500f0 /* KEY_UNKNOWN */
+ 0x06060019 /* KEY_P */
+ 0x06070067 /* KEY_UP */
+ 0x07000094 /* KEY_PROG1 */
+ 0x07010095 /* KEY_PROG2 */
+ 0x070200ca /* KEY_PROG3 */
+ 0x070300cb /* KEY_PROG4 */
+ 0x0704003e /* KEY_F4 */
+ 0x070500f0 /* KEY_UNKNOWN */
+ 0x07060160 /* KEY_OK */
+ 0x0707006c >; /* KEY_DOWN */
+ linux,input-no-autorepeat;
+ };
};
&i2c1 {
diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
index 04cbbcb..e78a1f1 100644
--- a/arch/arm/boot/dts/omap4.dtsi
+++ b/arch/arm/boot/dts/omap4.dtsi
@@ -296,4 +296,9 @@
ti,hwmods = "dmic";
};
};
+
+ keypad {
+ compatible = "ti,omap4-keypad";
+ ti,hwmods = "kbd";
+ };
};
--
1.7.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCHv7] arm/dts: omap4-sdp: Add keypad data
2012-08-14 11:15 [PATCHv7] arm/dts: omap4-sdp: Add keypad data Sourav Poddar
@ 2012-08-20 11:19 ` Poddar, Sourav
2012-08-20 11:34 ` Felipe Balbi
2012-08-20 11:53 ` Benoit Cousson
0 siblings, 2 replies; 5+ messages in thread
From: Poddar, Sourav @ 2012-08-20 11:19 UTC (permalink / raw)
To: devicetree-discuss, linux-arm-kernel, linux-omap, linux-kernel
Cc: Sourav Poddar, Tony Lindgren, Benoit Cousson, Rob Herring,
Grant Likely, Felipe Balbi
Hi All,
On Tue, Aug 14, 2012 at 4:45 PM, Sourav Poddar <sourav.poddar@ti.com> wrote:
> Add keypad data node in omap4 device tree file.
> Also fill the device tree binding parameters
> with the required value in "omap4-sdp" dts file.
>
> Tested on omap4430 sdp.
>
> Cc: Tony Lindgren <tony@atomide.com>
> Cc: Benoit Cousson <b-cousson@ti.com>
> Cc: Rob Herring <rob.herring@calxeda.com>
> Cc: Grant Likely <grant.likely@secretlab.ca>
> Cc: Felipe Balbi <balbi@ti.com>
> Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
> ---
> v6->v7
> Add key name as a comment.
> arch/arm/boot/dts/omap4-sdp.dts | 70 +++++++++++++++++++++++++++++++++++++++
> arch/arm/boot/dts/omap4.dtsi | 5 +++
> 2 files changed, 75 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/boot/dts/omap4-sdp.dts b/arch/arm/boot/dts/omap4-sdp.dts
> index 72216e9..8e200f4 100644
> --- a/arch/arm/boot/dts/omap4-sdp.dts
> +++ b/arch/arm/boot/dts/omap4-sdp.dts
> @@ -113,6 +113,76 @@
> "DMic", "Digital Mic",
> "Digital Mic", "Digital Mic1 Bias";
> };
> +
> + keypad {
> + keypad,num-rows = <8>;
> + keypad,num-columns = <8>;
> + linux,keymap = < 0x00000012 /* KEY_E */
> + 0x00010013 /* KEY_R */
> + 0x00020014 /* KEY_T */
> + 0x00030066 /* KEY_HOME */
> + 0x0004003f /* KEY_F5 */
> + 0x000500f0 /* KEY_UNKNOWN */
> + 0x00060017 /* KEY_I */
> + 0x0007002a /* KEY_LEFTSHIFT */
> + 0x01000020 /* KEY_D*/
> + 0x01010021 /* KEY_F */
> + 0x01020022 /* KEY_G */
> + 0x010300e7 /* KEY_SEND */
> + 0x01040040 /* KEY_F6 */
> + 0x010500f0 /* KEY_UNKNOWN */
> + 0x01060025 /* KEY_K */
> + 0x0107001c /* KEY_ENTER */
> + 0x0200002d /* KEY_X */
> + 0x0201002e /* KEY_C */
> + 0x0202002f /* KEY_V */
> + 0x0203006b /* KEY_END */
> + 0x02040041 /* KEY_F7 */
> + 0x020500f0 /* KEY_UNKNOWN */
> + 0x02060034 /* KEY_DOT */
> + 0x0207003a /* KEY_CAPSLOCK */
> + 0x0300002c /* KEY_Z */
> + 0x0301004e /* KEY_KPLUS */
> + 0x03020030 /* KEY_B */
> + 0x0303003b /* KEY_F1 */
> + 0x03040042 /* KEY_F8 */
> + 0x030500f0 /* KEY_UNKNOWN */
> + 0x03060018 /* KEY_O */
> + 0x03070039 /* KEY_SPACE */
> + 0x04000011 /* KEY_W */
> + 0x04010015 /* KEY_Y */
> + 0x04020016 /* KEY_U */
> + 0x0403003c /* KEY_F2 */
> + 0x04040073 /* KEY_VOLUMEUP */
> + 0x040500f0 /* KEY_UNKNOWN */
> + 0x04060026 /* KEY_L */
> + 0x04070069 /* KEY_LEFT */
> + 0x0500001f /* KEY_S */
> + 0x05010023 /* KEY_H */
> + 0x05020024 /* KEY_J */
> + 0x0503003d /* KEY_F3 */
> + 0x05040043 /* KEY_F9 */
> + 0x05050072 /* KEY_VOLUMEDOWN */
> + 0x05060032 /* KEY_M */
> + 0x0507006a /* KEY_RIGHT */
> + 0x06000010 /* KEY_Q */
> + 0x0601001e /* KEY_A */
> + 0x06020031 /* KEY_N */
> + 0x0603009e /* KEY_BACK */
> + 0x0604000e /* KEY_BACKSPACE */
> + 0x060500f0 /* KEY_UNKNOWN */
> + 0x06060019 /* KEY_P */
> + 0x06070067 /* KEY_UP */
> + 0x07000094 /* KEY_PROG1 */
> + 0x07010095 /* KEY_PROG2 */
> + 0x070200ca /* KEY_PROG3 */
> + 0x070300cb /* KEY_PROG4 */
> + 0x0704003e /* KEY_F4 */
> + 0x070500f0 /* KEY_UNKNOWN */
> + 0x07060160 /* KEY_OK */
> + 0x0707006c >; /* KEY_DOWN */
> + linux,input-no-autorepeat;
> + };
> };
>
> &i2c1 {
> diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
> index 04cbbcb..e78a1f1 100644
> --- a/arch/arm/boot/dts/omap4.dtsi
> +++ b/arch/arm/boot/dts/omap4.dtsi
> @@ -296,4 +296,9 @@
> ti,hwmods = "dmic";
> };
> };
> +
> + keypad {
> + compatible = "ti,omap4-keypad";
> + ti,hwmods = "kbd";
> + };
> };
> --
> 1.7.1
>
Any more comments on this?
~Sourav
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCHv7] arm/dts: omap4-sdp: Add keypad data
2012-08-20 11:19 ` Poddar, Sourav
@ 2012-08-20 11:34 ` Felipe Balbi
2012-08-20 11:53 ` Benoit Cousson
1 sibling, 0 replies; 5+ messages in thread
From: Felipe Balbi @ 2012-08-20 11:34 UTC (permalink / raw)
To: Poddar, Sourav
Cc: devicetree-discuss, linux-arm-kernel, linux-omap, linux-kernel,
Tony Lindgren, Benoit Cousson, Rob Herring, Grant Likely,
Felipe Balbi
[-- Attachment #1: Type: text/plain, Size: 5606 bytes --]
On Mon, Aug 20, 2012 at 04:49:36PM +0530, Poddar, Sourav wrote:
> Hi All,
>
> On Tue, Aug 14, 2012 at 4:45 PM, Sourav Poddar <sourav.poddar@ti.com> wrote:
> > Add keypad data node in omap4 device tree file.
> > Also fill the device tree binding parameters
> > with the required value in "omap4-sdp" dts file.
> >
> > Tested on omap4430 sdp.
> >
> > Cc: Tony Lindgren <tony@atomide.com>
> > Cc: Benoit Cousson <b-cousson@ti.com>
> > Cc: Rob Herring <rob.herring@calxeda.com>
> > Cc: Grant Likely <grant.likely@secretlab.ca>
> > Cc: Felipe Balbi <balbi@ti.com>
to me it looks good. you've even added comments on each key for easy
human parsing.
Acked-by: Felipe Balbi <balbi@ti.com>
> > Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
> > ---
> > v6->v7
> > Add key name as a comment.
> > arch/arm/boot/dts/omap4-sdp.dts | 70 +++++++++++++++++++++++++++++++++++++++
> > arch/arm/boot/dts/omap4.dtsi | 5 +++
> > 2 files changed, 75 insertions(+), 0 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/omap4-sdp.dts b/arch/arm/boot/dts/omap4-sdp.dts
> > index 72216e9..8e200f4 100644
> > --- a/arch/arm/boot/dts/omap4-sdp.dts
> > +++ b/arch/arm/boot/dts/omap4-sdp.dts
> > @@ -113,6 +113,76 @@
> > "DMic", "Digital Mic",
> > "Digital Mic", "Digital Mic1 Bias";
> > };
> > +
> > + keypad {
> > + keypad,num-rows = <8>;
> > + keypad,num-columns = <8>;
> > + linux,keymap = < 0x00000012 /* KEY_E */
> > + 0x00010013 /* KEY_R */
> > + 0x00020014 /* KEY_T */
> > + 0x00030066 /* KEY_HOME */
> > + 0x0004003f /* KEY_F5 */
> > + 0x000500f0 /* KEY_UNKNOWN */
> > + 0x00060017 /* KEY_I */
> > + 0x0007002a /* KEY_LEFTSHIFT */
> > + 0x01000020 /* KEY_D*/
> > + 0x01010021 /* KEY_F */
> > + 0x01020022 /* KEY_G */
> > + 0x010300e7 /* KEY_SEND */
> > + 0x01040040 /* KEY_F6 */
> > + 0x010500f0 /* KEY_UNKNOWN */
> > + 0x01060025 /* KEY_K */
> > + 0x0107001c /* KEY_ENTER */
> > + 0x0200002d /* KEY_X */
> > + 0x0201002e /* KEY_C */
> > + 0x0202002f /* KEY_V */
> > + 0x0203006b /* KEY_END */
> > + 0x02040041 /* KEY_F7 */
> > + 0x020500f0 /* KEY_UNKNOWN */
> > + 0x02060034 /* KEY_DOT */
> > + 0x0207003a /* KEY_CAPSLOCK */
> > + 0x0300002c /* KEY_Z */
> > + 0x0301004e /* KEY_KPLUS */
> > + 0x03020030 /* KEY_B */
> > + 0x0303003b /* KEY_F1 */
> > + 0x03040042 /* KEY_F8 */
> > + 0x030500f0 /* KEY_UNKNOWN */
> > + 0x03060018 /* KEY_O */
> > + 0x03070039 /* KEY_SPACE */
> > + 0x04000011 /* KEY_W */
> > + 0x04010015 /* KEY_Y */
> > + 0x04020016 /* KEY_U */
> > + 0x0403003c /* KEY_F2 */
> > + 0x04040073 /* KEY_VOLUMEUP */
> > + 0x040500f0 /* KEY_UNKNOWN */
> > + 0x04060026 /* KEY_L */
> > + 0x04070069 /* KEY_LEFT */
> > + 0x0500001f /* KEY_S */
> > + 0x05010023 /* KEY_H */
> > + 0x05020024 /* KEY_J */
> > + 0x0503003d /* KEY_F3 */
> > + 0x05040043 /* KEY_F9 */
> > + 0x05050072 /* KEY_VOLUMEDOWN */
> > + 0x05060032 /* KEY_M */
> > + 0x0507006a /* KEY_RIGHT */
> > + 0x06000010 /* KEY_Q */
> > + 0x0601001e /* KEY_A */
> > + 0x06020031 /* KEY_N */
> > + 0x0603009e /* KEY_BACK */
> > + 0x0604000e /* KEY_BACKSPACE */
> > + 0x060500f0 /* KEY_UNKNOWN */
> > + 0x06060019 /* KEY_P */
> > + 0x06070067 /* KEY_UP */
> > + 0x07000094 /* KEY_PROG1 */
> > + 0x07010095 /* KEY_PROG2 */
> > + 0x070200ca /* KEY_PROG3 */
> > + 0x070300cb /* KEY_PROG4 */
> > + 0x0704003e /* KEY_F4 */
> > + 0x070500f0 /* KEY_UNKNOWN */
> > + 0x07060160 /* KEY_OK */
> > + 0x0707006c >; /* KEY_DOWN */
> > + linux,input-no-autorepeat;
> > + };
> > };
> >
> > &i2c1 {
> > diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
> > index 04cbbcb..e78a1f1 100644
> > --- a/arch/arm/boot/dts/omap4.dtsi
> > +++ b/arch/arm/boot/dts/omap4.dtsi
> > @@ -296,4 +296,9 @@
> > ti,hwmods = "dmic";
> > };
> > };
> > +
> > + keypad {
> > + compatible = "ti,omap4-keypad";
> > + ti,hwmods = "kbd";
> > + };
> > };
> > --
> > 1.7.1
> >
> Any more comments on this?
>
> ~Sourav
--
balbi
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCHv7] arm/dts: omap4-sdp: Add keypad data
2012-08-20 11:19 ` Poddar, Sourav
2012-08-20 11:34 ` Felipe Balbi
@ 2012-08-20 11:53 ` Benoit Cousson
2012-08-20 11:56 ` Poddar, Sourav
1 sibling, 1 reply; 5+ messages in thread
From: Benoit Cousson @ 2012-08-20 11:53 UTC (permalink / raw)
To: Poddar, Sourav
Cc: devicetree-discuss, linux-arm-kernel, linux-omap, linux-kernel,
Tony Lindgren, Rob Herring, Grant Likely, Felipe Balbi
Hi Sourav,
On 08/20/2012 01:19 PM, Poddar, Sourav wrote:
> Hi All,
>
> On Tue, Aug 14, 2012 at 4:45 PM, Sourav Poddar <sourav.poddar@ti.com> wrote:
>> Add keypad data node in omap4 device tree file.
>> Also fill the device tree binding parameters
>> with the required value in "omap4-sdp" dts file.
>>
>> Tested on omap4430 sdp.
>>
>> Cc: Tony Lindgren <tony@atomide.com>
>> Cc: Benoit Cousson <b-cousson@ti.com>
>> Cc: Rob Herring <rob.herring@calxeda.com>
>> Cc: Grant Likely <grant.likely@secretlab.ca>
>> Cc: Felipe Balbi <balbi@ti.com>
>> Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
>> ---
>> v6->v7
>> Add key name as a comment.
>> arch/arm/boot/dts/omap4-sdp.dts | 70 +++++++++++++++++++++++++++++++++++++++
>> arch/arm/boot/dts/omap4.dtsi | 5 +++
>> 2 files changed, 75 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/omap4-sdp.dts b/arch/arm/boot/dts/omap4-sdp.dts
>> index 72216e9..8e200f4 100644
>> --- a/arch/arm/boot/dts/omap4-sdp.dts
>> +++ b/arch/arm/boot/dts/omap4-sdp.dts
>> @@ -113,6 +113,76 @@
>> "DMic", "Digital Mic",
>> "Digital Mic", "Digital Mic1 Bias";
>> };
>> +
>> + keypad {
>> + keypad,num-rows = <8>;
>> + keypad,num-columns = <8>;
>> + linux,keymap = < 0x00000012 /* KEY_E */
>> + 0x00010013 /* KEY_R */
>> + 0x00020014 /* KEY_T */
>> + 0x00030066 /* KEY_HOME */
>> + 0x0004003f /* KEY_F5 */
>> + 0x000500f0 /* KEY_UNKNOWN */
>> + 0x00060017 /* KEY_I */
>> + 0x0007002a /* KEY_LEFTSHIFT */
>> + 0x01000020 /* KEY_D*/
>> + 0x01010021 /* KEY_F */
>> + 0x01020022 /* KEY_G */
>> + 0x010300e7 /* KEY_SEND */
>> + 0x01040040 /* KEY_F6 */
>> + 0x010500f0 /* KEY_UNKNOWN */
>> + 0x01060025 /* KEY_K */
>> + 0x0107001c /* KEY_ENTER */
>> + 0x0200002d /* KEY_X */
>> + 0x0201002e /* KEY_C */
>> + 0x0202002f /* KEY_V */
>> + 0x0203006b /* KEY_END */
>> + 0x02040041 /* KEY_F7 */
>> + 0x020500f0 /* KEY_UNKNOWN */
>> + 0x02060034 /* KEY_DOT */
>> + 0x0207003a /* KEY_CAPSLOCK */
>> + 0x0300002c /* KEY_Z */
>> + 0x0301004e /* KEY_KPLUS */
>> + 0x03020030 /* KEY_B */
>> + 0x0303003b /* KEY_F1 */
>> + 0x03040042 /* KEY_F8 */
>> + 0x030500f0 /* KEY_UNKNOWN */
>> + 0x03060018 /* KEY_O */
>> + 0x03070039 /* KEY_SPACE */
>> + 0x04000011 /* KEY_W */
>> + 0x04010015 /* KEY_Y */
>> + 0x04020016 /* KEY_U */
>> + 0x0403003c /* KEY_F2 */
>> + 0x04040073 /* KEY_VOLUMEUP */
>> + 0x040500f0 /* KEY_UNKNOWN */
>> + 0x04060026 /* KEY_L */
>> + 0x04070069 /* KEY_LEFT */
>> + 0x0500001f /* KEY_S */
>> + 0x05010023 /* KEY_H */
>> + 0x05020024 /* KEY_J */
>> + 0x0503003d /* KEY_F3 */
>> + 0x05040043 /* KEY_F9 */
>> + 0x05050072 /* KEY_VOLUMEDOWN */
>> + 0x05060032 /* KEY_M */
>> + 0x0507006a /* KEY_RIGHT */
>> + 0x06000010 /* KEY_Q */
>> + 0x0601001e /* KEY_A */
>> + 0x06020031 /* KEY_N */
>> + 0x0603009e /* KEY_BACK */
>> + 0x0604000e /* KEY_BACKSPACE */
>> + 0x060500f0 /* KEY_UNKNOWN */
>> + 0x06060019 /* KEY_P */
>> + 0x06070067 /* KEY_UP */
>> + 0x07000094 /* KEY_PROG1 */
>> + 0x07010095 /* KEY_PROG2 */
>> + 0x070200ca /* KEY_PROG3 */
>> + 0x070300cb /* KEY_PROG4 */
>> + 0x0704003e /* KEY_F4 */
>> + 0x070500f0 /* KEY_UNKNOWN */
>> + 0x07060160 /* KEY_OK */
>> + 0x0707006c >; /* KEY_DOWN */
>> + linux,input-no-autorepeat;
>> + };
>> };
>>
>> &i2c1 {
>> diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
>> index 04cbbcb..e78a1f1 100644
>> --- a/arch/arm/boot/dts/omap4.dtsi
>> +++ b/arch/arm/boot/dts/omap4.dtsi
>> @@ -296,4 +296,9 @@
>> ti,hwmods = "dmic";
>> };
>> };
>> +
>> + keypad {
>> + compatible = "ti,omap4-keypad";
>> + ti,hwmods = "kbd";
>> + };
>> };
>> --
>> 1.7.1
>>
> Any more comments on this?
That's perfect.
I'll take it and send it with other DT patches to Tony.
Regards,
Benoit
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCHv7] arm/dts: omap4-sdp: Add keypad data
2012-08-20 11:53 ` Benoit Cousson
@ 2012-08-20 11:56 ` Poddar, Sourav
0 siblings, 0 replies; 5+ messages in thread
From: Poddar, Sourav @ 2012-08-20 11:56 UTC (permalink / raw)
To: Benoit Cousson
Cc: devicetree-discuss, linux-arm-kernel, linux-omap, linux-kernel,
Tony Lindgren, Rob Herring, Grant Likely, Felipe Balbi
On Mon, Aug 20, 2012 at 5:23 PM, Benoit Cousson <b-cousson@ti.com> wrote:
> Hi Sourav,
>
> On 08/20/2012 01:19 PM, Poddar, Sourav wrote:
>> Hi All,
>>
>> On Tue, Aug 14, 2012 at 4:45 PM, Sourav Poddar <sourav.poddar@ti.com> wrote:
>>> Add keypad data node in omap4 device tree file.
>>> Also fill the device tree binding parameters
>>> with the required value in "omap4-sdp" dts file.
>>>
>>> Tested on omap4430 sdp.
>>>
>>> Cc: Tony Lindgren <tony@atomide.com>
>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>> Cc: Rob Herring <rob.herring@calxeda.com>
>>> Cc: Grant Likely <grant.likely@secretlab.ca>
>>> Cc: Felipe Balbi <balbi@ti.com>
>>> Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
>>> ---
>>> v6->v7
>>> Add key name as a comment.
>>> arch/arm/boot/dts/omap4-sdp.dts | 70 +++++++++++++++++++++++++++++++++++++++
>>> arch/arm/boot/dts/omap4.dtsi | 5 +++
>>> 2 files changed, 75 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/arch/arm/boot/dts/omap4-sdp.dts b/arch/arm/boot/dts/omap4-sdp.dts
>>> index 72216e9..8e200f4 100644
>>> --- a/arch/arm/boot/dts/omap4-sdp.dts
>>> +++ b/arch/arm/boot/dts/omap4-sdp.dts
>>> @@ -113,6 +113,76 @@
>>> "DMic", "Digital Mic",
>>> "Digital Mic", "Digital Mic1 Bias";
>>> };
>>> +
>>> + keypad {
>>> + keypad,num-rows = <8>;
>>> + keypad,num-columns = <8>;
>>> + linux,keymap = < 0x00000012 /* KEY_E */
>>> + 0x00010013 /* KEY_R */
>>> + 0x00020014 /* KEY_T */
>>> + 0x00030066 /* KEY_HOME */
>>> + 0x0004003f /* KEY_F5 */
>>> + 0x000500f0 /* KEY_UNKNOWN */
>>> + 0x00060017 /* KEY_I */
>>> + 0x0007002a /* KEY_LEFTSHIFT */
>>> + 0x01000020 /* KEY_D*/
>>> + 0x01010021 /* KEY_F */
>>> + 0x01020022 /* KEY_G */
>>> + 0x010300e7 /* KEY_SEND */
>>> + 0x01040040 /* KEY_F6 */
>>> + 0x010500f0 /* KEY_UNKNOWN */
>>> + 0x01060025 /* KEY_K */
>>> + 0x0107001c /* KEY_ENTER */
>>> + 0x0200002d /* KEY_X */
>>> + 0x0201002e /* KEY_C */
>>> + 0x0202002f /* KEY_V */
>>> + 0x0203006b /* KEY_END */
>>> + 0x02040041 /* KEY_F7 */
>>> + 0x020500f0 /* KEY_UNKNOWN */
>>> + 0x02060034 /* KEY_DOT */
>>> + 0x0207003a /* KEY_CAPSLOCK */
>>> + 0x0300002c /* KEY_Z */
>>> + 0x0301004e /* KEY_KPLUS */
>>> + 0x03020030 /* KEY_B */
>>> + 0x0303003b /* KEY_F1 */
>>> + 0x03040042 /* KEY_F8 */
>>> + 0x030500f0 /* KEY_UNKNOWN */
>>> + 0x03060018 /* KEY_O */
>>> + 0x03070039 /* KEY_SPACE */
>>> + 0x04000011 /* KEY_W */
>>> + 0x04010015 /* KEY_Y */
>>> + 0x04020016 /* KEY_U */
>>> + 0x0403003c /* KEY_F2 */
>>> + 0x04040073 /* KEY_VOLUMEUP */
>>> + 0x040500f0 /* KEY_UNKNOWN */
>>> + 0x04060026 /* KEY_L */
>>> + 0x04070069 /* KEY_LEFT */
>>> + 0x0500001f /* KEY_S */
>>> + 0x05010023 /* KEY_H */
>>> + 0x05020024 /* KEY_J */
>>> + 0x0503003d /* KEY_F3 */
>>> + 0x05040043 /* KEY_F9 */
>>> + 0x05050072 /* KEY_VOLUMEDOWN */
>>> + 0x05060032 /* KEY_M */
>>> + 0x0507006a /* KEY_RIGHT */
>>> + 0x06000010 /* KEY_Q */
>>> + 0x0601001e /* KEY_A */
>>> + 0x06020031 /* KEY_N */
>>> + 0x0603009e /* KEY_BACK */
>>> + 0x0604000e /* KEY_BACKSPACE */
>>> + 0x060500f0 /* KEY_UNKNOWN */
>>> + 0x06060019 /* KEY_P */
>>> + 0x06070067 /* KEY_UP */
>>> + 0x07000094 /* KEY_PROG1 */
>>> + 0x07010095 /* KEY_PROG2 */
>>> + 0x070200ca /* KEY_PROG3 */
>>> + 0x070300cb /* KEY_PROG4 */
>>> + 0x0704003e /* KEY_F4 */
>>> + 0x070500f0 /* KEY_UNKNOWN */
>>> + 0x07060160 /* KEY_OK */
>>> + 0x0707006c >; /* KEY_DOWN */
>>> + linux,input-no-autorepeat;
>>> + };
>>> };
>>>
>>> &i2c1 {
>>> diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
>>> index 04cbbcb..e78a1f1 100644
>>> --- a/arch/arm/boot/dts/omap4.dtsi
>>> +++ b/arch/arm/boot/dts/omap4.dtsi
>>> @@ -296,4 +296,9 @@
>>> ti,hwmods = "dmic";
>>> };
>>> };
>>> +
>>> + keypad {
>>> + compatible = "ti,omap4-keypad";
>>> + ti,hwmods = "kbd";
>>> + };
>>> };
>>> --
>>> 1.7.1
>>>
>> Any more comments on this?
>
> That's perfect.
>
> I'll take it and send it with other DT patches to Tony.
>
Thanks Benoit !!
> Regards,
> Benoit
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-08-20 11:56 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-14 11:15 [PATCHv7] arm/dts: omap4-sdp: Add keypad data Sourav Poddar
2012-08-20 11:19 ` Poddar, Sourav
2012-08-20 11:34 ` Felipe Balbi
2012-08-20 11:53 ` Benoit Cousson
2012-08-20 11:56 ` Poddar, Sourav
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox