Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753713Ab3CHG5X (ORCPT ); Fri, 8 Mar 2013 01:57:23 -0500 Received: from mail-oa0-f54.google.com ([209.85.219.54]:55434 "EHLO mail-oa0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753400Ab3CHG5U (ORCPT ); Fri, 8 Mar 2013 01:57:20 -0500 MIME-Version: 1.0 In-Reply-To: <1362714214-32405-2-git-send-email-bleung@chromium.org> References: <1362187254-9227-1-git-send-email-bleung@chromium.org> <1362714214-32405-1-git-send-email-bleung@chromium.org> <1362714214-32405-2-git-send-email-bleung@chromium.org> Date: Thu, 7 Mar 2013 22:57:19 -0800 X-Google-Sender-Auth: uNS4GzfjHDo6aT23jBLk4woHdeY Message-ID: Subject: Re: [PATCH v3 1/2] Input: atmel_mxt_ts - Support for touchpad variant From: Benson Leung To: Linus Torvalds , Olof Johansson , Yufeng Shen , Dmitry Torokhov , Matthew Garrett , Henrik Rydberg , Daniel Kurtz , Joonyoung Shim , wfp5p@virginia.edu, linux-input@vger.kernel.org, "linux-kernel@vger.kernel.org" , "platform-driver-x86@vger.kernel.org" Cc: Benson Leung , David Miller , Nick Dyer Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1030 Lines: 32 On Thu, Mar 7, 2013 at 7:43 PM, Benson Leung wrote: > +static void mxt_input_button(struct mxt_data *data, struct mxt_message *message) > +{ > + struct device *dev = &data->client->dev; Oops. I missed a warning: unused variable 'dev' here. > + struct input_dev *input = data->input_dev; > + bool button; > + int i; > + > + /* Active-low switch */ > + for (i = 0; i < MXT_NUM_GPIO; i++) { > + if (data->pdata->key_map[i] == KEY_RESERVED) > + continue; > + button = !(message->message[0] & MXT_GPIO0_MASK << i); > + input_report_key(input, data->pdata->key_map[i], button); > + } > +} > + -- Benson Leung Software Engineer, Chrom* OS bleung@chromium.org -- 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/