From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754753Ab1LIUKv (ORCPT ); Fri, 9 Dec 2011 15:10:51 -0500 Received: from hqemgate04.nvidia.com ([216.228.121.35]:6309 "EHLO hqemgate04.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753938Ab1LIUKt convert rfc822-to-8bit (ORCPT ); Fri, 9 Dec 2011 15:10:49 -0500 X-PGP-Universal: processed; by hqnvupgp06.nvidia.com on Fri, 09 Dec 2011 12:10:44 -0800 From: Stephen Warren To: Shridhar Rasal , "dmitry.torokhov@gmail.com" CC: "rydberg@euromail.se" , Rakesh Iyer , "linux-kernel@vger.kernel.org" , "linux-input@vger.kernel.org" , "linux-tegra@vger.kernel.org" Date: Fri, 9 Dec 2011 12:10:43 -0800 Subject: RE: [PATCH 1/1] input: keyboard: Set configuration registers Thread-Topic: [PATCH 1/1] input: keyboard: Set configuration registers Thread-Index: Acy2Q9NDEw2BXwnlRtCzLCIg5Pl5PAAamu3A Message-ID: <74CDBE0F657A3D45AFBB94109FB122FF1751860550@HQMAIL01.nvidia.com> References: <1323415758-16822-1-git-send-email-srasal@nvidia.com> In-Reply-To: <1323415758-16822-1-git-send-email-srasal@nvidia.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Shridhar Rasal wrote at Friday, December 09, 2011 12:29 AM: > To: dmitry.torokhov@gmail.com > Cc: rydberg@euromail.se; Stephen Warren; Rakesh Iyer; linux-kernel@vger.kernel.org; linux- > input@vger.kernel.org; linux-tegra@vger.kernel.org; Shridhar Rasal > Subject: [PATCH 1/1] input: keyboard: Set configuration registers > > -Set only REQUIRED row and column configuration register to > PROPER values to avoid continuously generating KBC input events. > -Use *en* field in pin_cfg, to check GPIO_x_ROW_EN register > should be set or clear. > > Signed-off-by: Shridhar Rasal I wondered if num==0 could be used instead of a new en field, but 0 is a valid row/column number, so no. As such, Acked-by: Stephen Warren -- nvpublic