Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754475Ab2HGMyI (ORCPT ); Tue, 7 Aug 2012 08:54:08 -0400 Received: from mail-ee0-f46.google.com ([74.125.83.46]:45940 "EHLO mail-ee0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751690Ab2HGMyG convert rfc822-to-8bit (ORCPT ); Tue, 7 Aug 2012 08:54:06 -0400 Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 6.0 \(1485\)) Subject: Re: [PATCH 3/4] arm/dts: omap5-evm: Add keypad data From: Koen Kooi In-Reply-To: <1343997486-710-4-git-send-email-sourav.poddar@ti.com> Date: Tue, 7 Aug 2012 14:54:22 +0200 Cc: , , , , Benoit Cousson , Felipe Balbi , Santosh Shilimkar Content-Transfer-Encoding: 8BIT Message-Id: <2AE404CF-E456-408A-B7A2-D9B3D331C58E@dominion.thruhere.net> References: <1343997486-710-1-git-send-email-sourav.poddar@ti.com> <1343997486-710-4-git-send-email-sourav.poddar@ti.com> To: Sourav Poddar X-Mailer: Apple Mail (2.1485) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1603 Lines: 53 Op 3 aug. 2012, om 14:38 heeft Sourav Poddar het volgende geschreven: > Add keypad data node in omap5 device tree file. > Also fill the device tree binding parameters > with the required value in "omap5-evm" dts file. > > Tested on omap5430 evm with 3.5 custom kernel. > > Cc: Benoit Cousson > Cc: Felipe Balbi > Cc: Santosh Shilimkar > Acked-by: Felipe Balbi > Signed-off-by: Sourav Poddar > --- > arch/arm/boot/dts/omap5-evm.dts | 12 ++++++++++++ > arch/arm/boot/dts/omap5.dtsi | 5 +++++ > 2 files changed, 17 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/boot/dts/omap5-evm.dts b/arch/arm/boot/dts/omap5-evm.dts > index 45a8aeb..09fe941 100644 > --- a/arch/arm/boot/dts/omap5-evm.dts > +++ b/arch/arm/boot/dts/omap5-evm.dts > @@ -17,6 +17,18 @@ > device_type = "memory"; > reg = <0x80000000 0x40000000>; /* 1 GB */ > }; > + > + keypad { > + keypad,num-rows = <8>; > + keypad,num-columns = <8>; > + linux,keymap = < 0x02020073 > + 0x02030072 > + 0x020400e7 > + 0x02050066 > + 0x0206006b > + 0x020700d9 >; > + linux,input-no-autorepeat; > + }; Again not a coment on your patch, but on DT: If DT bindings must be OS independent, what then, is that linux keycode doing there? regards, Koen -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/